fix: update digest import path for 0.11.0-rc.11 compatibility - #878
fix: update digest import path for 0.11.0-rc.11 compatibility#878Jimmyh-world wants to merge 1 commit into
Conversation
The digest 0.11.0-rc.11 crate moved BlockSizeUser from `digest::crypto_common` to `digest::common`. This fixes the compilation error: error[E0432]: unresolved import `digest::crypto_common` This is a minimal backport of the fix from main branch to the 5.0.0-pre.1 release for users who need this specific version. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
|
I realize the main branch already has this fix in newer pre-releases (5.0.0-pre.5). This PR is mainly to highlight that 5.0.0-pre.1 on crates.io remains broken for users who depend on it transitively (e.g., via Would it be possible to either:
Either would unblock downstream users without requiring Cargo patches. Happy to close this PR if there's a better path forward. |
|
any update on this, we have quite a lot of broken code and need to a lot of manual pinning, based on the recent round of upgrades, and we can't yet upgrade to the latest as not everything has releases with rand@0.10 |
|
As far as I can tell this was fixed in #875 and released as I think this PR is just stale? The lines on #[cfg(feature = "digest")]
use digest::{
FixedOutput, HashMarker, array::typenum::U64, common::BlockSizeUser, consts::True,
typenum::IsGreater,
};@dignifiedquire have you tried upgrading to @Jimmyh-world I have no idea what you're requesting here:
|
|
|
Thanks for the update @tarcieri — understood, we'll look at moving to pre.6 when the time is right. @dignifiedquire yeah, we hit the same wall with the rand version change. We ended up creating patched crates as a workaround while we're still in active development, so we're unblocked for now. Appreciate you flagging the rand issue here though — good to know it's not just us seeing that as a blocker for the upgrade path. No rush on our end, happy to close this one out. Cheers! |
It changed the The breaking changes in |
Summary
This PR fixes a compilation error when using
curve25519-dalek 5.0.0-pre.1withdigest 0.11.0-rc.11.The
digestcrate movedBlockSizeUserfromdigest::crypto_commontodigest::common, causing:Changes
digest::crypto_common::BlockSizeUsertodigest::common::BlockSizeUserContext
This affects users of
iroh(tested with 0.95.x and 0.96.0) which depends oncurve25519-dalek =5.0.0-pre.1. The fix already exists in main branch (pre.5+) but the 5.0.0-pre.1 tag published to crates.io has the broken import.Testing
Verified that
iroh 0.96compiles successfully with this fix applied via Cargo patch.