From 26cdab42d6197121588af263d23c37a7f7ac8f69 Mon Sep 17 00:00:00 2001 From: jaoleal Date: Mon, 2 Mar 2026 20:33:25 -0300 Subject: [PATCH] fix(build): improve error messages when cmake is not found Replace bare .unwrap() calls on cmake Command invocations with expects with expressive messages. --- libbitcoinkernel-sys/build.rs | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/libbitcoinkernel-sys/build.rs b/libbitcoinkernel-sys/build.rs index 7a120fe0..5bdd6b40 100644 --- a/libbitcoinkernel-sys/build.rs +++ b/libbitcoinkernel-sys/build.rs @@ -42,7 +42,7 @@ fn main() { .arg("-DENABLE_IPC=OFF") .arg(format!("-DCMAKE_INSTALL_PREFIX={}", install_dir.display())) .status() - .unwrap(); + .expect("cmake should be installed and available in PATH."); let num_jobs = env::var("NUM_JOBS") .ok() @@ -56,7 +56,7 @@ fn main() { .arg(build_config) .arg(format!("--parallel={num_jobs}")) .status() - .unwrap(); + .expect("cmake build should succeed after configure"); Command::new("cmake") .arg("--install") @@ -64,7 +64,7 @@ fn main() { .arg("--config") .arg(build_config) .status() - .unwrap(); + .expect("cmake install should succeed after build"); // Check if the build system used a multi-config generator let lib_dir = if install_dir.join("lib").join(build_config).exists() {