-
Notifications
You must be signed in to change notification settings - Fork 38.7k
Add option to avoid warning on certain network upgrades #10994
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
Conversation
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.
Missing update to help message.
|
Concept ACK Ping @gmaxwell which requested this. |
e65d827 to
5e16a25
Compare
|
Rebased against master; updated help text; moved structure definition to versionbits.h; use emplace_back instead of push_back. Added additional commit that removes the BIP109/bit 28 warning on testnet by default, without the user needing to specify the parameter. |
|
It looks like interest in this got lost a bit due to recent drama, because it is not sure how feasible BIP9 still is in the first place. But anyhow ping @gmaxwell (again, sorry). |
5e16a25 to
38a3ebd
Compare
|
Rebased; use an enum class instead of a bool for use_ignorebits argument to ComputeBlockVersion. FWIW, I think version bits would still be useful for BIP8 or BIP148/BIP91-style UASF's; full node users can require miners to signal support (or to not signal support if they want to opt-out of a change) so as to allow two potential chains to be easily distinguished by SPV clients. |
The "-vbignore=bit:start:end" option can be used to prevent bitcoin from alerting the user of known but uninteresting proposed upgrades that are being signalled via the version field.
This provides the ability to specify irrelevant versionbit signalling as part of the chain parameters, in order to avoid issuing warnings for possible consensus changes. The "-novbignoredefaults" option can be used to disable this behaviour. This patch ignores signalling for BIP109 via bit 28 in 2016/2017 on testnet via this mechanism.
38a3ebd to
98b8d66
Compare
|
Going to close this until there's some clarity on what method might get used for soft-forks in future (bip 9, bip 8, something else)... |
The "-vbignore=bit:start:end" option can be used to prevent bitcoin
from alerting the user of known but uninteresting proposed upgrades
that are being signalled via the version field.
This hopefully provides the feature requested in issue #8266, allow version bits to have an "ignore button". It requires specifying start and end times rather than a final block height, which should map better to BIP 9 implementations. It also allows multiple deployments on a single bit to be ignored independently -- if the bit is used in between any deployments, it will still generate warnings.
This setting will also avoid the bit from triggering the "Unknown block versions being mined" warning during the set period.