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

  • Committer: Robert Collins
  • Date: 2009-08-26 03:20:32 UTC
  • mfrom: (4637 +trunk)
  • mto: (4634.6.2 2.0)
  • mto: This revision was merged to the branch mainline in revision 4660.
  • Revision ID: robertc@robertcollins.net-20090826032032-mx5kiog3eihaoy13
Merge and cherrypick outstanding 2.0 relevant patches from bzr.dev: Up to rev
4637, then 4639,4641,4643,4646,4649,4650,4651. (Robert Collins)

Log messages of the incorporated fixes:

revno: 4651 [merge]
  (robertc) Enable commit via record_iter_changes for specific file
        comments. (Robert Collins)

revno: 4650 [merge]
  Fix shelve on windows. (Robert Collins, #305006)

revno: 4649 [merge]
  (robertc) Make iter_changes produce output that is always safe to
        generate inventory deltas of in the same direction as the
        changes. (Robert Collins, #347649)

revno: 4646 [merge]
  (mbp) developer documentation about content filtering

revno: 4643 [merge]
  (mbp) small tweaks to release documentation

revno: 4641 [merge]
  (abentley) Shelve will not remove tree root.

revno: 4639 [merge]
  (andrew) Fix 'Revision ... not present' errors when upgrading stacked
        branches.

revno: 4637 [merge]
  Fix upgrade of branches in repositories.

revno: 4636 [merge]
  (mbp) fix crash formatting CannotBindAddress

revno: 4635 [merge]
  (robertc) Fix many locking errors on windows due to a small bug in
        merge.transform_tree. (Robert Collins)

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/revisiontree.py
242
242
        lookup_trees = [self.source]
243
243
        if extra_trees:
244
244
             lookup_trees.extend(extra_trees)
 
245
        # The ids of items we need to examine to insure delta consistency.
 
246
        precise_file_ids = set()
 
247
        discarded_changes = {}
245
248
        if specific_files == []:
246
249
            specific_file_ids = []
247
250
        else:
248
251
            specific_file_ids = self.target.paths2ids(specific_files,
249
252
                lookup_trees, require_versioned=require_versioned)
250
 
 
251
253
        # FIXME: It should be possible to delegate include_unchanged handling
252
254
        # to CHKInventory.iter_changes and do a better job there -- vila
253
255
        # 20090304
254
 
        if include_unchanged:
255
 
            changed_file_ids = []
 
256
        changed_file_ids = set()
256
257
        for result in self.target.inventory.iter_changes(self.source.inventory):
257
 
            if (specific_file_ids is not None
258
 
                and not result[0] in specific_file_ids):
259
 
                # CHKMap.iter_changes is clean and fast. Better filter out
260
 
                # the specific files *after* it did its job.
261
 
                continue
 
258
            if specific_file_ids is not None:
 
259
                file_id = result[0]
 
260
                if file_id not in specific_file_ids:
 
261
                    # A change from the whole tree that we don't want to show yet.
 
262
                    # We may find that we need to show it for delta consistency, so
 
263
                    # stash it.
 
264
                    discarded_changes[result[0]] = result
 
265
                    continue
 
266
                new_parent_id = result[4][1]
 
267
                precise_file_ids.add(new_parent_id)
262
268
            yield result
263
 
            if include_unchanged:
264
 
                # Keep track of yielded results (cheaper than building the
265
 
                # whole inventory).
266
 
                changed_file_ids.append(result[0])
 
269
            changed_file_ids.add(result[0])
 
270
        if specific_file_ids is not None:
 
271
            for result in self._handle_precise_ids(precise_file_ids,
 
272
                changed_file_ids, discarded_changes=discarded_changes):
 
273
                yield result
267
274
        if include_unchanged:
268
275
            # CHKMap avoid being O(tree), so we go to O(tree) only if
269
276
            # required to.