From e2a8e86e988b287234c32c916a310e1059281384 Mon Sep 17 00:00:00 2001
From: Andrew Liebenow <andrewliebenow@gmail.com>
Date: Sun, 15 Sep 2024 09:21:52 -0500
Subject: [PATCH] cp: show mode if target does not have S_IWUSR

If the target exists, and does not have the user write bit (S_IWUSR)
set, additional information should be added to the overwrite
confirmation prompt.

This should get the "i-2" test to pass. See
https://github.com/uutils/coreutils/issues/6658.
---
 src/uu/cp/src/cp.rs | 51 ++++++++++++++++++++++++++++++++++++++++++++-
 1 file changed, 50 insertions(+), 1 deletion(-)

diff --git a/src/uu/cp/src/cp.rs b/src/uu/cp/src/cp.rs
index 06f49520f6..bb83283c58 100644
--- a/src/uu/cp/src/cp.rs
+++ b/src/uu/cp/src/cp.rs
@@ -1410,7 +1410,56 @@ impl OverwriteMode {
                 Err(Error::Skipped(false))
             }
             Self::Interactive(_) => {
-                if prompt_yes!("overwrite {}?", path.quote()) {
+                // TODO
+                // The destination metadata can be read multiple times in the course of a single execution of `cp`.
+                // This fix adds yet another metadata read.
+                // Should this metadata be read once and then reused throughout the execution?
+                // https://github.com/uutils/coreutils/issues/6658
+                let extra_prompt_information: Option<(String, String)> = {
+                    #[cfg(unix)]
+                    {
+                        use libc::{mode_t, S_IWUSR};
+                        use std::os::unix::prelude::MetadataExt;
+
+                        match path.metadata() {
+                            Ok(me) => {
+                                let mode: mode_t = me.mode();
+
+                                // It looks like this extra information is added to the prompt iff the file's user write bit is 0
+                                if uucore::has!(mode, S_IWUSR) {
+                                    None
+                                } else {
+                                    // Discard leading digits
+                                    let mode_without_leading_digits = mode & 0o7777;
+
+                                    Some((
+                                        format!("{mode_without_leading_digits:04o}"),
+                                        uucore::fs::display_permissions_unix(mode, false),
+                                    ))
+                                }
+                            }
+                            // TODO: How should failure to read the metadata be handled? Ignoring for now.
+                            Err(_) => None,
+                        }
+                    }
+
+                    #[cfg(not(unix))]
+                    {
+                        None
+                    }
+                };
+
+                let prompt_yes_result =
+                    if let Some((octal, human_readable)) = extra_prompt_information {
+                        prompt_yes!(
+                            "replace {}, overriding mode {octal} ({human_readable})?",
+                            path.quote()
+                        )
+                    } else {
+                        prompt_yes!("overwrite {}?", path.quote())
+                    };
+
+                if prompt_yes_result {
                     Ok(())
                 } else {
                     Err(Error::Skipped(true))