Skip to content

ci: Update RUSTFLAGS to attempt to resolve cache issues #4537

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
Jul 7, 2025

Conversation

tgross35
Copy link
Contributor

@tgross35 tgross35 commented Jul 7, 2025

Our cache does not seem to be working. rust-cache takes RUSTFLAGS into
account, so it is possible that there is a problem due to it getting
set dynamically.

This is more than likely not the cause, but it is worth a try and is
cleaner anyway.

@rustbot rustbot added A-CI Area: CI-related items S-waiting-on-review labels Jul 7, 2025
@tgross35 tgross35 force-pushed the ci-cache branch 2 times, most recently from 37fa791 to 3b17052 Compare July 7, 2025 23:02
@tgross35 tgross35 enabled auto-merge July 7, 2025 23:02
@tgross35 tgross35 changed the title Try to fix CI caching ci: Update RUSTFLAGS to attempt to resolve cache issues Jul 7, 2025
Our cache does not seem to be working. rust-cache takes RUSTFLAGS into
account, so  it is possible that there is a problem due to it getting
set dynamically.

This is more than likely not the cause, but it is worth a try and is
cleaner anyway.
@tgross35 tgross35 added this pull request to the merge queue Jul 7, 2025
Merged via the queue into rust-lang:main with commit 8a39dcf Jul 7, 2025
48 of 52 checks passed
@tgross35 tgross35 added the stable-nominated This PR should be considered for cherry-pick to libc's stable release branch label Jul 8, 2025
@tgross35 tgross35 deleted the ci-cache branch July 8, 2025 04:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-CI Area: CI-related items stable-nominated This PR should be considered for cherry-pick to libc's stable release branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants