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.

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

[WIP] Bump digest, ed25519, signature, and sha2 #676

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

tarcieri
Copy link
Contributor

@tarcieri tarcieri commented Jul 28, 2024

Bumps the aforementioned dependencies to their latest (pre)releases.

See also: #620

tarcieri added 2 commits July 28, 2024 10:31
Bumps the aforementioned dependencies to their latest (pre)releases
tarcieri added a commit to RustCrypto/SSH that referenced this pull request Jul 28, 2024
This makes it possible to publish a prerelease, which we otherwise can't
do since we're sourcing `ed25519-dalek` from this git branch:

dalek-cryptography/curve25519-dalek#676

This commit can be reverted immediately after we publish a crate
release.
tarcieri added a commit to RustCrypto/SSH that referenced this pull request Jul 28, 2024
This makes it possible to publish a prerelease, which we otherwise can't
do since we're sourcing `ed25519-dalek` from this git branch:

dalek-cryptography/curve25519-dalek#676

This commit can be reverted immediately after we publish a crate
release.
tarcieri added a commit to RustCrypto/SSH that referenced this pull request Jul 28, 2024
This makes it possible to publish a prerelease, which we otherwise can't
do since we're sourcing `ed25519-dalek` from this git branch:

dalek-cryptography/curve25519-dalek#676

This commit can be reverted immediately after we publish a crate
release.
@wiktor-k
Copy link
Contributor

Hi,

Are there any blockers for un-drafting this PR and merging it? FWIW I'm using -pre versions of these crates and they work really well :)

Thanks for your time! 👋

@tarcieri
Copy link
Contributor Author

We'd need to create separate branches for stable vs development so not as to block stable work

@cvengler
Copy link

What is the current state of this? I am currently trying to integrate ed25519-dalek in an already existing larger project, that already moved to the pre-release versions in its development branch, thereby causing conflicts. ☹️

@tarcieri
Copy link
Contributor Author

We are discussing how to proceed. We might need to do a major version bump due to how aggressively people are reacting to MSRV bumps

@robjtede
Copy link
Contributor

We might need to do a major version bump

For 1.60 -> 1.72? Surely 1.72 (Aug 2023) is old enough to not warrant that.

how aggressively people are reacting

Yikes, really?

@tarcieri
Copy link
Contributor Author

The new MSRV will be 1.85 when the digest feature is active, due to 2024 edition bumps in the @RustCrypto crates.

You can see how people reacted here: RustCrypto/formats#1684

@robjtede
Copy link
Contributor

It's slightly surprising that /formats hopped on 2024 so quickly but I assume it had a good reason and was basically waiting for it.

Still, lockfiles exist to solve this already; doesn't require pinning in manifests. And post-1.84 you can opt in to resolver=3 to get MSRV resolving automatically.

Sucks that people are reacting like that instead of learning about the tools provided to help them.

@tarcieri
Copy link
Contributor Author

I pointed that out, but it seems some people were complaining about uncached cargo install failing on old Rust versions (where they could still use --locked).

It seems it doesn't matter if there are solutions, people have poorly configured builds and will show up to bitch you out with "Your shitty software which I don't care about broke my build"

@robjtede
Copy link
Contributor

robjtede commented Feb 28, 2025

I expect (and indeed have experienced) the same vocal minority objecting to (cascading) major library releases. The solutions are much simpler for the MSRV case.

I'll leave it at that.

@tarcieri
Copy link
Contributor Author

tarcieri commented Feb 28, 2025

Yeah, it's definitely much simpler not to bump the version, though I'm not sure you can necessarily dismiss them as a "vocal minority" (especially when you're not the one dealing with them)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants