Skip to content

Commit 41cfa6b

Browse files
committed
Don't watch PYTHON_SYS_EXECUTABLE and PATH when unnecessary
1 parent 1749747 commit 41cfa6b

File tree

1 file changed

+12
-9
lines changed

1 file changed

+12
-9
lines changed

build.rs

+12-9
Original file line numberDiff line numberDiff line change
@@ -893,17 +893,20 @@ fn main() -> Result<()> {
893893
// TODO: Find out how we can set -undefined dynamic_lookup here (if this is possible)
894894
}
895895

896-
let env_vars = [
897-
"LD_LIBRARY_PATH",
898-
"PATH",
899-
"PYTHON_SYS_EXECUTABLE",
900-
"PYO3_PYTHON",
901-
"LIB",
902-
];
903-
904-
for var in env_vars.iter() {
896+
for var in ["LIB", "LD_LIBRARY_PATH", "PYO3_PYTHON"].iter() {
905897
println!("cargo:rerun-if-env-changed={}", var);
906898
}
907899

900+
if env::var_os("PYO3_PYTHON").is_none() {
901+
// When PYO3_PYTHON is not used, PYTHON_SYS_EXECUTABLE has the highest priority.
902+
// Let's watch it.
903+
println!("cargo:rerun-if-env-changed=PYTHON_SYS_EXECUTABLE");
904+
if env::var_os("PYTHON_SYS_EXECUTABLE").is_none() {
905+
// When PYTHON_SYS_EXECUTABLE is also not used, then we use PATH.
906+
// Let's watch this, too.
907+
println!("cargo:rerun-if-env-changed=PATH");
908+
}
909+
}
910+
908911
Ok(())
909912
}

0 commit comments

Comments
 (0)