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.

Sign up for GitHub

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

Jump to bottom

Upgrade rust-bitcoin to 0.32 #1422

Open
2 of 6 tasks
notmandatory opened this issue May 2, 2024 · 7 comments · May be fixed by bitcoindevkit/rust-electrum-client#133, bitcoindevkit/rust-esplora-client#85, RCasatta/electrsd#80 or bitcoindevkit/rust-hwi#99
Open
2 of 6 tasks

Upgrade rust-bitcoin to 0.32 #1422

notmandatory opened this issue May 2, 2024 · 7 comments · May be fixed by bitcoindevkit/rust-electrum-client#133, bitcoindevkit/rust-esplora-client#85, RCasatta/electrsd#80 or bitcoindevkit/rust-hwi#99
Labels
api A breaking API change enhancement New feature or request
Milestone
1.0.0-alpha

Comments

@notmandatory
Copy link
Member

notmandatory commented May 2, 2024 •
edited by RCasatta

Describe the enhancement

The LDK team would like to use rust-bitcoin version 0.32 so BDK should update too. To do so we also need to update the following crates to the versions that depend on rust-bitcoin 0.32:

Use case

Updating to rust-bitcoin 0.32 removes the need for a specific no-std feature, it will be implied by not enabling the std feature. It also includes a new io crate.

Additional context

https://discord.com/channels/753336465005608961/753367451319926827/1228314439133171795
@notmandatory notmandatory added the enhancement New feature or request label May 2, 2024
@notmandatory notmandatory added this to the 1.0.0-alpha milestone May 2, 2024
@notmandatory notmandatory mentioned this issue May 2, 2024
Merged
3 tasks
@oleonardolima
Copy link
Contributor

I'm willing to give a try on this one.

@notmandatory
Copy link
Member Author

Thanks @oleonardolima you'll need to keep an eye to see when rust-miniscript and rust-bitcoincore-rpc are ready. Feel free to start making PRs for any of the crate that only need rust-bitcoin updated to 0.32.

@notmandatory
Copy link
Member Author

Another thing this PR should fix is the broken docs.rs for bdk_persist. I think they're breaking because it doesn't build with default features.

https://docs.rs/crate/bdk_persist/0.1.0/builds/1208655

@oleonardolima
Copy link
Contributor

Thanks @oleonardolima you'll need to keep an eye to see when rust-miniscript and rust-bitcoincore-rpc are ready. Feel free to start making PRs for any of the crate that only need rust-bitcoin updated to 0.32.

Sure, I'll keep an eye and initially start from other crates.

Another thing this PR should fix is the broken docs.rs for bdk_persist. I think they're breaking because it doesn't build with default features.

https://docs.rs/crate/bdk_persist/0.1.0/builds/1208655

Cool, I'll check if it solves the docs once the PR is done.

This was referenced May 3, 2024
Open
Open
@ValuedMammal
Copy link
Contributor

ValuedMammal commented May 4, 2024 •
edited

@notmandatory Would #1423 fix the docs.rs fail? - Oh I see you mentioned it wouldn't be needed after the rust-bitcoin upgrade. Still maybe a quick fix in the interim

This was referenced May 4, 2024
Open
Open
Draft
notmandatory added a commit that referenced this issue May 8, 2024
@notmandatory
Merge #1423: fix(persist): add default feature to enable bdk_chain/std
66abc73 
a577c22 fix(persist): add default feature to enable bdk_chain/std (Steve Myers)

Pull request description:

  ### Description

  This PR adds a `default` feature to `bdk_persist` so it can be build on its own.  Once #1422 is done we can remove the `default`again.

  ### Notes to the reviewers

  I need to be able to build `bdk_persist` on its own so I can publish it to crates.io.

  ### Checklists

  #### All Submissions:

  * [x] I've signed all my commits
  * [x] I followed the [contribution guidelines](https://github.com/bitcoindevkit/bdk/blob/master/CONTRIBUTING.md)
  * [x] I ran `cargo fmt` and `cargo clippy` before committing

ACKs for top commit:
  ValuedMammal:
    ACK a577c22
  oleonardolima:
    ACK a577c22
  storopoli:
    ACK a577c22

Tree-SHA512: 8b07a9e4974dec8812ca19ce7226dcaece064270a0be8b83d3c326fdf1e89b051eb0bd8aa0eda9362b2c8233ecd6003b70c92ee046603973d8d40611418c3841
@ValuedMammal
Copy link
Contributor

ValuedMammal commented May 8, 2024 •
edited

I forgot to mention there was a suggestion #1412 (comment) for a better crate description for bdk_persist that maybe we can fix when the opportunity arises.

@jirijakes jirijakes mentioned this issue May 10, 2024
Open
@notmandatory notmandatory added the api A breaking API change label May 13, 2024
@oleonardolima
Copy link
Contributor

oleonardolima commented May 16, 2024 •
edited

update: Some of the rust-bitcoin crates have been released, and the rust-electrum-client, and electrsd are ready to review (depends on each other).

EDIT: habemus rust-miniscript 🥳

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
api A breaking API change enhancement New feature or request
Projects
BDK
Status: In Progress
Milestone
1.0.0-alpha
3 participants
@notmandatory @oleonardolima @ValuedMammal