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

elliptic-curve: bump MSRV to 1.56 #795

Merged
merged 1 commit into from
Oct 21, 2021

Conversation

tarcieri
Copy link
Member

@tarcieri tarcieri commented Oct 21, 2021

The RustCrypto/formats repo was bumped to Rust 2021 edition, and some of those crates (e.g. der, sec1) are hard dependencies for the elliptic-curve crate:

RustCrypto/formats#136

To actually bump the crate to the 2021 edition, we need to bump the MSRV of the rest of the crates in the repo, so this commit only changes the MSRV and doesn't actually do a 2021 edition upgrade for the elliptic-curve crate yet.

@tarcieri
Copy link
Member Author

tarcieri commented Oct 21, 2021

Oh wow, that's annoying. It seems when we bump edition, we need to do entire repos at a time, as otherwise cargo will error out on edition = "2021" in e.g. elliptic-curve/Cargo.toml as being unsupported.

Given that, I'm going to leave elliptic-curve (and crypto) as 2018 edition crates, but bump their MSRV as they now use 2021 edition dependencies.

@tarcieri tarcieri force-pushed the elliptic-curve/2021-edition-upgrade branch from 79448bd to 9de917d Compare October 21, 2021 18:47
@tarcieri tarcieri changed the title elliptic-curve: Rust 2021 edition upgrade elliptic-curve: bump MSRV to 1.56 Oct 21, 2021
@tarcieri tarcieri force-pushed the elliptic-curve/2021-edition-upgrade branch 4 times, most recently from 2946252 to 6b59dd6 Compare October 21, 2021 19:42
The RustCrypto/formats repo was bumped to Rust 2021 edition, and some of
those crates (e.g. `der`, `sec1`) are hard dependencies for the
`elliptic-curve` crate:

RustCrypto/formats#136

This is a corresponding Rust 2021 edition bump for the `elliptic-curve`
crate, as well as the omnibus `crypto` crate whose MSRV is determined by
the highest MSRV crate in this repo.
@tarcieri tarcieri force-pushed the elliptic-curve/2021-edition-upgrade branch from 6b59dd6 to efc36a7 Compare October 21, 2021 19:57
@tarcieri
Copy link
Member Author

tarcieri commented Oct 21, 2021

I found a solution that allows elliptic-curve to bumped to the 2018 edition: remove it from the toplevel workspace.yml and put it in its own workspace.

That keeps the 2021 edition crates nicely out-of-band from the 2018 edition ones until they can be upgraded.

@tarcieri tarcieri merged commit b168e8c into master Oct 21, 2021
@tarcieri tarcieri deleted the elliptic-curve/2021-edition-upgrade branch October 21, 2021 20:04
tarcieri added a commit to RustCrypto/signatures that referenced this pull request Oct 21, 2021
The `elliptic-curve` crate as well as format parsers including `der`,
`sec1`, and `pkcs8` have all been bumped to the 2021 edition:

RustCrypto/traits#795

This commit updates the `ecdsa` crate to the 2021 edition accordingly.
tarcieri added a commit to RustCrypto/signatures that referenced this pull request Oct 21, 2021
The `elliptic-curve` crate as well as format parsers including `der`,
`sec1`, and `pkcs8` have all been bumped to the 2021 edition:

RustCrypto/traits#795

This commit updates the `ecdsa` crate to the 2021 edition accordingly.
tarcieri added a commit to RustCrypto/signatures that referenced this pull request Oct 21, 2021
The `elliptic-curve` crate as well as format parsers including `der`,
`sec1`, and `pkcs8` have all been bumped to the 2021 edition:

RustCrypto/traits#795

This commit updates the `ecdsa` crate to the 2021 edition accordingly.
tarcieri added a commit to RustCrypto/elliptic-curves that referenced this pull request Oct 21, 2021
The `elliptic-curve` and `ecdsa` crates as well as format parsers
including `der`, `sec1`, and `pkcs8` have all been bumped to
Rust 2021 edition:

- RustCrypto/traits#795
- RustCrypto/signatures#384

This commit updates the crates in this repo accordingly.
tarcieri added a commit to RustCrypto/elliptic-curves that referenced this pull request Oct 21, 2021
The `elliptic-curve` and `ecdsa` crates as well as format parsers
including `der`, `sec1`, and `pkcs8` have all been bumped to
Rust 2021 edition:

- RustCrypto/traits#795
- RustCrypto/signatures#384

This commit updates the crates in this repo accordingly.
tarcieri added a commit to RustCrypto/elliptic-curves that referenced this pull request Oct 21, 2021
The `elliptic-curve` and `ecdsa` crates as well as format parsers
including `der`, `sec1`, and `pkcs8` have all been bumped to
Rust 2021 edition:

- RustCrypto/traits#795
- RustCrypto/signatures#384

This commit updates the crates in this repo accordingly.
tarcieri added a commit to RustCrypto/elliptic-curves that referenced this pull request Oct 21, 2021
The `elliptic-curve` and `ecdsa` crates as well as format parsers
including `der`, `sec1`, and `pkcs8` have all been bumped to
Rust 2021 edition:

- RustCrypto/traits#795
- RustCrypto/signatures#384

This commit updates the crates in this repo accordingly.
@tarcieri tarcieri mentioned this pull request Nov 20, 2021
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