Skip to content
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

package.mask: re-mask bitcoin >=0.21.1 #22818

Closed
wants to merge 1 commit into from

Conversation

luke-jr
Copy link
Contributor

@luke-jr luke-jr commented Nov 4, 2021

Without a mask, automatic upgrades would trigger, which is unethical (at least at this time).

This reverts commit ae7251a.

Without a mask, automatic upgrades would trigger, which is unethical (at least at this time).

This reverts commit ae7251a.

Closes: https://bugs.gentoo.org/821709
Signed-off-by: Luke Dashjr <luke-jr+git@utopios.org>
@gentoo-bot
Copy link

Pull Request assignment

Submitter: @luke-jr
Areas affected: profiles
Packages affected: (none)

@gentoo/github

Linked bugs

Bugs linked: 821709

New packages

This Pull Request appears to be introducing new packages only. Due to limited manpower, adding new packages is considered low priority. This does not mean that your Pull Request will not receive any attention, however, it might take quite some time for it to be reviewed. In the meantime, your new ebuild might find a home in the GURU project repository: the ebuild repository maintained collaboratively by Gentoo users. GURU offers your ebuild a place to be reviewed and improved by other Gentoo users, while making it easy for Gentoo users to install it and enjoy the software it adds.


In order to force reassignment and/or bug reference scan, please append [please reassign] to the pull request title.

Docs: Code of ConductCopyright policy (expl.) ● DevmanualGitHub PRsProxy-maint guide

@gentoo-bot gentoo-bot added need assignment It was impossible to assign the PR correctly. Please assign it manually. bug linked Bug/Closes found in footer, and cross-linked with the PR. labels Nov 4, 2021
@luke-jr
Copy link
Contributor Author

luke-jr commented Nov 4, 2021

Quick fix to avoid users being upgraded without actively consenting.

Later, we can discuss other solutions (see also #21490 for some prior discussion)

@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2021-11-04 20:25 UTC
Newest commit scanned: 3116863
Status: ✅ good

There are existing issues already. Please look into the report to make sure none of them affect the packages in question:
https://qa-reports.gentoo.org/output/gentoo-ci/f0c2de261d/output.html

@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2021-11-04 20:40 UTC
Newest commit scanned: 3116863
Status: ✅ good

There are existing issues already. Please look into the report to make sure none of them affect the packages in question:
https://qa-reports.gentoo.org/output/gentoo-ci/7a6baa6cac/output.html

@gentoo-bot gentoo-bot closed this in deab2f8 Nov 5, 2021
@gyakovlev
Copy link
Member

@candrews FYI

merged since sounds like a big deal.
please find consensus on this bitcoin stuff guys.

and just a disclaimer again: I have no opinion on this topic or cryptocurrency in general,
just merging this PR as a request due to time-sensitivity.

@candrews
Copy link
Member

candrews commented Nov 5, 2021

NACK.

@gyakovlev I strongly disagree that this mask should be preserved.

Please see #21490 where it was agreed that the mask would be removed as November approached - I removed that mask as agreed.

Also, this is not an emergency and this mask should not have been re-added so quickly, especially since I'm listed as co-maintainer and had not yet responded.

Taproot is happening. There is no stopping it. And I'm not seeing any ethical argument here. The bitcoin community talked about taproot for over a year, it was scheduled for implementation through a community driven approval process, and it is happening. As far as I can tell, this is all unjustified FUD and it either it needs to stop, or real evidence needs to be cited and raised with the bitcoin project (and honestly, the media outlets).

@candrews
Copy link
Member

candrews commented Nov 5, 2021

By blocking upgrades to taproot supporting version of bitcoin, we are hurting our users.

Taproot happens on November 16; there is no stopping it.

What happens if I don't upgrade in time?
If you have not upgraded before Taproot activates (around November 12th), your node will no longer be fully secure and you may become vulnerable to certain attacks. In such a case, your node would be trusting and relying on miners to protect you until you do upgrade.

-- https://bitcointaproot.cc/

@luke-jr
Copy link
Contributor Author

luke-jr commented Nov 5, 2021

Please see #21490 where it was agreed that the mask would be removed as November approached - I removed that mask as agreed.

That was not agreed on. In fact, I explicitly rejected your suggestion to do so Nov 1st and explained why it was unethical, and you changed it to imply it was not decided yet. Removing it should have been discussed and done only when ethical to do so (it isn't today).

Also, this is not an emergency and this mask should not have been re-added so quickly, especially since I'm listed as co-maintainer and had not yet responded.

You're not... and I don't think I would be comfortable with adding you at this point. This is the second time you've done this.

Taproot is happening. There is no stopping it.

This is false. If users do not choose to enforce it (which requires they make that choice), Taproot doesn't happen.

By blocking upgrades to taproot supporting version of bitcoin, we are hurting our users.

The intent is not to block, but to ensure users actively consent. That is why they are advised to make a decision and unmask it themselves.

Better (IMO) alternatives to ensure user opt-in were suggested, but this is the method we ended up using this time. Next time, we should use something cleaner. But not getting user consent at all, is not ethical.

@gyakovlev
Copy link
Member

like I said - no opinion here from me on switch or not.
mask serves a purpose of a message to user, true.

also consider creating a news item. there has been a lot of points raised last time this discussion happened.

looks like there's still time.

@floppym
Copy link
Contributor

floppym commented Nov 5, 2021

@candrews You are not named as a "co-maintainer" on any of these packages. You are a member of the proxy-maint project, but I don't think that really makes you a co-maintainer. If you really want to take responsibility for bitcoin, add yourself to metadata.xml please.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug linked Bug/Closes found in footer, and cross-linked with the PR. need assignment It was impossible to assign the PR correctly. Please assign it manually.
Projects
None yet
6 participants