/brz/remove-bazaar

To get this branch, use:
bzr branch http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar

« back to all changes in this revision

Viewing changes to bzrlib/smart/protocol.py

  • Committer: Martin von Gagern
  • Date: 2011-05-27 12:01:22 UTC
  • mto: (5939.1.1 integration)
  • mto: This revision was merged to the branch mainline in revision 5940.
  • Revision ID: martin.vgagern@gmx.net-20110527120122-7f2yvh17wrech14v
https://launchpad.net/bugs/421776
Use move instead of rename when adding packs to repository.

A rename might fail (e.g. on sftp) if the target file does exist. This is
because rename is guaranteed to be atomic.  The move operation, on the other
hand, does guarantee deletion of any existing target if possible, so it is
more appropriate here, where we are not dealing with locks but with content
files.

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/smart/protocol.py
654
654
        """Make a remote call with a readv array.
655
655
 
656
656
        The body is encoded with one line per readv offset pair. The numbers in
657
 
        each pair are separated by a comma, and no trailing \n is emitted.
 
657
        each pair are separated by a comma, and no trailing \\n is emitted.
658
658
        """
659
659
        if 'hpss' in debug.debug_flags:
660
660
            mutter('hpss call w/readv: %s', repr(args)[1:-1])
1231
1231
                    if first_chunk is None:
1232
1232
                        first_chunk = chunk
1233
1233
                    self._write_prefixed_body(chunk)
 
1234
                    self.flush()
1234
1235
                    if 'hpssdetail' in debug.debug_flags:
1235
1236
                        # Not worth timing separately, as _write_func is
1236
1237
                        # actually buffered
1331
1332
        """Make a remote call with a readv array.
1332
1333
 
1333
1334
        The body is encoded with one line per readv offset pair. The numbers in
1334
 
        each pair are separated by a comma, and no trailing \n is emitted.
 
1335
        each pair are separated by a comma, and no trailing \\n is emitted.
1335
1336
        """
1336
1337
        if 'hpss' in debug.debug_flags:
1337
1338
            mutter('hpss call w/readv: %s', repr(args)[1:-1])