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

diff --git a/old/fileutils/ChangeLog b/old/fileutils/ChangeLog
index fa211058947192ee421f879022d1e0934891c4b5..e74e52ed7e1df057eae26f82422745a79d469032 100644 (file)
--- 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 20931f63b0bd1d040ba0ee30ec8b7e197934d155..b34909b76f167a7301215b96fe5ebf528261b1d9 100644 (file)
--- 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;
 }