From d4f47340a3553417d3aefd14735a9c509acb0c9c Mon Sep 17 00:00:00 2001 From: Elnard Utiushev Date: Fri, 15 Sep 2023 19:53:33 -0400 Subject: [PATCH] libafl_libfuzzer: replace `cargo:error` with assert in build.rs (#1517) It is very confusing if you are trying to build on MacOS and it is telling you that it worked, but actually it didn't. --- libafl_libfuzzer/build.rs | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/libafl_libfuzzer/build.rs b/libafl_libfuzzer/build.rs index f5797523f0..f4c701c4fa 100644 --- a/libafl_libfuzzer/build.rs +++ b/libafl_libfuzzer/build.rs @@ -4,12 +4,11 @@ fn main() { if cfg!(any(feature = "cargo-clippy", docsrs)) { return; // skip when clippy or docs is running } - if cfg!(not(target_os = "linux")) { - println!( - "cargo:error=The libafl_libfuzzer runtime may only be built for linux; failing fast." - ); - return; - } + assert!( + cfg!(target_os = "linux"), + "The libafl_libfuzzer runtime may only be built for linux; failing fast." + ); + println!("cargo:rerun-if-changed=libafl_libfuzzer_runtime/src"); println!("cargo:rerun-if-changed=libafl_libfuzzer_runtime/Cargo.toml"); println!("cargo:rerun-if-changed=libafl_libfuzzer_runtime/build.rs");