← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/merge.py

  • Committer: Martin
  • Date: 2010-05-03 20:57:39 UTC
  • mto: This revision was merged to the branch mainline in revision 5204.
  • Revision ID: gzlist@googlemail.com-20100503205739-n326zdvevv0rmruh
Retain original stack and error message when translating to ValueError in bencode

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/merge.py
1
 
# Copyright (C) 2005-2011 Canonical Ltd
 
1
# Copyright (C) 2005-2010 Canonical Ltd
2
2
#
3
3
# This program is free software; you can redistribute it and/or modify
4
4
# it under the terms of the GNU General Public License as published by
14
14
# along with this program; if not, write to the Free Software
15
15
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
16
16
 
17
 
from __future__ import absolute_import
18
 
 
19
17
import warnings
20
18
 
21
 
from bzrlib.lazy_import import lazy_import
22
 
lazy_import(globals(), """
23
19
from bzrlib import (
24
20
    branch as _mod_branch,
25
 
    cleanup,
26
21
    conflicts as _mod_conflicts,
27
22
    debug,
28
 
    generate_ids,
 
23
    decorators,
 
24
    errors,
29
25
    graph as _mod_graph,
 
26
    hooks,
30
27
    merge3,
31
28
    osutils,
32
29
    patiencediff,
37
34
    tree as _mod_tree,
38
35
    tsort,
39
36
    ui,
40
 
    versionedfile,
41
 
    workingtree,
42
 
    )
43
 
from bzrlib.i18n import gettext
44
 
""")
45
 
from bzrlib import (
46
 
    decorators,
47
 
    errors,
48
 
    hooks,
49
 
    registry,
50
 
    )
 
37
    versionedfile
 
38
    )
 
39
from bzrlib.cleanup import OperationWithCleanups
51
40
from bzrlib.symbol_versioning import (
52
41
    deprecated_in,
53
42
    deprecated_method,
57
46
 
58
47
def transform_tree(from_tree, to_tree, interesting_ids=None):
59
48
    from_tree.lock_tree_write()
60
 
    operation = cleanup.OperationWithCleanups(merge_inner)
 
49
    operation = OperationWithCleanups(merge_inner)
61
50
    operation.add_cleanup(from_tree.unlock)
62
51
    operation.run_simple(from_tree.branch, to_tree, from_tree,
63
52
        ignore_zero=True, interesting_ids=interesting_ids, this_tree=from_tree)
66
55
class MergeHooks(hooks.Hooks):
67
56
 
68
57
    def __init__(self):
69
 
        hooks.Hooks.__init__(self, "bzrlib.merge", "Merger.hooks")
70
 
        self.add_hook('merge_file_content',
 
58
        hooks.Hooks.__init__(self)
 
59
        self.create_hook(hooks.HookPoint('merge_file_content',
71
60
            "Called with a bzrlib.merge.Merger object to create a per file "
72
61
            "merge object when starting a merge. "
73
62
            "Should return either None or a subclass of "
77
66
            "side has deleted the file and the other has changed it). "
78
67
            "See the AbstractPerFileMerger API docs for details on how it is "
79
68
            "used by merge.",
80
 
            (2, 1))
81
 
        self.add_hook('pre_merge',
82
 
            'Called before a merge. '
83
 
            'Receives a Merger object as the single argument.',
84
 
            (2, 5))
85
 
        self.add_hook('post_merge',
86
 
            'Called after a merge. '
87
 
            'Receives a Merger object as the single argument. '
88
 
            'The return value is ignored.',
89
 
            (2, 5))
 
69
            (2, 1), None))
90
70
 
91
71
 
92
72
class AbstractPerFileMerger(object):
104
84
    def merge_contents(self, merge_params):
105
85
        """Attempt to merge the contents of a single file.
106
86
        
107
 
        :param merge_params: A bzrlib.merge.MergeFileHookParams
108
 
        :return: A tuple of (status, chunks), where status is one of
 
87
        :param merge_params: A bzrlib.merge.MergeHookParams
 
88
        :return : A tuple of (status, chunks), where status is one of
109
89
            'not_applicable', 'success', 'conflicted', or 'delete'.  If status
110
90
            is 'success' or 'conflicted', then chunks should be an iterable of
111
91
            strings for the new file contents.
131
111
 
132
112
    def get_filename(self, params, tree):
133
113
        """Lookup the filename (i.e. basename, not path), given a Tree (e.g.
134
 
        self.merger.this_tree) and a MergeFileHookParams.
 
114
        self.merger.this_tree) and a MergeHookParams.
135
115
        """
136
116
        return osutils.basename(tree.id2path(params.file_id))
137
117
 
138
118
    def get_filepath(self, params, tree):
139
119
        """Calculate the path to the file in a tree.
140
120
 
141
 
        :param params: A MergeFileHookParams describing the file to merge
 
121
        :param params: A MergeHookParams describing the file to merge
142
122
        :param tree: a Tree, e.g. self.merger.this_tree.
143
123
        """
144
124
        return tree.id2path(params.file_id)
151
131
            params.winner == 'other' or
152
132
            # THIS and OTHER aren't both files.
153
133
            not params.is_file_merge() or
154
 
            # The filename doesn't match
 
134
            # The filename doesn't match *.xml
155
135
            not self.file_matches(params)):
156
136
            return 'not_applicable', None
157
137
        return self.merge_matching(params)
233
213
        raise NotImplementedError(self.merge_text)
234
214
 
235
215
 
236
 
class MergeFileHookParams(object):
 
216
class MergeHookParams(object):
237
217
    """Object holding parameters passed to merge_file_content hooks.
238
218
 
239
219
    There are some fields hooks can access:
441
421
        return self._cached_trees[revision_id]
442
422
 
443
423
    def _get_tree(self, treespec, possible_transports=None):
 
424
        from bzrlib import workingtree
444
425
        location, revno = treespec
445
426
        if revno is None:
446
427
            tree = workingtree.WorkingTree.open_containing(location)[0]
454
435
        revision_id = _mod_revision.ensure_null(revision_id)
455
436
        return branch, self.revision_tree(revision_id, branch)
456
437
 
 
438
    @deprecated_method(deprecated_in((2, 1, 0)))
 
439
    def ensure_revision_trees(self):
 
440
        if self.this_revision_tree is None:
 
441
            self.this_basis_tree = self.revision_tree(self.this_basis)
 
442
            if self.this_basis == self.this_rev_id:
 
443
                self.this_revision_tree = self.this_basis_tree
 
444
 
 
445
        if self.other_rev_id is None:
 
446
            other_basis_tree = self.revision_tree(self.other_basis)
 
447
            if other_basis_tree.has_changes(self.other_tree):
 
448
                raise errors.WorkingTreeNotRevision(self.this_tree)
 
449
            other_rev_id = self.other_basis
 
450
            self.other_tree = other_basis_tree
 
451
 
 
452
    @deprecated_method(deprecated_in((2, 1, 0)))
 
453
    def file_revisions(self, file_id):
 
454
        self.ensure_revision_trees()
 
455
        def get_id(tree, file_id):
 
456
            revision_id = tree.inventory[file_id].revision
 
457
            return revision_id
 
458
        if self.this_rev_id is None:
 
459
            if self.this_basis_tree.get_file_sha1(file_id) != \
 
460
                self.this_tree.get_file_sha1(file_id):
 
461
                raise errors.WorkingTreeNotRevision(self.this_tree)
 
462
 
 
463
        trees = (self.this_basis_tree, self.other_tree)
 
464
        return [get_id(tree, file_id) for tree in trees]
 
465
 
 
466
    @deprecated_method(deprecated_in((2, 1, 0)))
 
467
    def check_basis(self, check_clean, require_commits=True):
 
468
        if self.this_basis is None and require_commits is True:
 
469
            raise errors.BzrCommandError(
 
470
                "This branch has no commits."
 
471
                " (perhaps you would prefer 'bzr pull')")
 
472
        if check_clean:
 
473
            self.compare_basis()
 
474
            if self.this_basis != self.this_rev_id:
 
475
                raise errors.UncommittedChanges(self.this_tree)
 
476
 
 
477
    @deprecated_method(deprecated_in((2, 1, 0)))
 
478
    def compare_basis(self):
 
479
        try:
 
480
            basis_tree = self.revision_tree(self.this_tree.last_revision())
 
481
        except errors.NoSuchRevision:
 
482
            basis_tree = self.this_tree.basis_tree()
 
483
        if not self.this_tree.has_changes(basis_tree):
 
484
            self.this_rev_id = self.this_basis
 
485
 
457
486
    def set_interesting_files(self, file_list):
458
487
        self.interesting_files = file_list
459
488
 
466
495
    def _add_parent(self):
467
496
        new_parents = self.this_tree.get_parent_ids() + [self.other_rev_id]
468
497
        new_parent_trees = []
469
 
        operation = cleanup.OperationWithCleanups(
470
 
            self.this_tree.set_parent_trees)
 
498
        operation = OperationWithCleanups(self.this_tree.set_parent_trees)
471
499
        for revision_id in new_parents:
472
500
            try:
473
501
                tree = self.revision_tree(revision_id)
504
532
                raise errors.NoCommits(self.other_branch)
505
533
        if self.other_rev_id is not None:
506
534
            self._cached_trees[self.other_rev_id] = self.other_tree
507
 
        self._maybe_fetch(self.other_branch, self.this_branch, self.other_basis)
 
535
        self._maybe_fetch(self.other_branch,self.this_branch, self.other_basis)
508
536
 
509
537
    def set_other_revision(self, revision_id, other_branch):
510
538
        """Set 'other' based on a branch and revision id
548
576
            elif len(lcas) == 1:
549
577
                self.base_rev_id = list(lcas)[0]
550
578
            else: # len(lcas) > 1
551
 
                self._is_criss_cross = True
552
579
                if len(lcas) > 2:
553
580
                    # find_unique_lca can only handle 2 nodes, so we have to
554
581
                    # start back at the beginning. It is a shame to traverse
559
586
                else:
560
587
                    self.base_rev_id = self.revision_graph.find_unique_lca(
561
588
                                            *lcas)
562
 
                sorted_lca_keys = self.revision_graph.find_merge_order(
563
 
                    revisions[0], lcas)
564
 
                if self.base_rev_id == _mod_revision.NULL_REVISION:
565
 
                    self.base_rev_id = sorted_lca_keys[0]
566
 
 
 
589
                self._is_criss_cross = True
567
590
            if self.base_rev_id == _mod_revision.NULL_REVISION:
568
591
                raise errors.UnrelatedBranches()
569
592
            if self._is_criss_cross:
570
593
                trace.warning('Warning: criss-cross merge encountered.  See bzr'
571
594
                              ' help criss-cross.')
572
595
                trace.mutter('Criss-cross lcas: %r' % lcas)
573
 
                if self.base_rev_id in lcas:
574
 
                    trace.mutter('Unable to find unique lca. '
575
 
                                 'Fallback %r as best option.'
576
 
                                 % self.base_rev_id)
577
 
                interesting_revision_ids = set(lcas)
578
 
                interesting_revision_ids.add(self.base_rev_id)
 
596
                interesting_revision_ids = [self.base_rev_id]
 
597
                interesting_revision_ids.extend(lcas)
579
598
                interesting_trees = dict((t.get_revision_id(), t)
580
599
                    for t in self.this_branch.repository.revision_trees(
581
600
                        interesting_revision_ids))
582
601
                self._cached_trees.update(interesting_trees)
583
 
                if self.base_rev_id in lcas:
584
 
                    self.base_tree = interesting_trees[self.base_rev_id]
585
 
                else:
586
 
                    self.base_tree = interesting_trees.pop(self.base_rev_id)
 
602
                self.base_tree = interesting_trees.pop(self.base_rev_id)
 
603
                sorted_lca_keys = self.revision_graph.find_merge_order(
 
604
                    revisions[0], lcas)
587
605
                self._lca_trees = [interesting_trees[key]
588
606
                                   for key in sorted_lca_keys]
589
607
            else:
612
630
            self._maybe_fetch(base_branch, self.this_branch, self.base_rev_id)
613
631
 
614
632
    def make_merger(self):
615
 
        kwargs = {'working_tree': self.this_tree, 'this_tree': self.this_tree,
 
633
        kwargs = {'working_tree':self.this_tree, 'this_tree': self.this_tree,
616
634
                  'other_tree': self.other_tree,
617
635
                  'interesting_ids': self.interesting_ids,
618
636
                  'interesting_files': self.interesting_files,
619
637
                  'this_branch': self.this_branch,
620
 
                  'other_branch': self.other_branch,
621
638
                  'do_merge': False}
622
639
        if self.merge_type.requires_base:
623
640
            kwargs['base_tree'] = self.base_tree
649
666
        merge = self.make_merger()
650
667
        if self.other_branch is not None:
651
668
            self.other_branch.update_references(self.this_branch)
652
 
        for hook in Merger.hooks['pre_merge']:
653
 
            hook(merge)
654
669
        merge.do_merge()
655
 
        for hook in Merger.hooks['post_merge']:
656
 
            hook(merge)
657
670
        if self.recurse == 'down':
658
671
            for relpath, file_id in self.this_tree.iter_references():
659
672
                sub_tree = self.this_tree.get_nested_tree(file_id, relpath)
663
676
                    continue
664
677
                sub_merge = Merger(sub_tree.branch, this_tree=sub_tree)
665
678
                sub_merge.merge_type = self.merge_type
666
 
                other_branch = self.other_branch.reference_parent(file_id,
667
 
                                                                  relpath)
 
679
                other_branch = self.other_branch.reference_parent(file_id, relpath)
668
680
                sub_merge.set_other_revision(other_revision, other_branch)
669
681
                base_revision = self.base_tree.get_reference_revision(file_id)
670
682
                sub_merge.base_tree = \
674
686
        return merge
675
687
 
676
688
    def do_merge(self):
677
 
        operation = cleanup.OperationWithCleanups(self._do_merge_to)
 
689
        operation = OperationWithCleanups(self._do_merge_to)
678
690
        self.this_tree.lock_tree_write()
679
691
        operation.add_cleanup(self.this_tree.unlock)
680
692
        if self.base_tree is not None:
686
698
        merge = operation.run_simple()
687
699
        if len(merge.cooked_conflicts) == 0:
688
700
            if not self.ignore_zero and not trace.is_quiet():
689
 
                trace.note(gettext("All changes applied successfully."))
 
701
                trace.note("All changes applied successfully.")
690
702
        else:
691
 
            trace.note(gettext("%d conflicts encountered.")
 
703
            trace.note("%d conflicts encountered."
692
704
                       % len(merge.cooked_conflicts))
693
705
 
694
706
        return len(merge.cooked_conflicts)
726
738
                 interesting_ids=None, reprocess=False, show_base=False,
727
739
                 pb=None, pp=None, change_reporter=None,
728
740
                 interesting_files=None, do_merge=True,
729
 
                 cherrypick=False, lca_trees=None, this_branch=None,
730
 
                 other_branch=None):
 
741
                 cherrypick=False, lca_trees=None, this_branch=None):
731
742
        """Initialize the merger object and perform the merge.
732
743
 
733
744
        :param working_tree: The working tree to apply the merge to
736
747
        :param other_tree: The other tree to merge changes from
737
748
        :param this_branch: The branch associated with this_tree.  Defaults to
738
749
            this_tree.branch if not supplied.
739
 
        :param other_branch: The branch associated with other_tree, if any.
740
750
        :param interesting_ids: The file_ids of files that should be
741
751
            participate in the merge.  May not be combined with
742
752
            interesting_files.
764
774
            this_branch = this_tree.branch
765
775
        self.interesting_ids = interesting_ids
766
776
        self.interesting_files = interesting_files
767
 
        self.working_tree = working_tree
768
 
        self.this_tree = this_tree
 
777
        self.this_tree = working_tree
769
778
        self.base_tree = base_tree
770
779
        self.other_tree = other_tree
771
780
        self.this_branch = this_branch
772
 
        self.other_branch = other_branch
773
781
        self._raw_conflicts = []
774
782
        self.cooked_conflicts = []
775
783
        self.reprocess = reprocess
790
798
            warnings.warn("pb argument to Merge3Merger is deprecated")
791
799
 
792
800
    def do_merge(self):
793
 
        operation = cleanup.OperationWithCleanups(self._do_merge)
794
 
        self.working_tree.lock_tree_write()
795
 
        operation.add_cleanup(self.working_tree.unlock)
796
 
        self.this_tree.lock_read()
 
801
        operation = OperationWithCleanups(self._do_merge)
 
802
        self.this_tree.lock_tree_write()
797
803
        operation.add_cleanup(self.this_tree.unlock)
798
804
        self.base_tree.lock_read()
799
805
        operation.add_cleanup(self.base_tree.unlock)
802
808
        operation.run()
803
809
 
804
810
    def _do_merge(self, operation):
805
 
        self.tt = transform.TreeTransform(self.working_tree, None)
 
811
        self.tt = transform.TreeTransform(self.this_tree, None)
806
812
        operation.add_cleanup(self.tt.finalize)
807
813
        self._compute_transform()
808
814
        results = self.tt.apply(no_conflicts=True)
809
815
        self.write_modified(results)
810
816
        try:
811
 
            self.working_tree.add_conflicts(self.cooked_conflicts)
 
817
            self.this_tree.add_conflicts(self.cooked_conflicts)
812
818
        except errors.UnsupportedOperation:
813
819
            pass
814
820
 
815
821
    def make_preview_transform(self):
816
 
        operation = cleanup.OperationWithCleanups(self._make_preview_transform)
 
822
        operation = OperationWithCleanups(self._make_preview_transform)
817
823
        self.base_tree.lock_read()
818
824
        operation.add_cleanup(self.base_tree.unlock)
819
825
        self.other_tree.lock_read()
821
827
        return operation.run_simple()
822
828
 
823
829
    def _make_preview_transform(self):
824
 
        self.tt = transform.TransformPreview(self.working_tree)
 
830
        self.tt = transform.TransformPreview(self.this_tree)
825
831
        self._compute_transform()
826
832
        return self.tt
827
833
 
832
838
        else:
833
839
            entries = self._entries_lca()
834
840
            resolver = self._lca_multi_way
835
 
        # Prepare merge hooks
836
 
        factories = Merger.hooks['merge_file_content']
837
 
        # One hook for each registered one plus our default merger
838
 
        hooks = [factory(self) for factory in factories] + [self]
839
 
        self.active_hooks = [hook for hook in hooks if hook is not None]
840
841
        child_pb = ui.ui_factory.nested_progress_bar()
841
842
        try:
 
843
            factories = Merger.hooks['merge_file_content']
 
844
            hooks = [factory(self) for factory in factories] + [self]
 
845
            self.active_hooks = [hook for hook in hooks if hook is not None]
842
846
            for num, (file_id, changed, parents3, names3,
843
847
                      executable3) in enumerate(entries):
844
 
                # Try merging each entry
845
 
                child_pb.update(gettext('Preparing file merge'),
846
 
                                num, len(entries))
 
848
                child_pb.update('Preparing file merge', num, len(entries))
847
849
                self._merge_names(file_id, parents3, names3, resolver=resolver)
848
850
                if changed:
849
851
                    file_status = self._do_merge_contents(file_id)
853
855
                    executable3, file_status, resolver=resolver)
854
856
        finally:
855
857
            child_pb.finished()
856
 
        self.tt.fixup_new_roots()
857
 
        self._finish_computing_transform()
858
 
 
859
 
    def _finish_computing_transform(self):
860
 
        """Finalize the transform and report the changes.
861
 
 
862
 
        This is the second half of _compute_transform.
863
 
        """
 
858
        self.fix_root()
864
859
        child_pb = ui.ui_factory.nested_progress_bar()
865
860
        try:
866
861
            fs_conflicts = transform.resolve_conflicts(self.tt, child_pb,
873
868
                self.tt.iter_changes(), self.change_reporter)
874
869
        self.cook_conflicts(fs_conflicts)
875
870
        for conflict in self.cooked_conflicts:
876
 
            trace.warning(unicode(conflict))
 
871
            trace.warning(conflict)
877
872
 
878
873
    def _entries3(self):
879
874
        """Gather data about files modified between three trees.
886
881
        """
887
882
        result = []
888
883
        iterator = self.other_tree.iter_changes(self.base_tree,
889
 
                specific_files=self.interesting_files,
 
884
                include_unchanged=True, specific_files=self.interesting_files,
890
885
                extra_trees=[self.this_tree])
891
886
        this_entries = dict((e.file_id, e) for p, e in
892
887
                            self.this_tree.iter_entries_by_dir(
918
913
        it then compares with THIS and BASE.
919
914
 
920
915
        For the multi-valued entries, the format will be (BASE, [lca1, lca2])
921
 
 
922
 
        :return: [(file_id, changed, parents, names, executable)], where:
923
 
 
924
 
            * file_id: Simple file_id of the entry
925
 
            * changed: Boolean, True if the kind or contents changed else False
926
 
            * parents: ((base, [parent_id, in, lcas]), parent_id_other,
927
 
                        parent_id_this)
928
 
            * names:   ((base, [name, in, lcas]), name_in_other, name_in_this)
929
 
            * executable: ((base, [exec, in, lcas]), exec_in_other,
930
 
                        exec_in_this)
 
916
        :return: [(file_id, changed, parents, names, executable)]
 
917
            file_id     Simple file_id of the entry
 
918
            changed     Boolean, True if the kind or contents changed
 
919
                        else False
 
920
            parents     ((base, [parent_id, in, lcas]), parent_id_other,
 
921
                         parent_id_this)
 
922
            names       ((base, [name, in, lcas]), name_in_other, name_in_this)
 
923
            executable  ((base, [exec, in, lcas]), exec_in_other, exec_in_this)
931
924
        """
932
925
        if self.interesting_files is not None:
933
926
            lookup_trees = [self.this_tree, self.base_tree]
940
933
        result = []
941
934
        walker = _mod_tree.MultiWalker(self.other_tree, self._lca_trees)
942
935
 
943
 
        base_inventory = self.base_tree.root_inventory
944
 
        this_inventory = self.this_tree.root_inventory
 
936
        base_inventory = self.base_tree.inventory
 
937
        this_inventory = self.this_tree.inventory
945
938
        for path, file_id, other_ie, lca_values in walker.iter_all():
946
939
            # Is this modified at all from any of the other trees?
947
940
            if other_ie is None:
975
968
                else:
976
969
                    lca_entries.append(lca_ie)
977
970
 
978
 
            if base_inventory.has_id(file_id):
 
971
            if file_id in base_inventory:
979
972
                base_ie = base_inventory[file_id]
980
973
            else:
981
974
                base_ie = _none_entry
982
975
 
983
 
            if this_inventory.has_id(file_id):
 
976
            if file_id in this_inventory:
984
977
                this_ie = this_inventory[file_id]
985
978
            else:
986
979
                this_ie = _none_entry
1078
1071
                          ))
1079
1072
        return result
1080
1073
 
 
1074
 
 
1075
    def fix_root(self):
 
1076
        try:
 
1077
            self.tt.final_kind(self.tt.root)
 
1078
        except errors.NoSuchFile:
 
1079
            self.tt.cancel_deletion(self.tt.root)
 
1080
        if self.tt.final_file_id(self.tt.root) is None:
 
1081
            self.tt.version_file(self.tt.tree_file_id(self.tt.root),
 
1082
                                 self.tt.root)
 
1083
        other_root_file_id = self.other_tree.get_root_id()
 
1084
        if other_root_file_id is None:
 
1085
            return
 
1086
        other_root = self.tt.trans_id_file_id(other_root_file_id)
 
1087
        if other_root == self.tt.root:
 
1088
            return
 
1089
        if self.other_tree.inventory.root.file_id in self.this_tree.inventory:
 
1090
            # the other tree's root is a non-root in the current tree (as when
 
1091
            # a previously unrelated branch is merged into another)
 
1092
            return
 
1093
        try:
 
1094
            self.tt.final_kind(other_root)
 
1095
            other_root_is_present = True
 
1096
        except errors.NoSuchFile:
 
1097
            # other_root doesn't have a physical representation. We still need
 
1098
            # to move any references to the actual root of the tree.
 
1099
            other_root_is_present = False
 
1100
        # 'other_tree.inventory.root' is not present in this tree. We are
 
1101
        # calling adjust_path for children which *want* to be present with a
 
1102
        # correct place to go.
 
1103
        for thing, child in self.other_tree.inventory.root.children.iteritems():
 
1104
            trans_id = self.tt.trans_id_file_id(child.file_id)
 
1105
            if not other_root_is_present:
 
1106
                # FIXME: Make final_kind returns None instead of raising
 
1107
                # NoSuchFile to avoid the ugly construct below -- vila 20100402
 
1108
                try:
 
1109
                    self.tt.final_kind(trans_id)
 
1110
                    # The item exist in the final tree and has a defined place
 
1111
                    # to go already.
 
1112
                    continue
 
1113
                except errors.NoSuchFile, e:
 
1114
                    pass
 
1115
            # Move the item into the root
 
1116
            self.tt.adjust_path(self.tt.final_name(trans_id),
 
1117
                                self.tt.root, trans_id)
 
1118
        if other_root_is_present:
 
1119
            self.tt.cancel_creation(other_root)
 
1120
            self.tt.cancel_versioning(other_root)
 
1121
 
1081
1122
    def write_modified(self, results):
1082
1123
        modified_hashes = {}
1083
1124
        for path in results.modified_paths:
1084
 
            file_id = self.working_tree.path2id(self.working_tree.relpath(path))
 
1125
            file_id = self.this_tree.path2id(self.this_tree.relpath(path))
1085
1126
            if file_id is None:
1086
1127
                continue
1087
 
            hash = self.working_tree.get_file_sha1(file_id)
 
1128
            hash = self.this_tree.get_file_sha1(file_id)
1088
1129
            if hash is None:
1089
1130
                continue
1090
1131
            modified_hashes[file_id] = hash
1091
 
        self.working_tree.set_merge_modified(modified_hashes)
 
1132
        self.this_tree.set_merge_modified(modified_hashes)
1092
1133
 
1093
1134
    @staticmethod
1094
1135
    def parent(entry, file_id):
1107
1148
    @staticmethod
1108
1149
    def contents_sha1(tree, file_id):
1109
1150
        """Determine the sha1 of the file contents (used as a key method)."""
1110
 
        if not tree.has_id(file_id):
 
1151
        if file_id not in tree:
1111
1152
            return None
1112
1153
        return tree.get_file_sha1(file_id)
1113
1154
 
1193
1234
        # At this point, the lcas disagree, and the tip disagree
1194
1235
        return 'conflict'
1195
1236
 
 
1237
    @staticmethod
 
1238
    @deprecated_method(deprecated_in((2, 2, 0)))
 
1239
    def scalar_three_way(this_tree, base_tree, other_tree, file_id, key):
 
1240
        """Do a three-way test on a scalar.
 
1241
        Return "this", "other" or "conflict", depending whether a value wins.
 
1242
        """
 
1243
        key_base = key(base_tree, file_id)
 
1244
        key_other = key(other_tree, file_id)
 
1245
        #if base == other, either they all agree, or only THIS has changed.
 
1246
        if key_base == key_other:
 
1247
            return "this"
 
1248
        key_this = key(this_tree, file_id)
 
1249
        # "Ambiguous clean merge"
 
1250
        if key_this == key_other:
 
1251
            return "this"
 
1252
        elif key_this == key_base:
 
1253
            return "other"
 
1254
        else:
 
1255
            return "conflict"
 
1256
 
1196
1257
    def merge_names(self, file_id):
1197
1258
        def get_entry(tree):
1198
 
            try:
1199
 
                return tree.root_inventory[file_id]
1200
 
            except errors.NoSuchId:
 
1259
            if tree.has_id(file_id):
 
1260
                return tree.inventory[file_id]
 
1261
            else:
1201
1262
                return None
1202
1263
        this_entry = get_entry(self.this_tree)
1203
1264
        other_entry = get_entry(self.other_tree)
1238
1299
            self._raw_conflicts.append(('path conflict', trans_id, file_id,
1239
1300
                                        this_parent, this_name,
1240
1301
                                        other_parent, other_name))
1241
 
        if not self.other_tree.has_id(file_id):
 
1302
        if other_name is None:
1242
1303
            # it doesn't matter whether the result was 'other' or
1243
 
            # 'conflict'-- if it has no file id, we leave it alone.
 
1304
            # 'conflict'-- if there's no 'other', we leave it alone.
1244
1305
            return
1245
1306
        parent_id = parents[self.winner_idx[parent_id_winner]]
1246
 
        name = names[self.winner_idx[name_winner]]
1247
 
        if parent_id is not None or name is not None:
 
1307
        if parent_id is not None:
1248
1308
            # if we get here, name_winner and parent_winner are set to safe
1249
1309
            # values.
1250
 
            if parent_id is None and name is not None:
1251
 
                # if parent_id is None and name is non-None, current file is
1252
 
                # the tree root.
1253
 
                if names[self.winner_idx[parent_id_winner]] != '':
1254
 
                    raise AssertionError(
1255
 
                        'File looks like a root, but named %s' %
1256
 
                        names[self.winner_idx[parent_id_winner]])
1257
 
                parent_trans_id = transform.ROOT_PARENT
1258
 
            else:
1259
 
                parent_trans_id = self.tt.trans_id_file_id(parent_id)
1260
 
            self.tt.adjust_path(name, parent_trans_id,
 
1310
            self.tt.adjust_path(names[self.winner_idx[name_winner]],
 
1311
                                self.tt.trans_id_file_id(parent_id),
1261
1312
                                self.tt.trans_id_file_id(file_id))
1262
1313
 
1263
1314
    def _do_merge_contents(self, file_id):
1264
1315
        """Performs a merge on file_id contents."""
1265
1316
        def contents_pair(tree):
1266
 
            if not tree.has_id(file_id):
 
1317
            if file_id not in tree:
1267
1318
                return (None, None)
1268
1319
            kind = tree.kind(file_id)
1269
1320
            if kind == "file":
1298
1349
        # We have a hypothetical conflict, but if we have files, then we
1299
1350
        # can try to merge the content
1300
1351
        trans_id = self.tt.trans_id_file_id(file_id)
1301
 
        params = MergeFileHookParams(self, file_id, trans_id, this_pair[0],
 
1352
        params = MergeHookParams(self, file_id, trans_id, this_pair[0],
1302
1353
            other_pair[0], winner)
1303
1354
        hooks = self.active_hooks
1304
1355
        hook_status = 'not_applicable'
1307
1358
            if hook_status != 'not_applicable':
1308
1359
                # Don't try any more hooks, this one applies.
1309
1360
                break
1310
 
        # If the merge ends up replacing the content of the file, we get rid of
1311
 
        # it at the end of this method (this variable is used to track the
1312
 
        # exceptions to this rule).
1313
 
        keep_this = False
1314
1361
        result = "modified"
1315
1362
        if hook_status == 'not_applicable':
1316
 
            # No merge hook was able to resolve the situation. Two cases exist:
1317
 
            # a content conflict or a duplicate one.
 
1363
            # This is a contents conflict, because none of the available
 
1364
            # functions could merge it.
1318
1365
            result = None
1319
1366
            name = self.tt.final_name(trans_id)
1320
1367
            parent_id = self.tt.final_parent(trans_id)
1321
 
            duplicate = False
1322
 
            inhibit_content_conflict = False
1323
 
            if params.this_kind is None: # file_id is not in THIS
1324
 
                # Is the name used for a different file_id ?
1325
 
                dupe_path = self.other_tree.id2path(file_id)
1326
 
                this_id = self.this_tree.path2id(dupe_path)
1327
 
                if this_id is not None:
1328
 
                    # Two entries for the same path
1329
 
                    keep_this = True
1330
 
                    # versioning the merged file will trigger a duplicate
1331
 
                    # conflict
1332
 
                    self.tt.version_file(file_id, trans_id)
1333
 
                    transform.create_from_tree(
1334
 
                        self.tt, trans_id, self.other_tree, file_id,
1335
 
                        filter_tree_path=self._get_filter_tree_path(file_id))
1336
 
                    inhibit_content_conflict = True
1337
 
            elif params.other_kind is None: # file_id is not in OTHER
1338
 
                # Is the name used for a different file_id ?
1339
 
                dupe_path = self.this_tree.id2path(file_id)
1340
 
                other_id = self.other_tree.path2id(dupe_path)
1341
 
                if other_id is not None:
1342
 
                    # Two entries for the same path again, but here, the other
1343
 
                    # entry will also be merged.  We simply inhibit the
1344
 
                    # 'content' conflict creation because we know OTHER will
1345
 
                    # create (or has already created depending on ordering) an
1346
 
                    # entry at the same path. This will trigger a 'duplicate'
1347
 
                    # conflict later.
1348
 
                    keep_this = True
1349
 
                    inhibit_content_conflict = True
1350
 
            if not inhibit_content_conflict:
1351
 
                if params.this_kind is not None:
1352
 
                    self.tt.unversion_file(trans_id)
1353
 
                # This is a contents conflict, because none of the available
1354
 
                # functions could merge it.
1355
 
                file_group = self._dump_conflicts(name, parent_id, file_id,
1356
 
                                                  set_version=True)
1357
 
                self._raw_conflicts.append(('contents conflict', file_group))
 
1368
            if self.this_tree.has_id(file_id):
 
1369
                self.tt.unversion_file(trans_id)
 
1370
            file_group = self._dump_conflicts(name, parent_id, file_id,
 
1371
                                              set_version=True)
 
1372
            self._raw_conflicts.append(('contents conflict', file_group))
1358
1373
        elif hook_status == 'success':
1359
1374
            self.tt.create_file(lines, trans_id)
1360
1375
        elif hook_status == 'conflicted':
1376
1391
            raise AssertionError('unknown hook_status: %r' % (hook_status,))
1377
1392
        if not self.this_tree.has_id(file_id) and result == "modified":
1378
1393
            self.tt.version_file(file_id, trans_id)
1379
 
        if not keep_this:
1380
 
            # The merge has been performed and produced a new content, so the
1381
 
            # old contents should not be retained.
 
1394
        # The merge has been performed, so the old contents should not be
 
1395
        # retained.
 
1396
        try:
1382
1397
            self.tt.delete_contents(trans_id)
 
1398
        except errors.NoSuchFile:
 
1399
            pass
1383
1400
        return result
1384
1401
 
1385
1402
    def _default_other_winner_merge(self, merge_hook_params):
1386
1403
        """Replace this contents with other."""
1387
1404
        file_id = merge_hook_params.file_id
1388
1405
        trans_id = merge_hook_params.trans_id
 
1406
        file_in_this = self.this_tree.has_id(file_id)
1389
1407
        if self.other_tree.has_id(file_id):
1390
1408
            # OTHER changed the file
1391
 
            transform.create_from_tree(
1392
 
                self.tt, trans_id, self.other_tree, file_id,
1393
 
                filter_tree_path=self._get_filter_tree_path(file_id))
 
1409
            wt = self.this_tree
 
1410
            if wt.supports_content_filtering():
 
1411
                # We get the path from the working tree if it exists.
 
1412
                # That fails though when OTHER is adding a file, so
 
1413
                # we fall back to the other tree to find the path if
 
1414
                # it doesn't exist locally.
 
1415
                try:
 
1416
                    filter_tree_path = wt.id2path(file_id)
 
1417
                except errors.NoSuchId:
 
1418
                    filter_tree_path = self.other_tree.id2path(file_id)
 
1419
            else:
 
1420
                # Skip the id2path lookup for older formats
 
1421
                filter_tree_path = None
 
1422
            transform.create_from_tree(self.tt, trans_id,
 
1423
                             self.other_tree, file_id,
 
1424
                             filter_tree_path=filter_tree_path)
1394
1425
            return 'done', None
1395
 
        elif self.this_tree.has_id(file_id):
 
1426
        elif file_in_this:
1396
1427
            # OTHER deleted the file
1397
1428
            return 'delete', None
1398
1429
        else:
1424
1455
    def get_lines(self, tree, file_id):
1425
1456
        """Return the lines in a file, or an empty list."""
1426
1457
        if tree.has_id(file_id):
1427
 
            return tree.get_file_lines(file_id)
 
1458
            return tree.get_file(file_id).readlines()
1428
1459
        else:
1429
1460
            return []
1430
1461
 
1472
1503
                                              other_lines)
1473
1504
            file_group.append(trans_id)
1474
1505
 
1475
 
 
1476
 
    def _get_filter_tree_path(self, file_id):
1477
 
        if self.this_tree.supports_content_filtering():
1478
 
            # We get the path from the working tree if it exists.
1479
 
            # That fails though when OTHER is adding a file, so
1480
 
            # we fall back to the other tree to find the path if
1481
 
            # it doesn't exist locally.
1482
 
            try:
1483
 
                return self.this_tree.id2path(file_id)
1484
 
            except errors.NoSuchId:
1485
 
                return self.other_tree.id2path(file_id)
1486
 
        # Skip the id2path lookup for older formats
1487
 
        return None
1488
 
 
1489
1506
    def _dump_conflicts(self, name, parent_id, file_id, this_lines=None,
1490
1507
                        base_lines=None, other_lines=None, set_version=False,
1491
1508
                        no_base=False):
1557
1574
        if winner == 'this' and file_status != "modified":
1558
1575
            return
1559
1576
        trans_id = self.tt.trans_id_file_id(file_id)
1560
 
        if self.tt.final_kind(trans_id) != "file":
 
1577
        try:
 
1578
            if self.tt.final_kind(trans_id) != "file":
 
1579
                return
 
1580
        except errors.NoSuchFile:
1561
1581
            return
1562
1582
        if winner == "this":
1563
1583
            executability = this_executable
1574
1594
 
1575
1595
    def cook_conflicts(self, fs_conflicts):
1576
1596
        """Convert all conflicts into a form that doesn't depend on trans_id"""
1577
 
        content_conflict_file_ids = set()
1578
 
        cooked_conflicts = transform.cook_conflicts(fs_conflicts, self.tt)
 
1597
        self.cooked_conflicts.extend(transform.cook_conflicts(
 
1598
                fs_conflicts, self.tt))
1579
1599
        fp = transform.FinalPaths(self.tt)
1580
1600
        for conflict in self._raw_conflicts:
1581
1601
            conflict_type = conflict[0]
1592
1612
                if other_parent is None or other_name is None:
1593
1613
                    other_path = '<deleted>'
1594
1614
                else:
1595
 
                    if other_parent == self.other_tree.get_root_id():
1596
 
                        # The tree transform doesn't know about the other root,
1597
 
                        # so we special case here to avoid a NoFinalPath
1598
 
                        # exception
1599
 
                        parent_path = ''
1600
 
                    else:
1601
 
                        parent_path =  fp.get_path(
1602
 
                            self.tt.trans_id_file_id(other_parent))
 
1615
                    parent_path =  fp.get_path(
 
1616
                        self.tt.trans_id_file_id(other_parent))
1603
1617
                    other_path = osutils.pathjoin(parent_path, other_name)
1604
1618
                c = _mod_conflicts.Conflict.factory(
1605
1619
                    'path conflict', path=this_path,
1609
1623
                for trans_id in conflict[1]:
1610
1624
                    file_id = self.tt.final_file_id(trans_id)
1611
1625
                    if file_id is not None:
1612
 
                        # Ok we found the relevant file-id
1613
1626
                        break
1614
1627
                path = fp.get_path(trans_id)
1615
1628
                for suffix in ('.BASE', '.THIS', '.OTHER'):
1616
1629
                    if path.endswith(suffix):
1617
 
                        # Here is the raw path
1618
1630
                        path = path[:-len(suffix)]
1619
1631
                        break
1620
1632
                c = _mod_conflicts.Conflict.factory(conflict_type,
1621
1633
                                                    path=path, file_id=file_id)
1622
 
                content_conflict_file_ids.add(file_id)
1623
1634
            elif conflict_type == 'text conflict':
1624
1635
                trans_id = conflict[1]
1625
1636
                path = fp.get_path(trans_id)
1628
1639
                                                    path=path, file_id=file_id)
1629
1640
            else:
1630
1641
                raise AssertionError('bad conflict type: %r' % (conflict,))
1631
 
            cooked_conflicts.append(c)
1632
 
 
1633
 
        self.cooked_conflicts = []
1634
 
        # We want to get rid of path conflicts when a corresponding contents
1635
 
        # conflict exists. This can occur when one branch deletes a file while
1636
 
        # the other renames *and* modifies it. In this case, the content
1637
 
        # conflict is enough.
1638
 
        for c in cooked_conflicts:
1639
 
            if (c.typestring == 'path conflict'
1640
 
                and c.file_id in content_conflict_file_ids):
1641
 
                continue
1642
1642
            self.cooked_conflicts.append(c)
1643
1643
        self.cooked_conflicts.sort(key=_mod_conflicts.Conflict.sort_key)
1644
1644
 
1750
1750
            osutils.rmtree(temp_dir)
1751
1751
 
1752
1752
 
1753
 
class PathNotInTree(errors.BzrError):
1754
 
 
1755
 
    _fmt = """Merge-into failed because %(tree)s does not contain %(path)s."""
1756
 
 
1757
 
    def __init__(self, path, tree):
1758
 
        errors.BzrError.__init__(self, path=path, tree=tree)
1759
 
 
1760
 
 
1761
 
class MergeIntoMerger(Merger):
1762
 
    """Merger that understands other_tree will be merged into a subdir.
1763
 
 
1764
 
    This also changes the Merger api so that it uses real Branch, revision_id,
1765
 
    and RevisonTree objects, rather than using revision specs.
1766
 
    """
1767
 
 
1768
 
    def __init__(self, this_tree, other_branch, other_tree, target_subdir,
1769
 
            source_subpath, other_rev_id=None):
1770
 
        """Create a new MergeIntoMerger object.
1771
 
 
1772
 
        source_subpath in other_tree will be effectively copied to
1773
 
        target_subdir in this_tree.
1774
 
 
1775
 
        :param this_tree: The tree that we will be merging into.
1776
 
        :param other_branch: The Branch we will be merging from.
1777
 
        :param other_tree: The RevisionTree object we want to merge.
1778
 
        :param target_subdir: The relative path where we want to merge
1779
 
            other_tree into this_tree
1780
 
        :param source_subpath: The relative path specifying the subtree of
1781
 
            other_tree to merge into this_tree.
1782
 
        """
1783
 
        # It is assumed that we are merging a tree that is not in our current
1784
 
        # ancestry, which means we are using the "EmptyTree" as our basis.
1785
 
        null_ancestor_tree = this_tree.branch.repository.revision_tree(
1786
 
                                _mod_revision.NULL_REVISION)
1787
 
        super(MergeIntoMerger, self).__init__(
1788
 
            this_branch=this_tree.branch,
1789
 
            this_tree=this_tree,
1790
 
            other_tree=other_tree,
1791
 
            base_tree=null_ancestor_tree,
1792
 
            )
1793
 
        self._target_subdir = target_subdir
1794
 
        self._source_subpath = source_subpath
1795
 
        self.other_branch = other_branch
1796
 
        if other_rev_id is None:
1797
 
            other_rev_id = other_tree.get_revision_id()
1798
 
        self.other_rev_id = self.other_basis = other_rev_id
1799
 
        self.base_is_ancestor = True
1800
 
        self.backup_files = True
1801
 
        self.merge_type = Merge3Merger
1802
 
        self.show_base = False
1803
 
        self.reprocess = False
1804
 
        self.interesting_ids = None
1805
 
        self.merge_type = _MergeTypeParameterizer(MergeIntoMergeType,
1806
 
              target_subdir=self._target_subdir,
1807
 
              source_subpath=self._source_subpath)
1808
 
        if self._source_subpath != '':
1809
 
            # If this isn't a partial merge make sure the revisions will be
1810
 
            # present.
1811
 
            self._maybe_fetch(self.other_branch, self.this_branch,
1812
 
                self.other_basis)
1813
 
 
1814
 
    def set_pending(self):
1815
 
        if self._source_subpath != '':
1816
 
            return
1817
 
        Merger.set_pending(self)
1818
 
 
1819
 
 
1820
 
class _MergeTypeParameterizer(object):
1821
 
    """Wrap a merge-type class to provide extra parameters.
1822
 
    
1823
 
    This is hack used by MergeIntoMerger to pass some extra parameters to its
1824
 
    merge_type.  Merger.do_merge() sets up its own set of parameters to pass to
1825
 
    the 'merge_type' member.  It is difficult override do_merge without
1826
 
    re-writing the whole thing, so instead we create a wrapper which will pass
1827
 
    the extra parameters.
1828
 
    """
1829
 
 
1830
 
    def __init__(self, merge_type, **kwargs):
1831
 
        self._extra_kwargs = kwargs
1832
 
        self._merge_type = merge_type
1833
 
 
1834
 
    def __call__(self, *args, **kwargs):
1835
 
        kwargs.update(self._extra_kwargs)
1836
 
        return self._merge_type(*args, **kwargs)
1837
 
 
1838
 
    def __getattr__(self, name):
1839
 
        return getattr(self._merge_type, name)
1840
 
 
1841
 
 
1842
 
class MergeIntoMergeType(Merge3Merger):
1843
 
    """Merger that incorporates a tree (or part of a tree) into another."""
1844
 
 
1845
 
    def __init__(self, *args, **kwargs):
1846
 
        """Initialize the merger object.
1847
 
 
1848
 
        :param args: See Merge3Merger.__init__'s args.
1849
 
        :param kwargs: See Merge3Merger.__init__'s keyword args, except for
1850
 
            source_subpath and target_subdir.
1851
 
        :keyword source_subpath: The relative path specifying the subtree of
1852
 
            other_tree to merge into this_tree.
1853
 
        :keyword target_subdir: The relative path where we want to merge
1854
 
            other_tree into this_tree
1855
 
        """
1856
 
        # All of the interesting work happens during Merge3Merger.__init__(),
1857
 
        # so we have have to hack in to get our extra parameters set.
1858
 
        self._source_subpath = kwargs.pop('source_subpath')
1859
 
        self._target_subdir = kwargs.pop('target_subdir')
1860
 
        super(MergeIntoMergeType, self).__init__(*args, **kwargs)
1861
 
 
1862
 
    def _compute_transform(self):
1863
 
        child_pb = ui.ui_factory.nested_progress_bar()
1864
 
        try:
1865
 
            entries = self._entries_to_incorporate()
1866
 
            entries = list(entries)
1867
 
            for num, (entry, parent_id) in enumerate(entries):
1868
 
                child_pb.update(gettext('Preparing file merge'), num, len(entries))
1869
 
                parent_trans_id = self.tt.trans_id_file_id(parent_id)
1870
 
                trans_id = transform.new_by_entry(self.tt, entry,
1871
 
                    parent_trans_id, self.other_tree)
1872
 
        finally:
1873
 
            child_pb.finished()
1874
 
        self._finish_computing_transform()
1875
 
 
1876
 
    def _entries_to_incorporate(self):
1877
 
        """Yields pairs of (inventory_entry, new_parent)."""
1878
 
        other_inv = self.other_tree.root_inventory
1879
 
        subdir_id = other_inv.path2id(self._source_subpath)
1880
 
        if subdir_id is None:
1881
 
            # XXX: The error would be clearer if it gave the URL of the source
1882
 
            # branch, but we don't have a reference to that here.
1883
 
            raise PathNotInTree(self._source_subpath, "Source tree")
1884
 
        subdir = other_inv[subdir_id]
1885
 
        parent_in_target = osutils.dirname(self._target_subdir)
1886
 
        target_id = self.this_tree.path2id(parent_in_target)
1887
 
        if target_id is None:
1888
 
            raise PathNotInTree(self._target_subdir, "Target tree")
1889
 
        name_in_target = osutils.basename(self._target_subdir)
1890
 
        merge_into_root = subdir.copy()
1891
 
        merge_into_root.name = name_in_target
1892
 
        if self.this_tree.has_id(merge_into_root.file_id):
1893
 
            # Give the root a new file-id.
1894
 
            # This can happen fairly easily if the directory we are
1895
 
            # incorporating is the root, and both trees have 'TREE_ROOT' as
1896
 
            # their root_id.  Users will expect this to Just Work, so we
1897
 
            # change the file-id here.
1898
 
            # Non-root file-ids could potentially conflict too.  That's really
1899
 
            # an edge case, so we don't do anything special for those.  We let
1900
 
            # them cause conflicts.
1901
 
            merge_into_root.file_id = generate_ids.gen_file_id(name_in_target)
1902
 
        yield (merge_into_root, target_id)
1903
 
        if subdir.kind != 'directory':
1904
 
            # No children, so we are done.
1905
 
            return
1906
 
        for ignored_path, entry in other_inv.iter_entries_by_dir(subdir_id):
1907
 
            parent_id = entry.parent_id
1908
 
            if parent_id == subdir.file_id:
1909
 
                # The root's parent ID has changed, so make sure children of
1910
 
                # the root refer to the new ID.
1911
 
                parent_id = merge_into_root.file_id
1912
 
            yield (entry, parent_id)
1913
 
 
1914
 
 
1915
1753
def merge_inner(this_branch, other_tree, base_tree, ignore_zero=False,
1916
1754
                backup_files=False,
1917
1755
                merge_type=Merge3Merger,
1925
1763
                change_reporter=None):
1926
1764
    """Primary interface for merging.
1927
1765
 
1928
 
    Typical use is probably::
1929
 
 
1930
 
        merge_inner(branch, branch.get_revision_tree(other_revision),
1931
 
                    branch.get_revision_tree(base_revision))
1932
 
    """
 
1766
        typical use is probably
 
1767
        'merge_inner(branch, branch.get_revision_tree(other_revision),
 
1768
                     branch.get_revision_tree(base_revision))'
 
1769
        """
1933
1770
    if this_tree is None:
1934
1771
        raise errors.BzrError("bzrlib.merge.merge_inner requires a this_tree "
1935
 
                              "parameter")
 
1772
                              "parameter as of bzrlib version 0.8.")
1936
1773
    merger = Merger(this_branch, other_tree, base_tree, this_tree=this_tree,
1937
1774
                    pb=pb, change_reporter=change_reporter)
1938
1775
    merger.backup_files = backup_files
1955
1792
    merger.set_base_revision(get_revision_id(), this_branch)
1956
1793
    return merger.do_merge()
1957
1794
 
1958
 
 
1959
 
merge_type_registry = registry.Registry()
1960
 
merge_type_registry.register('diff3', Diff3Merger,
1961
 
                             "Merge using external diff3.")
1962
 
merge_type_registry.register('lca', LCAMerger,
1963
 
                             "LCA-newness merge.")
1964
 
merge_type_registry.register('merge3', Merge3Merger,
1965
 
                             "Native diff3-style merge.")
1966
 
merge_type_registry.register('weave', WeaveMerger,
1967
 
                             "Weave-based merge.")
1968
 
 
1969
 
 
1970
1795
def get_merge_type_registry():
1971
 
    """Merge type registry was previously in bzrlib.option
 
1796
    """Merge type registry is in bzrlib.option to avoid circular imports.
1972
1797
 
1973
 
    This method provides a backwards compatible way to retrieve it.
 
1798
    This method provides a sanctioned way to retrieve it.
1974
1799
    """
1975
 
    return merge_type_registry
 
1800
    from bzrlib import option
 
1801
    return option._merge_type_registry
1976
1802
 
1977
1803
 
1978
1804
def _plan_annotate_merge(annotated_a, annotated_b, ancestors_a, ancestors_b):
2403
2229
class _PlanLCAMerge(_PlanMergeBase):
2404
2230
    """
2405
2231
    This merge algorithm differs from _PlanMerge in that:
2406
 
 
2407
2232
    1. comparisons are done against LCAs only
2408
2233
    2. cases where a contested line is new versus one LCA but old versus
2409
2234
       another are marked as conflicts, by emitting the line as conflicted-a
2450
2275
 
2451
2276
        If a line is killed and new, this indicates that the two merge
2452
2277
        revisions contain differing conflict resolutions.
2453
 
 
2454
2278
        :param revision_id: The id of the revision in which the lines are
2455
2279
            unique
2456
2280
        :param unique_line_numbers: The line numbers of unique lines.
2457
 
        :return: a tuple of (new_this, killed_other)
 
2281
        :return a tuple of (new_this, killed_other):
2458
2282
        """
2459
2283
        new = set()
2460
2284
        killed = set()