Skip to content

ci: Cache LLVM downloads separately - #42

Merged
tgross35 merged 1 commit into
rust-lang:mainfrom
tgross35:ci-cache
Sep 20, 2026
Merged

tgross35 merged 1 commit into
rust-lang:mainfrom
tgross35:ci-cache

Conversation

@tgross35

Copy link
Copy Markdown
Member

rust-cache made some changes (presumably 1) that delete most non-Rust files before caching, including the LLVM source, but leave the directories. This causes download-llvm.sh to think the source exists and skip a re-download, but the build fails because the needed files are missing. It's not clear whether the download was being cached previously, or if rust-cache deleted the directories so it was always re-downloaded.

Move the directory in CI and set up a separate cache step so rust-cache isn't responsible for these files anymore.

@tgross35
tgross35 force-pushed the ci-cache branch 4 times, most recently from 1a67072 to a9ced76 Compare September 20, 2026 08:59
rust-cache made some changes (presumably [1]) that delete most non-Rust
files before caching, including the LLVM source, but leave the
directories. This causes `download-llvm.sh` to think the source exists
and skip a re-download, but the build fails because the needed files are
missing. It's not clear whether the download was being cached
previously, or if rust-cache deleted the directories so it was always
re-downloaded.

Move the directory in CI and set up a separate cache step so rust-cache
isn't responsible for these files anymore.

[1]: Swatinem/rust-cache@3bf42ac
@tgross35
tgross35 marked this pull request as ready for review September 20, 2026 09:09
@tgross35
tgross35 merged commit a6fa80d into rust-lang:main Sep 20, 2026
4 checks passed
@tgross35
tgross35 deleted the ci-cache branch September 20, 2026 09:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant