The patch looks incomplete to me:

+               if written != len(data):
+                       os.unlink(file)
+                       os.rename(backup, file)
+                       raise IOError

have you tried saving when the remote machine has no space left?

Also, sicne gnomevfs doesn't have a rename() function we need top copy the whole
file. This means:

        a) (in the ssh case) it travels down the pipe to revelation and up again
           to the server.
        b) in a cornercase we may not be able to restore the backup file if
           there is not enough space for a copy of it.

There isn't much we can do about a), I need to think about b) for a while, maybe
we just error out in that case ...

-- 
You can rent this space for only $5 a week.



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to