Page MenuHomePhabricator

Merge #18412: script: fix SCRIPT_ERR_SIG_PUSHONLY error string
ClosedPublic

Authored by jasonbcox on Wed, Jun 24, 00:44.

Details

Summary

41ff4992e57f8626019c0b2ab3d024db71e4c20f script: fix SCRIPT_ERR_SIG_PUSHONLY error string (Sebastian Falbesoner)

Pull request description:

Fixes #18411, changing the error message from `"Only non-push operators allowed in signatures"` to `"Only push operators allowed in signatures"`.

ACKs for top commit:

laanwj:
  ACK 41ff4992e57f8626019c0b2ab3d024db71e4c20f

Tree-SHA512: 3b75d83e2198d638d599ef6a4a8da986f0158600fe3f89f55b3759554588157acf2b0cba3f6a907164617264e7aee727d6d460b510c8b37ca7728aa79e11ad80

Backport of Core PR18412

Although our error message is functionally correct, this brings us more inline with Core (per feedback from https://github.com/Bitcoin-ABC/bitcoin-abc/issues/393)

Test Plan

ninja check check-functional

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

jasonbcox created this revision.Wed, Jun 24, 00:44
Herald added a reviewer: Restricted Project. · View Herald TranscriptWed, Jun 24, 00:44
jasonbcox requested review of this revision.Wed, Jun 24, 00:44
teamcity edited the summary of this revision. (Show Details)Wed, Jun 24, 00:44

[Bot Message]
One or more PR numbers were detected in the summary.
Links to those PRs have been inserted into the summary for reference.

deadalnix accepted this revision.Wed, Jun 24, 00:57
This revision is now accepted and ready to land.Wed, Jun 24, 00:57