From 9321a34dbe4c243b2438e59a43977944ae9d2d97 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Bj=C3=B6rn=20Quentin?= Date: Thu, 26 Sep 2024 10:15:41 +0200 Subject: [PATCH] Reduce (unnecessary) rebuilds (#2232) * Reduce (unnecessary) rebuilds * Avoid redundant clone * Prefer `.display()` over `.to_str().unwrap()` --- esp-config/src/generate.rs | 20 ++++++++++++-------- 1 file changed, 12 insertions(+), 8 deletions(-) diff --git a/esp-config/src/generate.rs b/esp-config/src/generate.rs index 1df1a642b..1afa1c925 100644 --- a/esp-config/src/generate.rs +++ b/esp-config/src/generate.rs @@ -126,14 +126,18 @@ fn env_change_work_around() { let dotcargo = out_dir.join(".cargo/"); if dotcargo.exists() { - println!( - "cargo:rerun-if-changed={}", - dotcargo.clone().join("config.toml").to_str().unwrap() - ); - println!( - "cargo:rerun-if-changed={}", - dotcargo.clone().join("config").to_str().unwrap() - ); + if dotcargo.join("config.toml").exists() { + println!( + "cargo:rerun-if-changed={}", + dotcargo.join("config.toml").display() + ); + } + if dotcargo.join("config").exists() { + println!( + "cargo:rerun-if-changed={}", + dotcargo.join("config").display() + ); + } } }