[MERGE][bug #75721](0.17) "bzr push" should only connect 1 time

Martin Pool mbp at canonical.com
Wed May 2 02:30:14 BST 2007


Martin Pool has voted +1 (conditional).
Status is now: Conditionally approved
Comment:

+    def _make_tail_on_transport(self, transport):
+        """Create the final directory in transport if it doesn't exist.
+
+        We use "Easier to ask for Permission". And just create it, and 
ignore
+        if the directory already exists.
+        """

I think this should be in Transport not here.
I think you mean "easier to seek forgiveness than permission".

Normally when we make new directories we need to be careful to use the 
right permissions, which is taken care of by LockableFiles.  (Not really 
a great place for it I think but that's a separate matter.)  I wonder if 
that can also be a problem here?  I think probably not, because we're 
starting a new repository and there's nothing it needs to be consistent 
with.

OK with me for 0.16 if we move that method.

For details, see: 
http://bundlebuggy.aaronbentley.com/request/%3C4637C696.8030409%40arbash-meinel.com%3E



More information about the bazaar mailing list