Skip to content

Conversation

@starius
Copy link
Contributor

@starius starius commented Jan 7, 2026

See https://groups.google.com/g/bitcoindev/c/QsC_FWPNGr4/m/t6MSDT6pBQAJ

This undoes #476387 and #451606

Bitcoin 30.1 and 30.0 are affected by a rare bug in the wallet compound, which may result in a loss of funds. Rolling back to safety.

Note that code signer willcl-ark which we added in #476387 is still in the list. He has signed 29.1, so we should keep him.

I propose to use 29.1 and not update to 29.2, because Ava Chow mentioned in the mailing list, that version 30 will be fixed in 30.2 and we can update right to it.

Analyzing release notes for 30.0 and 30.1 I haven't found any downgrade risks. The only caveat is coinstat index rebuilding, but they have accounted for it in the release notes.

Things done

  • Built on platform:
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • Tested, as applicable:
  • Ran nixpkgs-review on this PR. See nixpkgs-review usage.
  • Tested basic functionality of all binary files, usually in ./result/bin/.
  • Nixpkgs Release Notes
    • Package update: when the change is major or breaking.
  • NixOS Release Notes
    • Module addition: when adding a new NixOS module.
    • Module update: when the change is significant.
  • Fits CONTRIBUTING.md, pkgs/README.md, maintainers/README.md and other READMEs.

Add a 👍 reaction to pull requests you find important.

This reverts commit c9ccd15.

Note that willcl-ark.gpg is still in the list of checked signers, see the
discussion in NixOS#476387
@starius starius requested review from 0xB10C, prusnak and roconnor January 7, 2026 01:48
@nixpkgs-ci nixpkgs-ci bot requested review from RaghavSood and mmahut January 7, 2026 01:53
@nixpkgs-ci nixpkgs-ci bot added 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. labels Jan 7, 2026
Copy link
Contributor

@roconnor roconnor left a comment

Choose a reason for hiding this comment

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

I reviewed the diff but didn't test it.

@nixpkgs-ci nixpkgs-ci bot added 12.approvals: 1 This PR was reviewed and approved by one person. 12.approved-by: package-maintainer This PR was reviewed and approved by a maintainer listed in any of the changed packages. labels Jan 7, 2026
@0xB10C
Copy link
Contributor

0xB10C commented Jan 7, 2026

While bitcoin-core/bitcoincore.org#1213 is still a draft, I think it's a priority of the project to get v30.2 released. I wonder if it's worth downgrading now and then upgrading to v30.2 ~next week instead of just upgrading to v30.2 once it's released?

@prusnak
Copy link
Member

prusnak commented Jan 7, 2026

+1 to wait for 30.2

@roconnor
Copy link
Contributor

roconnor commented Jan 7, 2026

My view is that the package is currently broken; cached binaries could cause users to lose funds; and we don't actually know when 30.2 will be available.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 12.approvals: 1 This PR was reviewed and approved by one person. 12.approved-by: package-maintainer This PR was reviewed and approved by a maintainer listed in any of the changed packages.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants