cp: add selinux cfg alias

Author: xtqqczze

Cargo.lock

@@ -18,6 +18,9 @@ workspace = true
 [lib]
 path = "src/cp.rs"
 
+[build-dependencies]
+cfg_aliases.workspace = true
+
 [dependencies]
 clap = { workspace = true }
 filetime = { workspace = true }

src/uu/cp/Cargo.toml

@@ -0,0 +1,12 @@
+// This file is part of the uutils coreutils package.
+//
+// For the full copyright and license information, please view the LICENSE
+// file that was distributed with this source code.
+
+use cfg_aliases::cfg_aliases;
+
+pub fn main() {
+    cfg_aliases! {
+        selinux: { all(feature = "selinux", any(target_os = "android", target_os = "linux")) },
+    }
+}

src/uu/cp/build.rs

@@ -26,7 +26,7 @@ use uucore::translate;
 use uucore::uio_error;
 use walkdir::{DirEntry, WalkDir};
 
-#[cfg(all(feature = "selinux", target_os = "linux"))]
+#[cfg(selinux)]
 use crate::set_selinux_context;
 use crate::{
     CopyMode, CopyResult, CpError, Options, aligned_ancestors, context_for, copy_attributes,
@@ -562,7 +562,7 @@ pub(crate) fn copy_directory(
             options.set_selinux_context,
         )?;
 
-        #[cfg(all(feature = "selinux", target_os = "linux"))]
+        #[cfg(selinux)]
         if options.set_selinux_context {
             set_selinux_context(&dir.dest, options.context.as_ref())?;
         }
@@ -582,7 +582,7 @@ pub(crate) fn copy_directory(
                     options.set_selinux_context,
                 )?;
 
-                #[cfg(all(feature = "selinux", target_os = "linux"))]
+                #[cfg(selinux)]
                 if options.set_selinux_context {
                     set_selinux_context(y, options.context.as_ref())?;
                 }

src/uu/cp/src/copydir.rs

@@ -896,11 +896,11 @@ impl Attributes {
         mode: Preserve::Yes { required: true },
         timestamps: Preserve::Yes { required: true },
         context: {
-            #[cfg(feature = "feat_selinux")]
+            #[cfg(selinux)]
             {
                 Preserve::Yes { required: false }
             }
-            #[cfg(not(feature = "feat_selinux"))]
+            #[cfg(not(selinux))]
             {
                 Preserve::No { explicit: false }
             }
@@ -1141,7 +1141,7 @@ impl Options {
             }
         }
 
-        #[cfg(not(feature = "selinux"))]
+        #[cfg(not(selinux))]
         if let Preserve::Yes { required } = attributes.context {
             let selinux_disabled_error = CpError::Error(translate!("cp-error-selinux-not-enabled"));
             if required {
@@ -1692,7 +1692,7 @@ fn handle_preserve<F: Fn() -> CopyResult<()>>(p: Preserve, f: F) -> CopyResult<(
     Ok(())
 }
 
-#[cfg(all(feature = "selinux", target_os = "linux"))]
+#[cfg(selinux)]
 pub(crate) fn set_selinux_context(path: &Path, context: Option<&String>) -> CopyResult<()> {
     if !uucore::selinux::is_selinux_enabled() {
         return Ok(());
@@ -1846,7 +1846,7 @@ pub(crate) fn copy_attributes(
         Ok(())
     })?;
 
-    #[cfg(all(feature = "selinux", any(target_os = "linux", target_os = "android")))]
+    #[cfg(selinux)]
     handle_preserve(attributes.context, || -> CopyResult<()> {
         // Get the source context and apply it to the destination
         if let Ok(context) = selinux::SecurityContext::of_path(source, false, false) {
@@ -2658,7 +2658,7 @@ fn copy_file(
         fs::File::create(dest).map(|f| f.set_len(0)).ok();
     })?;
 
-    #[cfg(all(feature = "selinux", target_os = "linux"))]
+    #[cfg(selinux)]
     if options.set_selinux_context {
         set_selinux_context(dest, options.context.as_ref())?;
     }