/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: John Arbash Meinel
  • Date: 2009-06-17 19:08:25 UTC
  • mto: This revision was merged to the branch mainline in revision 4460.
  • Revision ID: john@arbash-meinel.com-20090617190825-ktfk82li57rf2im6
It seems that fetch() no longer returns the number of revisions fetched.
It still does for *some* InterRepository fetch paths, but the generic one does not.
It is also not easy to get it to, since the Source and Sink are the ones
that would know how many keys were transmitted, and they are potentially 'remote'
objects.

This was also only tested to occur as a by-product in a random 'test_commit' test.
I assume if we really wanted the assurance, we would have a per_repo or interrepo
test for it.

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/revisiontree.py
1
 
# Copyright (C) 2006-2010 Canonical Ltd
 
1
# Copyright (C) 2005, 2007 Canonical Ltd
2
2
#
3
3
# This program is free software; you can redistribute it and/or modify
4
4
# it under the terms of the GNU General Public License as published by
20
20
 
21
21
from bzrlib import (
22
22
    errors,
 
23
    osutils,
23
24
    revision,
 
25
    symbol_versioning,
24
26
    tree,
25
27
    )
26
28
 
85
87
                      default_revision=revision.CURRENT_REVISION):
86
88
        """See Tree.annotate_iter"""
87
89
        text_key = (file_id, self.inventory[file_id].revision)
88
 
        annotator = self._repository.texts.get_annotator()
89
 
        annotations = annotator.annotate_flat(text_key)
 
90
        annotations = self._repository.texts.annotate(text_key)
90
91
        return [(key[-1], line) for key, line in annotations]
91
92
 
92
93
    def get_file_size(self, file_id):
101
102
 
102
103
    def get_file_mtime(self, file_id, path=None):
103
104
        ie = self._inventory[file_id]
104
 
        try:
105
 
            revision = self._repository.get_revision(ie.revision)
106
 
        except errors.NoSuchRevision:
107
 
            raise errors.FileTimestampUnavailable(self.id2path(file_id))
 
105
        revision = self._repository.get_revision(ie.revision)
108
106
        return revision.timestamp
109
107
 
110
108
    def is_executable(self, file_id, path=None):
243
241
        lookup_trees = [self.source]
244
242
        if extra_trees:
245
243
             lookup_trees.extend(extra_trees)
246
 
        # The ids of items we need to examine to insure delta consistency.
247
 
        precise_file_ids = set()
248
 
        discarded_changes = {}
249
244
        if specific_files == []:
250
245
            specific_file_ids = []
251
246
        else:
252
247
            specific_file_ids = self.target.paths2ids(specific_files,
253
248
                lookup_trees, require_versioned=require_versioned)
 
249
 
254
250
        # FIXME: It should be possible to delegate include_unchanged handling
255
251
        # to CHKInventory.iter_changes and do a better job there -- vila
256
252
        # 20090304
257
 
        changed_file_ids = set()
 
253
        if include_unchanged:
 
254
            changed_file_ids = []
258
255
        for result in self.target.inventory.iter_changes(self.source.inventory):
259
 
            if specific_file_ids is not None:
260
 
                file_id = result[0]
261
 
                if file_id not in specific_file_ids:
262
 
                    # A change from the whole tree that we don't want to show yet.
263
 
                    # We may find that we need to show it for delta consistency, so
264
 
                    # stash it.
265
 
                    discarded_changes[result[0]] = result
266
 
                    continue
267
 
                new_parent_id = result[4][1]
268
 
                precise_file_ids.add(new_parent_id)
 
256
            if (specific_file_ids is not None
 
257
                and not result[0] in specific_file_ids):
 
258
                # CHKMap.iter_changes is clean and fast. Better filter out
 
259
                # the specific files *after* it did its job.
 
260
                continue
269
261
            yield result
270
 
            changed_file_ids.add(result[0])
271
 
        if specific_file_ids is not None:
272
 
            for result in self._handle_precise_ids(precise_file_ids,
273
 
                changed_file_ids, discarded_changes=discarded_changes):
274
 
                yield result
 
262
            if include_unchanged:
 
263
                # Keep track of yielded results (cheaper than building the
 
264
                # whole inventory).
 
265
                changed_file_ids.append(result[0])
275
266
        if include_unchanged:
276
267
            # CHKMap avoid being O(tree), so we go to O(tree) only if
277
268
            # required to.