(backup-buffer-copy): Revert 2007-08-22 change.

diff --git a/lisp/ChangeLog b/lisp/ChangeLog
index 3f77b64753b1fe454c888bf9ce591697b2ac3747..0703d5134c9304c75768dd98ece2d97f2e4fa29f 100644 (file)
--- a/lisp/ChangeLog
+++ b/lisp/ChangeLog
@@ -1,3 +1,8 @@
+2007-08-24  Ulrich Mueller  <ulm@gentoo.org>  (tiny change)
+
+       * files.el (backup-buffer-copy): Don't wrap delete in
+       condition-case, only try to delete if file exists.
+
 2007-08-24  Glenn Morris  <rgm@gnu.org>
 
        * files.el (backup-buffer-copy): Revert 2007-08-22 change.

diff --git a/lisp/files.el b/lisp/files.el
index 8281a18b34f99a5e168e4ec7904db868eadf32c6..dab90097119b7c67abca163f44d51a9b26078cd0 100644 (file)
--- a/lisp/files.el
+++ b/lisp/files.el
@@ -3123,9 +3123,6 @@ BACKUPNAME is the backup file name, which is the old file renamed."
   (let ((umask (default-file-modes))
        (dir (or (file-name-directory to-name)
                 default-directory)))
-    ;; Can't delete or create files in a read-only directory.
-    (unless (file-writable-p dir)
-      (signal 'file-error (list "Directory is not writable" dir)))
     (unwind-protect
        (progn
          ;; Create temp files with strict access rights.  It's easy to
@@ -3134,11 +3131,6 @@ BACKUPNAME is the backup file name, which is the old file renamed."
          (set-default-file-modes ?\700)
          (while (condition-case ()
                     (progn
-                      ;; If we allow for the possibility of something
-                      ;; creating the file between delete and copy
-                      ;; (below), we must also allow for the
-                      ;; possibility of something deleting it between
-                      ;; a file-exists-p check and a delete.
                       (condition-case nil
                           (delete-file to-name)
                         (file-error nil))
@@ -3147,8 +3139,6 @@ BACKUPNAME is the backup file name, which is the old file renamed."
                   (file-already-exists t))
            ;; The file was somehow created by someone else between
            ;; `delete-file' and `copy-file', so let's try again.
-           ;; Does that every actually happen in practice?
-           ;; This is a potential infloop, which seems bad...
            ;; rms says "I think there is also a possible race
            ;; condition for making backup files" (emacs-devel 20070821).
            nil))