-
Notifications
You must be signed in to change notification settings - Fork 5.8k
bip-0054: update forward compat section with Bitcoin Core v30 #2044
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.
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
base: master
Are you sure you want to change the base?
Conversation
| that makes this type of transaction non-standard should be widely adopted before this soft fork is | ||
| considered for activation. | ||
| Bitcoin Core version [30.0][Core 30.0] and later will not generate a block template including a | ||
| transaction that violates the signature operations limit introduced in this BIP. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| transaction that violates the signature operations limit introduced in this BIP. | |
| transaction that violates the signature operations limit introduced in this BIP (see https://github.com/bitcoin/bitcoin/pull/32521). |
Do you think it would be useful to link to bitcoin/bitcoin#32521 here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah i thought about it but figured it was superfluous to link to implementation details.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is a rough threshold for "widely adopted" and, depending on what it is relative to current v30 adoption, should a mention about adoption remain for now?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am not sure what's a good threshold to give. Ideally all of them would be upgraded before activation, but you also do not want to give a too small minority of miners a veto power over consensus changes.
If something is to be kept it should apply to the whole section really, although the two most concerning items are the coinbase restriction and this one. I'm not sure. On the other hand having a whole section about miner forward compatibility sort of speaks for itself.
The BIP 54 sigops limit was made a standardness rules in Bitcoin Core 30.0.
8c526c0 to
1076d90
Compare
|
Rebased after #2015 was merged (thanks!). |
jonatack
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ACK 1076d90
The BIP 54 sigops limit was made a standardness rule in Bitcoin Core 30.0 with the merge of bitcoin/bitcoin#32521.