Update hash algorithm for Rust >= 1.85

[JohnCMoon]

Fixes #138.

[jyn514]

thanks for the PR :) this seems roughly on the right track but it needs a couple tweaks.

[JohnCMoon]

@jyn514, I took another pass at this and tested locally - I think it's ready for review.

[jyn514]

thanks :) lgtm with the comment addressed

[reneleonhardt]

@JohnCMoon Can master be released so Rust edition 2024 users can benefit from cargo-sweep? 🙂

[JohnCMoon]

@reneleonhardt, I'm not a maintainer on this project, but agree a release would be helpful. @jyn514 , what do you think?

[tjol]

I just wanted to say that it would be great if somebody who is able to make a release (@jyn514? @marcospb19?) found the time to publish a new version to crates.io, specifically for this change.

[marcospb19]

Hi @tjol, I tried doing so, but got this error:

$ cargo publish --dry-run
    Updating crates.io index
error: all dependencies must have a version specified when publishing.
dependency `rustc-stable-hash` does not specify a version
...

We are using this commit of rustc-stable-hash: rust-lang/rustc-stable-hash@3805516, that needs to be released, at the time of writing, their last release was done 7 months ago and doesn't include that commit.

So, can't do it yet.

[tjol]

Thanks for the update, @marcospb19

[JohnMoon-Voyager]

@marcospb19, looking at rustc-stable-hash, it appears the commit you linked is included in releases since v0.1.0. I believe you can just switch to the latest release (0.1.2) and things should work fine. I tried locally and all cargo test runs pass.

[marcospb19]

XD thanks @JohnMoon-Voyager for double checking and catching my error, I missed the "2024" part, thought that was from this year's July, not last year's.

Done!

https://github.com/holmgr/cargo-sweep/releases/tag/v0.8.0
https://crates.io/crates/cargo-sweep

[reneleonhardt]

Only a typo fix has been merged since then, 0.1.2 wasn't working?
rust-lang/rustc-stable-hash@v0.1.2...main

[marcospb19]

@reneleonhardt the new release is already out