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

  • Committer: Robert Collins
  • Date: 2010-05-06 11:08:10 UTC
  • mto: This revision was merged to the branch mainline in revision 5223.
  • Revision ID: robertc@robertcollins.net-20100506110810-h3j07fh5gmw54s25
Cleaner matcher matching revised unlocking protocol.

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2005-2011 Canonical Ltd
 
1
# Copyright (C) 2005-2010 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
18
18
"""
19
19
 
20
20
import os
21
 
 
22
 
from bzrlib.lazy_import import lazy_import
23
 
lazy_import(globals(), """
24
 
import collections
25
 
 
 
21
from collections import deque
 
22
 
 
23
import bzrlib
26
24
from bzrlib import (
27
25
    conflicts as _mod_conflicts,
28
26
    debug,
29
27
    delta,
30
 
    errors,
31
28
    filters,
32
 
    inventory,
33
29
    osutils,
34
30
    revision as _mod_revision,
35
31
    rules,
36
 
    trace,
37
32
    )
38
 
""")
39
 
 
40
33
from bzrlib.decorators import needs_read_lock
 
34
from bzrlib.errors import BzrError, NoSuchId
 
35
from bzrlib import errors
 
36
from bzrlib.inventory import InventoryFile
41
37
from bzrlib.inter import InterObject
42
 
from bzrlib.symbol_versioning import (
43
 
    deprecated_in,
44
 
    deprecated_method,
45
 
    )
 
38
from bzrlib.osutils import fingerprint_file
 
39
from bzrlib.symbol_versioning import deprecated_function, deprecated_in
 
40
from bzrlib.trace import note
46
41
 
47
42
 
48
43
class Tree(object):
54
49
 
55
50
    * `RevisionTree` is a tree as recorded at some point in the past.
56
51
 
 
52
    Trees contain an `Inventory` object, and also know how to retrieve
 
53
    file texts mentioned in the inventory, either from a working
 
54
    directory or from a store.
 
55
 
 
56
    It is possible for trees to contain files that are not described
 
57
    in their inventory or vice versa; for this use `filenames()`.
 
58
 
57
59
    Trees can be compared, etc, regardless of whether they are working
58
60
    trees or versioned trees.
59
61
    """
125
127
        raise NotImplementedError(self.has_filename)
126
128
 
127
129
    def has_id(self, file_id):
128
 
        raise NotImplementedError(self.has_id)
 
130
        return self.inventory.has_id(file_id)
129
131
 
130
132
    def __contains__(self, file_id):
131
133
        return self.has_id(file_id)
132
134
 
133
135
    def has_or_had_id(self, file_id):
134
 
        raise NotImplementedError(self.has_or_had_id)
 
136
        return self.inventory.has_id(file_id)
135
137
 
136
138
    def is_ignored(self, filename):
137
139
        """Check whether the filename is ignored by this tree.
141
143
        """
142
144
        return False
143
145
 
 
146
    def __iter__(self):
 
147
        return iter(self.inventory)
 
148
 
144
149
    def all_file_ids(self):
145
150
        """Iterate through all file ids, including ids for missing files."""
146
 
        raise NotImplementedError(self.all_file_ids)
 
151
        return set(self.inventory)
147
152
 
148
153
    def id2path(self, file_id):
149
154
        """Return the path for a file id.
150
155
 
151
156
        :raises NoSuchId:
152
157
        """
153
 
        raise NotImplementedError(self.id2path)
154
 
 
 
158
        return self.inventory.id2path(file_id)
 
159
 
 
160
    def is_control_filename(self, filename):
 
161
        """True if filename is the name of a control file in this tree.
 
162
 
 
163
        :param filename: A filename within the tree. This is a relative path
 
164
        from the root of this tree.
 
165
 
 
166
        This is true IF and ONLY IF the filename is part of the meta data
 
167
        that bzr controls in this tree. I.E. a random .bzr directory placed
 
168
        on disk will not be a control file for this tree.
 
169
        """
 
170
        return self.bzrdir.is_control_filename(filename)
 
171
 
 
172
    @needs_read_lock
155
173
    def iter_entries_by_dir(self, specific_file_ids=None, yield_parents=False):
156
174
        """Walk the tree in 'by_dir' order.
157
175
 
174
192
             g
175
193
 
176
194
        The yield order (ignoring root) would be::
177
 
 
178
195
          a, f, a/b, a/d, a/b/c, a/d/e, f/g
179
196
 
180
197
        :param yield_parents: If True, yield the parents from the root leading
181
198
            down to specific_file_ids that have been requested. This has no
182
199
            impact if specific_file_ids is None.
183
200
        """
184
 
        raise NotImplementedError(self.iter_entries_by_dir)
185
 
 
186
 
    def list_files(self, include_root=False, from_dir=None, recursive=True):
187
 
        """List all files in this tree.
188
 
 
189
 
        :param include_root: Whether to include the entry for the tree root
190
 
        :param from_dir: Directory under which to list files
191
 
        :param recursive: Whether to list files recursively
192
 
        :return: iterator over tuples of (path, versioned, kind, file_id,
193
 
            inventory entry)
194
 
        """
195
 
        raise NotImplementedError(self.list_files)
 
201
        return self.inventory.iter_entries_by_dir(
 
202
            specific_file_ids=specific_file_ids, yield_parents=yield_parents)
196
203
 
197
204
    def iter_references(self):
198
205
        if self.supports_tree_reference():
249
256
    def _file_size(self, entry, stat_value):
250
257
        raise NotImplementedError(self._file_size)
251
258
 
 
259
    def _get_inventory(self):
 
260
        return self._inventory
 
261
 
252
262
    def get_file(self, file_id, path=None):
253
263
        """Return a file object for the file file_id in the tree.
254
264
 
290
300
 
291
301
        :param file_id: The file_id of the file.
292
302
        :param path: The path of the file.
293
 
 
294
303
        If both file_id and path are supplied, an implementation may use
295
304
        either one.
296
305
        """
297
306
        return osutils.split_lines(self.get_file_text(file_id, path))
298
307
 
299
 
    def get_file_sha1(self, file_id, path=None):
300
 
        """Return the SHA1 file for a file.
301
 
 
302
 
        :param file_id: The handle for this file.
303
 
        :param path: The path that this file can be found at.
304
 
            These must point to the same object.
305
 
        """
306
 
        raise NotImplementedError(self.get_file_sha1)
307
 
 
308
308
    def get_file_mtime(self, file_id, path=None):
309
309
        """Return the modification time for a file.
310
310
 
324
324
        raise NotImplementedError(self.get_file_size)
325
325
 
326
326
    def get_file_by_path(self, path):
327
 
        raise NotImplementedError(self.get_file_by_path)
328
 
 
329
 
    def is_executable(self, file_id, path=None):
330
 
        """Check if a file is executable.
331
 
 
332
 
        :param file_id: The handle for this file.
333
 
        :param path: The path that this file can be found at.
334
 
            These must point to the same object.
335
 
        """
336
 
        raise NotImplementedError(self.is_executable)
 
327
        return self.get_file(self._inventory.path2id(path), path)
337
328
 
338
329
    def iter_files_bytes(self, desired_files):
339
330
        """Iterate through file contents.
361
352
            cur_file = (self.get_file_text(file_id),)
362
353
            yield identifier, cur_file
363
354
 
364
 
    def get_symlink_target(self, file_id, path=None):
 
355
    def get_symlink_target(self, file_id):
365
356
        """Get the target for a given file_id.
366
357
 
367
358
        It is assumed that the caller already knows that file_id is referencing
368
359
        a symlink.
369
360
        :param file_id: Handle for the symlink entry.
370
 
        :param path: The path of the file.
371
 
        If both file_id and path are supplied, an implementation may use
372
 
        either one.
373
361
        :return: The path the symlink points to.
374
362
        """
375
363
        raise NotImplementedError(self.get_symlink_target)
376
364
 
 
365
    def get_canonical_inventory_paths(self, paths):
 
366
        """Like get_canonical_inventory_path() but works on multiple items.
 
367
 
 
368
        :param paths: A sequence of paths relative to the root of the tree.
 
369
        :return: A list of paths, with each item the corresponding input path
 
370
        adjusted to account for existing elements that match case
 
371
        insensitively.
 
372
        """
 
373
        return list(self._yield_canonical_inventory_paths(paths))
 
374
 
 
375
    def get_canonical_inventory_path(self, path):
 
376
        """Returns the first inventory item that case-insensitively matches path.
 
377
 
 
378
        If a path matches exactly, it is returned. If no path matches exactly
 
379
        but more than one path matches case-insensitively, it is implementation
 
380
        defined which is returned.
 
381
 
 
382
        If no path matches case-insensitively, the input path is returned, but
 
383
        with as many path entries that do exist changed to their canonical
 
384
        form.
 
385
 
 
386
        If you need to resolve many names from the same tree, you should
 
387
        use get_canonical_inventory_paths() to avoid O(N) behaviour.
 
388
 
 
389
        :param path: A paths relative to the root of the tree.
 
390
        :return: The input path adjusted to account for existing elements
 
391
        that match case insensitively.
 
392
        """
 
393
        return self._yield_canonical_inventory_paths([path]).next()
 
394
 
 
395
    def _yield_canonical_inventory_paths(self, paths):
 
396
        for path in paths:
 
397
            # First, if the path as specified exists exactly, just use it.
 
398
            if self.path2id(path) is not None:
 
399
                yield path
 
400
                continue
 
401
            # go walkin...
 
402
            cur_id = self.get_root_id()
 
403
            cur_path = ''
 
404
            bit_iter = iter(path.split("/"))
 
405
            for elt in bit_iter:
 
406
                lelt = elt.lower()
 
407
                new_path = None
 
408
                for child in self.iter_children(cur_id):
 
409
                    try:
 
410
                        # XXX: it seem like if the child is known to be in the
 
411
                        # tree, we shouldn't need to go from its id back to
 
412
                        # its path -- mbp 2010-02-11
 
413
                        #
 
414
                        # XXX: it seems like we could be more efficient
 
415
                        # by just directly looking up the original name and
 
416
                        # only then searching all children; also by not
 
417
                        # chopping paths so much. -- mbp 2010-02-11
 
418
                        child_base = os.path.basename(self.id2path(child))
 
419
                        if (child_base == elt):
 
420
                            # if we found an exact match, we can stop now; if
 
421
                            # we found an approximate match we need to keep
 
422
                            # searching because there might be an exact match
 
423
                            # later.  
 
424
                            cur_id = child
 
425
                            new_path = osutils.pathjoin(cur_path, child_base)
 
426
                            break
 
427
                        elif child_base.lower() == lelt:
 
428
                            cur_id = child
 
429
                            new_path = osutils.pathjoin(cur_path, child_base)
 
430
                    except NoSuchId:
 
431
                        # before a change is committed we can see this error...
 
432
                        continue
 
433
                if new_path:
 
434
                    cur_path = new_path
 
435
                else:
 
436
                    # got to the end of this directory and no entries matched.
 
437
                    # Return what matched so far, plus the rest as specified.
 
438
                    cur_path = osutils.pathjoin(cur_path, elt, *list(bit_iter))
 
439
                    break
 
440
            yield cur_path
 
441
        # all done.
 
442
 
377
443
    def get_root_id(self):
378
444
        """Return the file_id for the root of this tree."""
379
445
        raise NotImplementedError(self.get_root_id)
437
503
            except errors.NoSuchRevisionInTree:
438
504
                yield self.repository.revision_tree(revision_id)
439
505
 
 
506
    @staticmethod
 
507
    def _file_revision(revision_tree, file_id):
 
508
        """Determine the revision associated with a file in a given tree."""
 
509
        revision_tree.lock_read()
 
510
        try:
 
511
            return revision_tree.inventory[file_id].revision
 
512
        finally:
 
513
            revision_tree.unlock()
 
514
 
440
515
    def _get_file_revision(self, file_id, vf, tree_revision):
441
516
        """Ensure that file_id, tree_revision is in vf to plan the merge."""
442
517
 
443
518
        if getattr(self, '_repository', None) is None:
444
519
            last_revision = tree_revision
445
 
            parent_keys = [(file_id, t.get_file_revision(file_id)) for t in
 
520
            parent_keys = [(file_id, self._file_revision(t, file_id)) for t in
446
521
                self._iter_parent_trees()]
447
522
            vf.add_lines((file_id, last_revision), parent_keys,
448
 
                         self.get_file_lines(file_id))
 
523
                         self.get_file(file_id).readlines())
449
524
            repo = self.branch.repository
450
525
            base_vf = repo.texts
451
526
        else:
452
 
            last_revision = self.get_file_revision(file_id)
 
527
            last_revision = self._file_revision(self, file_id)
453
528
            base_vf = self._repository.texts
454
529
        if base_vf not in vf.fallback_versionedfiles:
455
530
            vf.fallback_versionedfiles.append(base_vf)
456
531
        return last_revision
457
532
 
 
533
    inventory = property(_get_inventory,
 
534
                         doc="Inventory of this Tree")
 
535
 
458
536
    def _check_retrieved(self, ie, f):
459
537
        if not __debug__:
460
538
            return
461
 
        fp = osutils.fingerprint_file(f)
 
539
        fp = fingerprint_file(f)
462
540
        f.seek(0)
463
541
 
464
542
        if ie.text_size is not None:
465
543
            if ie.text_size != fp['size']:
466
 
                raise errors.BzrError(
467
 
                        "mismatched size for file %r in %r" %
468
 
                        (ie.file_id, self._store),
 
544
                raise BzrError("mismatched size for file %r in %r" % (ie.file_id, self._store),
469
545
                        ["inventory expects %d bytes" % ie.text_size,
470
546
                         "file is actually %d bytes" % fp['size'],
471
547
                         "store is probably damaged/corrupt"])
472
548
 
473
549
        if ie.text_sha1 != fp['sha1']:
474
 
            raise errors.BzrError("wrong SHA-1 for file %r in %r" %
475
 
                    (ie.file_id, self._store),
 
550
            raise BzrError("wrong SHA-1 for file %r in %r" % (ie.file_id, self._store),
476
551
                    ["inventory expects %s" % ie.text_sha1,
477
552
                     "file is actually %s" % fp['sha1'],
478
553
                     "store is probably damaged/corrupt"])
479
554
 
 
555
    @needs_read_lock
480
556
    def path2id(self, path):
481
557
        """Return the id for path in this tree."""
482
 
        raise NotImplementedError(self.path2id)
 
558
        return self._inventory.path2id(path)
483
559
 
484
560
    def paths2ids(self, paths, trees=[], require_versioned=True):
485
561
        """Return all the ids that can be reached by walking from paths.
506
582
            yield child.file_id
507
583
 
508
584
    def lock_read(self):
509
 
        """Lock this tree for multiple read only operations.
510
 
        
511
 
        :return: A bzrlib.lock.LogicalLockResult.
512
 
        """
513
585
        pass
514
586
 
515
587
    def revision_tree(self, revision_id):
542
614
 
543
615
        :return: set of paths.
544
616
        """
545
 
        raise NotImplementedError(self.filter_unversioned_files)
 
617
        # NB: we specifically *don't* call self.has_filename, because for
 
618
        # WorkingTrees that can indicate files that exist on disk but that
 
619
        # are not versioned.
 
620
        pred = self.inventory.has_filename
 
621
        return set((p for p in paths if not pred(p)))
546
622
 
547
623
    def walkdirs(self, prefix=""):
548
624
        """Walk the contents of this tree from path down.
600
676
        prefs = self.iter_search_rules([path], filter_pref_names).next()
601
677
        stk = filters._get_filter_stack_for(prefs)
602
678
        if 'filters' in debug.debug_flags:
603
 
            trace.note("*** %s content-filter: %s => %r" % (path,prefs,stk))
 
679
            note("*** %s content-filter: %s => %r" % (path,prefs,stk))
604
680
        return stk
605
681
 
606
682
    def _content_filter_stack_provider(self):
645
721
        return searcher
646
722
 
647
723
 
648
 
class InventoryTree(Tree):
649
 
    """A tree that relies on an inventory for its metadata.
650
 
 
651
 
    Trees contain an `Inventory` object, and also know how to retrieve
652
 
    file texts mentioned in the inventory, either from a working
653
 
    directory or from a store.
654
 
 
655
 
    It is possible for trees to contain files that are not described
656
 
    in their inventory or vice versa; for this use `filenames()`.
657
 
 
658
 
    Subclasses should set the _inventory attribute, which is considered
659
 
    private to external API users.
660
 
    """
661
 
 
662
 
    def get_canonical_inventory_paths(self, paths):
663
 
        """Like get_canonical_inventory_path() but works on multiple items.
664
 
 
665
 
        :param paths: A sequence of paths relative to the root of the tree.
666
 
        :return: A list of paths, with each item the corresponding input path
667
 
        adjusted to account for existing elements that match case
668
 
        insensitively.
669
 
        """
670
 
        return list(self._yield_canonical_inventory_paths(paths))
671
 
 
672
 
    def get_canonical_inventory_path(self, path):
673
 
        """Returns the first inventory item that case-insensitively matches path.
674
 
 
675
 
        If a path matches exactly, it is returned. If no path matches exactly
676
 
        but more than one path matches case-insensitively, it is implementation
677
 
        defined which is returned.
678
 
 
679
 
        If no path matches case-insensitively, the input path is returned, but
680
 
        with as many path entries that do exist changed to their canonical
681
 
        form.
682
 
 
683
 
        If you need to resolve many names from the same tree, you should
684
 
        use get_canonical_inventory_paths() to avoid O(N) behaviour.
685
 
 
686
 
        :param path: A paths relative to the root of the tree.
687
 
        :return: The input path adjusted to account for existing elements
688
 
        that match case insensitively.
689
 
        """
690
 
        return self._yield_canonical_inventory_paths([path]).next()
691
 
 
692
 
    def _yield_canonical_inventory_paths(self, paths):
693
 
        for path in paths:
694
 
            # First, if the path as specified exists exactly, just use it.
695
 
            if self.path2id(path) is not None:
696
 
                yield path
697
 
                continue
698
 
            # go walkin...
699
 
            cur_id = self.get_root_id()
700
 
            cur_path = ''
701
 
            bit_iter = iter(path.split("/"))
702
 
            for elt in bit_iter:
703
 
                lelt = elt.lower()
704
 
                new_path = None
705
 
                for child in self.iter_children(cur_id):
706
 
                    try:
707
 
                        # XXX: it seem like if the child is known to be in the
708
 
                        # tree, we shouldn't need to go from its id back to
709
 
                        # its path -- mbp 2010-02-11
710
 
                        #
711
 
                        # XXX: it seems like we could be more efficient
712
 
                        # by just directly looking up the original name and
713
 
                        # only then searching all children; also by not
714
 
                        # chopping paths so much. -- mbp 2010-02-11
715
 
                        child_base = os.path.basename(self.id2path(child))
716
 
                        if (child_base == elt):
717
 
                            # if we found an exact match, we can stop now; if
718
 
                            # we found an approximate match we need to keep
719
 
                            # searching because there might be an exact match
720
 
                            # later.  
721
 
                            cur_id = child
722
 
                            new_path = osutils.pathjoin(cur_path, child_base)
723
 
                            break
724
 
                        elif child_base.lower() == lelt:
725
 
                            cur_id = child
726
 
                            new_path = osutils.pathjoin(cur_path, child_base)
727
 
                    except errors.NoSuchId:
728
 
                        # before a change is committed we can see this error...
729
 
                        continue
730
 
                if new_path:
731
 
                    cur_path = new_path
732
 
                else:
733
 
                    # got to the end of this directory and no entries matched.
734
 
                    # Return what matched so far, plus the rest as specified.
735
 
                    cur_path = osutils.pathjoin(cur_path, elt, *list(bit_iter))
736
 
                    break
737
 
            yield cur_path
738
 
        # all done.
739
 
 
740
 
    def _get_inventory(self):
741
 
        return self._inventory
742
 
 
743
 
    inventory = property(_get_inventory,
744
 
                         doc="Inventory of this Tree")
745
 
 
746
 
    @needs_read_lock
747
 
    def path2id(self, path):
748
 
        """Return the id for path in this tree."""
749
 
        return self._inventory.path2id(path)
750
 
 
751
 
    def id2path(self, file_id):
752
 
        """Return the path for a file id.
753
 
 
754
 
        :raises NoSuchId:
755
 
        """
756
 
        return self.inventory.id2path(file_id)
757
 
 
758
 
    def has_id(self, file_id):
759
 
        return self.inventory.has_id(file_id)
760
 
 
761
 
    def has_or_had_id(self, file_id):
762
 
        return self.inventory.has_id(file_id)
763
 
 
764
 
    def all_file_ids(self):
765
 
        return set(self.inventory)
766
 
 
767
 
    @deprecated_method(deprecated_in((2, 4, 0)))
768
 
    def __iter__(self):
769
 
        return iter(self.inventory)
770
 
 
771
 
    def filter_unversioned_files(self, paths):
772
 
        """Filter out paths that are versioned.
773
 
 
774
 
        :return: set of paths.
775
 
        """
776
 
        # NB: we specifically *don't* call self.has_filename, because for
777
 
        # WorkingTrees that can indicate files that exist on disk but that
778
 
        # are not versioned.
779
 
        pred = self.inventory.has_filename
780
 
        return set((p for p in paths if not pred(p)))
781
 
 
782
 
    @needs_read_lock
783
 
    def iter_entries_by_dir(self, specific_file_ids=None, yield_parents=False):
784
 
        """Walk the tree in 'by_dir' order.
785
 
 
786
 
        This will yield each entry in the tree as a (path, entry) tuple.
787
 
        The order that they are yielded is:
788
 
 
789
 
        See Tree.iter_entries_by_dir for details.
790
 
 
791
 
        :param yield_parents: If True, yield the parents from the root leading
792
 
            down to specific_file_ids that have been requested. This has no
793
 
            impact if specific_file_ids is None.
794
 
        """
795
 
        return self.inventory.iter_entries_by_dir(
796
 
            specific_file_ids=specific_file_ids, yield_parents=yield_parents)
797
 
 
798
 
    def get_file_by_path(self, path):
799
 
        return self.get_file(self._inventory.path2id(path), path)
800
 
 
801
 
 
802
724
######################################################################
803
725
# diff
804
726
 
852
774
    return 'wtf?'
853
775
 
854
776
 
 
777
@deprecated_function(deprecated_in((1, 9, 0)))
 
778
def find_renames(old_inv, new_inv):
 
779
    for file_id in old_inv:
 
780
        if file_id not in new_inv:
 
781
            continue
 
782
        old_name = old_inv.id2path(file_id)
 
783
        new_name = new_inv.id2path(file_id)
 
784
        if old_name != new_name:
 
785
            yield (old_name, new_name)
 
786
 
 
787
 
855
788
def find_ids_across_trees(filenames, trees, require_versioned=True):
856
789
    """Find the ids corresponding to specified filenames.
857
790
 
862
795
        None)
863
796
    :param trees: The trees to find file_ids within
864
797
    :param require_versioned: if true, all specified filenames must occur in
865
 
        at least one tree.
 
798
    at least one tree.
866
799
    :return: a set of file ids for the specified filenames and their children.
867
800
    """
868
801
    if not filenames:
944
877
 
945
878
    _optimisers = []
946
879
 
947
 
    @classmethod
948
 
    def is_compatible(kls, source, target):
949
 
        # The default implementation is naive and uses the public API, so
950
 
        # it works for all trees.
951
 
        return True
952
 
 
953
880
    def _changes_from_entries(self, source_entry, target_entry,
954
881
        source_path=None, target_path=None):
955
882
        """Generate a iter_changes tuple between source_entry and target_entry.
1060
987
            # All files are unversioned, so just return an empty delta
1061
988
            # _compare_trees would think we want a complete delta
1062
989
            result = delta.TreeDelta()
1063
 
            fake_entry = inventory.InventoryFile('unused', 'unused', 'unused')
 
990
            fake_entry = InventoryFile('unused', 'unused', 'unused')
1064
991
            result.unversioned = [(path, None,
1065
992
                self.target._comparison_data(fake_entry, path)[0]) for path in
1066
993
                specific_files]
1131
1058
                                     self.target.extras()
1132
1059
                if specific_files is None or
1133
1060
                    osutils.is_inside_any(specific_files, p)])
1134
 
            all_unversioned = collections.deque(all_unversioned)
 
1061
            all_unversioned = deque(all_unversioned)
1135
1062
        else:
1136
 
            all_unversioned = collections.deque()
 
1063
            all_unversioned = deque()
1137
1064
        to_paths = {}
1138
1065
        from_entries_by_dir = list(self.source.iter_entries_by_dir(
1139
1066
            specific_file_ids=specific_file_ids))
1145
1072
        # the unversioned path lookup only occurs on real trees - where there
1146
1073
        # can be extras. So the fake_entry is solely used to look up
1147
1074
        # executable it values when execute is not supported.
1148
 
        fake_entry = inventory.InventoryFile('unused', 'unused', 'unused')
 
1075
        fake_entry = InventoryFile('unused', 'unused', 'unused')
1149
1076
        for target_path, target_entry in to_entries_by_dir:
1150
1077
            while (all_unversioned and
1151
1078
                all_unversioned[0][0] < target_path.split('/')):
1199
1126
            if file_id in to_paths:
1200
1127
                # already returned
1201
1128
                continue
1202
 
            if not self.target.has_id(file_id):
 
1129
            if file_id not in self.target.all_file_ids():
1203
1130
                # common case - paths we have not emitted are not present in
1204
1131
                # target.
1205
1132
                to_path = None
1324
1251
                    yield result
1325
1252
 
1326
1253
 
1327
 
InterTree.register_optimiser(InterTree)
1328
 
 
1329
 
 
1330
1254
class MultiWalker(object):
1331
1255
    """Walk multiple trees simultaneously, getting combined results."""
1332
1256