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

  • Committer: Jelmer Vernooij
  • Date: 2017-09-01 07:15:43 UTC
  • mfrom: (6770.3.2 py3_test_cleanup)
  • Revision ID: jelmer@jelmer.uk-20170901071543-1t83321xkog9qrxh
Merge lp:~gz/brz/py3_test_cleanup

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
17
17
"""Tree classes, representing directory at point in time.
18
18
"""
19
19
 
 
20
from __future__ import absolute_import
 
21
 
20
22
import os
21
 
from collections import deque
22
 
 
23
 
import bzrlib
24
 
from bzrlib import (
 
23
 
 
24
from .lazy_import import lazy_import
 
25
lazy_import(globals(), """
 
26
import collections
 
27
 
 
28
from breezy import (
25
29
    conflicts as _mod_conflicts,
26
30
    debug,
27
31
    delta,
28
32
    filters,
29
 
    osutils,
30
33
    revision as _mod_revision,
31
34
    rules,
32
 
    )
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
37
 
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
 
35
    trace,
 
36
    )
 
37
from breezy.bzr import (
 
38
    inventory,
 
39
    )
 
40
from breezy.i18n import gettext
 
41
""")
 
42
 
 
43
from . import (
 
44
    errors,
 
45
    lock,
 
46
    osutils,
 
47
    )
 
48
from .decorators import needs_read_lock
 
49
from .inter import InterObject
 
50
from .sixish import (
 
51
    viewvalues,
 
52
    )
 
53
 
 
54
 
 
55
class FileTimestampUnavailable(errors.BzrError):
 
56
 
 
57
    _fmt = "The filestamp for %(path)s is not available."
 
58
 
 
59
    internal_error = True
 
60
 
 
61
    def __init__(self, path):
 
62
        self.path = path
41
63
 
42
64
 
43
65
class Tree(object):
49
71
 
50
72
    * `RevisionTree` is a tree as recorded at some point in the past.
51
73
 
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
74
    Trees can be compared, etc, regardless of whether they are working
60
75
    trees or versioned trees.
61
76
    """
62
77
 
 
78
    def has_versioned_directories(self):
 
79
        """Whether this tree can contain explicitly versioned directories.
 
80
 
 
81
        This defaults to True, but some implementations may want to override
 
82
        it.
 
83
        """
 
84
        return True
 
85
 
63
86
    def changes_from(self, other, want_unchanged=False, specific_files=None,
64
87
        extra_trees=None, require_versioned=False, include_root=False,
65
88
        want_unversioned=False):
105
128
    def conflicts(self):
106
129
        """Get a list of the conflicts in the tree.
107
130
 
108
 
        Each conflict is an instance of bzrlib.conflicts.Conflict.
 
131
        Each conflict is an instance of breezy.conflicts.Conflict.
109
132
        """
110
133
        return _mod_conflicts.ConflictList()
111
134
 
127
150
        raise NotImplementedError(self.has_filename)
128
151
 
129
152
    def has_id(self, file_id):
130
 
        return self.inventory.has_id(file_id)
131
 
 
132
 
    def __contains__(self, file_id):
133
 
        return self.has_id(file_id)
 
153
        raise NotImplementedError(self.has_id)
134
154
 
135
155
    def has_or_had_id(self, file_id):
136
 
        return self.inventory.has_id(file_id)
 
156
        raise NotImplementedError(self.has_or_had_id)
137
157
 
138
158
    def is_ignored(self, filename):
139
159
        """Check whether the filename is ignored by this tree.
143
163
        """
144
164
        return False
145
165
 
146
 
    def __iter__(self):
147
 
        return iter(self.inventory)
148
 
 
149
166
    def all_file_ids(self):
150
167
        """Iterate through all file ids, including ids for missing files."""
151
 
        return set(self.inventory)
 
168
        raise NotImplementedError(self.all_file_ids)
152
169
 
153
170
    def id2path(self, file_id):
154
171
        """Return the path for a file id.
155
172
 
156
173
        :raises NoSuchId:
157
174
        """
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
 
175
        raise NotImplementedError(self.id2path)
 
176
 
173
177
    def iter_entries_by_dir(self, specific_file_ids=None, yield_parents=False):
174
178
        """Walk the tree in 'by_dir' order.
175
179
 
192
196
             g
193
197
 
194
198
        The yield order (ignoring root) would be::
 
199
 
195
200
          a, f, a/b, a/d, a/b/c, a/d/e, f/g
196
201
 
197
202
        :param yield_parents: If True, yield the parents from the root leading
198
203
            down to specific_file_ids that have been requested. This has no
199
204
            impact if specific_file_ids is None.
200
205
        """
201
 
        return self.inventory.iter_entries_by_dir(
202
 
            specific_file_ids=specific_file_ids, yield_parents=yield_parents)
 
206
        raise NotImplementedError(self.iter_entries_by_dir)
 
207
 
 
208
    def iter_child_entries(self, file_id, path=None):
 
209
        """Iterate over the children of a directory or tree reference.
 
210
 
 
211
        :param file_id: File id of the directory/tree-reference
 
212
        :param path: Optional path of the directory
 
213
        :raise NoSuchId: When the file_id does not exist
 
214
        :return: Iterator over entries in the directory
 
215
        """
 
216
        raise NotImplementedError(self.iter_child_entries)
 
217
 
 
218
    def list_files(self, include_root=False, from_dir=None, recursive=True):
 
219
        """List all files in this tree.
 
220
 
 
221
        :param include_root: Whether to include the entry for the tree root
 
222
        :param from_dir: Directory under which to list files
 
223
        :param recursive: Whether to list files recursively
 
224
        :return: iterator over tuples of (path, versioned, kind, file_id,
 
225
            inventory entry)
 
226
        """
 
227
        raise NotImplementedError(self.list_files)
203
228
 
204
229
    def iter_references(self):
205
230
        if self.supports_tree_reference():
256
281
    def _file_size(self, entry, stat_value):
257
282
        raise NotImplementedError(self._file_size)
258
283
 
259
 
    def _get_inventory(self):
260
 
        return self._inventory
261
 
 
262
284
    def get_file(self, file_id, path=None):
263
285
        """Return a file object for the file file_id in the tree.
264
286
 
286
308
 
287
309
        :param file_id: The file_id of the file.
288
310
        :param path: The path of the file.
 
311
 
289
312
        If both file_id and path are supplied, an implementation may use
290
313
        either one.
 
314
 
 
315
        :returns: A single byte string for the whole file.
291
316
        """
292
317
        my_file = self.get_file(file_id, path)
293
318
        try:
300
325
 
301
326
        :param file_id: The file_id of the file.
302
327
        :param path: The path of the file.
 
328
 
303
329
        If both file_id and path are supplied, an implementation may use
304
330
        either one.
305
331
        """
306
332
        return osutils.split_lines(self.get_file_text(file_id, path))
307
333
 
 
334
    def get_file_verifier(self, file_id, path=None, stat_value=None):
 
335
        """Return a verifier for a file.
 
336
 
 
337
        The default implementation returns a sha1.
 
338
 
 
339
        :param file_id: The handle for this file.
 
340
        :param path: The path that this file can be found at.
 
341
            These must point to the same object.
 
342
        :param stat_value: Optional stat value for the object
 
343
        :return: Tuple with verifier name and verifier data
 
344
        """
 
345
        return ("SHA1", self.get_file_sha1(file_id, path=path,
 
346
            stat_value=stat_value))
 
347
 
 
348
    def get_file_sha1(self, file_id, path=None, stat_value=None):
 
349
        """Return the SHA1 file for a file.
 
350
 
 
351
        :note: callers should use get_file_verifier instead
 
352
            where possible, as the underlying repository implementation may
 
353
            have quicker access to a non-sha1 verifier.
 
354
 
 
355
        :param file_id: The handle for this file.
 
356
        :param path: The path that this file can be found at.
 
357
            These must point to the same object.
 
358
        :param stat_value: Optional stat value for the object
 
359
        """
 
360
        raise NotImplementedError(self.get_file_sha1)
 
361
 
308
362
    def get_file_mtime(self, file_id, path=None):
309
363
        """Return the modification time for a file.
310
364
 
323
377
        """
324
378
        raise NotImplementedError(self.get_file_size)
325
379
 
326
 
    def get_file_by_path(self, path):
327
 
        return self.get_file(self._inventory.path2id(path), path)
 
380
    def is_executable(self, file_id, path=None):
 
381
        """Check if a file is executable.
 
382
 
 
383
        :param file_id: The handle for this file.
 
384
        :param path: The path that this file can be found at.
 
385
            These must point to the same object.
 
386
        """
 
387
        raise NotImplementedError(self.is_executable)
328
388
 
329
389
    def iter_files_bytes(self, desired_files):
330
390
        """Iterate through file contents.
352
412
            cur_file = (self.get_file_text(file_id),)
353
413
            yield identifier, cur_file
354
414
 
355
 
    def get_symlink_target(self, file_id):
 
415
    def get_symlink_target(self, file_id, path=None):
356
416
        """Get the target for a given file_id.
357
417
 
358
418
        It is assumed that the caller already knows that file_id is referencing
359
419
        a symlink.
360
420
        :param file_id: Handle for the symlink entry.
 
421
        :param path: The path of the file.
 
422
        If both file_id and path are supplied, an implementation may use
 
423
        either one.
361
424
        :return: The path the symlink points to.
362
425
        """
363
426
        raise NotImplementedError(self.get_symlink_target)
364
427
 
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
428
    def get_root_id(self):
444
429
        """Return the file_id for the root of this tree."""
445
430
        raise NotImplementedError(self.get_root_id)
459
444
        raise NotImplementedError(self.annotate_iter)
460
445
 
461
446
    def _get_plan_merge_data(self, file_id, other, base):
462
 
        from bzrlib import versionedfile
 
447
        from .bzr import versionedfile
463
448
        vf = versionedfile._PlanMergeVersionedFile(file_id)
464
449
        last_revision_a = self._get_file_revision(file_id, vf, 'this:')
465
450
        last_revision_b = other._get_file_revision(file_id, vf, 'other:')
503
488
            except errors.NoSuchRevisionInTree:
504
489
                yield self.repository.revision_tree(revision_id)
505
490
 
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
491
    def _get_file_revision(self, file_id, vf, tree_revision):
516
492
        """Ensure that file_id, tree_revision is in vf to plan the merge."""
517
493
 
518
494
        if getattr(self, '_repository', None) is None:
519
495
            last_revision = tree_revision
520
 
            parent_keys = [(file_id, self._file_revision(t, file_id)) for t in
 
496
            parent_keys = [(file_id, t.get_file_revision(file_id)) for t in
521
497
                self._iter_parent_trees()]
522
498
            vf.add_lines((file_id, last_revision), parent_keys,
523
 
                         self.get_file(file_id).readlines())
 
499
                         self.get_file_lines(file_id))
524
500
            repo = self.branch.repository
525
501
            base_vf = repo.texts
526
502
        else:
527
 
            last_revision = self._file_revision(self, file_id)
 
503
            last_revision = self.get_file_revision(file_id)
528
504
            base_vf = self._repository.texts
529
505
        if base_vf not in vf.fallback_versionedfiles:
530
506
            vf.fallback_versionedfiles.append(base_vf)
531
507
        return last_revision
532
508
 
533
 
    inventory = property(_get_inventory,
534
 
                         doc="Inventory of this Tree")
535
 
 
536
509
    def _check_retrieved(self, ie, f):
537
510
        if not __debug__:
538
511
            return
539
 
        fp = fingerprint_file(f)
 
512
        fp = osutils.fingerprint_file(f)
540
513
        f.seek(0)
541
514
 
542
515
        if ie.text_size is not None:
543
516
            if ie.text_size != fp['size']:
544
 
                raise BzrError("mismatched size for file %r in %r" % (ie.file_id, self._store),
 
517
                raise errors.BzrError(
 
518
                        "mismatched size for file %r in %r" %
 
519
                        (ie.file_id, self._store),
545
520
                        ["inventory expects %d bytes" % ie.text_size,
546
521
                         "file is actually %d bytes" % fp['size'],
547
522
                         "store is probably damaged/corrupt"])
548
523
 
549
524
        if ie.text_sha1 != fp['sha1']:
550
 
            raise BzrError("wrong SHA-1 for file %r in %r" % (ie.file_id, self._store),
 
525
            raise errors.BzrError("wrong SHA-1 for file %r in %r" %
 
526
                    (ie.file_id, self._store),
551
527
                    ["inventory expects %s" % ie.text_sha1,
552
528
                     "file is actually %s" % fp['sha1'],
553
529
                     "store is probably damaged/corrupt"])
554
530
 
555
 
    @needs_read_lock
556
531
    def path2id(self, path):
557
532
        """Return the id for path in this tree."""
558
 
        return self._inventory.path2id(path)
 
533
        raise NotImplementedError(self.path2id)
559
534
 
560
535
    def paths2ids(self, paths, trees=[], require_versioned=True):
561
536
        """Return all the ids that can be reached by walking from paths.
577
552
        return find_ids_across_trees(paths, [self] + list(trees), require_versioned)
578
553
 
579
554
    def iter_children(self, file_id):
580
 
        entry = self.iter_entries_by_dir([file_id]).next()[1]
581
 
        for child in getattr(entry, 'children', {}).itervalues():
582
 
            yield child.file_id
 
555
        """Iterate over the file ids of the children of an entry.
 
556
 
 
557
        :param file_id: File id of the entry
 
558
        :return: Iterator over child file ids.
 
559
        """
 
560
        raise NotImplementedError(self.iter_children)
583
561
 
584
562
    def lock_read(self):
585
 
        pass
 
563
        """Lock this tree for multiple read only operations.
 
564
 
 
565
        :return: A breezy.lock.LogicalLockResult.
 
566
        """
 
567
        return lock.LogicalLockResult(self.unlock)
586
568
 
587
569
    def revision_tree(self, revision_id):
588
570
        """Obtain a revision tree for the revision revision_id.
614
596
 
615
597
        :return: set of paths.
616
598
        """
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)))
 
599
        raise NotImplementedError(self.filter_unversioned_files)
622
600
 
623
601
    def walkdirs(self, prefix=""):
624
602
        """Walk the contents of this tree from path down.
673
651
            return []
674
652
        if path is None:
675
653
            path = self.id2path(file_id)
676
 
        prefs = self.iter_search_rules([path], filter_pref_names).next()
 
654
        prefs = next(self.iter_search_rules([path], filter_pref_names))
677
655
        stk = filters._get_filter_stack_for(prefs)
678
656
        if 'filters' in debug.debug_flags:
679
 
            note("*** %s content-filter: %s => %r" % (path,prefs,stk))
 
657
            trace.note(gettext("*** {0} content-filter: {1} => {2!r}").format(path,prefs,stk))
680
658
        return stk
681
659
 
682
660
    def _content_filter_stack_provider(self):
721
699
        return searcher
722
700
 
723
701
 
724
 
######################################################################
725
 
# diff
726
 
 
727
 
# TODO: Merge these two functions into a single one that can operate
728
 
# on either a whole tree or a set of files.
729
 
 
730
 
# TODO: Return the diff in order by filename, not by category or in
731
 
# random order.  Can probably be done by lock-stepping through the
732
 
# filenames from both trees.
733
 
 
734
 
 
735
 
def file_status(filename, old_tree, new_tree):
736
 
    """Return single-letter status, old and new names for a file.
737
 
 
738
 
    The complexity here is in deciding how to represent renames;
739
 
    many complex cases are possible.
740
 
    """
741
 
    old_inv = old_tree.inventory
742
 
    new_inv = new_tree.inventory
743
 
    new_id = new_inv.path2id(filename)
744
 
    old_id = old_inv.path2id(filename)
745
 
 
746
 
    if not new_id and not old_id:
747
 
        # easy: doesn't exist in either; not versioned at all
748
 
        if new_tree.is_ignored(filename):
749
 
            return 'I', None, None
750
 
        else:
751
 
            return '?', None, None
752
 
    elif new_id:
753
 
        # There is now a file of this name, great.
754
 
        pass
755
 
    else:
756
 
        # There is no longer a file of this name, but we can describe
757
 
        # what happened to the file that used to have
758
 
        # this name.  There are two possibilities: either it was
759
 
        # deleted entirely, or renamed.
760
 
        if new_inv.has_id(old_id):
761
 
            return 'X', old_inv.id2path(old_id), new_inv.id2path(old_id)
762
 
        else:
763
 
            return 'D', old_inv.id2path(old_id), None
764
 
 
765
 
    # if the file_id is new in this revision, it is added
766
 
    if new_id and not old_inv.has_id(new_id):
767
 
        return 'A'
768
 
 
769
 
    # if there used to be a file of this name, but that ID has now
770
 
    # disappeared, it is deleted
771
 
    if old_id and not new_inv.has_id(old_id):
772
 
        return 'D'
773
 
 
774
 
    return 'wtf?'
775
 
 
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
 
 
788
702
def find_ids_across_trees(filenames, trees, require_versioned=True):
789
703
    """Find the ids corresponding to specified filenames.
790
704
 
795
709
        None)
796
710
    :param trees: The trees to find file_ids within
797
711
    :param require_versioned: if true, all specified filenames must occur in
798
 
    at least one tree.
 
712
        at least one tree.
799
713
    :return: a set of file ids for the specified filenames and their children.
800
714
    """
801
715
    if not filenames:
864
778
    Its instances have methods like 'compare' and contain references to the
865
779
    source and target trees these operations are to be carried out on.
866
780
 
867
 
    Clients of bzrlib should not need to use InterTree directly, rather they
 
781
    Clients of breezy should not need to use InterTree directly, rather they
868
782
    should use the convenience methods on Tree such as 'Tree.compare()' which
869
783
    will pass through to InterTree as appropriate.
870
784
    """
877
791
 
878
792
    _optimisers = []
879
793
 
 
794
    @classmethod
 
795
    def is_compatible(kls, source, target):
 
796
        # The default implementation is naive and uses the public API, so
 
797
        # it works for all trees.
 
798
        return True
 
799
 
880
800
    def _changes_from_entries(self, source_entry, target_entry,
881
801
        source_path=None, target_path=None):
882
802
        """Generate a iter_changes tuple between source_entry and target_entry.
930
850
        if source_kind != target_kind:
931
851
            changed_content = True
932
852
        elif source_kind == 'file':
933
 
            if (self.source.get_file_sha1(file_id, source_path, source_stat) !=
934
 
                self.target.get_file_sha1(file_id, target_path, target_stat)):
 
853
            if not self.file_content_matches(file_id, file_id, source_path,
 
854
                    target_path, source_stat, target_stat):
935
855
                changed_content = True
936
856
        elif source_kind == 'symlink':
937
857
            if (self.source.get_symlink_target(file_id) !=
938
858
                self.target.get_symlink_target(file_id)):
939
859
                changed_content = True
940
 
            # XXX: Yes, the indentation below is wrong. But fixing it broke
941
 
            # test_merge.TestMergerEntriesLCAOnDisk.
942
 
            # test_nested_tree_subtree_renamed_and_modified. We'll wait for
943
 
            # the fix from bzr.dev -- vila 2009026
944
 
            elif source_kind == 'tree-reference':
945
 
                if (self.source.get_reference_revision(file_id, source_path)
946
 
                    != self.target.get_reference_revision(file_id, target_path)):
 
860
        elif source_kind == 'tree-reference':
 
861
            if (self.source.get_reference_revision(file_id, source_path)
 
862
                != self.target.get_reference_revision(file_id, target_path)):
947
863
                    changed_content = True
948
864
        parent = (source_parent, target_parent)
949
865
        name = (source_name, target_name)
987
903
            # All files are unversioned, so just return an empty delta
988
904
            # _compare_trees would think we want a complete delta
989
905
            result = delta.TreeDelta()
990
 
            fake_entry = InventoryFile('unused', 'unused', 'unused')
 
906
            fake_entry = inventory.InventoryFile('unused', 'unused', 'unused')
991
907
            result.unversioned = [(path, None,
992
908
                self.target._comparison_data(fake_entry, path)[0]) for path in
993
909
                specific_files]
1058
974
                                     self.target.extras()
1059
975
                if specific_files is None or
1060
976
                    osutils.is_inside_any(specific_files, p)])
1061
 
            all_unversioned = deque(all_unversioned)
 
977
            all_unversioned = collections.deque(all_unversioned)
1062
978
        else:
1063
 
            all_unversioned = deque()
 
979
            all_unversioned = collections.deque()
1064
980
        to_paths = {}
1065
981
        from_entries_by_dir = list(self.source.iter_entries_by_dir(
1066
982
            specific_file_ids=specific_file_ids))
1072
988
        # the unversioned path lookup only occurs on real trees - where there
1073
989
        # can be extras. So the fake_entry is solely used to look up
1074
990
        # executable it values when execute is not supported.
1075
 
        fake_entry = InventoryFile('unused', 'unused', 'unused')
 
991
        fake_entry = inventory.InventoryFile('unused', 'unused', 'unused')
1076
992
        for target_path, target_entry in to_entries_by_dir:
1077
993
            while (all_unversioned and
1078
994
                all_unversioned[0][0] < target_path.split('/')):
1126
1042
            if file_id in to_paths:
1127
1043
                # already returned
1128
1044
                continue
1129
 
            if file_id not in self.target.all_file_ids():
 
1045
            if not self.target.has_id(file_id):
1130
1046
                # common case - paths we have not emitted are not present in
1131
1047
                # target.
1132
1048
                to_path = None
1164
1080
        :param file_id: The file_id to lookup.
1165
1081
        """
1166
1082
        try:
1167
 
            inventory = tree.inventory
 
1083
            inventory = tree.root_inventory
1168
1084
        except NotImplementedError:
1169
1085
            # No inventory available.
1170
1086
            try:
1245
1161
                        if old_entry is None:
1246
1162
                            # Reusing a discarded change.
1247
1163
                            old_entry = self._get_entry(self.source, file_id)
1248
 
                        for child in old_entry.children.values():
1249
 
                            precise_file_ids.add(child.file_id)
 
1164
                        precise_file_ids.update(
 
1165
                                self.source.iter_children(file_id))
1250
1166
                    changed_file_ids.add(result[0])
1251
1167
                    yield result
1252
1168
 
 
1169
    @needs_read_lock
 
1170
    def file_content_matches(self, source_file_id, target_file_id,
 
1171
            source_path=None, target_path=None, source_stat=None, target_stat=None):
 
1172
        """Check if two files are the same in the source and target trees.
 
1173
 
 
1174
        This only checks that the contents of the files are the same,
 
1175
        it does not touch anything else.
 
1176
 
 
1177
        :param source_file_id: File id of the file in the source tree
 
1178
        :param target_file_id: File id of the file in the target tree
 
1179
        :param source_path: Path of the file in the source tree
 
1180
        :param target_path: Path of the file in the target tree
 
1181
        :param source_stat: Optional stat value of the file in the source tree
 
1182
        :param target_stat: Optional stat value of the file in the target tree
 
1183
        :return: Boolean indicating whether the files have the same contents
 
1184
        """
 
1185
        source_verifier_kind, source_verifier_data = self.source.get_file_verifier(
 
1186
            source_file_id, source_path, source_stat)
 
1187
        target_verifier_kind, target_verifier_data = self.target.get_file_verifier(
 
1188
            target_file_id, target_path, target_stat)
 
1189
        if source_verifier_kind == target_verifier_kind:
 
1190
            return (source_verifier_data == target_verifier_data)
 
1191
        # Fall back to SHA1 for now
 
1192
        if source_verifier_kind != "SHA1":
 
1193
            source_sha1 = self.source.get_file_sha1(source_file_id,
 
1194
                    source_path, source_stat)
 
1195
        else:
 
1196
            source_sha1 = source_verifier_data
 
1197
        if target_verifier_kind != "SHA1":
 
1198
            target_sha1 = self.target.get_file_sha1(target_file_id,
 
1199
                    target_path, target_stat)
 
1200
        else:
 
1201
            target_sha1 = target_verifier_data
 
1202
        return (source_sha1 == target_sha1)
 
1203
 
 
1204
InterTree.register_optimiser(InterTree)
 
1205
 
1253
1206
 
1254
1207
class MultiWalker(object):
1255
1208
    """Walk multiple trees simultaneously, getting combined results."""
1289
1242
            If has_more is False, path and ie will be None.
1290
1243
        """
1291
1244
        try:
1292
 
            path, ie = iterator.next()
 
1245
            path, ie = next(iterator)
1293
1246
        except StopIteration:
1294
1247
            return False, None, None
1295
1248
        else:
1358
1311
            return (None, None)
1359
1312
        else:
1360
1313
            self._out_of_order_processed.add(file_id)
1361
 
            cur_ie = other_tree.inventory[file_id]
 
1314
            cur_ie = other_tree.root_inventory[file_id]
1362
1315
            return (cur_path, cur_ie)
1363
1316
 
1364
1317
    def iter_all(self):
1385
1338
                         for other in self._other_trees]
1386
1339
        other_entries = [self._step_one(walker) for walker in other_walkers]
1387
1340
        # Track extra nodes in the other trees
1388
 
        others_extra = [{} for i in xrange(len(self._other_trees))]
 
1341
        others_extra = [{} for _ in range(len(self._other_trees))]
1389
1342
 
1390
1343
        master_has_more = True
1391
1344
        step_one = self._step_one
1465
1418
        #       might ensure better ordering, in case a caller strictly
1466
1419
        #       requires parents before children.
1467
1420
        for idx, other_extra in enumerate(self._others_extra):
1468
 
            others = sorted(other_extra.itervalues(),
 
1421
            others = sorted(viewvalues(other_extra),
1469
1422
                            key=lambda x: self._path_to_key(x[0]))
1470
1423
            for other_path, other_ie in others:
1471
1424
                file_id = other_ie.file_id
1473
1426
                # the lookup_by_file_id will be removing anything processed
1474
1427
                # from the extras cache
1475
1428
                other_extra.pop(file_id)
1476
 
                other_values = [(None, None) for i in xrange(idx)]
 
1429
                other_values = [(None, None)] * idx
1477
1430
                other_values.append((other_path, other_ie))
1478
1431
                for alt_idx, alt_extra in enumerate(self._others_extra[idx+1:]):
1479
1432
                    alt_idx = alt_idx + idx + 1