Skip to content

Install cmake to restore lintcheck runs #14514

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 1 commit into from
Apr 1, 2025

Conversation

samueltardieu
Copy link
Contributor

@samueltardieu samueltardieu commented Apr 1, 2025

tokio-rustls requires aws-lc-sys which requires cmake 3.x to be built. This installs cmake 3.x from the repository, overriding the cmake 4.x installation.

The situation will be reexamined in a few weeks, as aws-lc-sys will get a new release in a few days with cmake 4.x support. We will then be able to remove this temporary fix.

changelog: none
r? @ghost

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Apr 1, 2025
@samueltardieu samueltardieu force-pushed the push-kzrzwyqlmutr branch 3 times, most recently from 97a78f4 to 6392ebe Compare April 1, 2025 14:10
@samueltardieu samueltardieu changed the title Try removing cmake to restore lintcheck runs Install cmake to restore lintcheck runs Apr 1, 2025
@samueltardieu samueltardieu marked this pull request as ready for review April 1, 2025 14:22
@samueltardieu
Copy link
Contributor Author

r? @flip1995

@samueltardieu
Copy link
Contributor Author

This seems to be blocking #14490 and #14397, and probably all newer PRs.

Copy link
Member

@flip1995 flip1995 left a comment

Choose a reason for hiding this comment

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

For me this is just an approval and merge. As lintcheck is not really in the common local developer flow, this shouldn't be noticable by most contributors

@samueltardieu
Copy link
Contributor Author

For me this is just an approval and merge. As lintcheck is not really in the common local developer flow, this shouldn't be noticable by most contributors

I've updated the description, and I've set a reminder on the second week of May to check if this fix can be removed.

@Alexendoo Alexendoo added this pull request to the merge queue Apr 1, 2025
Merged via the queue into rust-lang:master with commit 227e43a Apr 1, 2025
17 checks passed
@samueltardieu
Copy link
Contributor Author

samueltardieu commented Apr 1, 2025

Looks like this didn't help with #14397, while it did in the current PR, and the trace shows that cmake 3.x has been installed from packages.

@Alexendoo
Copy link
Member

Ah right, some runners are on different image versions it looks like and this PR happened to be on the slightly older one

In that case we should remove tokio-rustls from the lintcheck run for the time being

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants