Skip to content
This repository has been archived by the owner on Oct 21, 2022. It is now read-only.

move to rust 2021 #322

Merged
merged 3 commits into from Nov 21, 2021
Merged

move to rust 2021 #322

merged 3 commits into from Nov 21, 2021

Conversation

NukeManDan
Copy link
Contributor

No description provided.

@pepyakin
Copy link
Contributor

I know this is not something we keep an eye on, but what's the MSRV of that change?

@NukeManDan
Copy link
Contributor Author

Dumb question: MSRV is The "minimum supported Rust version" right?

"increments are a breaking Semver change" policy may applies to this repository 🤔 should it?

The relevant check should update to the minimal dependency versions first, in order to make sure newer compatible versions are ignored in this regard.

AFAIK minimal stable toolchain version with rust edition 2021 is rustc 1.56.0
https://blog.rust-lang.org/2021/10/21/Rust-1.56.0.html

This could be propagated in all change logs and maybe include a version bump. But what type? patch, minor or major?

@athei
Copy link
Member

athei commented Nov 15, 2021

Changing the edition isn't a breaking change because different edition crates can depend on each other. However, this also bumps the required Rust version.

I think we should apply these two changes to this PR:

  • Bump to 0.45
  • Add rust-version = "1.56"

@NukeManDan
Copy link
Contributor Author

THE version you want to target is 1.56.1 and the last .1 is important.

from Will, this is what srtool is using and is tested.

@athei
Copy link
Member

athei commented Nov 15, 2021

Not sure how this is important because that is only a security fix and nothing that interacts with compatibility.

edition = "2021"
rust-version = "1.56.1"
@NukeManDan
Copy link
Contributor Author

rust-version = "1.56.1" introduced

For reference, I tried to get this to cargo build with 1.56.0 and it failed with

error: package `...` cannot be built because it requires rustc 1.56.1 or newer, while the currently active rustc version is 1.56.0

With --ignore-rust-version it does build with 1.56.0, as expected.

@athei
Copy link
Member

athei commented Nov 21, 2021

For reference, I tried to get this to cargo build with 1.56.0 and it failed with

My post stated "1.56" and not "1.56.0". The fix version is just noise here.

@NukeManDan NukeManDan merged commit 1080066 into master Nov 21, 2021
@NukeManDan
Copy link
Contributor Author

Based on a quick DM, it was decided this PR should be merged 🤝

@athei athei deleted the NukeManDan-patch-1 branch November 21, 2021 20:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants