Enable `bitcoin_hashes v0.14.0` #76

tcharding · 2024-07-30T21:58:50Z

Hardware devices like the smallest binary possible, currently if one builds with latest rust-bitcoin and rust-bip39 they get two versions of bitcoin_hashes in the dependency graph because we don't support the latest bitcoin_hashes.

Note using the latest version causes the MSRV to increase to Rust v0.56.1.

tcharding · 2024-07-30T21:59:46Z

cc @afilini

afilini · 2024-07-31T19:03:15Z

ACK, that would be helpful for me, all the other libs like rust-bitcoin and bdk have already upgraded to hashes 0.14

Kixunil

This would be nice for me even just for cleanness sake.

Kixunil · 2024-08-26T12:03:41Z

Cargo.toml

 # Enabling the "rand" feature by default to run the benches
 bip39 = { path = ".", features = ["rand"] }
-bitcoin_hashes = ">=0.12,<0.14" # enable default features for test
+bitcoin_hashes = ">=0.12,<=0.14" # enable default features for test


This must be >= 0.12, <0.15, otherwise 0.14.1 will be unusable even though it'd be compatible.

Oh wow, TIL. Will fix, thanks.

I'm going to go right ahead and 'trust don't verify'.

The verification is easy, just evaluate 0 < 1 expression in your head. :)

Oh I thought we were talking about 0.14 working for 0.14.0 but not for 0.14.1.

0.14 implies 0.14.0

Ok, I have no idea what you were talking about in your original comment then. <= 14 is equivalent to < 15

It's not. 0.14 == 0.14.0
0.14.1 > 0.14.0
0.14.1 < 0.15.0
And the above implies 0.14.1 > 0.14 which is a negation of 0.14.1 <= 0.14
So for <= 14 to be equivalent to < 15 we require all x to satisfy x <= 0.14 <-> x <0.15 and we have a counter example for x = 0.14.1. QED

I get it now. Seems the confusion started when I wrote "I'm going to go right ahead and 'trust don't verify'." and I have no clue what I was thinking then. Your original review comment is actually perfectly clean.

Just call me retarded :)

Kixunil · 2024-08-26T12:04:04Z

Cargo.toml


 # Unexported dependnecies
-bitcoin_hashes = { version = ">=0.12, <=0.13", default-features = false }
+bitcoin_hashes = { version = ">=0.12, <=0.14", default-features = false }


This also needs to be <0.15, it was already broken.

Kixunil

ACK 2b47d40

tcharding · 2024-08-29T19:56:19Z

Oh this PR is no good anyway, bitcoin_hashes v14 has an incompatible MSRV.

tcharding · 2024-09-18T04:33:07Z

Raised #79 to discuss the MSRV bump. Converting to draft until that resolves.

kayabaNerve · 2025-02-04T11:07:28Z

Is this a MSRV bump? Anyone who wants an older MSRV can use an older version of bitcoin_hashes. This may cause some automated workflows to fail on first pass? But I wouldn't consider that breaking, as at worst it would be trivially correctable by an extra line in a Cargo.toml of bitcoin_hashes = "=0.12".

As someone who currently has two copies of bitcoin_hashes in-tree, I'd appreciate this.

Kixunil · 2025-02-04T11:20:23Z

@kayabaNerve that line wouldn't be great, better jut use Cargo.lock correctly.

kayabaNerve · 2025-02-04T12:43:25Z

Precise updates get overriden upon the next call to update. A line in the toml, next to the line adding this very crate as a dependency, would pin it. That's why I said what I said. It's irrelevant to the larger point I don't believe this is a MSRV break and should be able to move forward regardless though.

Kixunil · 2025-02-04T15:11:41Z

Precise updates get overriden upon the next call to update.

MSRV doesn't get bumped on update if you use recent cargo. But yes, this is not MSRV break in the sense that it'd justify major version.

benma · 2025-08-20T08:53:08Z

Could you take this PR out of draft and merge and release please? 😄 Seems like a simple change and I need it for the reason @tcharding outlined.

dmitrmax · 2025-08-28T15:52:53Z

Could you take this PR out of draft and merge and release please? 😄 Seems like a simple change and I need it for the reason @tcharding outlined.

I second this. The simplest fix is resting on the shelf for more than a year now.

Maintainer, please do the v2.2.1 release.

benma · 2025-09-03T08:38:42Z

Ping @tcharding @stevenroose, please let's merge, it's a simple change and very much needed. Thanks

tcharding · 2025-09-04T02:55:48Z

I'm not a maintainer here sorry mate. If you have a direct link to Steven you could hassle him.

benma · 2025-09-21T07:00:43Z

@tcharding I guess the first step would be for you to rebase it (there are conflicts) and take it out of draft.

tcharding · 2025-09-21T23:01:52Z

No sweat, done.

tcharding · 2025-09-21T23:07:31Z

Oh this cannot be undrafted because of the MSRV bump. At the risk of being rude, and @stevenroose you can slap me in person when you see me next, I'd suggest one of the following:

Someone pester Steven to maintain his lib (I'm not going to do this, I've hassled him enough about other libs more important to me).
Suggest to him to bring on another maintainer, or offer to be one.
Fork the crate and sort it out.

I'm not the don of the rust-bitcoin org but I do feel in someway responsible that we have crates in the org that are not maintained. I don't know the solution to that problem.

No one is required to do anything round here, so props to Steven for writing the crate, and all due respect.

elsirion · 2025-10-28T09:50:43Z

LGTM, just some syntax errors in the workflow file

Screenshot 2025-10-28 at 05-49-43 Enable `bitcoin_hashes v0 14 0` · rust-bitcoin_rust-bip39@b3f9c4d

tcharding · 2025-10-28T22:57:24Z

Used new syntax as suggested and fixed up the github action job names.

tcharding · 2025-10-28T23:24:13Z

And used valid Rust versions, will fix pinning ...

tnull

Seems CI is still broken due to dependency/pinning foo.

tcharding · 2025-11-05T04:23:26Z

Is any one waiting on this that wants to work it out? I f**king hate pinning.

tnull · 2025-11-05T10:41:01Z

Is any one waiting on this that wants to work it out? I f**king hate pinning.

Hmm, honestly not so much the actual changes, but having the additional CI coverage would be nice. But no huge rush rn.

tcharding · 2025-11-05T23:54:22Z

Instead of pinning I found a lock file that works for 1.41 and 1.48 and committed those, then used the 1.48 lock file for all the MSRV versions tested.

tnull

Instead of pinning I found a lock file that works for 1.41 and 1.48 and committed those, then used the 1.48 lock file for all the MSRV versions tested.

Hmm, not quite sure if I prefer that approach, especially given that we point users to test.sh for pinning instructions to make this work under a specific MSRV. And now, we don't pin there. If you prefer not to deal with the the pinning part, we can just land this (maybe only the first commit)? And I can take it from there.

tnull · 2025-11-06T09:38:58Z

README.md

+- `bitcoin_hashes v0.13`: MSRV `v1.48.0`
+- `bitcoin_hashes v0.14`: MSRV `v1.56.0`

 When using older version of Rust, you might have to pin the versions of several crates, for an up-to-date list refer to [`contrib/test.sh`](contrib/test.sh):


We refer here to test.sh for pinning instructions, and now there aren't any anymore.

Oh dear, good eye. Thanks man, ~~will fix.~~

tcharding · 2025-11-06T21:11:20Z

If you prefer not to deal with the the pinning part, we can just land this (maybe only the first commit)? And I can take it from there.

Great, I have zero interest in pinning or in this crate at all so I"d really appreciate not having to touch this PR anymore. Thanks for the offer. I've remove the second patch.

tnull

Okay, LGTM mod one comment regarding the added pins.sh.

Can revisit extending MSRV test coverage in CI as a follow-up sometime.

tnull · 2025-11-07T07:50:16Z

pins.sh

Seems this file isn't used anywhere? Can we drop it for now?

Bother, that was just supposed to be use while I was messing with the pinning. My bad. Removed.

tnull · 2025-11-10T08:39:43Z

README.md


 When using older version of Rust, you might have to pin the versions of several crates, for an up-to-date list refer to [`contrib/test.sh`](contrib/test.sh):
-
+ 


This whitespace could still be dropped, but shouldn't block this PR :>

Hardware devices like the smallest binary possible, currently if one builds with `rust-bitcoin v0.32` and `rust-bip39` they get two versions of `bitcoin_hashes` in the dependency graph because we don't support `bitcoin_hashes v0.14`. Note that using `bitcoin_hashes 0.14` bumps the MSRV to Rust v1.56.1

tcharding · 2025-11-10T23:21:34Z

My bad, done.

tcharding · 2025-11-10T23:21:50Z

I actually checked by diff this time ...

elsirion · 2025-11-11T21:43:39Z

README.md


 - `bitcoin_hashes v0.12`: MSRV v1.41.1
 - `bitcoin_hashes v0.13`: MSRV v1.48.0
+- `bitcoin_hashes v0.14`: MSRV v1.56.0


I haven't verified this claim given that we decided to forgo CI checks for now and manually figuring out the necessary pinning is a pita. It seems benign enough and a reasonable assumption that the MSRV will be dictated by bitcoin_hashes and there aren't any further interactions in the code base.

Per https://github.com/michalkucharczyk/rust-bip39/tree/mku-2.0.1-release, `parity-bip39` was a fork to publish a release of `bip39` with two specific PRs merged. Not only have those PRs been merged, yet `bip39` now accepts `bitcoin_hashes 0.14` (rust-bitcoin/rust-bip39#76), making this a great time to reconcile (even though it does technically add a git dependency until the new release is cut...).

Kixunil suggested changes Aug 26, 2024

View reviewed changes

tcharding force-pushed the 07-30-update-hashes branch from 46ef0ed to 2b47d40 Compare August 26, 2024 23:34

Kixunil approved these changes Aug 27, 2024

View reviewed changes

tcharding mentioned this pull request Sep 18, 2024

Bump MSRV to Rust 1.63 #79

Closed

tcharding marked this pull request as draft September 18, 2024 04:33

tcharding mentioned this pull request Nov 12, 2024

chore: allow using bitcoin_hashes v0.14 #82

Closed

tvolk131 mentioned this pull request Nov 13, 2024

Duplicate dependencies fedimint/fedimint#6293

Open

tcharding force-pushed the 07-30-update-hashes branch 2 times, most recently from 1ca2e1b to 26683c2 Compare September 21, 2025 23:01

tcharding marked this pull request as ready for review September 21, 2025 23:02

tcharding marked this pull request as draft September 21, 2025 23:03

tcharding force-pushed the 07-30-update-hashes branch 2 times, most recently from 5994a4c to 1357566 Compare October 26, 2025 22:33

tcharding marked this pull request as ready for review October 26, 2025 22:33

tcharding changed the title ~~Enable bitcoin_hashes v0.14.0~~ Enable bitcoin_hashes v0.14.0 Oct 26, 2025

tcharding force-pushed the 07-30-update-hashes branch from b3f9c4d to a20b9fa Compare October 28, 2025 22:57

tcharding force-pushed the 07-30-update-hashes branch 2 times, most recently from 1a83c40 to ec024a8 Compare October 28, 2025 23:00

tcharding force-pushed the 07-30-update-hashes branch from ec024a8 to 649c990 Compare October 28, 2025 23:28

tnull reviewed Oct 30, 2025

View reviewed changes

tcharding force-pushed the 07-30-update-hashes branch 2 times, most recently from e3144d2 to d80f73a Compare November 5, 2025 23:53

tcharding force-pushed the 07-30-update-hashes branch 2 times, most recently from ead6f20 to 56f25a7 Compare November 6, 2025 02:57

tnull reviewed Nov 6, 2025

View reviewed changes

tcharding force-pushed the 07-30-update-hashes branch 2 times, most recently from 20a77f1 to 14b4f8c Compare November 6, 2025 21:10

tnull reviewed Nov 7, 2025

View reviewed changes

tcharding force-pushed the 07-30-update-hashes branch from 14b4f8c to 2c573a2 Compare November 9, 2025 20:55

tnull approved these changes Nov 10, 2025

View reviewed changes

tcharding force-pushed the 07-30-update-hashes branch from 2c573a2 to d567f87 Compare November 10, 2025 23:21

elsirion approved these changes Nov 11, 2025

View reviewed changes

tnull merged commit f735e25 into rust-bitcoin:master Nov 13, 2025
7 checks passed


		When using older version of Rust, you might have to pin the versions of several crates, for an up-to-date list refer to [`contrib/test.sh`](contrib/test.sh):

Enable bitcoin_hashes v0.14.0 #76

Enable bitcoin_hashes v0.14.0 #76

Uh oh!

Conversation

tcharding commented Jul 30, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

tcharding commented Jul 30, 2024

Uh oh!

afilini commented Jul 31, 2024

Uh oh!

Kixunil left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Kixunil left a comment

Choose a reason for hiding this comment

Uh oh!

tcharding commented Aug 29, 2024

Uh oh!

tcharding commented Sep 18, 2024

Uh oh!

kayabaNerve commented Feb 4, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Kixunil commented Feb 4, 2025

Uh oh!

kayabaNerve commented Feb 4, 2025

Uh oh!

Kixunil commented Feb 4, 2025

Uh oh!

benma commented Aug 20, 2025

Uh oh!

dmitrmax commented Aug 28, 2025

Uh oh!

benma commented Sep 3, 2025

Uh oh!

tcharding commented Sep 4, 2025

Uh oh!

benma commented Sep 21, 2025

Uh oh!

tcharding commented Sep 21, 2025

Uh oh!

tcharding commented Sep 21, 2025

Uh oh!

elsirion commented Oct 28, 2025

Uh oh!

tcharding commented Oct 28, 2025

Uh oh!

tcharding commented Oct 28, 2025

Uh oh!

tnull left a comment

Choose a reason for hiding this comment

Uh oh!

tcharding commented Nov 5, 2025

Uh oh!

tnull commented Nov 5, 2025

Uh oh!

tcharding commented Nov 5, 2025

Uh oh!

tnull left a comment

Enable `bitcoin_hashes v0.14.0` #76

Enable `bitcoin_hashes v0.14.0` #76

tcharding commented Jul 30, 2024 •

edited

Loading

kayabaNerve commented Feb 4, 2025 •

edited

Loading

tcharding Nov 6, 2025 •

edited

Loading

tcharding commented Nov 6, 2025 •

edited

Loading