Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Sanity-check script sizes in bitcoin-tx #11554

Merged
merged 1 commit into from Nov 7, 2017
Merged

Sanity-check script sizes in bitcoin-tx #11554

merged 1 commit into from Nov 7, 2017

Conversation

TheBlueMatt
Copy link
Contributor

No description provided.

@promag
Copy link
Member

promag commented Oct 26, 2017

utACK a6f33ea.

Should add some test case to test/util/data/bitcoin-util-test.json?

@laanwj
Copy link
Member

laanwj commented Nov 6, 2017

concept and utACK a6f33ea, better error clarity is good

@maflcko
Copy link
Member

maflcko commented Nov 7, 2017

utACK a6f33ea

@maflcko maflcko merged commit a6f33ea into bitcoin:master Nov 7, 2017
maflcko pushed a commit that referenced this pull request Nov 7, 2017
Merge #11554: Sanity-check script sizes in bitcoin-tx
89cc4f9 
a6f33ea Sanity-check script sizes in bitcoin-tx (Matt Corallo)

Pull request description:

Tree-SHA512: bb8ecb628763af23816ab085758f6140920a6ff05dcb298129c2bbe584a02a759c700a05740eca77023292c98a5658b2a608fa27d5a948d183f87ed9ab827952
@luke-jr
Copy link
Member

luke-jr commented Nov 10, 2017

This should probably be backported...

luke-jr pushed a commit to luke-jr/bitcoin that referenced this pull request Nov 11, 2017
@TheBlueMatt @luke-jr
Sanity-check script sizes in bitcoin-tx
f455bfd 
Github-Pull: bitcoin#11554
Rebased-From: a6f33ea
@luke-jr luke-jr mentioned this pull request Nov 11, 2017
Merged
laanwj added a commit that referenced this pull request Nov 20, 2017
@laanwj
Merge #11662: [0.15] Sanity-check script sizes in bitcoin-tx
a69cc07 
f455bfd Sanity-check script sizes in bitcoin-tx (Matt Corallo)

Pull request description:

  Backport of #11554

  Cleanly merges into 0.14 too if we care.

Tree-SHA512: e8aff8bcde5925a73b2b9293f4782cdcf5ebc71a29001a592628b2d77e368db9905823fe7956c8e8b50ae942c77f64d48e6023a58f95b1457e106fca780308a3
@fanquake fanquake mentioned this pull request Sep 18, 2018
Merged
codablock pushed a commit to codablock/dash that referenced this pull request Sep 26, 2019
@codablock
Merge bitcoin#11554: Sanity-check script sizes in bitcoin-tx
7ab0ea2 
a6f33ea Sanity-check script sizes in bitcoin-tx (Matt Corallo)

Pull request description:

Tree-SHA512: bb8ecb628763af23816ab085758f6140920a6ff05dcb298129c2bbe584a02a759c700a05740eca77023292c98a5658b2a608fa27d5a948d183f87ed9ab827952
codablock pushed a commit to codablock/dash that referenced this pull request Sep 30, 2019
@codablock
Merge bitcoin#11554: Sanity-check script sizes in bitcoin-tx
569a11f 
a6f33ea Sanity-check script sizes in bitcoin-tx (Matt Corallo)

Pull request description:

Tree-SHA512: bb8ecb628763af23816ab085758f6140920a6ff05dcb298129c2bbe584a02a759c700a05740eca77023292c98a5658b2a608fa27d5a948d183f87ed9ab827952
jasonbcox pushed a commit to Bitcoin-ABC/bitcoin-abc that referenced this pull request Dec 20, 2019
@majcosta
Sanity-check script sizes in bitcoin-tx
d688432 
Summary:
"Sanity-check script sizes in bitcoin-tx (Matt Corallo)"

This is a backport from Core PR11554: bitcoin/bitcoin#11554

Test Plan:
  ninja check
  ./test/functional/test_runner.py

Reviewers: O1 Bitcoin ABC, #bitcoin_abc, deadalnix, Fabien

Reviewed By: O1 Bitcoin ABC, #bitcoin_abc, Fabien

Differential Revision: https://reviews.bitcoinabc.org/D4666
barrystyle pushed a commit to PACGlobalOfficial/PAC that referenced this pull request Jan 22, 2020
@barrystyle
Merge bitcoin#11554: Sanity-check script sizes in bitcoin-tx
08471c9 
a6f33ea Sanity-check script sizes in bitcoin-tx (Matt Corallo)

Pull request description:

Tree-SHA512: bb8ecb628763af23816ab085758f6140920a6ff05dcb298129c2bbe584a02a759c700a05740eca77023292c98a5658b2a608fa27d5a948d183f87ed9ab827952
jonspock pushed a commit to jonspock/devault that referenced this pull request Oct 2, 2020
@jonspock
Sanity-check script sizes in bitcoin-tx
86b1fe1 
Summary:
"Sanity-check script sizes in bitcoin-tx (Matt Corallo)"

This is a backport from Core PR11554: bitcoin/bitcoin#11554

Test Plan:
  ninja check
  ./test/functional/test_runner.py

Reviewers: O1 Bitcoin ABC, #bitcoin_abc, deadalnix, Fabien

Reviewed By: O1 Bitcoin ABC, #bitcoin_abc, Fabien

Differential Revision: https://reviews.bitcoinabc.org/D4666
jonspock pushed a commit to jonspock/devault that referenced this pull request Oct 5, 2020
@jonspock
Sanity-check script sizes in bitcoin-tx
ff02d80 
Summary:
"Sanity-check script sizes in bitcoin-tx (Matt Corallo)"

This is a backport from Core PR11554: bitcoin/bitcoin#11554

Test Plan:
  ninja check
  ./test/functional/test_runner.py

Reviewers: O1 Bitcoin ABC, #bitcoin_abc, deadalnix, Fabien

Reviewed By: O1 Bitcoin ABC, #bitcoin_abc, Fabien

Differential Revision: https://reviews.bitcoinabc.org/D4666
jonspock pushed a commit to devaultcrypto/devault that referenced this pull request Oct 10, 2020
@jonspock
Sanity-check script sizes in bitcoin-tx
b94db88 
Summary:
"Sanity-check script sizes in bitcoin-tx (Matt Corallo)"

This is a backport from Core PR11554: bitcoin/bitcoin#11554

Test Plan:
  ninja check
  ./test/functional/test_runner.py

Reviewers: O1 Bitcoin ABC, #bitcoin_abc, deadalnix, Fabien

Reviewed By: O1 Bitcoin ABC, #bitcoin_abc, Fabien

Differential Revision: https://reviews.bitcoinabc.org/D4666
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Dec 16, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@TheBlueMatt @promag @laanwj @maflcko @luke-jr