~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/tree.py

  • Committer: Canonical.com Patch Queue Manager
  • Date: 2011-05-19 10:51:37 UTC
  • mfrom: (5891.1.3 api-docs)
  • Revision ID: pqm@pqm.ubuntu.com-20110519105137-amzagrral2ldm1lq
(spiv) Fix the formatting of more docstrings. (Andrew Bennetts)

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2005-2010 Canonical Ltd
 
1
# Copyright (C) 2005-2011 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
 
from collections import deque
22
 
 
23
 
import bzrlib
 
21
 
 
22
from bzrlib.lazy_import import lazy_import
 
23
lazy_import(globals(), """
 
24
import collections
 
25
 
24
26
from bzrlib import (
25
27
    conflicts as _mod_conflicts,
26
28
    debug,
27
29
    delta,
 
30
    errors,
28
31
    filters,
 
32
    inventory,
29
33
    osutils,
30
34
    revision as _mod_revision,
31
35
    rules,
 
36
    trace,
32
37
    )
 
38
""")
 
39
 
33
40
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
37
41
from bzrlib.inter import InterObject
38
 
from bzrlib.osutils import fingerprint_file
39
 
from bzrlib.symbol_versioning import deprecated_function, deprecated_in
40
 
from bzrlib.trace import note
 
42
from bzrlib.symbol_versioning import (
 
43
    deprecated_in,
 
44
    deprecated_method,
 
45
    )
41
46
 
42
47
 
43
48
class Tree(object):
49
54
 
50
55
    * `RevisionTree` is a tree as recorded at some point in the past.
51
56
 
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
 
 
59
57
    Trees can be compared, etc, regardless of whether they are working
60
58
    trees or versioned trees.
61
59
    """
127
125
        raise NotImplementedError(self.has_filename)
128
126
 
129
127
    def has_id(self, file_id):
130
 
        return self.inventory.has_id(file_id)
 
128
        raise NotImplementedError(self.has_id)
131
129
 
132
130
    def __contains__(self, file_id):
133
131
        return self.has_id(file_id)
134
132
 
135
133
    def has_or_had_id(self, file_id):
136
 
        return self.inventory.has_id(file_id)
 
134
        raise NotImplementedError(self.has_or_had_id)
137
135
 
138
136
    def is_ignored(self, filename):
139
137
        """Check whether the filename is ignored by this tree.
143
141
        """
144
142
        return False
145
143
 
146
 
    def __iter__(self):
147
 
        return iter(self.inventory)
148
 
 
149
144
    def all_file_ids(self):
150
145
        """Iterate through all file ids, including ids for missing files."""
151
 
        return set(self.inventory)
 
146
        raise NotImplementedError(self.all_file_ids)
152
147
 
153
148
    def id2path(self, file_id):
154
149
        """Return the path for a file id.
155
150
 
156
151
        :raises NoSuchId:
157
152
        """
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
 
153
        raise NotImplementedError(self.id2path)
 
154
 
173
155
    def iter_entries_by_dir(self, specific_file_ids=None, yield_parents=False):
174
156
        """Walk the tree in 'by_dir' order.
175
157
 
192
174
             g
193
175
 
194
176
        The yield order (ignoring root) would be::
 
177
 
195
178
          a, f, a/b, a/d, a/b/c, a/d/e, f/g
196
179
 
197
180
        :param yield_parents: If True, yield the parents from the root leading
198
181
            down to specific_file_ids that have been requested. This has no
199
182
            impact if specific_file_ids is None.
200
183
        """
201
 
        return self.inventory.iter_entries_by_dir(
202
 
            specific_file_ids=specific_file_ids, yield_parents=yield_parents)
 
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)
203
196
 
204
197
    def iter_references(self):
205
198
        if self.supports_tree_reference():
256
249
    def _file_size(self, entry, stat_value):
257
250
        raise NotImplementedError(self._file_size)
258
251
 
259
 
    def _get_inventory(self):
260
 
        return self._inventory
261
 
 
262
252
    def get_file(self, file_id, path=None):
263
253
        """Return a file object for the file file_id in the tree.
264
254
 
300
290
 
301
291
        :param file_id: The file_id of the file.
302
292
        :param path: The path of the file.
 
293
 
303
294
        If both file_id and path are supplied, an implementation may use
304
295
        either one.
305
296
        """
306
297
        return osutils.split_lines(self.get_file_text(file_id, path))
307
298
 
 
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
 
        return self.get_file(self._inventory.path2id(path), 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)
328
337
 
329
338
    def iter_files_bytes(self, desired_files):
330
339
        """Iterate through file contents.
352
361
            cur_file = (self.get_file_text(file_id),)
353
362
            yield identifier, cur_file
354
363
 
355
 
    def get_symlink_target(self, file_id):
 
364
    def get_symlink_target(self, file_id, path=None):
356
365
        """Get the target for a given file_id.
357
366
 
358
367
        It is assumed that the caller already knows that file_id is referencing
359
368
        a symlink.
360
369
        :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.
361
373
        :return: The path the symlink points to.
362
374
        """
363
375
        raise NotImplementedError(self.get_symlink_target)
364
376
 
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
 
 
443
377
    def get_root_id(self):
444
378
        """Return the file_id for the root of this tree."""
445
379
        raise NotImplementedError(self.get_root_id)
503
437
            except errors.NoSuchRevisionInTree:
504
438
                yield self.repository.revision_tree(revision_id)
505
439
 
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
 
 
515
440
    def _get_file_revision(self, file_id, vf, tree_revision):
516
441
        """Ensure that file_id, tree_revision is in vf to plan the merge."""
517
442
 
518
443
        if getattr(self, '_repository', None) is None:
519
444
            last_revision = tree_revision
520
 
            parent_keys = [(file_id, self._file_revision(t, file_id)) for t in
 
445
            parent_keys = [(file_id, t.get_file_revision(file_id)) for t in
521
446
                self._iter_parent_trees()]
522
447
            vf.add_lines((file_id, last_revision), parent_keys,
523
 
                         self.get_file(file_id).readlines())
 
448
                         self.get_file_lines(file_id))
524
449
            repo = self.branch.repository
525
450
            base_vf = repo.texts
526
451
        else:
527
 
            last_revision = self._file_revision(self, file_id)
 
452
            last_revision = self.get_file_revision(file_id)
528
453
            base_vf = self._repository.texts
529
454
        if base_vf not in vf.fallback_versionedfiles:
530
455
            vf.fallback_versionedfiles.append(base_vf)
531
456
        return last_revision
532
457
 
533
 
    inventory = property(_get_inventory,
534
 
                         doc="Inventory of this Tree")
535
 
 
536
458
    def _check_retrieved(self, ie, f):
537
459
        if not __debug__:
538
460
            return
539
 
        fp = fingerprint_file(f)
 
461
        fp = osutils.fingerprint_file(f)
540
462
        f.seek(0)
541
463
 
542
464
        if ie.text_size is not None:
543
465
            if ie.text_size != fp['size']:
544
 
                raise BzrError("mismatched size for file %r in %r" % (ie.file_id, self._store),
 
466
                raise errors.BzrError(
 
467
                        "mismatched size for file %r in %r" %
 
468
                        (ie.file_id, self._store),
545
469
                        ["inventory expects %d bytes" % ie.text_size,
546
470
                         "file is actually %d bytes" % fp['size'],
547
471
                         "store is probably damaged/corrupt"])
548
472
 
549
473
        if ie.text_sha1 != fp['sha1']:
550
 
            raise BzrError("wrong SHA-1 for file %r in %r" % (ie.file_id, self._store),
 
474
            raise errors.BzrError("wrong SHA-1 for file %r in %r" %
 
475
                    (ie.file_id, self._store),
551
476
                    ["inventory expects %s" % ie.text_sha1,
552
477
                     "file is actually %s" % fp['sha1'],
553
478
                     "store is probably damaged/corrupt"])
554
479
 
555
 
    @needs_read_lock
556
480
    def path2id(self, path):
557
481
        """Return the id for path in this tree."""
558
 
        return self._inventory.path2id(path)
 
482
        raise NotImplementedError(self.path2id)
559
483
 
560
484
    def paths2ids(self, paths, trees=[], require_versioned=True):
561
485
        """Return all the ids that can be reached by walking from paths.
582
506
            yield child.file_id
583
507
 
584
508
    def lock_read(self):
 
509
        """Lock this tree for multiple read only operations.
 
510
        
 
511
        :return: A bzrlib.lock.LogicalLockResult.
 
512
        """
585
513
        pass
586
514
 
587
515
    def revision_tree(self, revision_id):
614
542
 
615
543
        :return: set of paths.
616
544
        """
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)))
 
545
        raise NotImplementedError(self.filter_unversioned_files)
622
546
 
623
547
    def walkdirs(self, prefix=""):
624
548
        """Walk the contents of this tree from path down.
676
600
        prefs = self.iter_search_rules([path], filter_pref_names).next()
677
601
        stk = filters._get_filter_stack_for(prefs)
678
602
        if 'filters' in debug.debug_flags:
679
 
            note("*** %s content-filter: %s => %r" % (path,prefs,stk))
 
603
            trace.note("*** %s content-filter: %s => %r" % (path,prefs,stk))
680
604
        return stk
681
605
 
682
606
    def _content_filter_stack_provider(self):
721
645
        return searcher
722
646
 
723
647
 
 
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
 
724
802
######################################################################
725
803
# diff
726
804
 
774
852
    return 'wtf?'
775
853
 
776
854
 
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
 
 
788
855
def find_ids_across_trees(filenames, trees, require_versioned=True):
789
856
    """Find the ids corresponding to specified filenames.
790
857
 
795
862
        None)
796
863
    :param trees: The trees to find file_ids within
797
864
    :param require_versioned: if true, all specified filenames must occur in
798
 
    at least one tree.
 
865
        at least one tree.
799
866
    :return: a set of file ids for the specified filenames and their children.
800
867
    """
801
868
    if not filenames:
877
944
 
878
945
    _optimisers = []
879
946
 
 
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
 
880
953
    def _changes_from_entries(self, source_entry, target_entry,
881
954
        source_path=None, target_path=None):
882
955
        """Generate a iter_changes tuple between source_entry and target_entry.
987
1060
            # All files are unversioned, so just return an empty delta
988
1061
            # _compare_trees would think we want a complete delta
989
1062
            result = delta.TreeDelta()
990
 
            fake_entry = InventoryFile('unused', 'unused', 'unused')
 
1063
            fake_entry = inventory.InventoryFile('unused', 'unused', 'unused')
991
1064
            result.unversioned = [(path, None,
992
1065
                self.target._comparison_data(fake_entry, path)[0]) for path in
993
1066
                specific_files]
1058
1131
                                     self.target.extras()
1059
1132
                if specific_files is None or
1060
1133
                    osutils.is_inside_any(specific_files, p)])
1061
 
            all_unversioned = deque(all_unversioned)
 
1134
            all_unversioned = collections.deque(all_unversioned)
1062
1135
        else:
1063
 
            all_unversioned = deque()
 
1136
            all_unversioned = collections.deque()
1064
1137
        to_paths = {}
1065
1138
        from_entries_by_dir = list(self.source.iter_entries_by_dir(
1066
1139
            specific_file_ids=specific_file_ids))
1072
1145
        # the unversioned path lookup only occurs on real trees - where there
1073
1146
        # can be extras. So the fake_entry is solely used to look up
1074
1147
        # executable it values when execute is not supported.
1075
 
        fake_entry = InventoryFile('unused', 'unused', 'unused')
 
1148
        fake_entry = inventory.InventoryFile('unused', 'unused', 'unused')
1076
1149
        for target_path, target_entry in to_entries_by_dir:
1077
1150
            while (all_unversioned and
1078
1151
                all_unversioned[0][0] < target_path.split('/')):
1126
1199
            if file_id in to_paths:
1127
1200
                # already returned
1128
1201
                continue
1129
 
            if file_id not in self.target.all_file_ids():
 
1202
            if not self.target.has_id(file_id):
1130
1203
                # common case - paths we have not emitted are not present in
1131
1204
                # target.
1132
1205
                to_path = None
1251
1324
                    yield result
1252
1325
 
1253
1326
 
 
1327
InterTree.register_optimiser(InterTree)
 
1328
 
 
1329
 
1254
1330
class MultiWalker(object):
1255
1331
    """Walk multiple trees simultaneously, getting combined results."""
1256
1332