Skip to content

Conversation

@maaku
Copy link
Contributor

@maaku maaku commented Sep 29, 2017

This prevents an unhelpful "unknown error" from being printed in test logs if the CLEANSTACK error condition is set.

…own error" if the CLEANSTACK error condition is set.
@jl2012
Copy link
Contributor

jl2012 commented Sep 29, 2017

utACK cee28fb

@achow101
Copy link
Member

utACK cee28fb

@instagibbs
Copy link
Member

utACK

@sipa sipa merged commit cee28fb into bitcoin:master Sep 29, 2017
sipa added a commit that referenced this pull request Sep 29, 2017
cee28fb Add error string for CLEANSTACK script violation, preventing an "unknown error" if the CLEANSTACK error condition is set. (Mark Friedenbach)

Pull request description:

  This prevents an unhelpful "unknown error" from being printed in test logs if the CLEANSTACK error condition is set.

Tree-SHA512: cd6764e930184aef3d662e40c67f2ea8aea8552a26d33a567d0315a19d707a82aa2afad9f48ecbb731aa5b77fbbfbd7a6a3a989fdb1424a1181350052ff2a9b5
luke-jr pushed a commit to bitcoinknots/bitcoin that referenced this pull request Nov 11, 2017
…own error" if the CLEANSTACK error condition is set.

Github-Pull: bitcoin#11418
Rebased-From: cee28fb
PastaPastaPasta pushed a commit to PastaPastaPasta/dash that referenced this pull request Jan 12, 2020
cee28fb Add error string for CLEANSTACK script violation, preventing an "unknown error" if the CLEANSTACK error condition is set. (Mark Friedenbach)

Pull request description:

  This prevents an unhelpful "unknown error" from being printed in test logs if the CLEANSTACK error condition is set.

Tree-SHA512: cd6764e930184aef3d662e40c67f2ea8aea8552a26d33a567d0315a19d707a82aa2afad9f48ecbb731aa5b77fbbfbd7a6a3a989fdb1424a1181350052ff2a9b5
deadalnix pushed a commit to Bitcoin-ABC/bitcoin-abc that referenced this pull request Jun 24, 2020
Summary:
cee28fb Add error string for CLEANSTACK script violation, preventing an "unknown error" if the CLEANSTACK error condition is set. (Mark Friedenbach)

Pull request description:

  This prevents an unhelpful "unknown error" from being printed in test logs if the CLEANSTACK error condition is set.

Tree-SHA512: cd6764e930184aef3d662e40c67f2ea8aea8552a26d33a567d0315a19d707a82aa2afad9f48ecbb731aa5b77fbbfbd7a6a3a989fdb1424a1181350052ff2a9b5

Backport of Core [[bitcoin/bitcoin#11418 | PR11418]]

We had applied a similar fix to this previously, but this brings our error message inline with Core's (per feedback from #393)

Test Plan: `ninja check check-functional`

Reviewers: #bitcoin_abc, deadalnix

Reviewed By: #bitcoin_abc, deadalnix

Differential Revision: https://reviews.bitcoinabc.org/D6697
ftrader pushed a commit to bitcoin-cash-node/bitcoin-cash-node that referenced this pull request Aug 17, 2020
Summary:
cee28fb Add error string for CLEANSTACK script violation, preventing an "unknown error" if the CLEANSTACK error condition is set. (Mark Friedenbach)

Pull request description:

  This prevents an unhelpful "unknown error" from being printed in test logs if the CLEANSTACK error condition is set.

Tree-SHA512: cd6764e930184aef3d662e40c67f2ea8aea8552a26d33a567d0315a19d707a82aa2afad9f48ecbb731aa5b77fbbfbd7a6a3a989fdb1424a1181350052ff2a9b5

Backport of Core [[bitcoin/bitcoin#11418 | PR11418]]

We had applied a similar fix to this previously, but this brings our error message inline with Core's (per feedback from Bitcoin-ABC#393)

Test Plan: `ninja check check-functional`

Reviewers: #bitcoin_abc, deadalnix

Reviewed By: #bitcoin_abc, deadalnix

Differential Revision: https://reviews.bitcoinabc.org/D6697
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants