Add rerun-if-changed directives to all build scripts

Author: Mark-Simulacrum

collector/benchmarks/ripgrep/build.rs

@@ -18,6 +18,7 @@ use app::{RGArg, RGArgKind};
 mod app;
 
 fn main() {
+    println!("cargo:rerun-if-changed=build.rs");
     // OUT_DIR is set by Cargo and it's where any additional build artifacts
     // are written.
     let outdir = match env::var_os("OUT_DIR") {
@@ -26,7 +27,8 @@ fn main() {
             eprintln!(
                 "OUT_DIR environment variable not defined. \
                  Please file a bug: \
-                 https://github.com/BurntSushi/ripgrep/issues/new");
+                 https://github.com/BurntSushi/ripgrep/issues/new"
+            );
             process::exit(1);
         }
     };
@@ -95,8 +97,10 @@ fn generate_man_page<P: AsRef<Path>>(outdir: P) -> io::Result<()> {
     File::create(&txt_path)?.write_all(tpl.as_bytes())?;
     let result = process::Command::new("a2x")
         .arg("--no-xmllint")
-        .arg("--doctype").arg("manpage")
-        .arg("--format").arg("manpage")
+        .arg("--doctype")
+        .arg("manpage")
+        .arg("--format")
+        .arg("manpage")
         .arg(&txt_path)
         .spawn()?
         .wait()?;
@@ -119,7 +123,7 @@ fn formatted_options() -> io::Result<String> {
         // ripgrep only has two positional arguments, and probably will only
         // ever have two positional arguments, so we just hardcode them into
         // the template.
-        if let app::RGArgKind::Positional{..} = arg.kind {
+        if let app::RGArgKind::Positional { .. } = arg.kind {
             continue;
         }
         formatted.push(formatted_arg(&arg)?);
@@ -129,8 +133,12 @@ fn formatted_options() -> io::Result<String> {
 
 fn formatted_arg(arg: &RGArg) -> io::Result<String> {
     match arg.kind {
-        RGArgKind::Positional{..} => panic!("unexpected positional argument"),
-        RGArgKind::Switch { long, short, multiple } => {
+        RGArgKind::Positional { .. } => panic!("unexpected positional argument"),
+        RGArgKind::Switch {
+            long,
+            short,
+            multiple,
+        } => {
             let mut out = vec![];
 
             let mut header = format!("--{}", long);
@@ -147,7 +155,13 @@ fn formatted_arg(arg: &RGArg) -> io::Result<String> {
 
             Ok(String::from_utf8(out).unwrap())
         }
-        RGArgKind::Flag { long, short, value_name, multiple, .. } => {
+        RGArgKind::Flag {
+            long,
+            short,
+            value_name,
+            multiple,
+            ..
+        } => {
             let mut out = vec![];
 
             let mut header = format!("--{}", long);

collector/benchmarks/script-servo-2/components/atoms/build.rs

@@ -28,4 +28,5 @@ fn main() {
         .atoms(static_atoms.lines().map(Result::unwrap))
         .write_to_file(&Path::new(&env::var_os("OUT_DIR").unwrap()).join("atom.rs"))
         .unwrap();
+    println!("cargo:rerun-if-changed=build.rs");
 }

collector/benchmarks/script-servo-2/components/compositing/build.rs

@@ -8,6 +8,7 @@ use std::io::{Read, Write};
 use std::path::Path;
 
 fn main() {
+    println!("cargo:rerun-if-changed=build.rs");
     let lockfile_path = Path::new(&env::var("CARGO_MANIFEST_DIR").unwrap())
         .join("..")
         .join("..")

collector/benchmarks/script-servo-2/components/net/build.rs

@@ -3,6 +3,7 @@
  * file, You can obtain one at https://mozilla.org/MPL/2.0/. */
 
 fn main() {
+    println!("cargo:rerun-if-changed=build.rs");
     let version =
         std::env::var("DEP_OPENSSL_VERSION_NUMBER").expect("missing DEP_OPENSSL_VERSION_NUMBER");
     let actual = u64::from_str_radix(&version, 16).unwrap();

collector/benchmarks/script-servo-2/components/script/build.rs

@@ -44,6 +44,7 @@ fn main() {
         map.build(),
     )
     .unwrap();
+    println!("cargo:rerun-if-changed=build.rs");
 }
 
 #[derive(Eq, Hash, PartialEq)]

collector/benchmarks/script-servo-2/ports/gstplugin/build.rs

@@ -3,5 +3,6 @@
  * file, You can obtain one at https://mozilla.org/MPL/2.0/. */
 
 fn main() {
+    println!("cargo:rerun-if-changed=build.rs");
     gst_plugin_version_helper::get_info()
 }

collector/benchmarks/script-servo-2/ports/winit/build.rs

@@ -22,4 +22,5 @@ fn main() {
             .file("platform/macos/count_threads.c")
             .compile("count_threads");
     }
+    println!("cargo:rerun-if-changed=build.rs");
 }

collector/benchmarks/style-servo/components/atoms/build.rs

@@ -6,7 +6,7 @@ extern crate string_cache_codegen;
 
 use std::env;
 use std::fs::File;
-use std::io::{BufReader, BufRead};
+use std::io::{BufRead, BufReader};
 use std::path::Path;
 
 fn main() {
@@ -29,4 +29,5 @@ fn main() {
         .atoms(static_atoms.lines().map(Result::unwrap))
         .write_to_file(&Path::new(&env::var("OUT_DIR").unwrap()).join("atom.rs"))
         .unwrap();
+    println!("cargo:rerun-if-changed=build.rs");
 }

collector/benchmarks/style-servo/components/script/build.rs

@@ -16,6 +16,7 @@ use std::path::PathBuf;
 use std::time::Instant;
 
 fn main() {
+    println!("cargo:rerun-if-changed=build.rs");
     let start = Instant::now();
 
     // This must use the Ninja generator -- it's the only one that
@@ -31,30 +32,46 @@ fn main() {
         build.generator("Ninja");
         // because we're using ninja, we need to explicitly set these
         // to VC++, otherwise it'll try to use cc
-        build.define("CMAKE_C_COMPILER", "cl.exe")
-             .define("CMAKE_CXX_COMPILER", "cl.exe");
+        build
+            .define("CMAKE_C_COMPILER", "cl.exe")
+            .define("CMAKE_CXX_COMPILER", "cl.exe");
         // We have to explicitly specify the full path to link.exe,
         // for reasons that I don't understand.  If we just give
         // link.exe, it tries to use script-*/out/link.exe, which of
         // course does not exist.
-        let link = std::process::Command::new("where").arg("link.exe").output().unwrap();
-        let link_path: Vec<&str> = std::str::from_utf8(&link.stdout).unwrap().split("\r\n").collect();
+        let link = std::process::Command::new("where")
+            .arg("link.exe")
+            .output()
+            .unwrap();
+        let link_path: Vec<&str> = std::str::from_utf8(&link.stdout)
+            .unwrap()
+            .split("\r\n")
+            .collect();
         build.define("CMAKE_LINKER", link_path[0]);
     }
 
     build.build();
 
-    println!("Binding generation completed in {}s", start.elapsed().as_secs());
+    println!(
+        "Binding generation completed in {}s",
+        start.elapsed().as_secs()
+    );
 
-    let json = PathBuf::from(env::var("OUT_DIR").unwrap()).join("build").join("InterfaceObjectMapData.json");
+    let json = PathBuf::from(env::var("OUT_DIR").unwrap())
+        .join("build")
+        .join("InterfaceObjectMapData.json");
     let json: Value = serde_json::from_reader(File::open(&json).unwrap()).unwrap();
     let mut map = phf_codegen::Map::new();
     for (key, value) in json.as_object().unwrap() {
         map.entry(Bytes(key), value.as_str().unwrap());
     }
     let phf = PathBuf::from(env::var("OUT_DIR").unwrap()).join("InterfaceObjectMapPhf.rs");
     let mut phf = File::create(&phf).unwrap();
-    write!(&mut phf, "pub static MAP: phf::Map<&'static [u8], unsafe fn(*mut JSContext, HandleObject)> = ").unwrap();
+    write!(
+        &mut phf,
+        "pub static MAP: phf::Map<&'static [u8], unsafe fn(*mut JSContext, HandleObject)> = "
+    )
+    .unwrap();
     map.build(&mut phf).unwrap();
     write!(&mut phf, ";\n").unwrap();
 }