Skip to content

Cache all crates for citool #142395

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

Closed
wants to merge 1 commit into from
Closed

Conversation

Kobzol
Copy link
Contributor

@Kobzol Kobzol commented Jun 12, 2025

Even the local workspace ones, since they almost never change.

Also remove the manual CARGO_INCREMENTAL=0 setting, it is set automatically by rust-cache.

r? @marcoieni

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jun 12, 2025
@Kobzol Kobzol force-pushed the citool-more-cache branch 5 times, most recently from f6b1558 to d49e8ad Compare June 12, 2025 05:11
@rustbot rustbot added A-testsuite Area: The testsuite used to check the correctness of rustc T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue. labels Jun 12, 2025
@Kobzol Kobzol force-pushed the citool-more-cache branch from d49e8ad to b5f6cc9 Compare June 12, 2025 05:16
Even the local workspace ones, since they almost never change.
@Kobzol Kobzol force-pushed the citool-more-cache branch from b5f6cc9 to c279dda Compare June 12, 2025 05:18
@Kobzol
Copy link
Contributor Author

Kobzol commented Jun 12, 2025

Okay, no, it seems like rustc-cache fundamentally cannot cache local crates. Also, it looks like without tests, the caching itself is pretty much useless and it takes ~the same time as building things from scratch 🤦‍♂️ Well, let's keep things as they are then.

@Kobzol Kobzol closed this Jun 12, 2025
@Kobzol Kobzol deleted the citool-more-cache branch June 12, 2025 05:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-testsuite Area: The testsuite used to check the correctness of rustc S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants