/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: John Arbash Meinel
  • Date: 2011-05-25 13:44:28 UTC
  • mto: This revision was merged to the branch mainline in revision 5913.
  • Revision ID: john@arbash-meinel.com-20110525134428-pgce0hkk91qhjyyc
Fix a test that was failing on win32.

You can't open() a file that is exclusive locked.
So just read it from the raw file.

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, stat_value=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
        :param stat_value: Optional stat value for the object
 
306
        """
 
307
        raise NotImplementedError(self.get_file_sha1)
 
308
 
308
309
    def get_file_mtime(self, file_id, path=None):
309
310
        """Return the modification time for a file.
310
311
 
324
325
        raise NotImplementedError(self.get_file_size)
325
326
 
326
327
    def get_file_by_path(self, path):
327
 
        return self.get_file(self._inventory.path2id(path), path)
 
328
        raise NotImplementedError(self.get_file_by_path)
 
329
 
 
330
    def is_executable(self, file_id, path=None):
 
331
        """Check if a file is executable.
 
332
 
 
333
        :param file_id: The handle for this file.
 
334
        :param path: The path that this file can be found at.
 
335
            These must point to the same object.
 
336
        """
 
337
        raise NotImplementedError(self.is_executable)
328
338
 
329
339
    def iter_files_bytes(self, desired_files):
330
340
        """Iterate through file contents.
352
362
            cur_file = (self.get_file_text(file_id),)
353
363
            yield identifier, cur_file
354
364
 
355
 
    def get_symlink_target(self, file_id):
 
365
    def get_symlink_target(self, file_id, path=None):
356
366
        """Get the target for a given file_id.
357
367
 
358
368
        It is assumed that the caller already knows that file_id is referencing
359
369
        a symlink.
360
370
        :param file_id: Handle for the symlink entry.
 
371
        :param path: The path of the file.
 
372
        If both file_id and path are supplied, an implementation may use
 
373
        either one.
361
374
        :return: The path the symlink points to.
362
375
        """
363
376
        raise NotImplementedError(self.get_symlink_target)
364
377
 
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
378
    def get_root_id(self):
444
379
        """Return the file_id for the root of this tree."""
445
380
        raise NotImplementedError(self.get_root_id)
503
438
            except errors.NoSuchRevisionInTree:
504
439
                yield self.repository.revision_tree(revision_id)
505
440
 
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
441
    def _get_file_revision(self, file_id, vf, tree_revision):
516
442
        """Ensure that file_id, tree_revision is in vf to plan the merge."""
517
443
 
518
444
        if getattr(self, '_repository', None) is None:
519
445
            last_revision = tree_revision
520
 
            parent_keys = [(file_id, self._file_revision(t, file_id)) for t in
 
446
            parent_keys = [(file_id, t.get_file_revision(file_id)) for t in
521
447
                self._iter_parent_trees()]
522
448
            vf.add_lines((file_id, last_revision), parent_keys,
523
 
                         self.get_file(file_id).readlines())
 
449
                         self.get_file_lines(file_id))
524
450
            repo = self.branch.repository
525
451
            base_vf = repo.texts
526
452
        else:
527
 
            last_revision = self._file_revision(self, file_id)
 
453
            last_revision = self.get_file_revision(file_id)
528
454
            base_vf = self._repository.texts
529
455
        if base_vf not in vf.fallback_versionedfiles:
530
456
            vf.fallback_versionedfiles.append(base_vf)
531
457
        return last_revision
532
458
 
533
 
    inventory = property(_get_inventory,
534
 
                         doc="Inventory of this Tree")
535
 
 
536
459
    def _check_retrieved(self, ie, f):
537
460
        if not __debug__:
538
461
            return
539
 
        fp = fingerprint_file(f)
 
462
        fp = osutils.fingerprint_file(f)
540
463
        f.seek(0)
541
464
 
542
465
        if ie.text_size is not None:
543
466
            if ie.text_size != fp['size']:
544
 
                raise BzrError("mismatched size for file %r in %r" % (ie.file_id, self._store),
 
467
                raise errors.BzrError(
 
468
                        "mismatched size for file %r in %r" %
 
469
                        (ie.file_id, self._store),
545
470
                        ["inventory expects %d bytes" % ie.text_size,
546
471
                         "file is actually %d bytes" % fp['size'],
547
472
                         "store is probably damaged/corrupt"])
548
473
 
549
474
        if ie.text_sha1 != fp['sha1']:
550
 
            raise BzrError("wrong SHA-1 for file %r in %r" % (ie.file_id, self._store),
 
475
            raise errors.BzrError("wrong SHA-1 for file %r in %r" %
 
476
                    (ie.file_id, self._store),
551
477
                    ["inventory expects %s" % ie.text_sha1,
552
478
                     "file is actually %s" % fp['sha1'],
553
479
                     "store is probably damaged/corrupt"])
554
480
 
555
 
    @needs_read_lock
556
481
    def path2id(self, path):
557
482
        """Return the id for path in this tree."""
558
 
        return self._inventory.path2id(path)
 
483
        raise NotImplementedError(self.path2id)
559
484
 
560
485
    def paths2ids(self, paths, trees=[], require_versioned=True):
561
486
        """Return all the ids that can be reached by walking from paths.
582
507
            yield child.file_id
583
508
 
584
509
    def lock_read(self):
 
510
        """Lock this tree for multiple read only operations.
 
511
        
 
512
        :return: A bzrlib.lock.LogicalLockResult.
 
513
        """
585
514
        pass
586
515
 
587
516
    def revision_tree(self, revision_id):
614
543
 
615
544
        :return: set of paths.
616
545
        """
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
        raise NotImplementedError(self.filter_unversioned_files)
622
547
 
623
548
    def walkdirs(self, prefix=""):
624
549
        """Walk the contents of this tree from path down.
676
601
        prefs = self.iter_search_rules([path], filter_pref_names).next()
677
602
        stk = filters._get_filter_stack_for(prefs)
678
603
        if 'filters' in debug.debug_flags:
679
 
            note("*** %s content-filter: %s => %r" % (path,prefs,stk))
 
604
            trace.note("*** %s content-filter: %s => %r" % (path,prefs,stk))
680
605
        return stk
681
606
 
682
607
    def _content_filter_stack_provider(self):
721
646
        return searcher
722
647
 
723
648
 
 
649
class InventoryTree(Tree):
 
650
    """A tree that relies on an inventory for its metadata.
 
651
 
 
652
    Trees contain an `Inventory` object, and also know how to retrieve
 
653
    file texts mentioned in the inventory, either from a working
 
654
    directory or from a store.
 
655
 
 
656
    It is possible for trees to contain files that are not described
 
657
    in their inventory or vice versa; for this use `filenames()`.
 
658
 
 
659
    Subclasses should set the _inventory attribute, which is considered
 
660
    private to external API users.
 
661
    """
 
662
 
 
663
    def get_canonical_inventory_paths(self, paths):
 
664
        """Like get_canonical_inventory_path() but works on multiple items.
 
665
 
 
666
        :param paths: A sequence of paths relative to the root of the tree.
 
667
        :return: A list of paths, with each item the corresponding input path
 
668
        adjusted to account for existing elements that match case
 
669
        insensitively.
 
670
        """
 
671
        return list(self._yield_canonical_inventory_paths(paths))
 
672
 
 
673
    def get_canonical_inventory_path(self, path):
 
674
        """Returns the first inventory item that case-insensitively matches path.
 
675
 
 
676
        If a path matches exactly, it is returned. If no path matches exactly
 
677
        but more than one path matches case-insensitively, it is implementation
 
678
        defined which is returned.
 
679
 
 
680
        If no path matches case-insensitively, the input path is returned, but
 
681
        with as many path entries that do exist changed to their canonical
 
682
        form.
 
683
 
 
684
        If you need to resolve many names from the same tree, you should
 
685
        use get_canonical_inventory_paths() to avoid O(N) behaviour.
 
686
 
 
687
        :param path: A paths relative to the root of the tree.
 
688
        :return: The input path adjusted to account for existing elements
 
689
        that match case insensitively.
 
690
        """
 
691
        return self._yield_canonical_inventory_paths([path]).next()
 
692
 
 
693
    def _yield_canonical_inventory_paths(self, paths):
 
694
        for path in paths:
 
695
            # First, if the path as specified exists exactly, just use it.
 
696
            if self.path2id(path) is not None:
 
697
                yield path
 
698
                continue
 
699
            # go walkin...
 
700
            cur_id = self.get_root_id()
 
701
            cur_path = ''
 
702
            bit_iter = iter(path.split("/"))
 
703
            for elt in bit_iter:
 
704
                lelt = elt.lower()
 
705
                new_path = None
 
706
                for child in self.iter_children(cur_id):
 
707
                    try:
 
708
                        # XXX: it seem like if the child is known to be in the
 
709
                        # tree, we shouldn't need to go from its id back to
 
710
                        # its path -- mbp 2010-02-11
 
711
                        #
 
712
                        # XXX: it seems like we could be more efficient
 
713
                        # by just directly looking up the original name and
 
714
                        # only then searching all children; also by not
 
715
                        # chopping paths so much. -- mbp 2010-02-11
 
716
                        child_base = os.path.basename(self.id2path(child))
 
717
                        if (child_base == elt):
 
718
                            # if we found an exact match, we can stop now; if
 
719
                            # we found an approximate match we need to keep
 
720
                            # searching because there might be an exact match
 
721
                            # later.  
 
722
                            cur_id = child
 
723
                            new_path = osutils.pathjoin(cur_path, child_base)
 
724
                            break
 
725
                        elif child_base.lower() == lelt:
 
726
                            cur_id = child
 
727
                            new_path = osutils.pathjoin(cur_path, child_base)
 
728
                    except errors.NoSuchId:
 
729
                        # before a change is committed we can see this error...
 
730
                        continue
 
731
                if new_path:
 
732
                    cur_path = new_path
 
733
                else:
 
734
                    # got to the end of this directory and no entries matched.
 
735
                    # Return what matched so far, plus the rest as specified.
 
736
                    cur_path = osutils.pathjoin(cur_path, elt, *list(bit_iter))
 
737
                    break
 
738
            yield cur_path
 
739
        # all done.
 
740
 
 
741
    def _get_inventory(self):
 
742
        return self._inventory
 
743
 
 
744
    inventory = property(_get_inventory,
 
745
                         doc="Inventory of this Tree")
 
746
 
 
747
    @needs_read_lock
 
748
    def path2id(self, path):
 
749
        """Return the id for path in this tree."""
 
750
        return self._inventory.path2id(path)
 
751
 
 
752
    def id2path(self, file_id):
 
753
        """Return the path for a file id.
 
754
 
 
755
        :raises NoSuchId:
 
756
        """
 
757
        return self.inventory.id2path(file_id)
 
758
 
 
759
    def has_id(self, file_id):
 
760
        return self.inventory.has_id(file_id)
 
761
 
 
762
    def has_or_had_id(self, file_id):
 
763
        return self.inventory.has_id(file_id)
 
764
 
 
765
    def all_file_ids(self):
 
766
        return set(self.inventory)
 
767
 
 
768
    @deprecated_method(deprecated_in((2, 4, 0)))
 
769
    def __iter__(self):
 
770
        return iter(self.inventory)
 
771
 
 
772
    def filter_unversioned_files(self, paths):
 
773
        """Filter out paths that are versioned.
 
774
 
 
775
        :return: set of paths.
 
776
        """
 
777
        # NB: we specifically *don't* call self.has_filename, because for
 
778
        # WorkingTrees that can indicate files that exist on disk but that
 
779
        # are not versioned.
 
780
        pred = self.inventory.has_filename
 
781
        return set((p for p in paths if not pred(p)))
 
782
 
 
783
    @needs_read_lock
 
784
    def iter_entries_by_dir(self, specific_file_ids=None, yield_parents=False):
 
785
        """Walk the tree in 'by_dir' order.
 
786
 
 
787
        This will yield each entry in the tree as a (path, entry) tuple.
 
788
        The order that they are yielded is:
 
789
 
 
790
        See Tree.iter_entries_by_dir for details.
 
791
 
 
792
        :param yield_parents: If True, yield the parents from the root leading
 
793
            down to specific_file_ids that have been requested. This has no
 
794
            impact if specific_file_ids is None.
 
795
        """
 
796
        return self.inventory.iter_entries_by_dir(
 
797
            specific_file_ids=specific_file_ids, yield_parents=yield_parents)
 
798
 
 
799
    def get_file_by_path(self, path):
 
800
        return self.get_file(self._inventory.path2id(path), path)
 
801
 
 
802
 
724
803
######################################################################
725
804
# diff
726
805
 
774
853
    return 'wtf?'
775
854
 
776
855
 
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
856
def find_ids_across_trees(filenames, trees, require_versioned=True):
789
857
    """Find the ids corresponding to specified filenames.
790
858
 
795
863
        None)
796
864
    :param trees: The trees to find file_ids within
797
865
    :param require_versioned: if true, all specified filenames must occur in
798
 
    at least one tree.
 
866
        at least one tree.
799
867
    :return: a set of file ids for the specified filenames and their children.
800
868
    """
801
869
    if not filenames:
877
945
 
878
946
    _optimisers = []
879
947
 
 
948
    @classmethod
 
949
    def is_compatible(kls, source, target):
 
950
        # The default implementation is naive and uses the public API, so
 
951
        # it works for all trees.
 
952
        return True
 
953
 
880
954
    def _changes_from_entries(self, source_entry, target_entry,
881
955
        source_path=None, target_path=None):
882
956
        """Generate a iter_changes tuple between source_entry and target_entry.
987
1061
            # All files are unversioned, so just return an empty delta
988
1062
            # _compare_trees would think we want a complete delta
989
1063
            result = delta.TreeDelta()
990
 
            fake_entry = InventoryFile('unused', 'unused', 'unused')
 
1064
            fake_entry = inventory.InventoryFile('unused', 'unused', 'unused')
991
1065
            result.unversioned = [(path, None,
992
1066
                self.target._comparison_data(fake_entry, path)[0]) for path in
993
1067
                specific_files]
1058
1132
                                     self.target.extras()
1059
1133
                if specific_files is None or
1060
1134
                    osutils.is_inside_any(specific_files, p)])
1061
 
            all_unversioned = deque(all_unversioned)
 
1135
            all_unversioned = collections.deque(all_unversioned)
1062
1136
        else:
1063
 
            all_unversioned = deque()
 
1137
            all_unversioned = collections.deque()
1064
1138
        to_paths = {}
1065
1139
        from_entries_by_dir = list(self.source.iter_entries_by_dir(
1066
1140
            specific_file_ids=specific_file_ids))
1072
1146
        # the unversioned path lookup only occurs on real trees - where there
1073
1147
        # can be extras. So the fake_entry is solely used to look up
1074
1148
        # executable it values when execute is not supported.
1075
 
        fake_entry = InventoryFile('unused', 'unused', 'unused')
 
1149
        fake_entry = inventory.InventoryFile('unused', 'unused', 'unused')
1076
1150
        for target_path, target_entry in to_entries_by_dir:
1077
1151
            while (all_unversioned and
1078
1152
                all_unversioned[0][0] < target_path.split('/')):
1126
1200
            if file_id in to_paths:
1127
1201
                # already returned
1128
1202
                continue
1129
 
            if file_id not in self.target.all_file_ids():
 
1203
            if not self.target.has_id(file_id):
1130
1204
                # common case - paths we have not emitted are not present in
1131
1205
                # target.
1132
1206
                to_path = None
1251
1325
                    yield result
1252
1326
 
1253
1327
 
 
1328
InterTree.register_optimiser(InterTree)
 
1329
 
 
1330
 
1254
1331
class MultiWalker(object):
1255
1332
    """Walk multiple trees simultaneously, getting combined results."""
1256
1333