/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/transform.py

https://launchpad.net/bugs/597686
Merge spiv's fix replacing use of _new_contents with _limbo_files in finalize

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/transform.py
1172
1172
        if self._tree is None:
1173
1173
            return
1174
1174
        try:
1175
 
            entries = [(self._limbo_name(t), t, k) for t, k in
1176
 
                       self._new_contents.iteritems()]
1177
 
            entries.sort(reverse=True)
1178
 
            for path, trans_id, kind in entries:
1179
 
                delete_any(path)
 
1175
            limbo_paths = sorted(self._limbo_files.values(), reverse=True)
 
1176
            for path in limbo_paths:
 
1177
                try:
 
1178
                    delete_any(path)
 
1179
                except OSError, e:
 
1180
                    if e.errno != errno.ENOENT:
 
1181
                        raise
 
1182
                    # XXX: warn? perhaps we just got interrupted at an
 
1183
                    # inconvenient moment, but perhaps files are disappearing
 
1184
                    # from under us?
1180
1185
            try:
1181
1186
                delete_any(self._limbodir)
1182
1187
            except OSError:
1265
1270
        name = self._limbo_name(trans_id)
1266
1271
        f = open(name, 'wb')
1267
1272
        try:
1268
 
            try:
1269
 
                unique_add(self._new_contents, trans_id, 'file')
1270
 
            except:
1271
 
                # Clean up the file, it never got registered so
1272
 
                # TreeTransform.finalize() won't clean it up.
1273
 
                f.close()
1274
 
                os.unlink(name)
1275
 
                raise
 
1273
            unique_add(self._new_contents, trans_id, 'file')
1276
1274
            f.writelines(contents)
1277
1275
        finally:
1278
1276
            f.close()
1851
1849
                    self._observed_sha1s[trans_id] = (o_sha1, st)
1852
1850
        finally:
1853
1851
            child_pb.finished()
 
1852
        for trans_id in self._new_contents:
 
1853
            del self._limbo_files[trans_id]
1854
1854
        self._new_contents.clear()
1855
1855
        return modified_paths
1856
1856