Skip to content

Commit eceb834

Browse files
committed
change download-ci-llvm default from if-unchanged to true
Signed-off-by: onur-ozkan <[email protected]>
1 parent 35195f5 commit eceb834

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

src/bootstrap/src/core/config/config.rs

+5-4
Original file line numberDiff line numberDiff line change
@@ -2738,6 +2738,8 @@ impl Config {
27382738
download_ci_llvm: Option<StringOrBool>,
27392739
asserts: bool,
27402740
) -> bool {
2741+
let download_ci_llvm = download_ci_llvm.unwrap_or(StringOrBool::Bool(true));
2742+
27412743
let if_unchanged = || {
27422744
if self.rust_info.is_from_tarball() {
27432745
// Git is needed for running "if-unchanged" logic.
@@ -2761,8 +2763,7 @@ impl Config {
27612763
};
27622764

27632765
match download_ci_llvm {
2764-
None => self.channel == "dev" || self.download_rustc_commit.is_some(),
2765-
Some(StringOrBool::Bool(b)) => {
2766+
StringOrBool::Bool(b) => {
27662767
if !b && self.download_rustc_commit.is_some() {
27672768
panic!(
27682769
"`llvm.download-ci-llvm` cannot be set to `false` if `rust.download-rustc` is set to `true` or `if-unchanged`."
@@ -2772,8 +2773,8 @@ impl Config {
27722773
// If download-ci-llvm=true we also want to check that CI llvm is available
27732774
b && llvm::is_ci_llvm_available(self, asserts)
27742775
}
2775-
Some(StringOrBool::String(s)) if s == "if-unchanged" => if_unchanged(),
2776-
Some(StringOrBool::String(other)) => {
2776+
StringOrBool::String(s) if s == "if-unchanged" => if_unchanged(),
2777+
StringOrBool::String(other) => {
27772778
panic!("unrecognized option for download-ci-llvm: {:?}", other)
27782779
}
27792780
}

0 commit comments

Comments
 (0)