~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/merge.py

  • Committer: Canonical.com Patch Queue Manager
  • Date: 2008-06-05 04:05:05 UTC
  • mfrom: (3473.1.1 ianc-integration)
  • Revision ID: pqm@pqm.ubuntu.com-20080605040505-i9kqxg2fps2qjdi0
Add the 'alias' command (Tim Penhey)

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2005, 2006, 2008 Canonical Ltd
 
1
# Copyright (C) 2005, 2006 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
15
15
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
16
16
 
17
17
 
 
18
import os
18
19
import errno
19
 
from itertools import chain
20
 
import os
21
20
import warnings
22
21
 
23
22
from bzrlib import (
24
23
    debug,
25
24
    errors,
26
 
    graph as _mod_graph,
27
25
    osutils,
28
26
    patiencediff,
29
27
    registry,
30
28
    revision as _mod_revision,
31
 
    tree as _mod_tree,
32
 
    tsort,
33
29
    )
34
30
from bzrlib.branch import Branch
35
31
from bzrlib.conflicts import ConflictList, Conflict
46
42
                           WorkingTreeNotRevision,
47
43
                           BinaryFile,
48
44
                           )
49
 
from bzrlib.graph import Graph
50
45
from bzrlib.merge3 import Merge3
51
46
from bzrlib.osutils import rename, pathjoin
52
47
from progress import DummyProgress, ProgressPhase
55
50
from bzrlib.trace import mutter, warning, note, is_quiet
56
51
from bzrlib.transform import (TransformPreview, TreeTransform,
57
52
                              resolve_conflicts, cook_conflicts,
58
 
                              conflict_pass, FinalPaths, create_from_tree,
 
53
                              conflict_pass, FinalPaths, create_by_entry,
59
54
                              unique_add, ROOT_PARENT)
60
55
from bzrlib.versionedfile import PlanWeaveMerge
61
56
from bzrlib import ui
97
92
        self._revision_graph = revision_graph
98
93
        self._base_is_ancestor = None
99
94
        self._base_is_other_ancestor = None
100
 
        self._is_criss_cross = None
101
 
        self._lca_trees = None
102
95
 
103
96
    @property
104
97
    def revision_graph(self):
132
125
                                      _set_base_is_other_ancestor)
133
126
 
134
127
    @staticmethod
135
 
    def from_uncommitted(tree, other_tree, pb, base_tree=None):
 
128
    def from_uncommitted(tree, other_tree, pb):
136
129
        """Return a Merger for uncommitted changes in other_tree.
137
130
 
138
131
        :param tree: The tree to merge into
139
132
        :param other_tree: The tree to get uncommitted changes from
140
133
        :param pb: A progress indicator
141
 
        :param base_tree: The basis to use for the merge.  If unspecified,
142
 
            other_tree.basis_tree() will be used.
143
134
        """
144
 
        if base_tree is None:
145
 
            base_tree = other_tree.basis_tree()
146
 
        merger = Merger(tree.branch, other_tree, base_tree, tree, pb)
 
135
        merger = Merger(tree.branch, other_tree, other_tree.basis_tree(), tree,
 
136
                        pb)
147
137
        merger.base_rev_id = merger.base_tree.get_revision_id()
148
138
        merger.other_rev_id = None
149
139
        merger.other_basis = merger.base_rev_id
175
165
 
176
166
    @staticmethod
177
167
    def from_revision_ids(pb, tree, other, base=None, other_branch=None,
178
 
                          base_branch=None, revision_graph=None,
179
 
                          tree_branch=None):
 
168
                          base_branch=None, revision_graph=None):
180
169
        """Return a Merger for revision-ids.
181
170
 
182
 
        :param pb: A progress indicator
183
171
        :param tree: The tree to merge changes into
184
172
        :param other: The revision-id to use as OTHER
185
173
        :param base: The revision-id to use as BASE.  If not specified, will
190
178
            not supplied, other_branch or tree.branch will be used.
191
179
        :param revision_graph: If you have a revision_graph precomputed, pass
192
180
            it in, otherwise it will be created for you.
193
 
        :param tree_branch: The branch associated with tree.  If not supplied,
194
 
            tree.branch will be used.
 
181
        :param pb: A progress indicator
195
182
        """
196
 
        if tree_branch is None:
197
 
            tree_branch = tree.branch
198
 
        merger = Merger(tree_branch, this_tree=tree, pb=pb,
 
183
        merger = Merger(tree.branch, this_tree=tree, pb=pb,
199
184
                        revision_graph=revision_graph)
200
185
        if other_branch is None:
201
186
            other_branch = tree.branch
272
257
    def compare_basis(self):
273
258
        try:
274
259
            basis_tree = self.revision_tree(self.this_tree.last_revision())
275
 
        except errors.NoSuchRevision:
 
260
        except errors.RevisionNotPresent:
276
261
            basis_tree = self.this_tree.basis_tree()
277
262
        changes = self.this_tree.changes_from(basis_tree)
278
263
        if not changes.has_changed():
292
277
        for revision_id in new_parents:
293
278
            try:
294
279
                tree = self.revision_tree(revision_id)
295
 
            except errors.NoSuchRevision:
 
280
            except errors.RevisionNotPresent:
296
281
                tree = None
297
282
            else:
298
283
                tree.lock_read()
364
349
                     ensure_null(self.other_basis)]
365
350
        if NULL_REVISION in revisions:
366
351
            self.base_rev_id = NULL_REVISION
367
 
            self.base_tree = self.revision_tree(self.base_rev_id)
368
 
            self._is_criss_cross = False
369
352
        else:
370
 
            lcas = self.revision_graph.find_lca(revisions[0], revisions[1])
371
 
            self._is_criss_cross = False
372
 
            if len(lcas) == 0:
373
 
                self.base_rev_id = NULL_REVISION
374
 
            elif len(lcas) == 1:
375
 
                self.base_rev_id = list(lcas)[0]
376
 
            else: # len(lcas) > 1
377
 
                if len(lcas) > 2:
378
 
                    # find_unique_lca can only handle 2 nodes, so we have to
379
 
                    # start back at the beginning. It is a shame to traverse
380
 
                    # the graph again, but better than re-implementing
381
 
                    # find_unique_lca.
382
 
                    self.base_rev_id = self.revision_graph.find_unique_lca(
383
 
                                            revisions[0], revisions[1])
384
 
                else:
385
 
                    self.base_rev_id = self.revision_graph.find_unique_lca(
386
 
                                            *lcas)
387
 
                self._is_criss_cross = True
 
353
            self.base_rev_id, steps = self.revision_graph.find_unique_lca(
 
354
                revisions[0], revisions[1], count_steps=True)
388
355
            if self.base_rev_id == NULL_REVISION:
389
356
                raise UnrelatedBranches()
390
 
            if self._is_criss_cross:
 
357
            if steps > 1:
391
358
                warning('Warning: criss-cross merge encountered.  See bzr'
392
359
                        ' help criss-cross.')
393
 
                interesting_revision_ids = [self.base_rev_id]
394
 
                interesting_revision_ids.extend(lcas)
395
 
                interesting_trees = dict((t.get_revision_id(), t)
396
 
                    for t in self.this_branch.repository.revision_trees(
397
 
                        interesting_revision_ids))
398
 
                self._cached_trees.update(interesting_trees)
399
 
                self.base_tree = interesting_trees.pop(self.base_rev_id)
400
 
                sorted_lca_keys = self.revision_graph.find_merge_order(
401
 
                    revisions[0], lcas)
402
 
                self._lca_trees = [interesting_trees[key]
403
 
                                   for key in sorted_lca_keys]
404
 
            else:
405
 
                self.base_tree = self.revision_tree(self.base_rev_id)
 
360
        self.base_tree = self.revision_tree(self.base_rev_id)
406
361
        self.base_is_ancestor = True
407
362
        self.base_is_other_ancestor = True
408
363
 
450
405
        if self.merge_type.supports_cherrypick:
451
406
            kwargs['cherrypick'] = (not self.base_is_ancestor or
452
407
                                    not self.base_is_other_ancestor)
453
 
        if self._is_criss_cross and getattr(self.merge_type,
454
 
                                            'supports_lca_trees', False):
455
 
            kwargs['lca_trees'] = self._lca_trees
456
408
        return self.merge_type(pb=self._pb,
457
409
                               change_reporter=self.change_reporter,
458
410
                               **kwargs)
459
411
 
460
 
    def _do_merge_to(self, merge):
461
 
        merge.do_merge()
462
 
        if self.recurse == 'down':
463
 
            for relpath, file_id in self.this_tree.iter_references():
464
 
                sub_tree = self.this_tree.get_nested_tree(file_id, relpath)
465
 
                other_revision = self.other_tree.get_reference_revision(
466
 
                    file_id, relpath)
467
 
                if  other_revision == sub_tree.last_revision():
468
 
                    continue
469
 
                sub_merge = Merger(sub_tree.branch, this_tree=sub_tree)
470
 
                sub_merge.merge_type = self.merge_type
471
 
                other_branch = self.other_branch.reference_parent(file_id, relpath)
472
 
                sub_merge.set_other_revision(other_revision, other_branch)
473
 
                base_revision = self.base_tree.get_reference_revision(file_id)
474
 
                sub_merge.base_tree = \
475
 
                    sub_tree.branch.repository.revision_tree(base_revision)
476
 
                sub_merge.base_rev_id = base_revision
477
 
                sub_merge.do_merge()
478
 
        
479
412
    def do_merge(self):
480
413
        self.this_tree.lock_tree_write()
 
414
        if self.base_tree is not None:
 
415
            self.base_tree.lock_read()
 
416
        if self.other_tree is not None:
 
417
            self.other_tree.lock_read()
481
418
        try:
 
419
            merge = self.make_merger()
 
420
            merge.do_merge()
 
421
            if self.recurse == 'down':
 
422
                for path, file_id in self.this_tree.iter_references():
 
423
                    sub_tree = self.this_tree.get_nested_tree(file_id, path)
 
424
                    other_revision = self.other_tree.get_reference_revision(
 
425
                        file_id, path)
 
426
                    if  other_revision == sub_tree.last_revision():
 
427
                        continue
 
428
                    sub_merge = Merger(sub_tree.branch, this_tree=sub_tree)
 
429
                    sub_merge.merge_type = self.merge_type
 
430
                    relpath = self.this_tree.relpath(path)
 
431
                    other_branch = self.other_branch.reference_parent(file_id, relpath)
 
432
                    sub_merge.set_other_revision(other_revision, other_branch)
 
433
                    base_revision = self.base_tree.get_reference_revision(file_id)
 
434
                    sub_merge.base_tree = \
 
435
                        sub_tree.branch.repository.revision_tree(base_revision)
 
436
                    sub_merge.base_rev_id = base_revision
 
437
                    sub_merge.do_merge()
 
438
 
 
439
        finally:
 
440
            if self.other_tree is not None:
 
441
                self.other_tree.unlock()
482
442
            if self.base_tree is not None:
483
 
                self.base_tree.lock_read()
484
 
            try:
485
 
                if self.other_tree is not None:
486
 
                    self.other_tree.lock_read()
487
 
                try:
488
 
                    merge = self.make_merger()
489
 
                    self._do_merge_to(merge)
490
 
                finally:
491
 
                    if self.other_tree is not None:
492
 
                        self.other_tree.unlock()
493
 
            finally:
494
 
                if self.base_tree is not None:
495
 
                    self.base_tree.unlock()
496
 
        finally:
 
443
                self.base_tree.unlock()
497
444
            self.this_tree.unlock()
498
445
        if len(merge.cooked_conflicts) == 0:
499
446
            if not self.ignore_zero and not is_quiet():
504
451
        return len(merge.cooked_conflicts)
505
452
 
506
453
 
507
 
class _InventoryNoneEntry(object):
508
 
    """This represents an inventory entry which *isn't there*.
509
 
 
510
 
    It simplifies the merging logic if we always have an InventoryEntry, even
511
 
    if it isn't actually present
512
 
    """
513
 
    executable = None
514
 
    kind = None
515
 
    name = None
516
 
    parent_id = None
517
 
    revision = None
518
 
    symlink_target = None
519
 
    text_sha1 = None
520
 
 
521
 
_none_entry = _InventoryNoneEntry()
522
 
 
523
 
 
524
454
class Merge3Merger(object):
525
455
    """Three-way merger that uses the merge3 text merger"""
526
456
    requires_base = True
530
460
    supports_cherrypick = True
531
461
    supports_reverse_cherrypick = True
532
462
    winner_idx = {"this": 2, "other": 1, "conflict": 1}
533
 
    supports_lca_trees = True
534
463
 
535
464
    def __init__(self, working_tree, this_tree, base_tree, other_tree, 
536
465
                 interesting_ids=None, reprocess=False, show_base=False,
537
466
                 pb=DummyProgress(), pp=None, change_reporter=None,
538
467
                 interesting_files=None, do_merge=True,
539
 
                 cherrypick=False, lca_trees=None):
 
468
                 cherrypick=False):
540
469
        """Initialize the merger object and perform the merge.
541
470
 
542
471
        :param working_tree: The working tree to apply the merge to
558
487
            be combined with interesting_ids.  If neither interesting_files nor
559
488
            interesting_ids is specified, all files may participate in the
560
489
            merge.
561
 
        :param lca_trees: Can be set to a dictionary of {revision_id:rev_tree}
562
 
            if the ancestry was found to include a criss-cross merge.
563
 
            Otherwise should be None.
564
490
        """
565
491
        object.__init__(self)
566
492
        if interesting_files is not None and interesting_ids is not None:
575
501
        self.cooked_conflicts = []
576
502
        self.reprocess = reprocess
577
503
        self.show_base = show_base
578
 
        self._lca_trees = lca_trees
579
 
        # Uncommenting this will change the default algorithm to always use
580
 
        # _entries_lca. This can be useful for running the test suite and
581
 
        # making sure we haven't missed any corner cases.
582
 
        # if lca_trees is None:
583
 
        #     self._lca_trees = [self.base_tree]
584
504
        self.pb = pb
585
505
        self.pp = pp
586
506
        self.change_reporter = change_reporter
627
547
        return self.tt
628
548
 
629
549
    def _compute_transform(self):
630
 
        if self._lca_trees is None:
631
 
            entries = self._entries3()
632
 
            resolver = self._three_way
633
 
        else:
634
 
            entries = self._entries_lca()
635
 
            resolver = self._lca_multi_way
 
550
        entries = self._entries3()
636
551
        child_pb = ui.ui_factory.nested_progress_bar()
637
552
        try:
638
553
            for num, (file_id, changed, parents3, names3,
639
554
                      executable3) in enumerate(entries):
640
555
                child_pb.update('Preparing file merge', num, len(entries))
641
 
                self._merge_names(file_id, parents3, names3, resolver=resolver)
 
556
                self._merge_names(file_id, parents3, names3)
642
557
                if changed:
643
558
                    file_status = self.merge_contents(file_id)
644
559
                else:
645
560
                    file_status = 'unmodified'
646
561
                self._merge_executable(file_id,
647
 
                    executable3, file_status, resolver=resolver)
 
562
                    executable3, file_status)
648
563
        finally:
649
564
            child_pb.finished()
650
565
        self.fix_root()
676
591
        iterator = self.other_tree.iter_changes(self.base_tree,
677
592
                include_unchanged=True, specific_files=self.interesting_files,
678
593
                extra_trees=[self.this_tree])
679
 
        this_entries = dict((e.file_id, e) for p, e in
680
 
                            self.this_tree.iter_entries_by_dir(
681
 
                            self.interesting_ids))
682
594
        for (file_id, paths, changed, versioned, parents, names, kind,
683
595
             executable) in iterator:
684
596
            if (self.interesting_ids is not None and
685
597
                file_id not in self.interesting_ids):
686
598
                continue
687
 
            entry = this_entries.get(file_id)
688
 
            if entry is not None:
 
599
            if file_id in self.this_tree.inventory:
 
600
                entry = self.this_tree.inventory[file_id]
689
601
                this_name = entry.name
690
602
                this_parent = entry.parent_id
691
603
                this_executable = entry.executable
699
611
            result.append((file_id, changed, parents3, names3, executable3))
700
612
        return result
701
613
 
702
 
    def _entries_lca(self):
703
 
        """Gather data about files modified between multiple trees.
704
 
 
705
 
        This compares OTHER versus all LCA trees, and for interesting entries,
706
 
        it then compares with THIS and BASE.
707
 
 
708
 
        For the multi-valued entries, the format will be (BASE, [lca1, lca2])
709
 
        :return: [(file_id, changed, parents, names, executable)]
710
 
            file_id     Simple file_id of the entry
711
 
            changed     Boolean, True if the kind or contents changed
712
 
                        else False
713
 
            parents     ((base, [parent_id, in, lcas]), parent_id_other,
714
 
                         parent_id_this)
715
 
            names       ((base, [name, in, lcas]), name_in_other, name_in_this)
716
 
            executable  ((base, [exec, in, lcas]), exec_in_other, exec_in_this)
717
 
        """
718
 
        if self.interesting_files is not None:
719
 
            lookup_trees = [self.this_tree, self.base_tree]
720
 
            lookup_trees.extend(self._lca_trees)
721
 
            # I think we should include the lca trees as well
722
 
            interesting_ids = self.other_tree.paths2ids(self.interesting_files,
723
 
                                                        lookup_trees)
724
 
        else:
725
 
            interesting_ids = self.interesting_ids
726
 
        result = []
727
 
        walker = _mod_tree.MultiWalker(self.other_tree, self._lca_trees)
728
 
 
729
 
        base_inventory = self.base_tree.inventory
730
 
        this_inventory = self.this_tree.inventory
731
 
        for path, file_id, other_ie, lca_values in walker.iter_all():
732
 
            # Is this modified at all from any of the other trees?
733
 
            if other_ie is None:
734
 
                other_ie = _none_entry
735
 
            if interesting_ids is not None and file_id not in interesting_ids:
736
 
                continue
737
 
 
738
 
            # If other_revision is found in any of the lcas, that means this
739
 
            # node is uninteresting. This is because when merging, if there are
740
 
            # multiple heads(), we have to create a new node. So if we didn't,
741
 
            # we know that the ancestry is linear, and that OTHER did not
742
 
            # modify anything
743
 
            # See doc/developers/lca_merge_resolution.txt for details
744
 
            other_revision = other_ie.revision
745
 
            if other_revision is not None:
746
 
                # We can't use this shortcut when other_revision is None,
747
 
                # because it may be None because things are WorkingTrees, and
748
 
                # not because it is *actually* None.
749
 
                is_unmodified = False
750
 
                for lca_path, ie in lca_values:
751
 
                    if ie is not None and ie.revision == other_revision:
752
 
                        is_unmodified = True
753
 
                        break
754
 
                if is_unmodified:
755
 
                    continue
756
 
 
757
 
            lca_entries = []
758
 
            for lca_path, lca_ie in lca_values:
759
 
                if lca_ie is None:
760
 
                    lca_entries.append(_none_entry)
761
 
                else:
762
 
                    lca_entries.append(lca_ie)
763
 
 
764
 
            if file_id in base_inventory:
765
 
                base_ie = base_inventory[file_id]
766
 
            else:
767
 
                base_ie = _none_entry
768
 
 
769
 
            if file_id in this_inventory:
770
 
                this_ie = this_inventory[file_id]
771
 
            else:
772
 
                this_ie = _none_entry
773
 
 
774
 
            lca_kinds = []
775
 
            lca_parent_ids = []
776
 
            lca_names = []
777
 
            lca_executable = []
778
 
            for lca_ie in lca_entries:
779
 
                lca_kinds.append(lca_ie.kind)
780
 
                lca_parent_ids.append(lca_ie.parent_id)
781
 
                lca_names.append(lca_ie.name)
782
 
                lca_executable.append(lca_ie.executable)
783
 
 
784
 
            kind_winner = self._lca_multi_way(
785
 
                (base_ie.kind, lca_kinds),
786
 
                other_ie.kind, this_ie.kind)
787
 
            parent_id_winner = self._lca_multi_way(
788
 
                (base_ie.parent_id, lca_parent_ids),
789
 
                other_ie.parent_id, this_ie.parent_id)
790
 
            name_winner = self._lca_multi_way(
791
 
                (base_ie.name, lca_names),
792
 
                other_ie.name, this_ie.name)
793
 
 
794
 
            content_changed = True
795
 
            if kind_winner == 'this':
796
 
                # No kind change in OTHER, see if there are *any* changes
797
 
                if other_ie.kind == None:
798
 
                    # No content and 'this' wins the kind, so skip this?
799
 
                    # continue
800
 
                    pass
801
 
                elif other_ie.kind == 'directory':
802
 
                    if parent_id_winner == 'this' and name_winner == 'this':
803
 
                        # No change for this directory in OTHER, skip
804
 
                        continue
805
 
                    content_changed = False
806
 
                elif other_ie.kind == 'file':
807
 
                    def get_sha1(ie, tree):
808
 
                        if ie.kind != 'file':
809
 
                            return None
810
 
                        return tree.get_file_sha1(file_id)
811
 
                    base_sha1 = get_sha1(base_ie, self.base_tree)
812
 
                    lca_sha1s = [get_sha1(ie, tree) for ie, tree
813
 
                                 in zip(lca_entries, self._lca_trees)]
814
 
                    this_sha1 = get_sha1(this_ie, self.this_tree)
815
 
                    other_sha1 = get_sha1(other_ie, self.other_tree)
816
 
                    sha1_winner = self._lca_multi_way(
817
 
                        (base_sha1, lca_sha1s), other_sha1, this_sha1,
818
 
                        allow_overriding_lca=False)
819
 
                    exec_winner = self._lca_multi_way(
820
 
                        (base_ie.executable, lca_executable),
821
 
                        other_ie.executable, this_ie.executable)
822
 
                    if (parent_id_winner == 'this' and name_winner == 'this'
823
 
                        and sha1_winner == 'this' and exec_winner == 'this'):
824
 
                        # No kind, parent, name, exec, or content change for
825
 
                        # OTHER, so this node is not considered interesting
826
 
                        continue
827
 
                    if sha1_winner == 'this':
828
 
                        content_changed = False
829
 
                elif other_ie.kind == 'symlink':
830
 
                    def get_target(ie, tree):
831
 
                        if ie.kind != 'symlink':
832
 
                            return None
833
 
                        return tree.get_symlink_target(file_id)
834
 
                    base_target = get_target(base_ie, self.base_tree)
835
 
                    lca_targets = [get_target(ie, tree) for ie, tree
836
 
                                   in zip(lca_entries, self._lca_trees)]
837
 
                    this_target = get_target(this_ie, self.this_tree)
838
 
                    other_target = get_target(other_ie, self.other_tree)
839
 
                    target_winner = self._lca_multi_way(
840
 
                        (base_target, lca_targets),
841
 
                        other_target, this_target)
842
 
                    if (parent_id_winner == 'this' and name_winner == 'this'
843
 
                        and target_winner == 'this'):
844
 
                        # No kind, parent, name, or symlink target change
845
 
                        # not interesting
846
 
                        continue
847
 
                    if target_winner == 'this':
848
 
                        content_changed = False
849
 
                elif other_ie.kind == 'tree-reference':
850
 
                    # The 'changed' information seems to be handled at a higher
851
 
                    # level. At least, _entries3 returns False for content
852
 
                    # changed, even when at a new revision_id.
853
 
                    content_changed = False
854
 
                    if (parent_id_winner == 'this' and name_winner == 'this'):
855
 
                        # Nothing interesting
856
 
                        continue
857
 
                else:
858
 
                    raise AssertionError('unhandled kind: %s' % other_ie.kind)
859
 
                # XXX: We need to handle kind == 'symlink'
860
 
 
861
 
            # If we have gotten this far, that means something has changed
862
 
            result.append((file_id, content_changed,
863
 
                           ((base_ie.parent_id, lca_parent_ids),
864
 
                            other_ie.parent_id, this_ie.parent_id),
865
 
                           ((base_ie.name, lca_names),
866
 
                            other_ie.name, this_ie.name),
867
 
                           ((base_ie.executable, lca_executable),
868
 
                            other_ie.executable, this_ie.executable)
869
 
                          ))
870
 
        return result
871
 
 
872
 
 
873
614
    def fix_root(self):
874
615
        try:
875
616
            self.tt.final_kind(self.tt.root)
878
619
        if self.tt.final_file_id(self.tt.root) is None:
879
620
            self.tt.version_file(self.tt.tree_file_id(self.tt.root), 
880
621
                                 self.tt.root)
 
622
        if self.other_tree.inventory.root is None:
 
623
            return
881
624
        other_root_file_id = self.other_tree.get_root_id()
882
 
        if other_root_file_id is None:
883
 
            return
884
625
        other_root = self.tt.trans_id_file_id(other_root_file_id)
885
626
        if other_root == self.tt.root:
886
627
            return
888
629
            self.tt.final_kind(other_root)
889
630
        except NoSuchFile:
890
631
            return
891
 
        if self.other_tree.inventory.root.file_id in self.this_tree.inventory:
892
 
            # the other tree's root is a non-root in the current tree
893
 
            return
894
632
        self.reparent_children(self.other_tree.inventory.root, self.tt.root)
895
633
        self.tt.cancel_creation(other_root)
896
634
        self.tt.cancel_versioning(other_root)
964
702
            return "other"
965
703
 
966
704
    @staticmethod
967
 
    def _lca_multi_way(bases, other, this, allow_overriding_lca=True):
968
 
        """Consider LCAs when determining whether a change has occurred.
969
 
 
970
 
        If LCAS are all identical, this is the same as a _three_way comparison.
971
 
 
972
 
        :param bases: value in (BASE, [LCAS])
973
 
        :param other: value in OTHER
974
 
        :param this: value in THIS
975
 
        :param allow_overriding_lca: If there is more than one unique lca
976
 
            value, allow OTHER to override THIS if it has a new value, and
977
 
            THIS only has an lca value, or vice versa. This is appropriate for
978
 
            truly scalar values, not as much for non-scalars.
979
 
        :return: 'this', 'other', or 'conflict' depending on whether an entry
980
 
            changed or not.
981
 
        """
982
 
        # See doc/developers/lca_merge_resolution.txt for details about this
983
 
        # algorithm.
984
 
        if other == this:
985
 
            # Either Ambiguously clean, or nothing was actually changed. We
986
 
            # don't really care
987
 
            return 'this'
988
 
        base_val, lca_vals = bases
989
 
        # Remove 'base_val' from the lca_vals, because it is not interesting
990
 
        filtered_lca_vals = [lca_val for lca_val in lca_vals
991
 
                                      if lca_val != base_val]
992
 
        if len(filtered_lca_vals) == 0:
993
 
            return Merge3Merger._three_way(base_val, other, this)
994
 
 
995
 
        unique_lca_vals = set(filtered_lca_vals)
996
 
        if len(unique_lca_vals) == 1:
997
 
            return Merge3Merger._three_way(unique_lca_vals.pop(), other, this)
998
 
 
999
 
        if allow_overriding_lca:
1000
 
            if other in unique_lca_vals:
1001
 
                if this in unique_lca_vals:
1002
 
                    # Each side picked a different lca, conflict
1003
 
                    return 'conflict'
1004
 
                else:
1005
 
                    # This has a value which supersedes both lca values, and
1006
 
                    # other only has an lca value
1007
 
                    return 'this'
1008
 
            elif this in unique_lca_vals:
1009
 
                # OTHER has a value which supersedes both lca values, and this
1010
 
                # only has an lca value
1011
 
                return 'other'
1012
 
 
1013
 
        # At this point, the lcas disagree, and the tips disagree
1014
 
        return 'conflict'
1015
 
 
1016
 
    @staticmethod
1017
705
    def scalar_three_way(this_tree, base_tree, other_tree, file_id, key):
1018
706
        """Do a three-way test on a scalar.
1019
707
        Return "this", "other" or "conflict", depending whether a value wins.
1051
739
            else:
1052
740
                names.append(entry.name)
1053
741
                parents.append(entry.parent_id)
1054
 
        return self._merge_names(file_id, parents, names,
1055
 
                                 resolver=self._three_way)
 
742
        return self._merge_names(file_id, parents, names)
1056
743
 
1057
 
    def _merge_names(self, file_id, parents, names, resolver):
 
744
    def _merge_names(self, file_id, parents, names):
1058
745
        """Perform a merge on file_id names and parents"""
1059
746
        base_name, other_name, this_name = names
1060
747
        base_parent, other_parent, this_parent = parents
1061
748
 
1062
 
        name_winner = resolver(*names)
 
749
        name_winner = self._three_way(*names)
1063
750
 
1064
 
        parent_id_winner = resolver(*parents)
 
751
        parent_id_winner = self._three_way(*parents)
1065
752
        if this_name is None:
1066
753
            if name_winner == "this":
1067
754
                name_winner = "other"
1136
823
                    self.tt.delete_contents(trans_id)
1137
824
                if file_id in self.other_tree:
1138
825
                    # OTHER changed the file
1139
 
                    create_from_tree(self.tt, trans_id,
1140
 
                                     self.other_tree, file_id)
1141
 
                    if file_id not in self.this_tree:
 
826
                    create_by_entry(self.tt, 
 
827
                                    self.other_tree.inventory[file_id], 
 
828
                                    self.other_tree, trans_id)
 
829
                    if file_id not in self.this_tree.inventory:
1142
830
                        self.tt.version_file(file_id, trans_id)
1143
831
                    return "modified"
1144
832
                elif file_id in self.this_tree.inventory:
1154
842
                    self.text_merge(file_id, trans_id)
1155
843
                except BinaryFile:
1156
844
                    return contents_conflict()
1157
 
                if file_id not in self.this_tree:
 
845
                if file_id not in self.this_tree.inventory:
1158
846
                    self.tt.version_file(file_id, trans_id)
1159
847
                try:
1160
848
                    self.tt.tree_kind(trans_id)
1241
929
                    versioned = True
1242
930
        return file_group
1243
931
           
1244
 
    def _conflict_file(self, name, parent_id, tree, file_id, suffix,
 
932
    def _conflict_file(self, name, parent_id, tree, file_id, suffix, 
1245
933
                       lines=None):
1246
934
        """Emit a single conflict file."""
1247
935
        name = name + '.' + suffix
1248
936
        trans_id = self.tt.create_path(name, parent_id)
1249
 
        create_from_tree(self.tt, trans_id, tree, file_id, lines)
 
937
        entry = tree.inventory[file_id]
 
938
        create_by_entry(self.tt, entry, tree, trans_id, lines)
1250
939
        return trans_id
1251
940
 
1252
941
    def merge_executable(self, file_id, file_status):
1253
942
        """Perform a merge on the execute bit."""
1254
943
        executable = [self.executable(t, file_id) for t in (self.base_tree,
1255
944
                      self.other_tree, self.this_tree)]
1256
 
        self._merge_executable(file_id, executable, file_status,
1257
 
                               resolver=self._three_way)
 
945
        self._merge_executable(file_id, executable, file_status)
1258
946
 
1259
 
    def _merge_executable(self, file_id, executable, file_status,
1260
 
                          resolver):
 
947
    def _merge_executable(self, file_id, executable, file_status):
1261
948
        """Perform a merge on the execute bit."""
1262
949
        base_executable, other_executable, this_executable = executable
1263
950
        if file_status == "deleted":
1264
951
            return
1265
 
        winner = resolver(*executable)
 
952
        winner = self._three_way(*executable)
1266
953
        if winner == "conflict":
1267
954
        # There must be a None in here, if we have a conflict, but we
1268
955
        # need executability since file status was not deleted.
1554
1241
 
1555
1242
class _PlanMergeBase(object):
1556
1243
 
1557
 
    def __init__(self, a_rev, b_rev, vf, key_prefix):
 
1244
    def __init__(self, a_rev, b_rev, vf):
1558
1245
        """Contructor.
1559
1246
 
1560
1247
        :param a_rev: Revision-id of one revision to merge
1561
1248
        :param b_rev: Revision-id of the other revision to merge
1562
 
        :param vf: A VersionedFiles containing both revisions
1563
 
        :param key_prefix: A prefix for accessing keys in vf, typically
1564
 
            (file_id,).
 
1249
        :param vf: A versionedfile containing both revisions
1565
1250
        """
1566
1251
        self.a_rev = a_rev
1567
1252
        self.b_rev = b_rev
 
1253
        self.lines_a = vf.get_lines(a_rev)
 
1254
        self.lines_b = vf.get_lines(b_rev)
1568
1255
        self.vf = vf
1569
1256
        self._last_lines = None
1570
1257
        self._last_lines_revision_id = None
1571
1258
        self._cached_matching_blocks = {}
1572
 
        self._key_prefix = key_prefix
1573
 
        self._precache_tip_lines()
1574
 
 
1575
 
    def _precache_tip_lines(self):
1576
 
        lines = self.get_lines([self.a_rev, self.b_rev])
1577
 
        self.lines_a = lines[self.a_rev]
1578
 
        self.lines_b = lines[self.b_rev]
1579
 
 
1580
 
    def get_lines(self, revisions):
1581
 
        """Get lines for revisions from the backing VersionedFiles.
1582
 
        
1583
 
        :raises RevisionNotPresent: on absent texts.
1584
 
        """
1585
 
        keys = [(self._key_prefix + (rev,)) for rev in revisions]
1586
 
        result = {}
1587
 
        for record in self.vf.get_record_stream(keys, 'unordered', True):
1588
 
            if record.storage_kind == 'absent':
1589
 
                raise errors.RevisionNotPresent(record.key, self.vf)
1590
 
            result[record.key[-1]] = osutils.split_lines(
1591
 
                record.get_bytes_as('fulltext'))
1592
 
        return result
1593
1259
 
1594
1260
    def plan_merge(self):
1595
1261
        """Generate a 'plan' for merging the two revisions.
1643
1309
            return cached
1644
1310
        if self._last_lines_revision_id == left_revision:
1645
1311
            left_lines = self._last_lines
1646
 
            right_lines = self.get_lines([right_revision])[right_revision]
1647
1312
        else:
1648
 
            lines = self.get_lines([left_revision, right_revision])
1649
 
            left_lines = lines[left_revision]
1650
 
            right_lines = lines[right_revision]
 
1313
            left_lines = self.vf.get_lines(left_revision)
 
1314
        right_lines = self.vf.get_lines(right_revision)
1651
1315
        self._last_lines = right_lines
1652
1316
        self._last_lines_revision_id = right_revision
1653
1317
        matcher = patiencediff.PatienceSequenceMatcher(None, left_lines,
1706
1370
class _PlanMerge(_PlanMergeBase):
1707
1371
    """Plan an annotate merge using on-the-fly annotation"""
1708
1372
 
1709
 
    def __init__(self, a_rev, b_rev, vf, key_prefix):
1710
 
        super(_PlanMerge, self).__init__(a_rev, b_rev, vf, key_prefix)
1711
 
        self.a_key = self._key_prefix + (self.a_rev,)
1712
 
        self.b_key = self._key_prefix + (self.b_rev,)
1713
 
        self.graph = Graph(self.vf)
1714
 
        heads = self.graph.heads((self.a_key, self.b_key))
1715
 
        if len(heads) == 1:
1716
 
            # one side dominates, so we can just return its values, yay for
1717
 
            # per-file graphs
1718
 
            # Ideally we would know that before we get this far
1719
 
            self._head_key = heads.pop()
1720
 
            if self._head_key == self.a_key:
1721
 
                other = b_rev
1722
 
            else:
1723
 
                other = a_rev
1724
 
            mutter('found dominating revision for %s\n%s > %s', self.vf,
1725
 
                   self._head_key[-1], other)
1726
 
            self._weave = None
1727
 
        else:
1728
 
            self._head_key = None
1729
 
            self._build_weave()
1730
 
 
1731
 
    def _precache_tip_lines(self):
1732
 
        # Turn this into a no-op, because we will do this later
1733
 
        pass
1734
 
 
1735
 
    def _find_recursive_lcas(self):
1736
 
        """Find all the ancestors back to a unique lca"""
1737
 
        cur_ancestors = (self.a_key, self.b_key)
1738
 
        # graph.find_lca(uncommon, keys) now returns plain NULL_REVISION,
1739
 
        # rather than a key tuple. We will just map that directly to no common
1740
 
        # ancestors.
1741
 
        parent_map = {}
1742
 
        while True:
1743
 
            next_lcas = self.graph.find_lca(*cur_ancestors)
1744
 
            # Map a plain NULL_REVISION to a simple no-ancestors
1745
 
            if next_lcas == set([NULL_REVISION]):
1746
 
                next_lcas = ()
1747
 
            # Order the lca's based on when they were merged into the tip
1748
 
            # While the actual merge portion of weave merge uses a set() of
1749
 
            # active revisions, the order of insertion *does* effect the
1750
 
            # implicit ordering of the texts.
1751
 
            for rev_key in cur_ancestors:
1752
 
                ordered_parents = tuple(self.graph.find_merge_order(rev_key,
1753
 
                                                                    next_lcas))
1754
 
                parent_map[rev_key] = ordered_parents
1755
 
            if len(next_lcas) == 0:
1756
 
                break
1757
 
            elif len(next_lcas) == 1:
1758
 
                parent_map[list(next_lcas)[0]] = ()
1759
 
                break
1760
 
            elif len(next_lcas) > 2:
1761
 
                # More than 2 lca's, fall back to grabbing all nodes between
1762
 
                # this and the unique lca.
1763
 
                mutter('More than 2 LCAs, falling back to all nodes for:'
1764
 
                       ' %s, %s\n=> %s', self.a_key, self.b_key, cur_ancestors)
1765
 
                cur_lcas = next_lcas
1766
 
                while len(cur_lcas) > 1:
1767
 
                    cur_lcas = self.graph.find_lca(*cur_lcas)
1768
 
                if len(cur_lcas) == 0:
1769
 
                    # No common base to find, use the full ancestry
1770
 
                    unique_lca = None
1771
 
                else:
1772
 
                    unique_lca = list(cur_lcas)[0]
1773
 
                    if unique_lca == NULL_REVISION:
1774
 
                        # find_lca will return a plain 'NULL_REVISION' rather
1775
 
                        # than a key tuple when there is no common ancestor, we
1776
 
                        # prefer to just use None, because it doesn't confuse
1777
 
                        # _get_interesting_texts()
1778
 
                        unique_lca = None
1779
 
                parent_map.update(self._find_unique_parents(next_lcas,
1780
 
                                                            unique_lca))
1781
 
                break
1782
 
            cur_ancestors = next_lcas
1783
 
        return parent_map
1784
 
 
1785
 
    def _find_unique_parents(self, tip_keys, base_key):
1786
 
        """Find ancestors of tip that aren't ancestors of base.
1787
 
        
1788
 
        :param tip_keys: Nodes that are interesting
1789
 
        :param base_key: Cull all ancestors of this node
1790
 
        :return: The parent map for all revisions between tip_keys and
1791
 
            base_key. base_key will be included. References to nodes outside of
1792
 
            the ancestor set will also be removed.
1793
 
        """
1794
 
        # TODO: this would be simpler if find_unique_ancestors took a list
1795
 
        #       instead of a single tip, internally it supports it, but it
1796
 
        #       isn't a "backwards compatible" api change.
1797
 
        if base_key is None:
1798
 
            parent_map = dict(self.graph.iter_ancestry(tip_keys))
1799
 
            # We remove NULL_REVISION because it isn't a proper tuple key, and
1800
 
            # thus confuses things like _get_interesting_texts, and our logic
1801
 
            # to add the texts into the memory weave.
1802
 
            if NULL_REVISION in parent_map:
1803
 
                parent_map.pop(NULL_REVISION)
1804
 
        else:
1805
 
            interesting = set()
1806
 
            for tip in tip_keys:
1807
 
                interesting.update(
1808
 
                    self.graph.find_unique_ancestors(tip, [base_key]))
1809
 
            parent_map = self.graph.get_parent_map(interesting)
1810
 
            parent_map[base_key] = ()
1811
 
        culled_parent_map, child_map, tails = self._remove_external_references(
1812
 
            parent_map)
1813
 
        # Remove all the tails but base_key
1814
 
        if base_key is not None:
1815
 
            tails.remove(base_key)
1816
 
            self._prune_tails(culled_parent_map, child_map, tails)
1817
 
        # Now remove all the uninteresting 'linear' regions
1818
 
        simple_map = _mod_graph.collapse_linear_regions(culled_parent_map)
1819
 
        return simple_map
1820
 
 
1821
 
    @staticmethod
1822
 
    def _remove_external_references(parent_map):
1823
 
        """Remove references that go outside of the parent map.
1824
 
 
1825
 
        :param parent_map: Something returned from Graph.get_parent_map(keys)
1826
 
        :return: (filtered_parent_map, child_map, tails)
1827
 
            filtered_parent_map is parent_map without external references
1828
 
            child_map is the {parent_key: [child_keys]} mapping
1829
 
            tails is a list of nodes that do not have any parents in the map
1830
 
        """
1831
 
        # TODO: The basic effect of this function seems more generic than
1832
 
        #       _PlanMerge. But the specific details of building a child_map,
1833
 
        #       and computing tails seems very specific to _PlanMerge.
1834
 
        #       Still, should this be in Graph land?
1835
 
        filtered_parent_map = {}
1836
 
        child_map = {}
1837
 
        tails = []
1838
 
        for key, parent_keys in parent_map.iteritems():
1839
 
            culled_parent_keys = [p for p in parent_keys if p in parent_map]
1840
 
            if not culled_parent_keys:
1841
 
                tails.append(key)
1842
 
            for parent_key in culled_parent_keys:
1843
 
                child_map.setdefault(parent_key, []).append(key)
1844
 
            # TODO: Do we want to do this, it adds overhead for every node,
1845
 
            #       just to say that the node has no children
1846
 
            child_map.setdefault(key, [])
1847
 
            filtered_parent_map[key] = culled_parent_keys
1848
 
        return filtered_parent_map, child_map, tails
1849
 
 
1850
 
    @staticmethod
1851
 
    def _prune_tails(parent_map, child_map, tails_to_remove):
1852
 
        """Remove tails from the parent map.
1853
 
        
1854
 
        This will remove the supplied revisions until no more children have 0
1855
 
        parents.
1856
 
 
1857
 
        :param parent_map: A dict of {child: [parents]}, this dictionary will
1858
 
            be modified in place.
1859
 
        :param tails_to_remove: A list of tips that should be removed,
1860
 
            this list will be consumed
1861
 
        :param child_map: The reverse dict of parent_map ({parent: [children]})
1862
 
            this dict will be modified
1863
 
        :return: None, parent_map will be modified in place.
1864
 
        """
1865
 
        while tails_to_remove:
1866
 
            next = tails_to_remove.pop()
1867
 
            parent_map.pop(next)
1868
 
            children = child_map.pop(next)
1869
 
            for child in children:
1870
 
                child_parents = parent_map[child]
1871
 
                child_parents.remove(next)
1872
 
                if len(child_parents) == 0:
1873
 
                    tails_to_remove.append(child)
1874
 
 
1875
 
    def _get_interesting_texts(self, parent_map):
1876
 
        """Return a dict of texts we are interested in.
1877
 
 
1878
 
        Note that the input is in key tuples, but the output is in plain
1879
 
        revision ids.
1880
 
 
1881
 
        :param parent_map: The output from _find_recursive_lcas
1882
 
        :return: A dict of {'revision_id':lines} as returned by
1883
 
            _PlanMergeBase.get_lines()
1884
 
        """
1885
 
        all_revision_keys = set(parent_map)
1886
 
        all_revision_keys.add(self.a_key)
1887
 
        all_revision_keys.add(self.b_key)
1888
 
 
1889
 
        # Everything else is in 'keys' but get_lines is in 'revision_ids'
1890
 
        all_texts = self.get_lines([k[-1] for k in all_revision_keys])
1891
 
        return all_texts
1892
 
 
1893
 
    def _build_weave(self):
1894
 
        from bzrlib import weave
1895
 
        self._weave = weave.Weave(weave_name='in_memory_weave',
1896
 
                                  allow_reserved=True)
1897
 
        parent_map = self._find_recursive_lcas()
1898
 
 
1899
 
        all_texts = self._get_interesting_texts(parent_map)
1900
 
 
1901
 
        # Note: Unfortunately, the order given by topo_sort will effect the
1902
 
        # ordering resolution in the output. Specifically, if you add A then B,
1903
 
        # then in the output text A lines will show up before B lines. And, of
1904
 
        # course, topo_sort doesn't guarantee any real ordering.
1905
 
        # So we use merge_sort, and add a fake node on the tip.
1906
 
        # This ensures that left-hand parents will always be inserted into the
1907
 
        # weave before right-hand parents.
1908
 
        tip_key = self._key_prefix + (_mod_revision.CURRENT_REVISION,)
1909
 
        parent_map[tip_key] = (self.a_key, self.b_key)
1910
 
 
1911
 
        for seq_num, key, depth, eom in reversed(tsort.merge_sort(parent_map,
1912
 
                                                                  tip_key)):
1913
 
            if key == tip_key:
1914
 
                continue
1915
 
        # for key in tsort.topo_sort(parent_map):
1916
 
            parent_keys = parent_map[key]
1917
 
            revision_id = key[-1]
1918
 
            parent_ids = [k[-1] for k in parent_keys]
1919
 
            self._weave.add_lines(revision_id, parent_ids,
1920
 
                                  all_texts[revision_id])
1921
 
 
1922
 
    def plan_merge(self):
1923
 
        """Generate a 'plan' for merging the two revisions.
1924
 
 
1925
 
        This involves comparing their texts and determining the cause of
1926
 
        differences.  If text A has a line and text B does not, then either the
1927
 
        line was added to text A, or it was deleted from B.  Once the causes
1928
 
        are combined, they are written out in the format described in
1929
 
        VersionedFile.plan_merge
1930
 
        """
1931
 
        if self._head_key is not None: # There was a single head
1932
 
            if self._head_key == self.a_key:
1933
 
                plan = 'new-a'
1934
 
            else:
1935
 
                if self._head_key != self.b_key:
1936
 
                    raise AssertionError('There was an invalid head: %s != %s'
1937
 
                                         % (self.b_key, self._head_key))
1938
 
                plan = 'new-b'
1939
 
            head_rev = self._head_key[-1]
1940
 
            lines = self.get_lines([head_rev])[head_rev]
1941
 
            return ((plan, line) for line in lines)
1942
 
        return self._weave.plan_merge(self.a_rev, self.b_rev)
 
1373
    def __init__(self, a_rev, b_rev, vf):
 
1374
       _PlanMergeBase.__init__(self, a_rev, b_rev, vf)
 
1375
       a_ancestry = set(vf.get_ancestry(a_rev, topo_sorted=False))
 
1376
       b_ancestry = set(vf.get_ancestry(b_rev, topo_sorted=False))
 
1377
       self.uncommon = a_ancestry.symmetric_difference(b_ancestry)
 
1378
 
 
1379
    def _determine_status(self, revision_id, unique_line_numbers):
 
1380
        """Determines the status unique lines versus all lcas.
 
1381
 
 
1382
        Basically, determines why the line is unique to this revision.
 
1383
 
 
1384
        A line may be determined new or killed, but not both.
 
1385
 
 
1386
        :param revision_id: The id of the revision in which the lines are
 
1387
            unique
 
1388
        :param unique_line_numbers: The line numbers of unique lines.
 
1389
        :return a tuple of (new_this, killed_other):
 
1390
        """
 
1391
        new = self._find_new(revision_id)
 
1392
        killed = set(unique_line_numbers).difference(new)
 
1393
        return new, killed
 
1394
 
 
1395
    def _find_new(self, version_id):
 
1396
        """Determine which lines are new in the ancestry of this version.
 
1397
 
 
1398
        If a lines is present in this version, and not present in any
 
1399
        common ancestor, it is considered new.
 
1400
        """
 
1401
        if version_id not in self.uncommon:
 
1402
            return set()
 
1403
        parents = self.vf.get_parent_map([version_id])[version_id]
 
1404
        if len(parents) == 0:
 
1405
            return set(range(len(self.vf.get_lines(version_id))))
 
1406
        new = None
 
1407
        for parent in parents:
 
1408
            blocks = self._get_matching_blocks(version_id, parent)
 
1409
            result, unused = self._unique_lines(blocks)
 
1410
            parent_new = self._find_new(parent)
 
1411
            for i, j, n in blocks:
 
1412
                for ii, jj in [(i+r, j+r) for r in range(n)]:
 
1413
                    if jj in parent_new:
 
1414
                        result.append(ii)
 
1415
            if new is None:
 
1416
                new = set(result)
 
1417
            else:
 
1418
                new.intersection_update(result)
 
1419
        return new
1943
1420
 
1944
1421
 
1945
1422
class _PlanLCAMerge(_PlanMergeBase):
1953
1430
    This is faster, and hopefully produces more useful output.
1954
1431
    """
1955
1432
 
1956
 
    def __init__(self, a_rev, b_rev, vf, key_prefix, graph):
1957
 
        _PlanMergeBase.__init__(self, a_rev, b_rev, vf, key_prefix)
1958
 
        lcas = graph.find_lca(key_prefix + (a_rev,), key_prefix + (b_rev,))
1959
 
        self.lcas = set()
1960
 
        for lca in lcas:
1961
 
            if lca == NULL_REVISION:
1962
 
                self.lcas.add(lca)
1963
 
            else:
1964
 
                self.lcas.add(lca[-1])
 
1433
    def __init__(self, a_rev, b_rev, vf, graph):
 
1434
        _PlanMergeBase.__init__(self, a_rev, b_rev, vf)
 
1435
        self.lcas = graph.find_lca(a_rev, b_rev)
1965
1436
        for lca in self.lcas:
1966
1437
            if _mod_revision.is_null(lca):
1967
1438
                lca_lines = []
1968
1439
            else:
1969
 
                lca_lines = self.get_lines([lca])[lca]
 
1440
                lca_lines = self.vf.get_lines(lca)
1970
1441
            matcher = patiencediff.PatienceSequenceMatcher(None, self.lines_a,
1971
1442
                                                           lca_lines)
1972
1443
            blocks = list(matcher.get_matching_blocks())