Skip to content

BIP 149: Segregated Witness (second deployment)#526

Merged
luke-jr merged 5 commits intobitcoin:masterfrom
shaolinfry:bip-segwit-uasf
May 1, 2017
Merged

BIP 149: Segregated Witness (second deployment)#526
luke-jr merged 5 commits intobitcoin:masterfrom
shaolinfry:bip-segwit-uasf

Conversation

@shaolinfry
Copy link
Copy Markdown
Contributor

No description provided.

@kanzure
Copy link
Copy Markdown
Contributor

kanzure commented Apr 29, 2017


==Deployment==

This BIP will be deployed by BIP8 with the name "uasegwit" and using bit 2.
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.

I think it would be best to keep "segwit" as the name here, if only deployment strategy is changed.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

It's unclear to be if each BIP9 deployment needs a completely different common name or not, but in any case, since we need a new service bit, I think there has to be a new name anyway. I'm going to leave this as is for the time being.


==Rationale==

This BIP can be deployed well in advance of the BIP8 '''starttime''' so that the '''timeout''' will be sufficiently far in the future to allow Bitcoin users to uprgade in preparation.
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.

upgrade*

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

fixed


==Deployment==

This BIP will be deployed by BIP8 with the name "uasegwit" and using bit 2.
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.

Normally bits aren't assigned until the proposal leaves Draft stage (at least in principle) I think? (There's also an allocations file that would need updating)

Considering that there may be changes to the proposal before re-deployment, it is probably better not to give the impression it is ready for implementation.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Changed to TBD. Will update deployments file once there is an actual deployment.


==Deployment==

This BIP will be deployed by BIP8 with the name "uasegwit" and using bit TBD.
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.

Note that a new name will require redeployment of mining software.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

fixed

@shaolinfry
Copy link
Copy Markdown
Contributor Author

Should be good to go now.

@luke-jr luke-jr changed the title BIP proposal: segwit-uasf BIP 149: segwit-uasf May 1, 2017
@luke-jr
Copy link
Copy Markdown
Member

luke-jr commented May 1, 2017

Use BIP 149. Title currently exceeds 44 character limit.

@luke-jr luke-jr added the New BIP label May 1, 2017
@luke-jr luke-jr changed the title BIP 149: segwit-uasf BIP 149: Segregated Witness (second deployment) May 1, 2017
@luke-jr luke-jr merged commit a27cb63 into bitcoin:master May 1, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants