fix: remove build-data dependency

Simplifies getting the information we actually need, without using another dependency.

Ref #1035

deny.toml

@@ -24,7 +24,6 @@ license-files = [
 [advisories]
 ignore = [
        "RUSTSEC-2020-0168", # mach is unmaintained
-       "RUSTSEC-2020-0071", # time advisory, see #1035
 ]
 
 [sources]

iroh-net/Cargo.toml

@@ -84,7 +84,7 @@ tokio = { version = "1", features = ["io-util", "sync", "rt", "net", "fs", "macr
 tracing-subscriber = { version = "0.3", features = ["env-filter"] }
 
 [build-dependencies]
-build-data = "0.1.3"
+duct = "0.13.6"
 
 [features]
 default = ["metrics"]

iroh-net/build.rs

@@ -1,6 +1,25 @@
+use std::io;
+
+use duct::cmd;
+
 fn main() {
-    build_data::set_GIT_COMMIT();
-    build_data::set_RUSTC_VERSION();
-    build_data::set_SOURCE_TIMESTAMP();
-    build_data::no_debug_rebuilds();
+    // Git commit
+    println!("cargo:rustc-env=GIT_COMMIT={}", get_git_commit().unwrap());
+
+    // Rustc version
+    println!(
+        "cargo:rustc-env=RUSTC_VERSION={}",
+        get_rustc_version().unwrap()
+    );
+}
+
+fn get_git_commit() -> std::io::Result<String> {
+    cmd!("git", "rev-parse", "HEAD").read()
+}
+
+fn get_rustc_version() -> io::Result<String> {
+    let rustc_var = std::env::var_os("RUSTC")
+        .filter(|s| !s.is_empty())
+        .ok_or_else(|| io::Error::new(io::ErrorKind::Other, "RUSTC env var is not set"))?;
+    cmd!(rustc_var, "--version").read()
 }

iroh-net/src/hp/hostinfo.rs

@@ -83,6 +83,7 @@ mod tests {
         let info = Hostinfo::default();
         println!("{:#?}", info);
 
+        assert!(!info.git_commit.is_empty());
         assert!(!info.rust_version.is_empty());
     }
 }