Only set PYTHONDONTWRITEBYTECODE env variable for subprocess

Signed-off-by: Nico Burns <[email protected]>

Author: nicoburns

style/build.rs

@@ -53,18 +53,17 @@ fn generate_properties(engine: &str) {
         }
     }
 
-    // `cargo publish` isn't happy with the `__pycache__` files that are created
-    // when we run the property generator.
-    //
-    // TODO(mrobinson): Is this happening because of how we run this script? It
-    // would be better to ensure are just placed in the output directory.
-    env::set_var("PYTHONDONTWRITEBYTECODE", "1");
-
     let script = Path::new(&env::var_os("CARGO_MANIFEST_DIR").unwrap())
         .join("properties")
         .join("build.py");
 
     let status = Command::new(&*PYTHON)
+        // `cargo publish` isn't happy with the `__pycache__` files that are created
+        // when we run the property generator.
+        //
+        // TODO(mrobinson): Is this happening because of how we run this script? It
+        // would be better to ensure are just placed in the output directory.
+        .env("PYTHONDONTWRITEBYTECODE", "1")
         .arg(&script)
         .arg(engine)
         .arg("style-crate")

style/build_gecko.rs

@@ -369,6 +369,7 @@ fn generate_atoms() {
         .join("regen_atoms.py");
     println!("cargo:rerun-if-changed={}", script.display());
     let status = Command::new(&*PYTHON)
+        .env("PYTHONDONTWRITEBYTECODE", "1")
         .arg(&script)
         .arg(DISTDIR_PATH.as_os_str())
         .arg(OUTDIR_PATH.as_os_str())