From: Jim Meyering <jim@meyering.net>
Date: Sat, 15 Sep 2001 12:34:17 +0000 (+0000)
Subject: (copy_internal) [move_mode]: Support the new
X-Git-Tag: TEXTUTILS-2_0_15~103
X-Git-Url: http://git.ipfire.org/gitweb/index.cgi?a=commitdiff_plain;h=cb40817c279db84c61ae1df5330670839e036a16;p=thirdparty%2Fcoreutils.git

(copy_internal) [move_mode]: Support the new
--reply=... option.
---

diff --git a/src/copy.c b/src/copy.c
index b7f535c7b1..ae9d6ad1d4 100644
--- a/src/copy.c
+++ b/src/copy.c
@@ -682,36 +682,57 @@ copy_internal (const char *src_path, const char *dst_path,
 		}
 	    }
 
+	  /* When there is an existing destination file, we may end up
+	     returning early, and hence not copying/moving the file.
+	     This may be due to an interactive `negative' reply to the
+	     prompt about the existing file.  It may also be due to the
+	     use of the --reply=no option.  */
 	  if (!S_ISDIR (src_type))
 	    {
 	      /* cp and mv treat -i and -f differently.  */
 	      if (x->move_mode)
 		{
-		  if (x->interactive != I_OFF
-		      && (x->interactive == I_ON
-			  || (x->stdin_tty
-			      /* euidaccess is not meaningful for symlinks */
-			      && ! S_ISLNK (dst_sb.st_mode)
-			      && euidaccess (dst_path, W_OK) != 0)))
+		  int do_move = 1;
+
+		  if (x->interactive == I_ALWAYS_NO)
+		    {
+		      if (! S_ISLNK (dst_sb.st_mode)
+			  && euidaccess (dst_path, W_OK) != 0)
+			{
+			  do_move = 0;
+			}
+		    }
+		  else if (x->interactive == I_ASK_USER
+			   || (x->interactive == I_UNSPECIFIED
+			       && (x->stdin_tty
+				   /* euidaccess is not meaningful for symlinks */
+				   && ! S_ISLNK (dst_sb.st_mode)
+				   && euidaccess (dst_path, W_OK) != 0)))
 		    {
 		      overwrite_prompt (dst_path, &dst_sb);
 		      if (!yesno ())
 			{
-			  /* Pretend the rename succeeded, so the caller (mv)
-			     doesn't end up removing the source file.  */
-			  if (rename_succeeded)
-			    *rename_succeeded = 1;
-			  return 0;
+			  do_move = 0;
 			}
 		    }
+
+		  if ( ! do_move)
+		    {
+		      /* Pretend the rename succeeded, so the caller (mv)
+			 doesn't end up removing the source file.  */
+		      if (rename_succeeded)
+			*rename_succeeded = 1;
+		      return 0;
+		    }
 		}
 	      else
 		{
-		  if (x->interactive == I_ON)
+		  if (x->interactive == I_ALWAYS_NO
+		      || (x->interactive == I_ASK_USER
+			  && (overwrite_prompt (dst_path, &dst_sb), 1)
+			  && ! yesno ()))
 		    {
-		      overwrite_prompt (dst_path, &dst_sb);
-		      if (!yesno ())
-			return 0;
+		      return 0;
 		    }
 		}
 	    }