Skip to content

Bump bitcoin dependency requirement to v0.32.6 to reflect used API #266

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

Conversation

tnull
Copy link
Contributor

@tnull tnull commented Jun 12, 2025

Description

Starting with commit f6fd9853e, bdk_wallet makes use of bitcoin::key::TweakedKeyPair::to_keypair which was introduced with v0.32.6. Here, we adjust the minimum API requirement in Cargo.toml, as BDK 2.0.0 builds could otherwise fail with:

error[E0599]: no method named `to_keypair` found for struct `TweakedKeypair` in the current scope
   --> /home/tnull/.cargo/registry/src/index.crates.io-1949cf8c6b5b557f/bdk_wallet-2.0.0/src/wallet/signer.rs:580:14
    |
578 |           None => keypair
    |  _________________-
579 | |             .tap_tweak(secp, psbt_input.tap_merkle_root)
580 | |             .to_keypair(),
    | |             -^^^^^^^^^^ method not found in `TweakedKeypair`
    | |_____________|
    |

For more information about this error, try `rustc --explain E0599`.

Copy link
Member

@thunderbiscuit thunderbiscuit left a comment

Choose a reason for hiding this comment

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

ACK c9c8806.

Copy link
Contributor

@oleonardolima oleonardolima left a comment

Choose a reason for hiding this comment

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

ACK c9c8806

@oleonardolima oleonardolima moved this to Needs Review in BDK Wallet Jun 12, 2025
@oleonardolima oleonardolima added the dependencies Pull requests that update a dependency file label Jun 12, 2025
@ValuedMammal
Copy link
Collaborator

ACK c9c8806

Thanks @tnull

I can see why this would have been smart to include in #214 @tvpeter

@ValuedMammal ValuedMammal added this to the Wallet 2.1.0 milestone Jun 13, 2025
Copy link
Member

@luisschwab luisschwab left a comment

Choose a reason for hiding this comment

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

ACK c9c8806

Starting with commit `f6fd9853e`, `bdk_wallet` makes use of
`bitcoin::key::TweakedKeyPair::to_keypair` which was introduced with
v0.32.6. Here, we adjust the minimum API requirement in `Cargo.toml`, as
BDK 2.0.0 builds could otherwise fail with:

```
error[E0599]: no method named `to_keypair` found for struct `TweakedKeypair` in the current scope
   --> /home/tnull/.cargo/registry/src/index.crates.io-1949cf8c6b5b557f/bdk_wallet-2.0.0/src/wallet/signer.rs:580:14
    |
578 |           None => keypair
    |  _________________-
579 | |             .tap_tweak(secp, psbt_input.tap_merkle_root)
580 | |             .to_keypair(),
    | |             -^^^^^^^^^^ method not found in `TweakedKeypair`
    | |_____________|
    |

For more information about this error, try `rustc --explain E0599`.
```
@ValuedMammal ValuedMammal force-pushed the 2025-06-bump-bitcoin-version-to-unbreak-semver branch from c9c8806 to af792ea Compare June 16, 2025 22:01
@ValuedMammal
Copy link
Collaborator

reACK af792ea

@ValuedMammal ValuedMammal merged commit ec6e59b into bitcoindevkit:master Jun 16, 2025
20 checks passed
@github-project-automation github-project-automation bot moved this from Needs Review to Done in BDK Wallet Jun 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

5 participants