Skip to content

Commit

Permalink
Rollup merge of #85762 - mati865:disable-zlib-on-windows, r=Mark-Simu…
Browse files Browse the repository at this point in the history
…lacrum

Do not try to build LLVM with Zlib on Windows

Fixes #85422
Fixes #85624

We do not install Zlib on the CI but recent builds somehow started picking it's shared version.
To avoid relying on CI binaries so let's explicitly disable it.
  • Loading branch information
GuillaumeGomez authored May 30, 2021
2 parents bdd7062 + 53bf79e commit 957badb
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/bootstrap/native.rs
Original file line number Diff line number Diff line change
Expand Up @@ -181,7 +181,7 @@ impl Step for Llvm {
.define("LLVM_TARGET_ARCH", target_native.split('-').next().unwrap())
.define("LLVM_DEFAULT_TARGET_TRIPLE", target_native);

if target != "aarch64-apple-darwin" {
if target != "aarch64-apple-darwin" && !target.contains("windows") {
cfg.define("LLVM_ENABLE_ZLIB", "ON");
} else {
cfg.define("LLVM_ENABLE_ZLIB", "OFF");
Expand Down

0 comments on commit 957badb

Please sign in to comment.