Skip to content

BIP-110: Clarify rule 2, update Deployment, and specify GBT usage#2115

Merged
luke-jr merged 2 commits into
bitcoin:masterfrom
dathonohm:bip110-update-rule2-and-deployment
Mar 5, 2026
Merged

BIP-110: Clarify rule 2, update Deployment, and specify GBT usage#2115
luke-jr merged 2 commits into
bitcoin:masterfrom
dathonohm:bip110-update-rule2-and-deployment

Conversation

@dathonohm

Copy link
Copy Markdown
Contributor

Updates to match the reference implementation and address community feedback:

  • Rule 2: Several people were confused about which witness stack elements are limited by rule 2, so I have added a definition for "script argument witness items" (@murchandamus' preferred nomenclature) with a FAQ entry explaining exactly which elements are excluded and why. See this thread on the original PR for more context.
  • Deployment: The reference implementation now has an EXPIRED terminal state in the state machine. Updated the deployment section to reflect this.
  • GBT: Added a GetBlockTemplate subsection to the specification, following BIP 9 conventions, since the reference implementation now includes full GBT support.

@luke-jr luke-jr left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@luke-jr luke-jr merged commit b3ab91f into bitcoin:master Mar 5, 2026
4 checks passed

@murchandamus murchandamus left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good improvements. If this had been left open long enough for others to comment, I would have suggested that you introduce a Changelog section and Version header, and consider advancing your proposal to Complete.

@murchandamus murchandamus added the BIP Update by Owner PR by Author or Deputy to modify their own BIP label Mar 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

BIP Update by Owner PR by Author or Deputy to modify their own BIP

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants