From 1180905b5e8746cc480638be4929aa524fcf725b Mon Sep 17 00:00:00 2001
From: Sylvestre Ledru <sylvestre@debian.org>
Date: Thu, 26 Dec 2024 16:43:53 +0100
Subject: [PATCH] cp: use the function from uucore

---
 src/uu/cp/Cargo.toml |  1 +
 src/uu/cp/src/cp.rs  | 13 +++++--------
 2 files changed, 6 insertions(+), 8 deletions(-)

diff --git a/src/uu/cp/Cargo.toml b/src/uu/cp/Cargo.toml
index 6801e6a09..3912f3308 100644
--- a/src/uu/cp/Cargo.toml
+++ b/src/uu/cp/Cargo.toml
@@ -30,6 +30,7 @@ uucore = { workspace = true, features = [
   "backup-control",
   "entries",
   "fs",
+  "fsxattr",
   "perms",
   "mode",
   "update-control",
diff --git a/src/uu/cp/src/cp.rs b/src/uu/cp/src/cp.rs
index 32168b090..2f1f1ef1e 100644
--- a/src/uu/cp/src/cp.rs
+++ b/src/uu/cp/src/cp.rs
@@ -17,6 +17,8 @@ use std::os::unix::ffi::OsStrExt;
 #[cfg(unix)]
 use std::os::unix::fs::{FileTypeExt, PermissionsExt};
 use std::path::{Path, PathBuf, StripPrefixError};
+#[cfg(all(unix, not(any(target_os = "android", target_os = "macos"))))]
+use uucore::fsxattr::copy_xattrs;
 
 use clap::{builder::ValueParser, crate_version, Arg, ArgAction, ArgMatches, Command};
 use filetime::FileTime;
@@ -1603,16 +1605,11 @@ pub(crate) fn copy_attributes(
     })?;
 
     handle_preserve(&attributes.xattr, || -> CopyResult<()> {
-        #[cfg(all(unix, not(target_os = "android")))]
+        #[cfg(all(unix, not(any(target_os = "android", target_os = "macos"))))]
         {
-            let xattrs = xattr::list(source)?;
-            for attr in xattrs {
-                if let Some(attr_value) = xattr::get(source, attr.clone())? {
-                    xattr::set(dest, attr, &attr_value[..])?;
-                }
-            }
+            copy_xattrs(source, dest)?;
         }
-        #[cfg(not(all(unix, not(target_os = "android"))))]
+        #[cfg(not(all(unix, not(any(target_os = "android", target_os = "macos")))))]
         {
             // The documentation for GNU cp states:
             //