Skip to content

bitcoind: add support for v31.0#596

Open
0xB10C wants to merge 1 commit into
rust-bitcoin:masterfrom
0xB10C:2026-05-bitcoind-v31.0
Open

bitcoind: add support for v31.0#596
0xB10C wants to merge 1 commit into
rust-bitcoin:masterfrom
0xB10C:2026-05-bitcoind-v31.0

Conversation

@0xB10C
Copy link
Copy Markdown
Contributor

@0xB10C 0xB10C commented May 20, 2026

This is a cherry-pick from #586 by @xyzconstant which only adds support for downloading Bitcoin Core v31 to the bitcoind crate. My hope is that this is faster and easier to review than the whole of #586.

It does not touch the RPC part nor the integration tests - this can be done elsewhere. The bitcoind v31.0 binary is needed in e.g. peer-observer/peer-observer#435 (where I currently work around this with a custom 0xb10c/corepc branch) for the bitcoin-node binary.

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.

Comment thread bitcoind/Cargo.toml Outdated
download = ["anyhow", "bitcoin_hashes", "flate2", "tar", "bitreq", "zip"]

latest = ["30_2"]
latest = ["31_0"]
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I don't think we should change the latest feature, since v31 is not fully implemented.

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.

Right. Good point. For anyone else wondering what it's missing:

in bitcoind/src/client_versions.rs

#[cfg(feature = "31_0")]
pub use corepc_client::{client_sync::v31::*, types::v31 as vtype};

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.

I went a head and added a comment for the next one.

@0xB10C 0xB10C force-pushed the 2026-05-bitcoind-v31.0 branch from 41128fe to 8bb308c Compare May 20, 2026 10:46
Copy link
Copy Markdown
Collaborator

@jamillambert jamillambert left a comment

Choose a reason for hiding this comment

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

ACK 8bb308c

CI failure is unrelated, I'm looking into it.

@jamillambert
Copy link
Copy Markdown
Collaborator

jamillambert commented May 20, 2026

Unrelated CI failure in this PR due to the below error fixed in #597:

21 | use webpki_roots::TLS_SERVER_ROOTS;
   |     ^^^^^^^^^^^^ use of unresolved module or unlinked crate `webpki_roots`

@satsfy
Copy link
Copy Markdown
Contributor

satsfy commented May 20, 2026

@0xB10C you can just git commit --amend --no-edit and force push here to try again.

@0xB10C 0xB10C force-pushed the 2026-05-bitcoind-v31.0 branch from 8bb308c to 8145739 Compare May 21, 2026 10:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants