cp: remove check for ownership perm for windows

uutils · May 10, 2023 · 2f4afc5 · 2f4afc5
1 parent a710805
commit 2f4afc5
Showing 1 changed file with 10 additions and 0 deletions.
diff --git a/src/uu/cp/src/cp.rs b/src/uu/cp/src/cp.rs
@@ -1759,9 +1759,19 @@ fn copy_helper(
     Ok(())
 }
 
+#[cfg(not(windows))]
 fn should_preserve_attribute(options: &Options) -> bool {
     matches!(options.attributes.mode, Preserve::Yes { .. })
+        || matches!(options.attributes.timestamps, Preserve::Yes { .. })
+        || matches!(options.attributes.links, Preserve::Yes { .. })
+        || matches!(options.attributes.context, Preserve::Yes { .. })
+        || matches!(options.attributes.xattr, Preserve::Yes { .. })
         || matches!(options.attributes.ownership, Preserve::Yes { .. })
+}
+
+#[cfg(windows)]
+fn should_preserve_attribute(options: &Options) -> bool {
+    matches!(options.attributes.mode, Preserve::Yes { .. })
         || matches!(options.attributes.timestamps, Preserve::Yes { .. })
         || matches!(options.attributes.links, Preserve::Yes { .. })
         || matches!(options.attributes.context, Preserve::Yes { .. })