From 6c6c8df86b7806bd2f7b938f70ddab4227cb9b13 Mon Sep 17 00:00:00 2001 From: Ed Page Date: Fri, 6 Dec 2024 09:27:38 -0600 Subject: [PATCH] fix(build-rs)!: Remove meaningless 'cargo_cfg_debug_assertions' The documentation that was added was pulled straight from a comment in `custom_build.rs`. --- Cargo.toml | 2 +- crates/build-rs-test-lib/build.rs | 1 - crates/build-rs/Cargo.toml | 2 +- crates/build-rs/src/input.rs | 4 ++++ 4 files changed, 6 insertions(+), 3 deletions(-) diff --git a/Cargo.toml b/Cargo.toml index 915fb30cef0..8e676d2984d 100644 --- a/Cargo.toml +++ b/Cargo.toml @@ -24,7 +24,7 @@ anstyle = "1.0.8" anyhow = "1.0.86" base64 = "0.22.1" blake3 = "1.5.2" -build-rs = { version = "0.2.0", path = "crates/build-rs" } +build-rs = { version = "0.3.0", path = "crates/build-rs" } bytesize = "1.3" cargo = { path = "" } cargo-credential = { version = "0.4.2", path = "credential/cargo-credential" } diff --git a/crates/build-rs-test-lib/build.rs b/crates/build-rs-test-lib/build.rs index 1167a34907e..eff04806a73 100644 --- a/crates/build-rs-test-lib/build.rs +++ b/crates/build-rs-test-lib/build.rs @@ -10,7 +10,6 @@ fn smoke_test_inputs() { use build_rs::input::*; dbg!(cargo()); dbg!(cargo_cfg("careful")); - dbg!(cargo_cfg_debug_assertions()); #[cfg(feature = "unstable")] dbg!(cargo_cfg_fmt_debug()); #[cfg(feature = "unstable")] diff --git a/crates/build-rs/Cargo.toml b/crates/build-rs/Cargo.toml index e2e8eb9fccf..24e42b09863 100644 --- a/crates/build-rs/Cargo.toml +++ b/crates/build-rs/Cargo.toml @@ -1,6 +1,6 @@ [package] name = "build-rs" -version = "0.2.0" +version = "0.3.0" rust-version.workspace = true edition.workspace = true license.workspace = true diff --git a/crates/build-rs/src/input.rs b/crates/build-rs/src/input.rs index 6f564f8c68b..8ab148bd2a5 100644 --- a/crates/build-rs/src/input.rs +++ b/crates/build-rs/src/input.rs @@ -109,6 +109,10 @@ mod cfg { } /// If we are compiling with debug assertions enabled. + /// + /// This cfg is always true and misleading, so avoid setting it. + /// That is because Cargo queries rustc without any profile settings. + #[cfg(any())] #[track_caller] pub fn cargo_cfg_debug_assertions() -> bool { is_present("CARGO_CFG_DEBUG_ASSERTIONS")