[merge][rfc] test that Transport.append sets permissions on existing files

John Arbash Meinel john at arbash-meinel.com
Thu Aug 16 18:18:50 BST 2007


John Arbash Meinel has voted approve.
Status is now: Approved
Comment:
In the current code, we never pass a mode flag to append. Because we 
assume that you did the right thing when creating the file.
It means that if you update the .bzr we won't update files underneath 
it, but I think that is okay.

I would be fine with deprecating the mode bits on Transport.append() and 
just expect them to be accepted for put_non_atomic() et al.


For details, see: 
http://bundlebuggy.aaronbentley.com/request/%3Ce01316480708142355y111a032fu18637e7aa6a3a701%40mail.gmail.com%3E



More information about the bazaar mailing list