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

chore: update rust-version to 1.63 in all crates #6126

Merged
merged 1 commit into from
Nov 4, 2023

Conversation

Darksonn
Copy link
Contributor

@Darksonn Darksonn commented Nov 3, 2023

Refs: #6125

@Darksonn Darksonn added A-tokio-util Area: The tokio-util crate A-tokio-macros Area: The tokio-macros crate A-tokio-test Area: The tokio-test crate A-tokio-stream Area: The tokio-stream crate labels Nov 3, 2023
@Darksonn Darksonn requested a review from taiki-e November 3, 2023 09:54
@@ -6,7 +6,7 @@ name = "tokio-macros"
# - Create "tokio-macros-1.x.y" git tag.
version = "2.1.0"
edition = "2021"
rust-version = "1.56"
rust-version = "1.63"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tokio-macros is a dependency of tokio, so I don't think it is a utility crate.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good point. I'll reword the commit message when I merge it. However, I don't see any issue in upgrading it, since the next release of tokio-macros will need at least the next version of Tokio, which is on 1.63.

@Darksonn Darksonn changed the title chore: update rust-version to 1.63 in utility crates chore: update rust-version to 1.63 in all crates Nov 3, 2023
@taiki-e
Copy link
Member

taiki-e commented Nov 3, 2023

We will need to figure out how to add a CI step to prevent this from happening again.

Cargo uses cargo-hack's --rust-version flag for this (it can handle multiple different MSRVs in workspace):
https://github.com/rust-lang/cargo/blob/6354bb3d813335731838fd7dce2010d040cc2545/.github/workflows/main.yml#L257

@Darksonn
Copy link
Contributor Author

Darksonn commented Nov 3, 2023

Ah, wow it supports that? Cargo-hack is awesome.

@@ -6,7 +6,7 @@ name = "tokio-util"
# - Create "tokio-util-0.7.x" git tag.
version = "0.7.10"
edition = "2021"
rust-version = "1.56"
rust-version = "1.63"
Copy link

@polarathene polarathene Nov 3, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just to clarify, is this no longer accurately representing the minimal version this would compile on? As the issue I raised mentioned it was 1.60 with an older tokio release. (EDIT: I had forgotten about changes to 0.7.10 bumping it again)

I understand with newer tokio releases that 1.56 isn't supported, and the current 1.56 for tokio-util is already inaccurate as a fallback for toolchains below 1.60.


So is rust-version within tokio crates likely not to represent a crates actual minimal toolchain version to compile on? But the MSRV for tokio crates as a whole?

EDIT: My bad, I forgot that a newer release of tokio-util also bumped to hashbrown 0.14 which has rust-version = "1.63" 🤦‍♂️

@Darksonn Darksonn merged commit 944024e into master Nov 4, 2023
74 of 79 checks passed
@Darksonn Darksonn deleted the alice/update-rust-version-1.63 branch November 4, 2023 08:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-tokio-macros Area: The tokio-macros crate A-tokio-stream Area: The tokio-stream crate A-tokio-test Area: The tokio-test crate A-tokio-util Area: The tokio-util crate
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants