bdk icon indicating copy to clipboard operation
bdk copied to clipboard

bump(bdk_chain): rusqlite to 0.36.0

Open binarybaron opened this issue 4 months ago • 3 comments

Description

This PR bumps the chain crates feature dependency on rusqlite to the more recent version 0.36.0.

Notes to the reviewers

I submitted this PR because the "outdated" dependency breaks dependency resolution in my project.

See https://github.com/bitcoindevkit/bdk/pull/1766

Changelog notice

Checklists

All Submissions:

New Features:

  • [ ] I've added tests for the new feature
  • [ ] I've added docs for the new feature

Bugfixes:

  • [ ] This pull request breaks the existing API
  • [ ] I've added tests to reproduce the issue which are now passing
  • [ ] I'm linking the issue being fixed by this PR

binarybaron avatar Jul 29 '25 17:07 binarybaron

Changed to 0.36.0 because arti-client uses 0.36.0

binarybaron avatar Jul 29 '25 17:07 binarybaron

@tnull & @binarybaron what would you think about extracting the bdk_sqlite persistence store from the bdk_chain crate and putting it in it's own crate. If we do that we could have a 1.0.0 version with the current 0.31.0 rusqlite and release a new 2.0.0 version with 0.36.0 rusqlite, while still back-porting bug / features for the 1.x. Or open to other ideas along these lines.

notmandatory avatar Jul 29 '25 17:07 notmandatory

@tnull & @binarybaron what would you think about extracting the bdk_sqlite persistence store from the bdk_chain crate and putting it in it's own crate. If we do that we could have a 1.0.0 version with the current 0.31.0 rusqlite and release a new 2.0.0 version with 0.36.0 rusqlite, while still back-porting bug / features for the 1.x. Or open to other ideas along these lines.

So, moving the SQLite store into an individual crate would generally be a good idea, IMO. I however don't think that it would solve the issue, at least if you don't want to go down the road of introducing features to enable individual rusqlite versions. As mentioned above, a lot of projects in the space by now depend on the exact rusqlite version, and I think it would be important to check in with most of them to coordinate such breakage. Apart from that, yes, it would of course also require a major version bump for all of the involved projects.

tnull avatar Jul 29 '25 17:07 tnull