From 3a196c3a08d84bc2977b5cace0f080afb11a3f2f Mon Sep 17 00:00:00 2001 From: Daniel Rainer Date: Mon, 15 Sep 2025 01:26:48 +0200 Subject: [PATCH] Use `rsconf::warn!` where appropriate This is a more explicit variant than directly prefixing build script output with `cargo:warning=`. Part of #11726 --- crates/build-man-pages/build.rs | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/crates/build-man-pages/build.rs b/crates/build-man-pages/build.rs index 86c2cefdf..d534706e0 100644 --- a/crates/build-man-pages/build.rs +++ b/crates/build-man-pages/build.rs @@ -48,7 +48,7 @@ fn build_man(man_dir: &Path) { rsconf::rebuild_if_env_changed("FISH_BUILD_DOCS"); if env::var("FISH_BUILD_DOCS") == Ok("0".to_string()) { - println!("cargo:warning=Skipping man pages because $FISH_BUILD_DOCS is set to 0"); + rsconf::warn!("Skipping man pages because $FISH_BUILD_DOCS is set to 0"); return; } @@ -62,8 +62,8 @@ fn build_man(man_dir: &Path) { if env::var("FISH_BUILD_DOCS") == Ok("1".to_string()) { panic!("Could not find sphinx-build to build man pages.\nInstall sphinx or disable building the docs by setting $FISH_BUILD_DOCS=0."); } - println!("cargo:warning=Cannot find sphinx-build to build man pages."); - println!("cargo:warning=If you install it now you need to run `cargo clean` and rebuild, or set $FISH_BUILD_DOCS=1 explicitly."); + rsconf::warn!("Cannot find sphinx-build to build man pages."); + rsconf::warn!("If you install it now you need to run `cargo clean` and rebuild, or set $FISH_BUILD_DOCS=1 explicitly."); } Err(x) => { // Another error - permissions wrong etc