user0/rust
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Simplify default value of download-ci-llvm

Browse source
This commit is contained in:
Jakub Beránek 2025-09-19 16:00:05 +02:00
parent 2f4dfc753f
commit 4b755141bc
No known key found for this signature in database
GPG key ID: 909CD0D26483516B
1 changed files with 4 additions and 11 deletions
Download patch file Download diff file Expand all files Collapse all files

15
src/bootstrap/src/core/config/config.rs
View file

@ -2134,15 +2134,7 @@ pub fn parse_download_ci_llvm<'a>(
asserts: bool,
) -> bool {
let dwn_ctx = dwn_ctx.as_ref();
// We don't ever want to use `true` on CI, as we should not
// download upstream artifacts if there are any local modifications.
let default = if dwn_ctx.is_running_on_ci {
StringOrBool::String("if-unchanged".to_string())
} else {
StringOrBool::Bool(true)
};
let download_ci_llvm = download_ci_llvm.unwrap_or(default);
let download_ci_llvm = download_ci_llvm.unwrap_or(StringOrBool::Bool(true));
let if_unchanged = || {
if rust_info.is_from_tarball() {
@ -2174,8 +2166,9 @@ pub fn parse_download_ci_llvm<'a>(
);
}
if b && dwn_ctx.is_running_on_ci {
// On CI, we must always rebuild LLVM if there were any modifications to it
#[cfg(not(test))]
if b && dwn_ctx.is_running_on_ci && CiEnv::is_rust_lang_managed_ci_job() {
// On rust-lang CI, we must always rebuild LLVM if there were any modifications to it
panic!(
"`llvm.download-ci-llvm` cannot be set to `true` on CI. Use `if-unchanged` instead."
);