Skip to content

Relax version requirements for various deps #482

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

Merged
merged 2 commits into from
Aug 26, 2022

Conversation

nicholasbishop
Copy link
Member

Allow older versions of some dependencies to be used if that's all
that's available. This change will have no effect for typical builds
with Cargo, as Cargo will just pick the latest version and download
it. But in more restrictive/sandboxed build environments where the
latest versions of packages might not be available, it's helpful to
allow a wider range of versions.

Only non-dev dependencies have been changed, and only in the three
published packages.

Tested locally by temporarily prefixing these depdency versions with =
to force Cargo to use exactly that version, and verifying that
build/test/run commands still worked.

Allow older versions of some dependencies to be used if that's all
that's available. This change will have no effect for typical builds
with Cargo, as Cargo will just pick the latest version and download
it. But in more restrictive/sandboxed build environments where the
latest versions of packages might not be available, it's helpful to
allow a wider range of versions.

Only non-dev dependencies have been changed, and only in the three
published packages.

Tested locally by temporarily prefixing these depdency versions with `=`
to force Cargo to use exactly that version, and verifying that
build/test/run commands still worked.
@nicholasbishop
Copy link
Member Author

Merging since I want to include this in #488, should be a very low-risk change since normally Cargo will still pick the same version in the end.

@nicholasbishop nicholasbishop merged commit 08ddf18 into rust-osdev:main Aug 26, 2022
@nicholasbishop nicholasbishop deleted the bishop-loosen-deps branch August 26, 2022 18:18
@nicholasbishop nicholasbishop mentioned this pull request Nov 17, 2022
2 tasks
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.

1 participant