(change_attributes): Don't remove the destination file upon failure.
authorJim Meyering <jim@meyering.net>
Mon, 1 May 2000 14:09:19 +0000 (14:09 +0000)
committerJim Meyering <jim@meyering.net>
Mon, 1 May 2000 14:09:19 +0000 (14:09 +0000)
old/fileutils/ChangeLog
src/install.c

index fa21105..e74e52e 100644 (file)
@@ -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.
index 20931f6..b34909b 100644 (file)
@@ -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;
 }