~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/merge.py

  • Committer: Vincent Ladeuil
  • Date: 2016-01-21 17:48:07 UTC
  • mto: This revision was merged to the branch mainline in revision 6613.
  • Revision ID: v.ladeuil+lp@free.fr-20160121174807-g4ybpaij9ln5wj6a
Make all transport put_bytes() raises TypeError when given unicode strings rather than bytes.

There was a mix of AssertionError or UnicodeEncodeError.

Also deleted test_put_file_unicode() which was bogus, files contain bytes not unicode strings.

Show diffs side-by-side

added added

removed removed

Lines of Context:
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
 
17
19
import warnings
18
20
 
19
21
from bzrlib.lazy_import import lazy_import
38
40
    versionedfile,
39
41
    workingtree,
40
42
    )
 
43
from bzrlib.i18n import gettext
41
44
""")
42
45
from bzrlib import (
43
46
    decorators,
44
47
    errors,
45
48
    hooks,
 
49
    registry,
46
50
    )
47
51
from bzrlib.symbol_versioning import (
48
52
    deprecated_in,
74
78
            "See the AbstractPerFileMerger API docs for details on how it is "
75
79
            "used by merge.",
76
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))
77
90
 
78
91
 
79
92
class AbstractPerFileMerger(object):
91
104
    def merge_contents(self, merge_params):
92
105
        """Attempt to merge the contents of a single file.
93
106
        
94
 
        :param merge_params: A bzrlib.merge.MergeHookParams
 
107
        :param merge_params: A bzrlib.merge.MergeFileHookParams
95
108
        :return: A tuple of (status, chunks), where status is one of
96
109
            'not_applicable', 'success', 'conflicted', or 'delete'.  If status
97
110
            is 'success' or 'conflicted', then chunks should be an iterable of
118
131
 
119
132
    def get_filename(self, params, tree):
120
133
        """Lookup the filename (i.e. basename, not path), given a Tree (e.g.
121
 
        self.merger.this_tree) and a MergeHookParams.
 
134
        self.merger.this_tree) and a MergeFileHookParams.
122
135
        """
123
136
        return osutils.basename(tree.id2path(params.file_id))
124
137
 
125
138
    def get_filepath(self, params, tree):
126
139
        """Calculate the path to the file in a tree.
127
140
 
128
 
        :param params: A MergeHookParams describing the file to merge
 
141
        :param params: A MergeFileHookParams describing the file to merge
129
142
        :param tree: a Tree, e.g. self.merger.this_tree.
130
143
        """
131
144
        return tree.id2path(params.file_id)
138
151
            params.winner == 'other' or
139
152
            # THIS and OTHER aren't both files.
140
153
            not params.is_file_merge() or
141
 
            # The filename doesn't match *.xml
 
154
            # The filename doesn't match
142
155
            not self.file_matches(params)):
143
156
            return 'not_applicable', None
144
157
        return self.merge_matching(params)
220
233
        raise NotImplementedError(self.merge_text)
221
234
 
222
235
 
223
 
class MergeHookParams(object):
 
236
class MergeFileHookParams(object):
224
237
    """Object holding parameters passed to merge_file_content hooks.
225
238
 
226
239
    There are some fields hooks can access:
441
454
        revision_id = _mod_revision.ensure_null(revision_id)
442
455
        return branch, self.revision_tree(revision_id, branch)
443
456
 
444
 
    @deprecated_method(deprecated_in((2, 1, 0)))
445
 
    def ensure_revision_trees(self):
446
 
        if self.this_revision_tree is None:
447
 
            self.this_basis_tree = self.revision_tree(self.this_basis)
448
 
            if self.this_basis == self.this_rev_id:
449
 
                self.this_revision_tree = self.this_basis_tree
450
 
 
451
 
        if self.other_rev_id is None:
452
 
            other_basis_tree = self.revision_tree(self.other_basis)
453
 
            if other_basis_tree.has_changes(self.other_tree):
454
 
                raise errors.WorkingTreeNotRevision(self.this_tree)
455
 
            other_rev_id = self.other_basis
456
 
            self.other_tree = other_basis_tree
457
 
 
458
 
    @deprecated_method(deprecated_in((2, 1, 0)))
459
 
    def file_revisions(self, file_id):
460
 
        self.ensure_revision_trees()
461
 
        if self.this_rev_id is None:
462
 
            if self.this_basis_tree.get_file_sha1(file_id) != \
463
 
                self.this_tree.get_file_sha1(file_id):
464
 
                raise errors.WorkingTreeNotRevision(self.this_tree)
465
 
 
466
 
        trees = (self.this_basis_tree, self.other_tree)
467
 
        return [tree.get_file_revision(file_id) for tree in trees]
468
 
 
469
 
    @deprecated_method(deprecated_in((2, 1, 0)))
470
 
    def check_basis(self, check_clean, require_commits=True):
471
 
        if self.this_basis is None and require_commits is True:
472
 
            raise errors.BzrCommandError(
473
 
                "This branch has no commits."
474
 
                " (perhaps you would prefer 'bzr pull')")
475
 
        if check_clean:
476
 
            self.compare_basis()
477
 
            if self.this_basis != self.this_rev_id:
478
 
                raise errors.UncommittedChanges(self.this_tree)
479
 
 
480
 
    @deprecated_method(deprecated_in((2, 1, 0)))
481
 
    def compare_basis(self):
482
 
        try:
483
 
            basis_tree = self.revision_tree(self.this_tree.last_revision())
484
 
        except errors.NoSuchRevision:
485
 
            basis_tree = self.this_tree.basis_tree()
486
 
        if not self.this_tree.has_changes(basis_tree):
487
 
            self.this_rev_id = self.this_basis
488
 
 
489
457
    def set_interesting_files(self, file_list):
490
458
        self.interesting_files = file_list
491
459
 
536
504
                raise errors.NoCommits(self.other_branch)
537
505
        if self.other_rev_id is not None:
538
506
            self._cached_trees[self.other_rev_id] = self.other_tree
539
 
        self._maybe_fetch(self.other_branch,self.this_branch, self.other_basis)
 
507
        self._maybe_fetch(self.other_branch, self.this_branch, self.other_basis)
540
508
 
541
509
    def set_other_revision(self, revision_id, other_branch):
542
510
        """Set 'other' based on a branch and revision id
591
559
                else:
592
560
                    self.base_rev_id = self.revision_graph.find_unique_lca(
593
561
                                            *lcas)
594
 
                sorted_lca_keys = self.revision_graph.find_merge_order(                
 
562
                sorted_lca_keys = self.revision_graph.find_merge_order(
595
563
                    revisions[0], lcas)
596
564
                if self.base_rev_id == _mod_revision.NULL_REVISION:
597
565
                    self.base_rev_id = sorted_lca_keys[0]
598
 
                
 
566
 
599
567
            if self.base_rev_id == _mod_revision.NULL_REVISION:
600
568
                raise errors.UnrelatedBranches()
601
569
            if self._is_criss_cross:
604
572
                trace.mutter('Criss-cross lcas: %r' % lcas)
605
573
                if self.base_rev_id in lcas:
606
574
                    trace.mutter('Unable to find unique lca. '
607
 
                                 'Fallback %r as best option.' % self.base_rev_id)
 
575
                                 'Fallback %r as best option.'
 
576
                                 % self.base_rev_id)
608
577
                interesting_revision_ids = set(lcas)
609
578
                interesting_revision_ids.add(self.base_rev_id)
610
579
                interesting_trees = dict((t.get_revision_id(), t)
643
612
            self._maybe_fetch(base_branch, self.this_branch, self.base_rev_id)
644
613
 
645
614
    def make_merger(self):
646
 
        kwargs = {'working_tree':self.this_tree, 'this_tree': self.this_tree,
 
615
        kwargs = {'working_tree': self.this_tree, 'this_tree': self.this_tree,
647
616
                  'other_tree': self.other_tree,
648
617
                  'interesting_ids': self.interesting_ids,
649
618
                  'interesting_files': self.interesting_files,
650
619
                  'this_branch': self.this_branch,
 
620
                  'other_branch': self.other_branch,
651
621
                  'do_merge': False}
652
622
        if self.merge_type.requires_base:
653
623
            kwargs['base_tree'] = self.base_tree
679
649
        merge = self.make_merger()
680
650
        if self.other_branch is not None:
681
651
            self.other_branch.update_references(self.this_branch)
 
652
        for hook in Merger.hooks['pre_merge']:
 
653
            hook(merge)
682
654
        merge.do_merge()
 
655
        for hook in Merger.hooks['post_merge']:
 
656
            hook(merge)
683
657
        if self.recurse == 'down':
684
658
            for relpath, file_id in self.this_tree.iter_references():
685
659
                sub_tree = self.this_tree.get_nested_tree(file_id, relpath)
689
663
                    continue
690
664
                sub_merge = Merger(sub_tree.branch, this_tree=sub_tree)
691
665
                sub_merge.merge_type = self.merge_type
692
 
                other_branch = self.other_branch.reference_parent(file_id, relpath)
 
666
                other_branch = self.other_branch.reference_parent(file_id,
 
667
                                                                  relpath)
693
668
                sub_merge.set_other_revision(other_revision, other_branch)
694
669
                base_revision = self.base_tree.get_reference_revision(file_id)
695
670
                sub_merge.base_tree = \
711
686
        merge = operation.run_simple()
712
687
        if len(merge.cooked_conflicts) == 0:
713
688
            if not self.ignore_zero and not trace.is_quiet():
714
 
                trace.note("All changes applied successfully.")
 
689
                trace.note(gettext("All changes applied successfully."))
715
690
        else:
716
 
            trace.note("%d conflicts encountered."
 
691
            trace.note(gettext("%d conflicts encountered.")
717
692
                       % len(merge.cooked_conflicts))
718
693
 
719
694
        return len(merge.cooked_conflicts)
751
726
                 interesting_ids=None, reprocess=False, show_base=False,
752
727
                 pb=None, pp=None, change_reporter=None,
753
728
                 interesting_files=None, do_merge=True,
754
 
                 cherrypick=False, lca_trees=None, this_branch=None):
 
729
                 cherrypick=False, lca_trees=None, this_branch=None,
 
730
                 other_branch=None):
755
731
        """Initialize the merger object and perform the merge.
756
732
 
757
733
        :param working_tree: The working tree to apply the merge to
760
736
        :param other_tree: The other tree to merge changes from
761
737
        :param this_branch: The branch associated with this_tree.  Defaults to
762
738
            this_tree.branch if not supplied.
 
739
        :param other_branch: The branch associated with other_tree, if any.
763
740
        :param interesting_ids: The file_ids of files that should be
764
741
            participate in the merge.  May not be combined with
765
742
            interesting_files.
787
764
            this_branch = this_tree.branch
788
765
        self.interesting_ids = interesting_ids
789
766
        self.interesting_files = interesting_files
790
 
        self.this_tree = working_tree
 
767
        self.working_tree = working_tree
 
768
        self.this_tree = this_tree
791
769
        self.base_tree = base_tree
792
770
        self.other_tree = other_tree
793
771
        self.this_branch = this_branch
 
772
        self.other_branch = other_branch
794
773
        self._raw_conflicts = []
795
774
        self.cooked_conflicts = []
796
775
        self.reprocess = reprocess
812
791
 
813
792
    def do_merge(self):
814
793
        operation = cleanup.OperationWithCleanups(self._do_merge)
815
 
        self.this_tree.lock_tree_write()
 
794
        self.working_tree.lock_tree_write()
 
795
        operation.add_cleanup(self.working_tree.unlock)
 
796
        self.this_tree.lock_read()
816
797
        operation.add_cleanup(self.this_tree.unlock)
817
798
        self.base_tree.lock_read()
818
799
        operation.add_cleanup(self.base_tree.unlock)
821
802
        operation.run()
822
803
 
823
804
    def _do_merge(self, operation):
824
 
        self.tt = transform.TreeTransform(self.this_tree, None)
 
805
        self.tt = transform.TreeTransform(self.working_tree, None)
825
806
        operation.add_cleanup(self.tt.finalize)
826
807
        self._compute_transform()
827
808
        results = self.tt.apply(no_conflicts=True)
828
809
        self.write_modified(results)
829
810
        try:
830
 
            self.this_tree.add_conflicts(self.cooked_conflicts)
 
811
            self.working_tree.add_conflicts(self.cooked_conflicts)
831
812
        except errors.UnsupportedOperation:
832
813
            pass
833
814
 
840
821
        return operation.run_simple()
841
822
 
842
823
    def _make_preview_transform(self):
843
 
        self.tt = transform.TransformPreview(self.this_tree)
 
824
        self.tt = transform.TransformPreview(self.working_tree)
844
825
        self._compute_transform()
845
826
        return self.tt
846
827
 
851
832
        else:
852
833
            entries = self._entries_lca()
853
834
            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]
854
840
        child_pb = ui.ui_factory.nested_progress_bar()
855
841
        try:
856
 
            factories = Merger.hooks['merge_file_content']
857
 
            hooks = [factory(self) for factory in factories] + [self]
858
 
            self.active_hooks = [hook for hook in hooks if hook is not None]
859
842
            for num, (file_id, changed, parents3, names3,
860
843
                      executable3) in enumerate(entries):
861
 
                child_pb.update('Preparing file merge', num, len(entries))
 
844
                # Try merging each entry
 
845
                child_pb.update(gettext('Preparing file merge'),
 
846
                                num, len(entries))
862
847
                self._merge_names(file_id, parents3, names3, resolver=resolver)
863
848
                if changed:
864
849
                    file_status = self._do_merge_contents(file_id)
955
940
        result = []
956
941
        walker = _mod_tree.MultiWalker(self.other_tree, self._lca_trees)
957
942
 
958
 
        base_inventory = self.base_tree.inventory
959
 
        this_inventory = self.this_tree.inventory
 
943
        base_inventory = self.base_tree.root_inventory
 
944
        this_inventory = self.this_tree.root_inventory
960
945
        for path, file_id, other_ie, lca_values in walker.iter_all():
961
946
            # Is this modified at all from any of the other trees?
962
947
            if other_ie is None:
1093
1078
                          ))
1094
1079
        return result
1095
1080
 
1096
 
    @deprecated_method(deprecated_in((2, 4, 0)))
1097
 
    def fix_root(self):
1098
 
        if self.tt.final_kind(self.tt.root) is None:
1099
 
            self.tt.cancel_deletion(self.tt.root)
1100
 
        if self.tt.final_file_id(self.tt.root) is None:
1101
 
            self.tt.version_file(self.tt.tree_file_id(self.tt.root),
1102
 
                                 self.tt.root)
1103
 
        other_root_file_id = self.other_tree.get_root_id()
1104
 
        if other_root_file_id is None:
1105
 
            return
1106
 
        other_root = self.tt.trans_id_file_id(other_root_file_id)
1107
 
        if other_root == self.tt.root:
1108
 
            return
1109
 
        if self.this_tree.inventory.has_id(
1110
 
            self.other_tree.inventory.root.file_id):
1111
 
            # the other tree's root is a non-root in the current tree (as
1112
 
            # when a previously unrelated branch is merged into another)
1113
 
            return
1114
 
        if self.tt.final_kind(other_root) is not None:
1115
 
            other_root_is_present = True
1116
 
        else:
1117
 
            # other_root doesn't have a physical representation. We still need
1118
 
            # to move any references to the actual root of the tree.
1119
 
            other_root_is_present = False
1120
 
        # 'other_tree.inventory.root' is not present in this tree. We are
1121
 
        # calling adjust_path for children which *want* to be present with a
1122
 
        # correct place to go.
1123
 
        for _, child in self.other_tree.inventory.root.children.iteritems():
1124
 
            trans_id = self.tt.trans_id_file_id(child.file_id)
1125
 
            if not other_root_is_present:
1126
 
                if self.tt.final_kind(trans_id) is not None:
1127
 
                    # The item exist in the final tree and has a defined place
1128
 
                    # to go already.
1129
 
                    continue
1130
 
            # Move the item into the root
1131
 
            try:
1132
 
                final_name = self.tt.final_name(trans_id)
1133
 
            except errors.NoFinalPath:
1134
 
                # This file is not present anymore, ignore it.
1135
 
                continue
1136
 
            self.tt.adjust_path(final_name, self.tt.root, trans_id)
1137
 
        if other_root_is_present:
1138
 
            self.tt.cancel_creation(other_root)
1139
 
            self.tt.cancel_versioning(other_root)
1140
 
 
1141
1081
    def write_modified(self, results):
1142
1082
        modified_hashes = {}
1143
1083
        for path in results.modified_paths:
1144
 
            file_id = self.this_tree.path2id(self.this_tree.relpath(path))
 
1084
            file_id = self.working_tree.path2id(self.working_tree.relpath(path))
1145
1085
            if file_id is None:
1146
1086
                continue
1147
 
            hash = self.this_tree.get_file_sha1(file_id)
 
1087
            hash = self.working_tree.get_file_sha1(file_id)
1148
1088
            if hash is None:
1149
1089
                continue
1150
1090
            modified_hashes[file_id] = hash
1151
 
        self.this_tree.set_merge_modified(modified_hashes)
 
1091
        self.working_tree.set_merge_modified(modified_hashes)
1152
1092
 
1153
1093
    @staticmethod
1154
1094
    def parent(entry, file_id):
1253
1193
        # At this point, the lcas disagree, and the tip disagree
1254
1194
        return 'conflict'
1255
1195
 
1256
 
    @staticmethod
1257
 
    @deprecated_method(deprecated_in((2, 2, 0)))
1258
 
    def scalar_three_way(this_tree, base_tree, other_tree, file_id, key):
1259
 
        """Do a three-way test on a scalar.
1260
 
        Return "this", "other" or "conflict", depending whether a value wins.
1261
 
        """
1262
 
        key_base = key(base_tree, file_id)
1263
 
        key_other = key(other_tree, file_id)
1264
 
        #if base == other, either they all agree, or only THIS has changed.
1265
 
        if key_base == key_other:
1266
 
            return "this"
1267
 
        key_this = key(this_tree, file_id)
1268
 
        # "Ambiguous clean merge"
1269
 
        if key_this == key_other:
1270
 
            return "this"
1271
 
        elif key_this == key_base:
1272
 
            return "other"
1273
 
        else:
1274
 
            return "conflict"
1275
 
 
1276
1196
    def merge_names(self, file_id):
1277
1197
        def get_entry(tree):
1278
 
            if tree.has_id(file_id):
1279
 
                return tree.inventory[file_id]
1280
 
            else:
 
1198
            try:
 
1199
                return tree.root_inventory[file_id]
 
1200
            except errors.NoSuchId:
1281
1201
                return None
1282
1202
        this_entry = get_entry(self.this_tree)
1283
1203
        other_entry = get_entry(self.other_tree)
1378
1298
        # We have a hypothetical conflict, but if we have files, then we
1379
1299
        # can try to merge the content
1380
1300
        trans_id = self.tt.trans_id_file_id(file_id)
1381
 
        params = MergeHookParams(self, file_id, trans_id, this_pair[0],
 
1301
        params = MergeFileHookParams(self, file_id, trans_id, this_pair[0],
1382
1302
            other_pair[0], winner)
1383
1303
        hooks = self.active_hooks
1384
1304
        hook_status = 'not_applicable'
1387
1307
            if hook_status != 'not_applicable':
1388
1308
                # Don't try any more hooks, this one applies.
1389
1309
                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
1390
1314
        result = "modified"
1391
1315
        if hook_status == 'not_applicable':
1392
 
            # This is a contents conflict, because none of the available
1393
 
            # functions could merge it.
 
1316
            # No merge hook was able to resolve the situation. Two cases exist:
 
1317
            # a content conflict or a duplicate one.
1394
1318
            result = None
1395
1319
            name = self.tt.final_name(trans_id)
1396
1320
            parent_id = self.tt.final_parent(trans_id)
1397
 
            if self.this_tree.has_id(file_id):
1398
 
                self.tt.unversion_file(trans_id)
1399
 
            file_group = self._dump_conflicts(name, parent_id, file_id,
1400
 
                                              set_version=True)
1401
 
            self._raw_conflicts.append(('contents conflict', file_group))
 
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))
1402
1358
        elif hook_status == 'success':
1403
1359
            self.tt.create_file(lines, trans_id)
1404
1360
        elif hook_status == 'conflicted':
1420
1376
            raise AssertionError('unknown hook_status: %r' % (hook_status,))
1421
1377
        if not self.this_tree.has_id(file_id) and result == "modified":
1422
1378
            self.tt.version_file(file_id, trans_id)
1423
 
        # The merge has been performed, so the old contents should not be
1424
 
        # retained.
1425
 
        self.tt.delete_contents(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.
 
1382
            self.tt.delete_contents(trans_id)
1426
1383
        return result
1427
1384
 
1428
1385
    def _default_other_winner_merge(self, merge_hook_params):
1429
1386
        """Replace this contents with other."""
1430
1387
        file_id = merge_hook_params.file_id
1431
1388
        trans_id = merge_hook_params.trans_id
1432
 
        file_in_this = self.this_tree.has_id(file_id)
1433
1389
        if self.other_tree.has_id(file_id):
1434
1390
            # OTHER changed the file
1435
 
            wt = self.this_tree
1436
 
            if wt.supports_content_filtering():
1437
 
                # We get the path from the working tree if it exists.
1438
 
                # That fails though when OTHER is adding a file, so
1439
 
                # we fall back to the other tree to find the path if
1440
 
                # it doesn't exist locally.
1441
 
                try:
1442
 
                    filter_tree_path = wt.id2path(file_id)
1443
 
                except errors.NoSuchId:
1444
 
                    filter_tree_path = self.other_tree.id2path(file_id)
1445
 
            else:
1446
 
                # Skip the id2path lookup for older formats
1447
 
                filter_tree_path = None
1448
 
            transform.create_from_tree(self.tt, trans_id,
1449
 
                             self.other_tree, file_id,
1450
 
                             filter_tree_path=filter_tree_path)
 
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))
1451
1394
            return 'done', None
1452
 
        elif file_in_this:
 
1395
        elif self.this_tree.has_id(file_id):
1453
1396
            # OTHER deleted the file
1454
1397
            return 'delete', None
1455
1398
        else:
1529
1472
                                              other_lines)
1530
1473
            file_group.append(trans_id)
1531
1474
 
 
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
 
1532
1489
    def _dump_conflicts(self, name, parent_id, file_id, this_lines=None,
1533
1490
                        base_lines=None, other_lines=None, set_version=False,
1534
1491
                        no_base=False):
1652
1609
                for trans_id in conflict[1]:
1653
1610
                    file_id = self.tt.final_file_id(trans_id)
1654
1611
                    if file_id is not None:
 
1612
                        # Ok we found the relevant file-id
1655
1613
                        break
1656
1614
                path = fp.get_path(trans_id)
1657
1615
                for suffix in ('.BASE', '.THIS', '.OTHER'):
1658
1616
                    if path.endswith(suffix):
 
1617
                        # Here is the raw path
1659
1618
                        path = path[:-len(suffix)]
1660
1619
                        break
1661
1620
                c = _mod_conflicts.Conflict.factory(conflict_type,
1906
1865
            entries = self._entries_to_incorporate()
1907
1866
            entries = list(entries)
1908
1867
            for num, (entry, parent_id) in enumerate(entries):
1909
 
                child_pb.update('Preparing file merge', num, len(entries))
 
1868
                child_pb.update(gettext('Preparing file merge'), num, len(entries))
1910
1869
                parent_trans_id = self.tt.trans_id_file_id(parent_id)
1911
1870
                trans_id = transform.new_by_entry(self.tt, entry,
1912
1871
                    parent_trans_id, self.other_tree)
1916
1875
 
1917
1876
    def _entries_to_incorporate(self):
1918
1877
        """Yields pairs of (inventory_entry, new_parent)."""
1919
 
        other_inv = self.other_tree.inventory
 
1878
        other_inv = self.other_tree.root_inventory
1920
1879
        subdir_id = other_inv.path2id(self._source_subpath)
1921
1880
        if subdir_id is None:
1922
1881
            # XXX: The error would be clearer if it gave the URL of the source
1924
1883
            raise PathNotInTree(self._source_subpath, "Source tree")
1925
1884
        subdir = other_inv[subdir_id]
1926
1885
        parent_in_target = osutils.dirname(self._target_subdir)
1927
 
        target_id = self.this_tree.inventory.path2id(parent_in_target)
 
1886
        target_id = self.this_tree.path2id(parent_in_target)
1928
1887
        if target_id is None:
1929
1888
            raise PathNotInTree(self._target_subdir, "Target tree")
1930
1889
        name_in_target = osutils.basename(self._target_subdir)
1931
1890
        merge_into_root = subdir.copy()
1932
1891
        merge_into_root.name = name_in_target
1933
 
        if self.this_tree.inventory.has_id(merge_into_root.file_id):
 
1892
        if self.this_tree.has_id(merge_into_root.file_id):
1934
1893
            # Give the root a new file-id.
1935
1894
            # This can happen fairly easily if the directory we are
1936
1895
            # incorporating is the root, and both trees have 'TREE_ROOT' as
1996
1955
    merger.set_base_revision(get_revision_id(), this_branch)
1997
1956
    return merger.do_merge()
1998
1957
 
 
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
 
1999
1970
def get_merge_type_registry():
2000
 
    """Merge type registry is in bzrlib.option to avoid circular imports.
 
1971
    """Merge type registry was previously in bzrlib.option
2001
1972
 
2002
 
    This method provides a sanctioned way to retrieve it.
 
1973
    This method provides a backwards compatible way to retrieve it.
2003
1974
    """
2004
 
    from bzrlib import option
2005
 
    return option._merge_type_registry
 
1975
    return merge_type_registry
2006
1976
 
2007
1977
 
2008
1978
def _plan_annotate_merge(annotated_a, annotated_b, ancestors_a, ancestors_b):