Skip to content

Bring fix for fully qualified path#238

Merged
Manishearth merged 1 commit into
rust-fuzz:mainfrom
baloo:baloo/push-lopsmyosqnpv
Jun 2, 2026
Merged

Bring fix for fully qualified path#238
Manishearth merged 1 commit into
rust-fuzz:mainfrom
baloo:baloo/push-lopsmyosqnpv

Conversation

@baloo

@baloo baloo commented Jun 2, 2026

Copy link
Copy Markdown
Contributor

When a downstream crate has a minimal-version check, the version of derive_arbitrary that gets pulled in is 1.4.0, which does not carry the fix the fully qualified path on result that was made in de2becf

This bumps the patch version of the derive from arbitrary so that a downstream crate does not have to make a direct dependency expression against derive_arbitrary.

minimal-version checks are useful for checking API stability.

When a downstream crate has a [minimal-version check], the
version of derive_arbitrary that gets pulled in is 1.4.0, which does not
carry the fix the fully qualified path on result that was made in
de2becf

This bumps the patch version of the derive from arbitrary so that a
downstream crate does not have to make a direct dependency expression
against `derive_arbitrary`.

minimal-version checks are useful for checking API stability.

[minimal-version check]: <https://github.com/RustCrypto/actions/blob/master/.github/workflows/minimal-versions.yml>
@Manishearth

Copy link
Copy Markdown
Member

Generally I don't think it actually makes sense to do things like this: the version dep is "the latest version dep you need" which makes it maximally flexible for people. When you use minimal-version you are opting in to not getting fixes. If you need a fix, specify it in the Cargo.toml.

That said, that fix is years old now, so I don't particularly mind merging this. I don't plan to cut a release for this crate soon though (some other maintainer might).

@Manishearth Manishearth merged commit df8afe9 into rust-fuzz:main Jun 2, 2026
6 checks passed
@baloo baloo deleted the baloo/push-lopsmyosqnpv branch June 2, 2026 22:23
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.

2 participants