From: Jim Meyering <jim@meyering.net>
Date: Mon, 1 May 2000 14:09:19 +0000 (+0000)
Subject: (change_attributes): Don't remove the destination file upon failure.
X-Git-Tag: SH-UTILS-2_0h~104
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=624dd54796656c89a82d359bebc42774f5e90549;p=thirdparty%2Fcoreutils.git

(change_attributes): Don't remove the destination file upon failure.
---

diff --git a/old/fileutils/ChangeLog b/old/fileutils/ChangeLog
index fa21105894..e74e52ed7e 100644
--- a/old/fileutils/ChangeLog
+++ b/old/fileutils/ChangeLog
@@ -1,3 +1,8 @@
+2000-05-01  Jim Meyering  <meyering@lucent.com>
+
+	* src/install.c (change_attributes): Don't even attempt the chmod
+	if the chown fails.  Reported by Marc Olzheim.
+
 2000-04-30  Jim Meyering  <meyering@lucent.com>
 
 	* src/touch.c (O_NOCTTY): Define if not defined already.
diff --git a/src/install.c b/src/install.c
index 20931f63b0..b34909b76f 100644
--- a/src/install.c
+++ b/src/install.c
@@ -517,13 +517,6 @@ change_attributes (const char *path)
       err = 1;
     }
 
-  if (err)
-    {
-      error (0, 0, "removing file: `%s'", path);
-      if (unlink (path))
-	error (0, errno, "cannot remove `%s'", path);
-    }
-
   return err;
 }