Skip to content

Reject BIP-0060 (three years inactivity) #792

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

Merged
merged 1 commit into from
Mar 21, 2025
Merged

Reject BIP-0060 (three years inactivity) #792

merged 1 commit into from
Mar 21, 2025

Conversation

ysangkok
Copy link
Contributor

@ysangkok ysangkok commented Jun 20, 2019

According to BIP-0002, any person can request this.

@luke-jr
Copy link
Member

luke-jr commented Jul 23, 2019

This is a confusing BIP. I guess it's proposing adding the relay-txs flag to all version messages. Current nodes do this.

@ysangkok
Copy link
Contributor Author

OK, if the BIP is bad, why not Reject it then? Proper documentation can be written in a separate BIP or elsewhere.

@luke-jr
Copy link
Member

luke-jr commented Jul 26, 2019

It's unclear to me if it's Reject or Final.

@ysangkok
Copy link
Contributor Author

@luke-jr it's silly to have a badly written BIP stuck in the Draft state... Can you explain the rationale for either Rejecting or marking as Final? Then those points could be discussed.

@maflcko
Copy link
Member

maflcko commented Jun 26, 2020

side-note: There is a protocol-version-bump collision between BIP 60 and BIP 61.

It is up to implementations whether to implement BIP 60. If there are some implementations that implement it, it should be Final.

@luke-jr
Copy link
Member

luke-jr commented Jun 26, 2020

I wouldn't call it a collision - you could just as well argue BIP 61 and BIP 60 must be implemented together?

@maflcko
Copy link
Member

maflcko commented Jun 26, 2020

The protocol version number can not be used as an indicator whether BIP 61 (or BIP 60) is implemented

@ysangkok
Copy link
Contributor Author

@kallewoof Is there any chance that this can move ahead or should I close it?

@murchandamus
Copy link
Contributor

It seems to me that BIP60 has not made progress in 11 years, and is not implemented by any software. According to BIP2, it should be moved to rejected. If it should rather be treated differently, this would require an amendment of our process.

ACK 3b6a929

@murchandamus
Copy link
Contributor

After reading the discussion on #1012, it may be better to hold off on merging this PR, as there is currently an on-going discussion about revising the BIP process per a successor to BIP2.

@murchandamus murchandamus added the Process Trying to update process or stuck due to disagreement about Process label May 9, 2024
Copy link
Contributor

@murchandamus murchandamus left a comment

Choose a reason for hiding this comment

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

It seems to me that this should be closed both per the BIP2 process or the BIP3 process, and the point that it might be implemented by some Bitcoin implementation has not been substantiated in 6 years, so I am going to merge this PR.

@murchandamus murchandamus merged commit b650373 into bitcoin:master Mar 21, 2025
@murchandamus murchandamus removed the Process Trying to update process or stuck due to disagreement about Process label Mar 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants