deps: gate selinux-related dependency set behind feat_selinux

build.rs

@@ -37,15 +37,19 @@ pub fn main() {
 
     let out_dir = env::var("OUT_DIR").unwrap();
 
+    let target_os = env::var_os("CARGO_CFG_TARGET_OS").unwrap();
+    let is_feat_selinux = env::var_os("CARGO_FEATURE_FEAT_SELINUX").is_some();
+
     let mut crates = Vec::new();
     for (key, val) in env::vars() {
         if val == "1" && key.starts_with(ENV_FEATURE_PREFIX) {
             let krate = key[ENV_FEATURE_PREFIX.len()..].to_lowercase();
             // Allow this as we have a bunch of info in the comments
             #[allow(clippy::match_same_arms)]
             match krate.as_ref() {
-                #[cfg(not(any(target_os = "linux", target_os = "android")))]
-                "chcon" | "runcon" => {
+                "chcon" | "runcon"
+                    if !(is_feat_selinux && (target_os == "linux" || target_os == "android")) =>
+                {
                     continue;
                 }
                 "default" | "macos" | "unix" | "windows" | "selinux" | "zip" | "clap_complete"

src/uu/chcon/Cargo.toml

@@ -19,15 +19,30 @@ path = "src/chcon.rs"
 test = false
 doctest = false
 
-# TODO: block fetching crates without feat_selinux
 [target.'cfg(any(target_os = "linux", target_os = "android"))'.dependencies]
-clap = { workspace = true }
-uucore = { workspace = true, features = ["entries", "fs", "perms"] }
-selinux = { workspace = true }
-thiserror = { workspace = true }
-libc = { workspace = true }
-fts-sys = { workspace = true }
-fluent = { workspace = true }
+clap = { workspace = true, optional = true }
+fluent = { workspace = true, optional = true }
+fts-sys = { workspace = true, optional = true }
+libc = { workspace = true, optional = true }
+selinux = { workspace = true, optional = true }
+thiserror = { workspace = true, optional = true }
+uucore = { workspace = true, features = [
+  "entries",
+  "fs",
+  "perms",
+], optional = true }
+
+[features]
+feat_selinux = ["selinux"]
+selinux = [
+  "dep:clap",
+  "dep:fluent",
+  "dep:fts-sys",
+  "dep:libc",
+  "dep:selinux",
+  "dep:thiserror",
+  "dep:uucore",
+]
 
 [[bin]]
 name = "chcon"

src/uu/chcon/src/chcon.rs

@@ -5,7 +5,7 @@
 
 // spell-checker:ignore (vars) RFILE
 
-#![cfg(any(target_os = "linux", target_os = "android"))]
+#![cfg(all(feature = "selinux", any(target_os = "linux", target_os = "android")))]
 #![allow(clippy::upper_case_acronyms)]
 
 use clap::builder::ValueParser;

src/uu/chcon/src/errors.rs

@@ -3,8 +3,6 @@
 // For the full copyright and license information, please view the LICENSE
 // file that was distributed with this source code.
 
-#![cfg(any(target_os = "linux", target_os = "android"))]
-
 use std::ffi::OsString;
 use std::fmt::Write;
 use std::io;

src/uu/chcon/src/fts.rs

@@ -3,8 +3,6 @@
 // For the full copyright and license information, please view the LICENSE
 // file that was distributed with this source code.
 
-#![cfg(any(target_os = "linux", target_os = "android"))]
-
 use std::ffi::{CStr, CString, OsStr};
 use std::marker::PhantomData;
 use std::os::raw::{c_int, c_long, c_short};

src/uu/chcon/src/main.rs

@@ -9,10 +9,10 @@
 //! entirely (via #![cfg(...)]), which can break tooling and cross builds that
 //! expect this binary to exist even when it's a no-op off Linux.
 
-#[cfg(any(target_os = "linux", target_os = "android"))]
+#[cfg(all(feature = "selinux", any(target_os = "linux", target_os = "android")))]
 uucore::bin!(uu_chcon);
 
-#[cfg(not(any(target_os = "linux", target_os = "android")))]
+#[cfg(not(all(feature = "selinux", any(target_os = "linux", target_os = "android"))))]
 fn main() {
     eprintln!("chcon: SELinux is not supported on this platform");
     std::process::exit(1);

src/uu/runcon/Cargo.toml

@@ -19,14 +19,29 @@ path = "src/runcon.rs"
 test = false
 doctest = false
 
-# TODO: block fetching crates without feat_selinux
 [target.'cfg(any(target_os = "linux", target_os = "android"))'.dependencies]
-clap = { workspace = true }
-uucore = { workspace = true, features = ["entries", "fs", "perms", "selinux"] }
-selinux = { workspace = true }
-thiserror = { workspace = true }
-libc = { workspace = true }
-fluent = { workspace = true }
+clap = { workspace = true, optional = true }
+fluent = { workspace = true, optional = true }
+libc = { workspace = true, optional = true }
+selinux = { workspace = true, optional = true }
+thiserror = { workspace = true, optional = true }
+uucore = { workspace = true, features = [
+  "entries",
+  "fs",
+  "perms",
+  "selinux",
+], optional = true }
+
+[features]
+feat_selinux = ["selinux"]
+selinux = [
+  "dep:clap",
+  "dep:fluent",
+  "dep:libc",
+  "dep:selinux",
+  "dep:thiserror",
+  "dep:uucore",
+]
 
 [[bin]]
 name = "runcon"

src/uu/runcon/src/errors.rs

@@ -3,8 +3,6 @@
 // For the full copyright and license information, please view the LICENSE
 // file that was distributed with this source code.
 
-#![cfg(any(target_os = "linux", target_os = "android"))]
-
 use std::ffi::OsString;
 use std::fmt::{Display, Formatter, Write};
 use std::io;

src/uu/runcon/src/main.rs

@@ -9,10 +9,10 @@
 //! entirely (via #![cfg(...)]), which can break tooling and cross builds that
 //! expect this binary to exist even when it's a no-op off Linux.
 
-#[cfg(any(target_os = "linux", target_os = "android"))]
+#[cfg(all(feature = "selinux", any(target_os = "linux", target_os = "android")))]
 uucore::bin!(uu_runcon);
 
-#[cfg(not(any(target_os = "linux", target_os = "android")))]
+#[cfg(not(all(feature = "selinux", any(target_os = "linux", target_os = "android"))))]
 fn main() {
     eprintln!("runcon: SELinux is not supported on this platform");
     std::process::exit(1);

src/uu/runcon/src/runcon.rs

@@ -5,7 +5,7 @@
 
 // spell-checker:ignore (vars) RFILE execv execvp
 
-#![cfg(any(target_os = "linux", target_os = "android"))]
+#![cfg(all(feature = "selinux", any(target_os = "linux", target_os = "android")))]
 
 use clap::builder::ValueParser;
 use uucore::error::{UError, UResult};