chore: use build-rs in build scripts (#1716)

Author: Mirko-von-Leipzig

Cargo.lock

@@ -77,6 +77,7 @@ miden-crypto = { default-features = false, version = "0.19" }
 anyhow            = { version = "1.0" }
 assert_matches    = { version = "1.5" }
 async-trait       = { version = "0.1" }
+build-rs          = { version = "0.3" }
 clap              = { features = ["derive"], version = "4.5" }
 deadpool          = { default-features = false, version = "0.12" }
 deadpool-diesel   = { version = "0.6" }

Cargo.toml

@@ -47,6 +47,7 @@ miden-testing   = { workspace = true }
 miden-tx        = { features = ["testing"], workspace = true }
 
 [build-dependencies]
+build-rs                           = { workspace = true }
 fs-err                             = { workspace = true }
 miden-node-proto-build             = { features = ["internal"], workspace = true }
 miden-node-rocksdb-cxx-linkage-fix = { workspace = true }

bin/remote-prover/Cargo.toml

@@ -1,4 +1,4 @@
-use std::path::{Path, PathBuf};
+use std::path::Path;
 
 use fs_err as fs;
 use miden_node_proto_build::remote_prover_api_descriptor;
@@ -9,8 +9,7 @@ use tonic_prost_build::FileDescriptorSet;
 fn main() -> miette::Result<()> {
     miden_node_rocksdb_cxx_linkage_fix::configure();
 
-    let dst_dir =
-        PathBuf::from(std::env::var("OUT_DIR").expect("OUT_DIR should be set")).join("generated");
+    let dst_dir = build_rs::input::out_dir().join("generated");
 
     // Remove all existing files.
     let _ = fs::remove_dir_all(&dst_dir);

bin/remote-prover/build.rs

@@ -34,6 +34,9 @@ tonic                      = { workspace = true }
 tracing                    = { workspace = true }
 url                        = { workspace = true }
 
+[build-dependencies]
+build-rs = { workspace = true }
+
 [dev-dependencies]
 miden-node-test-macro = { path = "../test-macro" }
 miden-node-utils      = { features = ["testing"], workspace = true }

crates/ntx-builder/Cargo.toml

@@ -3,9 +3,9 @@
 // <https://docs.rs/diesel_migrations/latest/diesel_migrations/macro.embed_migrations.html#automatic-rebuilds>.
 
 fn main() {
-    println!("cargo:rerun-if-changed=./src/db/migrations");
+    build_rs::output::rerun_if_changed("src/db/migrations");
     // If we do one re-write, the default rules are disabled,
     // hence we need to trigger explicitly on `Cargo.toml`.
     // <https://doc.rust-lang.org/cargo/reference/build-scripts.html#rerun-if-changed>
-    println!("cargo:rerun-if-changed=Cargo.toml");
+    build_rs::output::rerun_if_changed("Cargo.toml");
 }

crates/ntx-builder/build.rs

@@ -33,6 +33,7 @@ assert_matches = { workspace = true }
 proptest       = { version = "1.7" }
 
 [build-dependencies]
+build-rs                           = { workspace = true }
 fs-err                             = { workspace = true }
 miden-node-proto-build             = { features = ["internal"], workspace = true }
 miden-node-rocksdb-cxx-linkage-fix = { workspace = true }

crates/proto/Cargo.toml

@@ -1,5 +1,4 @@
-use std::env;
-use std::path::{Path, PathBuf};
+use std::path::Path;
 
 use fs_err as fs;
 use miden_node_proto_build::{
@@ -16,12 +15,9 @@ use tonic_prost_build::FileDescriptorSet;
 
 /// Generates Rust protobuf bindings using `miden-node-proto-build`.
 fn main() -> miette::Result<()> {
-    println!("cargo::rerun-if-changed=../../proto/proto");
-
     miden_node_rocksdb_cxx_linkage_fix::configure();
 
-    let dst_dir =
-        PathBuf::from(env::var("OUT_DIR").expect("OUT_DIR should be set")).join("generated");
+    let dst_dir = build_rs::input::out_dir().join("generated");
 
     // Remove all existing files.
     let _ = fs::remove_dir_all(&dst_dir);

crates/proto/build.rs

@@ -41,6 +41,7 @@ tokio          = { default-features = false, features = ["sync"], optional = tru
 tonic-prost    = { workspace = true }
 
 [build-dependencies]
+build-rs               = { workspace = true }
 fs-err                 = { workspace = true }
 miden-node-proto-build = { workspace = true }
 miette                 = { features = ["fancy"], version = "7.5" }

crates/remote-prover-client/Cargo.toml

@@ -1,5 +1,5 @@
 use std::io::Write;
-use std::path::{Path, PathBuf};
+use std::path::Path;
 
 use fs_err as fs;
 use miden_node_proto_build::remote_prover_api_descriptor;
@@ -8,9 +8,7 @@ use tonic_prost_build::FileDescriptorSet;
 
 /// Generates Rust protobuf bindings.
 fn main() -> miette::Result<()> {
-    let dst_dir =
-        PathBuf::from(std::env::var("OUT_DIR").expect("OUT_DIR is always set for build.rs"))
-            .join("generated");
+    let dst_dir = build_rs::input::out_dir().join("generated");
 
     // Remove all existing files.
     let _ = fs::remove_dir_all(&dst_dir);