/brz/remove-bazaar

To get this branch, use:
bzr branch http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar

« back to all changes in this revision

Viewing changes to bzrlib/merge.py

Merge the 2.1-simple-set branch

But revert the changes to bzrlib/_btree_serializer_pyx.pyx that didn't end up in that branch.
And restore the export and import headers.

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2005-2010 Canonical Ltd
 
1
# Copyright (C) 2005, 2006, 2008 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
 
import warnings
18
17
 
19
18
from bzrlib import (
20
19
    branch as _mod_branch,
21
20
    conflicts as _mod_conflicts,
22
21
    debug,
23
 
    decorators,
24
22
    errors,
25
23
    graph as _mod_graph,
26
 
    hooks,
27
24
    merge3,
28
25
    osutils,
29
26
    patiencediff,
 
27
    progress,
 
28
    registry,
30
29
    revision as _mod_revision,
31
30
    textfile,
32
31
    trace,
36
35
    ui,
37
36
    versionedfile
38
37
    )
39
 
from bzrlib.cleanup import OperationWithCleanups
40
 
from bzrlib.symbol_versioning import (
41
 
    deprecated_in,
42
 
    deprecated_method,
43
 
    )
44
38
# TODO: Report back as changes are merged in
45
39
 
46
40
 
47
41
def transform_tree(from_tree, to_tree, interesting_ids=None):
48
42
    from_tree.lock_tree_write()
49
 
    operation = OperationWithCleanups(merge_inner)
50
 
    operation.add_cleanup(from_tree.unlock)
51
 
    operation.run_simple(from_tree.branch, to_tree, from_tree,
52
 
        ignore_zero=True, interesting_ids=interesting_ids, this_tree=from_tree)
53
 
 
54
 
 
55
 
class MergeHooks(hooks.Hooks):
56
 
 
57
 
    def __init__(self):
58
 
        hooks.Hooks.__init__(self)
59
 
        self.create_hook(hooks.HookPoint('merge_file_content',
60
 
            "Called with a bzrlib.merge.Merger object to create a per file "
61
 
            "merge object when starting a merge. "
62
 
            "Should return either None or a subclass of "
63
 
            "``bzrlib.merge.AbstractPerFileMerger``. "
64
 
            "Such objects will then be called per file "
65
 
            "that needs to be merged (including when one "
66
 
            "side has deleted the file and the other has changed it). "
67
 
            "See the AbstractPerFileMerger API docs for details on how it is "
68
 
            "used by merge.",
69
 
            (2, 1), None))
70
 
 
71
 
 
72
 
class AbstractPerFileMerger(object):
73
 
    """PerFileMerger objects are used by plugins extending merge for bzrlib.
74
 
 
75
 
    See ``bzrlib.plugins.news_merge.news_merge`` for an example concrete class.
76
 
    
77
 
    :ivar merger: The Merge3Merger performing the merge.
78
 
    """
79
 
 
80
 
    def __init__(self, merger):
81
 
        """Create a PerFileMerger for use with merger."""
82
 
        self.merger = merger
83
 
 
84
 
    def merge_contents(self, merge_params):
85
 
        """Attempt to merge the contents of a single file.
86
 
        
87
 
        :param merge_params: A bzrlib.merge.MergeHookParams
88
 
        :return : A tuple of (status, chunks), where status is one of
89
 
            'not_applicable', 'success', 'conflicted', or 'delete'.  If status
90
 
            is 'success' or 'conflicted', then chunks should be an iterable of
91
 
            strings for the new file contents.
92
 
        """
93
 
        return ('not applicable', None)
94
 
 
95
 
 
96
 
class PerFileMerger(AbstractPerFileMerger):
97
 
    """Merge individual files when self.file_matches returns True.
98
 
 
99
 
    This class is intended to be subclassed.  The file_matches and
100
 
    merge_matching methods should be overridden with concrete implementations.
101
 
    """
102
 
 
103
 
    def file_matches(self, params):
104
 
        """Return True if merge_matching should be called on this file.
105
 
 
106
 
        Only called with merges of plain files with no clear winner.
107
 
 
108
 
        Subclasses must override this.
109
 
        """
110
 
        raise NotImplementedError(self.file_matches)
111
 
 
112
 
    def get_filename(self, params, tree):
113
 
        """Lookup the filename (i.e. basename, not path), given a Tree (e.g.
114
 
        self.merger.this_tree) and a MergeHookParams.
115
 
        """
116
 
        return osutils.basename(tree.id2path(params.file_id))
117
 
 
118
 
    def get_filepath(self, params, tree):
119
 
        """Calculate the path to the file in a tree.
120
 
 
121
 
        :param params: A MergeHookParams describing the file to merge
122
 
        :param tree: a Tree, e.g. self.merger.this_tree.
123
 
        """
124
 
        return tree.id2path(params.file_id)
125
 
 
126
 
    def merge_contents(self, params):
127
 
        """Merge the contents of a single file."""
128
 
        # Check whether this custom merge logic should be used.
129
 
        if (
130
 
            # OTHER is a straight winner, rely on default merge.
131
 
            params.winner == 'other' or
132
 
            # THIS and OTHER aren't both files.
133
 
            not params.is_file_merge() or
134
 
            # The filename doesn't match *.xml
135
 
            not self.file_matches(params)):
136
 
            return 'not_applicable', None
137
 
        return self.merge_matching(params)
138
 
 
139
 
    def merge_matching(self, params):
140
 
        """Merge the contents of a single file that has matched the criteria
141
 
        in PerFileMerger.merge_contents (is a conflict, is a file,
142
 
        self.file_matches is True).
143
 
 
144
 
        Subclasses must override this.
145
 
        """
146
 
        raise NotImplementedError(self.merge_matching)
147
 
 
148
 
 
149
 
class ConfigurableFileMerger(PerFileMerger):
150
 
    """Merge individual files when configured via a .conf file.
151
 
 
152
 
    This is a base class for concrete custom file merging logic. Concrete
153
 
    classes should implement ``merge_text``.
154
 
 
155
 
    See ``bzrlib.plugins.news_merge.news_merge`` for an example concrete class.
156
 
    
157
 
    :ivar affected_files: The configured file paths to merge.
158
 
 
159
 
    :cvar name_prefix: The prefix to use when looking up configuration
160
 
        details. <name_prefix>_merge_files describes the files targeted by the
161
 
        hook for example.
162
 
        
163
 
    :cvar default_files: The default file paths to merge when no configuration
164
 
        is present.
165
 
    """
166
 
 
167
 
    name_prefix = None
168
 
    default_files = None
169
 
 
170
 
    def __init__(self, merger):
171
 
        super(ConfigurableFileMerger, self).__init__(merger)
172
 
        self.affected_files = None
173
 
        self.default_files = self.__class__.default_files or []
174
 
        self.name_prefix = self.__class__.name_prefix
175
 
        if self.name_prefix is None:
176
 
            raise ValueError("name_prefix must be set.")
177
 
 
178
 
    def file_matches(self, params):
179
 
        """Check whether the file should call the merge hook.
180
 
 
181
 
        <name_prefix>_merge_files configuration variable is a list of files
182
 
        that should use the hook.
183
 
        """
184
 
        affected_files = self.affected_files
185
 
        if affected_files is None:
186
 
            config = self.merger.this_branch.get_config()
187
 
            # Until bzr provides a better policy for caching the config, we
188
 
            # just add the part we're interested in to the params to avoid
189
 
            # reading the config files repeatedly (bazaar.conf, location.conf,
190
 
            # branch.conf).
191
 
            config_key = self.name_prefix + '_merge_files'
192
 
            affected_files = config.get_user_option_as_list(config_key)
193
 
            if affected_files is None:
194
 
                # If nothing was specified in the config, use the default.
195
 
                affected_files = self.default_files
196
 
            self.affected_files = affected_files
197
 
        if affected_files:
198
 
            filepath = self.get_filepath(params, self.merger.this_tree)
199
 
            if filepath in affected_files:
200
 
                return True
201
 
        return False
202
 
 
203
 
    def merge_matching(self, params):
204
 
        return self.merge_text(params)
205
 
 
206
 
    def merge_text(self, params):
207
 
        """Merge the byte contents of a single file.
208
 
 
209
 
        This is called after checking that the merge should be performed in
210
 
        merge_contents, and it should behave as per
211
 
        ``bzrlib.merge.AbstractPerFileMerger.merge_contents``.
212
 
        """
213
 
        raise NotImplementedError(self.merge_text)
214
 
 
215
 
 
216
 
class MergeHookParams(object):
217
 
    """Object holding parameters passed to merge_file_content hooks.
218
 
 
219
 
    There are some fields hooks can access:
220
 
 
221
 
    :ivar file_id: the file ID of the file being merged
222
 
    :ivar trans_id: the transform ID for the merge of this file
223
 
    :ivar this_kind: kind of file_id in 'this' tree
224
 
    :ivar other_kind: kind of file_id in 'other' tree
225
 
    :ivar winner: one of 'this', 'other', 'conflict'
226
 
    """
227
 
 
228
 
    def __init__(self, merger, file_id, trans_id, this_kind, other_kind,
229
 
            winner):
230
 
        self._merger = merger
231
 
        self.file_id = file_id
232
 
        self.trans_id = trans_id
233
 
        self.this_kind = this_kind
234
 
        self.other_kind = other_kind
235
 
        self.winner = winner
236
 
 
237
 
    def is_file_merge(self):
238
 
        """True if this_kind and other_kind are both 'file'."""
239
 
        return self.this_kind == 'file' and self.other_kind == 'file'
240
 
 
241
 
    @decorators.cachedproperty
242
 
    def base_lines(self):
243
 
        """The lines of the 'base' version of the file."""
244
 
        return self._merger.get_lines(self._merger.base_tree, self.file_id)
245
 
 
246
 
    @decorators.cachedproperty
247
 
    def this_lines(self):
248
 
        """The lines of the 'this' version of the file."""
249
 
        return self._merger.get_lines(self._merger.this_tree, self.file_id)
250
 
 
251
 
    @decorators.cachedproperty
252
 
    def other_lines(self):
253
 
        """The lines of the 'other' version of the file."""
254
 
        return self._merger.get_lines(self._merger.other_tree, self.file_id)
 
43
    try:
 
44
        merge_inner(from_tree.branch, to_tree, from_tree, ignore_zero=True,
 
45
                    interesting_ids=interesting_ids, this_tree=from_tree)
 
46
    finally:
 
47
        from_tree.unlock()
255
48
 
256
49
 
257
50
class Merger(object):
258
 
 
259
 
    hooks = MergeHooks()
260
 
 
261
51
    def __init__(self, this_branch, other_tree=None, base_tree=None,
262
52
                 this_tree=None, pb=None, change_reporter=None,
263
53
                 recurse='down', revision_graph=None):
278
68
        self.interesting_files = None
279
69
        self.show_base = False
280
70
        self.reprocess = False
281
 
        if pb is not None:
282
 
            warnings.warn("pb parameter to Merger() is deprecated and ignored")
 
71
        if pb is None:
 
72
            pb = progress.DummyProgress()
 
73
        self._pb = pb
283
74
        self.pp = None
284
75
        self.recurse = recurse
285
76
        self.change_reporter = change_reporter
435
226
        revision_id = _mod_revision.ensure_null(revision_id)
436
227
        return branch, self.revision_tree(revision_id, branch)
437
228
 
438
 
    @deprecated_method(deprecated_in((2, 1, 0)))
439
229
    def ensure_revision_trees(self):
440
230
        if self.this_revision_tree is None:
441
231
            self.this_basis_tree = self.revision_tree(self.this_basis)
445
235
        if self.other_rev_id is None:
446
236
            other_basis_tree = self.revision_tree(self.other_basis)
447
237
            if other_basis_tree.has_changes(self.other_tree):
448
 
                raise errors.WorkingTreeNotRevision(self.this_tree)
 
238
                raise WorkingTreeNotRevision(self.this_tree)
449
239
            other_rev_id = self.other_basis
450
240
            self.other_tree = other_basis_tree
451
241
 
452
 
    @deprecated_method(deprecated_in((2, 1, 0)))
453
242
    def file_revisions(self, file_id):
454
243
        self.ensure_revision_trees()
455
244
        def get_id(tree, file_id):
463
252
        trees = (self.this_basis_tree, self.other_tree)
464
253
        return [get_id(tree, file_id) for tree in trees]
465
254
 
466
 
    @deprecated_method(deprecated_in((2, 1, 0)))
467
255
    def check_basis(self, check_clean, require_commits=True):
468
256
        if self.this_basis is None and require_commits is True:
469
257
            raise errors.BzrCommandError(
474
262
            if self.this_basis != self.this_rev_id:
475
263
                raise errors.UncommittedChanges(self.this_tree)
476
264
 
477
 
    @deprecated_method(deprecated_in((2, 1, 0)))
478
265
    def compare_basis(self):
479
266
        try:
480
267
            basis_tree = self.revision_tree(self.this_tree.last_revision())
487
274
        self.interesting_files = file_list
488
275
 
489
276
    def set_pending(self):
490
 
        if (not self.base_is_ancestor or not self.base_is_other_ancestor
491
 
            or self.other_rev_id is None):
 
277
        if not self.base_is_ancestor or not self.base_is_other_ancestor or self.other_rev_id is None:
492
278
            return
493
279
        self._add_parent()
494
280
 
495
281
    def _add_parent(self):
496
282
        new_parents = self.this_tree.get_parent_ids() + [self.other_rev_id]
497
283
        new_parent_trees = []
498
 
        operation = OperationWithCleanups(self.this_tree.set_parent_trees)
499
284
        for revision_id in new_parents:
500
285
            try:
501
286
                tree = self.revision_tree(revision_id)
503
288
                tree = None
504
289
            else:
505
290
                tree.lock_read()
506
 
                operation.add_cleanup(tree.unlock)
507
291
            new_parent_trees.append((revision_id, tree))
508
 
        operation.run_simple(new_parent_trees, allow_leftmost_as_ghost=True)
 
292
        try:
 
293
            self.this_tree.set_parent_trees(new_parent_trees,
 
294
                                            allow_leftmost_as_ghost=True)
 
295
        finally:
 
296
            for _revision_id, tree in new_parent_trees:
 
297
                if tree is not None:
 
298
                    tree.unlock()
509
299
 
510
300
    def set_other(self, other_revision, possible_transports=None):
511
301
        """Set the revision and tree to merge from.
634
424
                  'other_tree': self.other_tree,
635
425
                  'interesting_ids': self.interesting_ids,
636
426
                  'interesting_files': self.interesting_files,
637
 
                  'this_branch': self.this_branch,
 
427
                  'pp': self.pp,
638
428
                  'do_merge': False}
639
429
        if self.merge_type.requires_base:
640
430
            kwargs['base_tree'] = self.base_tree
658
448
        if self._is_criss_cross and getattr(self.merge_type,
659
449
                                            'supports_lca_trees', False):
660
450
            kwargs['lca_trees'] = self._lca_trees
661
 
        return self.merge_type(pb=None,
 
451
        return self.merge_type(pb=self._pb,
662
452
                               change_reporter=self.change_reporter,
663
453
                               **kwargs)
664
454
 
665
 
    def _do_merge_to(self):
666
 
        merge = self.make_merger()
 
455
    def _do_merge_to(self, merge):
667
456
        if self.other_branch is not None:
668
457
            self.other_branch.update_references(self.this_branch)
669
458
        merge.do_merge()
683
472
                    sub_tree.branch.repository.revision_tree(base_revision)
684
473
                sub_merge.base_rev_id = base_revision
685
474
                sub_merge.do_merge()
686
 
        return merge
687
475
 
688
476
    def do_merge(self):
689
 
        operation = OperationWithCleanups(self._do_merge_to)
690
477
        self.this_tree.lock_tree_write()
691
 
        operation.add_cleanup(self.this_tree.unlock)
692
 
        if self.base_tree is not None:
693
 
            self.base_tree.lock_read()
694
 
            operation.add_cleanup(self.base_tree.unlock)
695
 
        if self.other_tree is not None:
696
 
            self.other_tree.lock_read()
697
 
            operation.add_cleanup(self.other_tree.unlock)
698
 
        merge = operation.run_simple()
 
478
        try:
 
479
            if self.base_tree is not None:
 
480
                self.base_tree.lock_read()
 
481
            try:
 
482
                if self.other_tree is not None:
 
483
                    self.other_tree.lock_read()
 
484
                try:
 
485
                    merge = self.make_merger()
 
486
                    self._do_merge_to(merge)
 
487
                finally:
 
488
                    if self.other_tree is not None:
 
489
                        self.other_tree.unlock()
 
490
            finally:
 
491
                if self.base_tree is not None:
 
492
                    self.base_tree.unlock()
 
493
        finally:
 
494
            self.this_tree.unlock()
699
495
        if len(merge.cooked_conflicts) == 0:
700
496
            if not self.ignore_zero and not trace.is_quiet():
701
497
                trace.note("All changes applied successfully.")
736
532
 
737
533
    def __init__(self, working_tree, this_tree, base_tree, other_tree,
738
534
                 interesting_ids=None, reprocess=False, show_base=False,
739
 
                 pb=None, pp=None, change_reporter=None,
 
535
                 pb=progress.DummyProgress(), pp=None, change_reporter=None,
740
536
                 interesting_files=None, do_merge=True,
741
 
                 cherrypick=False, lca_trees=None, this_branch=None):
 
537
                 cherrypick=False, lca_trees=None):
742
538
        """Initialize the merger object and perform the merge.
743
539
 
744
540
        :param working_tree: The working tree to apply the merge to
745
541
        :param this_tree: The local tree in the merge operation
746
542
        :param base_tree: The common tree in the merge operation
747
543
        :param other_tree: The other tree to merge changes from
748
 
        :param this_branch: The branch associated with this_tree.  Defaults to
749
 
            this_tree.branch if not supplied.
750
544
        :param interesting_ids: The file_ids of files that should be
751
545
            participate in the merge.  May not be combined with
752
546
            interesting_files.
753
547
        :param: reprocess If True, perform conflict-reduction processing.
754
548
        :param show_base: If True, show the base revision in text conflicts.
755
549
            (incompatible with reprocess)
756
 
        :param pb: ignored
 
550
        :param pb: A Progress bar
757
551
        :param pp: A ProgressPhase object
758
552
        :param change_reporter: An object that should report changes made
759
553
        :param interesting_files: The tree-relative paths of files that should
770
564
        if interesting_files is not None and interesting_ids is not None:
771
565
            raise ValueError(
772
566
                'specify either interesting_ids or interesting_files')
773
 
        if this_branch is None:
774
 
            this_branch = this_tree.branch
775
567
        self.interesting_ids = interesting_ids
776
568
        self.interesting_files = interesting_files
777
569
        self.this_tree = working_tree
778
570
        self.base_tree = base_tree
779
571
        self.other_tree = other_tree
780
 
        self.this_branch = this_branch
781
572
        self._raw_conflicts = []
782
573
        self.cooked_conflicts = []
783
574
        self.reprocess = reprocess
788
579
        # making sure we haven't missed any corner cases.
789
580
        # if lca_trees is None:
790
581
        #     self._lca_trees = [self.base_tree]
 
582
        self.pb = pb
 
583
        self.pp = pp
791
584
        self.change_reporter = change_reporter
792
585
        self.cherrypick = cherrypick
 
586
        if self.pp is None:
 
587
            self.pp = progress.ProgressPhase("Merge phase", 3, self.pb)
793
588
        if do_merge:
794
589
            self.do_merge()
795
 
        if pp is not None:
796
 
            warnings.warn("pp argument to Merge3Merger is deprecated")
797
 
        if pb is not None:
798
 
            warnings.warn("pb argument to Merge3Merger is deprecated")
799
590
 
800
591
    def do_merge(self):
801
 
        operation = OperationWithCleanups(self._do_merge)
802
592
        self.this_tree.lock_tree_write()
803
 
        operation.add_cleanup(self.this_tree.unlock)
804
593
        self.base_tree.lock_read()
805
 
        operation.add_cleanup(self.base_tree.unlock)
806
594
        self.other_tree.lock_read()
807
 
        operation.add_cleanup(self.other_tree.unlock)
808
 
        operation.run()
809
 
 
810
 
    def _do_merge(self, operation):
811
 
        self.tt = transform.TreeTransform(self.this_tree, None)
812
 
        operation.add_cleanup(self.tt.finalize)
813
 
        self._compute_transform()
814
 
        results = self.tt.apply(no_conflicts=True)
815
 
        self.write_modified(results)
816
595
        try:
817
 
            self.this_tree.add_conflicts(self.cooked_conflicts)
818
 
        except errors.UnsupportedOperation:
819
 
            pass
 
596
            self.tt = transform.TreeTransform(self.this_tree, self.pb)
 
597
            try:
 
598
                self.pp.next_phase()
 
599
                self._compute_transform()
 
600
                self.pp.next_phase()
 
601
                results = self.tt.apply(no_conflicts=True)
 
602
                self.write_modified(results)
 
603
                try:
 
604
                    self.this_tree.add_conflicts(self.cooked_conflicts)
 
605
                except errors.UnsupportedOperation:
 
606
                    pass
 
607
            finally:
 
608
                self.tt.finalize()
 
609
        finally:
 
610
            self.other_tree.unlock()
 
611
            self.base_tree.unlock()
 
612
            self.this_tree.unlock()
 
613
            self.pb.clear()
820
614
 
821
615
    def make_preview_transform(self):
822
 
        operation = OperationWithCleanups(self._make_preview_transform)
823
616
        self.base_tree.lock_read()
824
 
        operation.add_cleanup(self.base_tree.unlock)
825
617
        self.other_tree.lock_read()
826
 
        operation.add_cleanup(self.other_tree.unlock)
827
 
        return operation.run_simple()
828
 
 
829
 
    def _make_preview_transform(self):
830
618
        self.tt = transform.TransformPreview(self.this_tree)
831
 
        self._compute_transform()
 
619
        try:
 
620
            self.pp.next_phase()
 
621
            self._compute_transform()
 
622
            self.pp.next_phase()
 
623
        finally:
 
624
            self.other_tree.unlock()
 
625
            self.base_tree.unlock()
 
626
            self.pb.clear()
832
627
        return self.tt
833
628
 
834
629
    def _compute_transform(self):
840
635
            resolver = self._lca_multi_way
841
636
        child_pb = ui.ui_factory.nested_progress_bar()
842
637
        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]
846
638
            for num, (file_id, changed, parents3, names3,
847
639
                      executable3) in enumerate(entries):
848
640
                child_pb.update('Preparing file merge', num, len(entries))
849
641
                self._merge_names(file_id, parents3, names3, resolver=resolver)
850
642
                if changed:
851
 
                    file_status = self._do_merge_contents(file_id)
 
643
                    file_status = self.merge_contents(file_id)
852
644
                else:
853
645
                    file_status = 'unmodified'
854
646
                self._merge_executable(file_id,
856
648
        finally:
857
649
            child_pb.finished()
858
650
        self.fix_root()
 
651
        self.pp.next_phase()
859
652
        child_pb = ui.ui_factory.nested_progress_bar()
860
653
        try:
861
654
            fs_conflicts = transform.resolve_conflicts(self.tt, child_pb,
1059
852
                        continue
1060
853
                else:
1061
854
                    raise AssertionError('unhandled kind: %s' % other_ie.kind)
 
855
                # XXX: We need to handle kind == 'symlink'
1062
856
 
1063
857
            # If we have gotten this far, that means something has changed
1064
858
            result.append((file_id, content_changed,
1086
880
        other_root = self.tt.trans_id_file_id(other_root_file_id)
1087
881
        if other_root == self.tt.root:
1088
882
            return
 
883
        try:
 
884
            self.tt.final_kind(other_root)
 
885
        except errors.NoSuchFile:
 
886
            return
1089
887
        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)
 
888
            # the other tree's root is a non-root in the current tree
1092
889
            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():
 
890
        self.reparent_children(self.other_tree.inventory.root, self.tt.root)
 
891
        self.tt.cancel_creation(other_root)
 
892
        self.tt.cancel_versioning(other_root)
 
893
 
 
894
    def reparent_children(self, ie, target):
 
895
        for thing, child in ie.children.iteritems():
1104
896
            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)
 
897
            self.tt.adjust_path(self.tt.final_name(trans_id), target, trans_id)
1121
898
 
1122
899
    def write_modified(self, results):
1123
900
        modified_hashes = {}
1155
932
    @staticmethod
1156
933
    def executable(tree, file_id):
1157
934
        """Determine the executability of a file-id (used as a key method)."""
1158
 
        if not tree.has_id(file_id):
 
935
        if file_id not in tree:
1159
936
            return None
1160
937
        if tree.kind(file_id) != "file":
1161
938
            return False
1164
941
    @staticmethod
1165
942
    def kind(tree, file_id):
1166
943
        """Determine the kind of a file-id (used as a key method)."""
1167
 
        if not tree.has_id(file_id):
 
944
        if file_id not in tree:
1168
945
            return None
1169
946
        return tree.kind(file_id)
1170
947
 
1171
948
    @staticmethod
1172
949
    def _three_way(base, other, this):
 
950
        #if base == other, either they all agree, or only THIS has changed.
1173
951
        if base == other:
1174
 
            # if 'base == other', either they all agree, or only 'this' has
1175
 
            # changed.
1176
952
            return 'this'
1177
953
        elif this not in (base, other):
1178
 
            # 'this' is neither 'base' nor 'other', so both sides changed
1179
954
            return 'conflict'
 
955
        # "Ambiguous clean merge" -- both sides have made the same change.
1180
956
        elif this == other:
1181
 
            # "Ambiguous clean merge" -- both sides have made the same change.
1182
957
            return "this"
 
958
        # this == base: only other has changed.
1183
959
        else:
1184
 
            # this == base: only other has changed.
1185
960
            return "other"
1186
961
 
1187
962
    @staticmethod
1231
1006
                # only has an lca value
1232
1007
                return 'other'
1233
1008
 
1234
 
        # At this point, the lcas disagree, and the tip disagree
 
1009
        # At this point, the lcas disagree, and the tips disagree
1235
1010
        return 'conflict'
1236
1011
 
1237
1012
    @staticmethod
1238
 
    @deprecated_method(deprecated_in((2, 2, 0)))
1239
1013
    def scalar_three_way(this_tree, base_tree, other_tree, file_id, key):
1240
1014
        """Do a three-way test on a scalar.
1241
1015
        Return "this", "other" or "conflict", depending whether a value wins.
1256
1030
 
1257
1031
    def merge_names(self, file_id):
1258
1032
        def get_entry(tree):
1259
 
            if tree.has_id(file_id):
 
1033
            if file_id in tree.inventory:
1260
1034
                return tree.inventory[file_id]
1261
1035
            else:
1262
1036
                return None
1291
1065
                parent_id_winner = "other"
1292
1066
        if name_winner == "this" and parent_id_winner == "this":
1293
1067
            return
1294
 
        if name_winner == 'conflict' or parent_id_winner == 'conflict':
1295
 
            # Creating helpers (.OTHER or .THIS) here cause problems down the
1296
 
            # road if a ContentConflict needs to be created so we should not do
1297
 
            # that
1298
 
            trans_id = self.tt.trans_id_file_id(file_id)
1299
 
            self._raw_conflicts.append(('path conflict', trans_id, file_id,
1300
 
                                        this_parent, this_name,
1301
 
                                        other_parent, other_name))
 
1068
        if name_winner == "conflict":
 
1069
            trans_id = self.tt.trans_id_file_id(file_id)
 
1070
            self._raw_conflicts.append(('name conflict', trans_id,
 
1071
                                        this_name, other_name))
 
1072
        if parent_id_winner == "conflict":
 
1073
            trans_id = self.tt.trans_id_file_id(file_id)
 
1074
            self._raw_conflicts.append(('parent conflict', trans_id,
 
1075
                                        this_parent, other_parent))
1302
1076
        if other_name is None:
1303
1077
            # it doesn't matter whether the result was 'other' or
1304
1078
            # 'conflict'-- if there's no 'other', we leave it alone.
1305
1079
            return
 
1080
        # if we get here, name_winner and parent_winner are set to safe values.
 
1081
        trans_id = self.tt.trans_id_file_id(file_id)
1306
1082
        parent_id = parents[self.winner_idx[parent_id_winner]]
1307
1083
        if parent_id is not None:
1308
 
            # if we get here, name_winner and parent_winner are set to safe
1309
 
            # values.
 
1084
            parent_trans_id = self.tt.trans_id_file_id(parent_id)
1310
1085
            self.tt.adjust_path(names[self.winner_idx[name_winner]],
1311
 
                                self.tt.trans_id_file_id(parent_id),
1312
 
                                self.tt.trans_id_file_id(file_id))
 
1086
                                parent_trans_id, trans_id)
1313
1087
 
1314
 
    def _do_merge_contents(self, file_id):
 
1088
    def merge_contents(self, file_id):
1315
1089
        """Performs a merge on file_id contents."""
1316
1090
        def contents_pair(tree):
1317
1091
            if file_id not in tree:
1325
1099
                contents = None
1326
1100
            return kind, contents
1327
1101
 
 
1102
        def contents_conflict():
 
1103
            trans_id = self.tt.trans_id_file_id(file_id)
 
1104
            name = self.tt.final_name(trans_id)
 
1105
            parent_id = self.tt.final_parent(trans_id)
 
1106
            if file_id in self.this_tree.inventory:
 
1107
                self.tt.unversion_file(trans_id)
 
1108
                if file_id in self.this_tree:
 
1109
                    self.tt.delete_contents(trans_id)
 
1110
            file_group = self._dump_conflicts(name, parent_id, file_id,
 
1111
                                              set_version=True)
 
1112
            self._raw_conflicts.append(('contents conflict', file_group))
 
1113
 
1328
1114
        # See SPOT run.  run, SPOT, run.
1329
1115
        # So we're not QUITE repeating ourselves; we do tricky things with
1330
1116
        # file kind...
1346
1132
        if winner == 'this':
1347
1133
            # No interesting changes introduced by OTHER
1348
1134
            return "unmodified"
1349
 
        # We have a hypothetical conflict, but if we have files, then we
1350
 
        # can try to merge the content
1351
1135
        trans_id = self.tt.trans_id_file_id(file_id)
1352
 
        params = MergeHookParams(self, file_id, trans_id, this_pair[0],
1353
 
            other_pair[0], winner)
1354
 
        hooks = self.active_hooks
1355
 
        hook_status = 'not_applicable'
1356
 
        for hook in hooks:
1357
 
            hook_status, lines = hook.merge_contents(params)
1358
 
            if hook_status != 'not_applicable':
1359
 
                # Don't try any more hooks, this one applies.
1360
 
                break
1361
 
        result = "modified"
1362
 
        if hook_status == 'not_applicable':
1363
 
            # This is a contents conflict, because none of the available
1364
 
            # functions could merge it.
1365
 
            result = None
1366
 
            name = self.tt.final_name(trans_id)
1367
 
            parent_id = self.tt.final_parent(trans_id)
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))
1373
 
        elif hook_status == 'success':
1374
 
            self.tt.create_file(lines, trans_id)
1375
 
        elif hook_status == 'conflicted':
1376
 
            # XXX: perhaps the hook should be able to provide
1377
 
            # the BASE/THIS/OTHER files?
1378
 
            self.tt.create_file(lines, trans_id)
1379
 
            self._raw_conflicts.append(('text conflict', trans_id))
1380
 
            name = self.tt.final_name(trans_id)
1381
 
            parent_id = self.tt.final_parent(trans_id)
1382
 
            self._dump_conflicts(name, parent_id, file_id)
1383
 
        elif hook_status == 'delete':
1384
 
            self.tt.unversion_file(trans_id)
1385
 
            result = "deleted"
1386
 
        elif hook_status == 'done':
1387
 
            # The hook function did whatever it needs to do directly, no
1388
 
            # further action needed here.
1389
 
            pass
1390
 
        else:
1391
 
            raise AssertionError('unknown hook_status: %r' % (hook_status,))
1392
 
        if not self.this_tree.has_id(file_id) and result == "modified":
1393
 
            self.tt.version_file(file_id, trans_id)
1394
 
        # The merge has been performed, so the old contents should not be
1395
 
        # retained.
1396
 
        try:
1397
 
            self.tt.delete_contents(trans_id)
1398
 
        except errors.NoSuchFile:
1399
 
            pass
1400
 
        return result
1401
 
 
1402
 
    def _default_other_winner_merge(self, merge_hook_params):
1403
 
        """Replace this contents with other."""
1404
 
        file_id = merge_hook_params.file_id
1405
 
        trans_id = merge_hook_params.trans_id
1406
 
        file_in_this = self.this_tree.has_id(file_id)
1407
 
        if self.other_tree.has_id(file_id):
1408
 
            # OTHER changed the file
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)
1425
 
            return 'done', None
1426
 
        elif file_in_this:
1427
 
            # OTHER deleted the file
1428
 
            return 'delete', None
1429
 
        else:
1430
 
            raise AssertionError(
1431
 
                'winner is OTHER, but file_id %r not in THIS or OTHER tree'
1432
 
                % (file_id,))
1433
 
 
1434
 
    def merge_contents(self, merge_hook_params):
1435
 
        """Fallback merge logic after user installed hooks."""
1436
 
        # This function is used in merge hooks as the fallback instance.
1437
 
        # Perhaps making this function and the functions it calls be a 
1438
 
        # a separate class would be better.
1439
 
        if merge_hook_params.winner == 'other':
 
1136
        if winner == 'other':
1440
1137
            # OTHER is a straight winner, so replace this contents with other
1441
 
            return self._default_other_winner_merge(merge_hook_params)
1442
 
        elif merge_hook_params.is_file_merge():
1443
 
            # THIS and OTHER are both files, so text merge.  Either
1444
 
            # BASE is a file, or both converted to files, so at least we
1445
 
            # have agreement that output should be a file.
1446
 
            try:
1447
 
                self.text_merge(merge_hook_params.file_id,
1448
 
                    merge_hook_params.trans_id)
1449
 
            except errors.BinaryFile:
1450
 
                return 'not_applicable', None
1451
 
            return 'done', None
 
1138
            file_in_this = file_id in self.this_tree
 
1139
            if file_in_this:
 
1140
                # Remove any existing contents
 
1141
                self.tt.delete_contents(trans_id)
 
1142
            if file_id in self.other_tree:
 
1143
                # OTHER changed the file
 
1144
                transform.create_from_tree(self.tt, trans_id,
 
1145
                                           self.other_tree, file_id)
 
1146
                if not file_in_this:
 
1147
                    self.tt.version_file(file_id, trans_id)
 
1148
                return "modified"
 
1149
            elif file_in_this:
 
1150
                # OTHER deleted the file
 
1151
                self.tt.unversion_file(trans_id)
 
1152
                return "deleted"
1452
1153
        else:
1453
 
            return 'not_applicable', None
 
1154
            # We have a hypothetical conflict, but if we have files, then we
 
1155
            # can try to merge the content
 
1156
            if this_pair[0] == 'file' and other_pair[0] == 'file':
 
1157
                # THIS and OTHER are both files, so text merge.  Either
 
1158
                # BASE is a file, or both converted to files, so at least we
 
1159
                # have agreement that output should be a file.
 
1160
                try:
 
1161
                    self.text_merge(file_id, trans_id)
 
1162
                except errors.BinaryFile:
 
1163
                    return contents_conflict()
 
1164
                if file_id not in self.this_tree:
 
1165
                    self.tt.version_file(file_id, trans_id)
 
1166
                try:
 
1167
                    self.tt.tree_kind(trans_id)
 
1168
                    self.tt.delete_contents(trans_id)
 
1169
                except errors.NoSuchFile:
 
1170
                    pass
 
1171
                return "modified"
 
1172
            else:
 
1173
                return contents_conflict()
1454
1174
 
1455
1175
    def get_lines(self, tree, file_id):
1456
1176
        """Return the lines in a file, or an empty list."""
1457
 
        if tree.has_id(file_id):
 
1177
        if file_id in tree:
1458
1178
            return tree.get_file(file_id).readlines()
1459
1179
        else:
1460
1180
            return []
1463
1183
        """Perform a three-way text merge on a file_id"""
1464
1184
        # it's possible that we got here with base as a different type.
1465
1185
        # if so, we just want two-way text conflicts.
1466
 
        if self.base_tree.has_id(file_id) and \
 
1186
        if file_id in self.base_tree and \
1467
1187
            self.base_tree.kind(file_id) == "file":
1468
1188
            base_lines = self.get_lines(self.base_tree, file_id)
1469
1189
        else:
1515
1235
                ('THIS', self.this_tree, this_lines)]
1516
1236
        if not no_base:
1517
1237
            data.append(('BASE', self.base_tree, base_lines))
1518
 
 
1519
 
        # We need to use the actual path in the working tree of the file here,
1520
 
        # ignoring the conflict suffixes
1521
 
        wt = self.this_tree
1522
 
        if wt.supports_content_filtering():
1523
 
            try:
1524
 
                filter_tree_path = wt.id2path(file_id)
1525
 
            except errors.NoSuchId:
1526
 
                # file has been deleted
1527
 
                filter_tree_path = None
1528
 
        else:
1529
 
            # Skip the id2path lookup for older formats
1530
 
            filter_tree_path = None
1531
 
 
1532
1238
        versioned = False
1533
1239
        file_group = []
1534
1240
        for suffix, tree, lines in data:
1535
 
            if tree.has_id(file_id):
 
1241
            if file_id in tree:
1536
1242
                trans_id = self._conflict_file(name, parent_id, tree, file_id,
1537
 
                                               suffix, lines, filter_tree_path)
 
1243
                                               suffix, lines)
1538
1244
                file_group.append(trans_id)
1539
1245
                if set_version and not versioned:
1540
1246
                    self.tt.version_file(file_id, trans_id)
1542
1248
        return file_group
1543
1249
 
1544
1250
    def _conflict_file(self, name, parent_id, tree, file_id, suffix,
1545
 
                       lines=None, filter_tree_path=None):
 
1251
                       lines=None):
1546
1252
        """Emit a single conflict file."""
1547
1253
        name = name + '.' + suffix
1548
1254
        trans_id = self.tt.create_path(name, parent_id)
1549
 
        transform.create_from_tree(self.tt, trans_id, tree, file_id, lines,
1550
 
            filter_tree_path)
 
1255
        transform.create_from_tree(self.tt, trans_id, tree, file_id, lines)
1551
1256
        return trans_id
1552
1257
 
1553
1258
    def merge_executable(self, file_id, file_status):
1582
1287
        if winner == "this":
1583
1288
            executability = this_executable
1584
1289
        else:
1585
 
            if self.other_tree.has_id(file_id):
 
1290
            if file_id in self.other_tree:
1586
1291
                executability = other_executable
1587
 
            elif self.this_tree.has_id(file_id):
 
1292
            elif file_id in self.this_tree:
1588
1293
                executability = this_executable
1589
 
            elif self.base_tree_has_id(file_id):
 
1294
            elif file_id in self.base_tree:
1590
1295
                executability = base_executable
1591
1296
        if executability is not None:
1592
1297
            trans_id = self.tt.trans_id_file_id(file_id)
1594
1299
 
1595
1300
    def cook_conflicts(self, fs_conflicts):
1596
1301
        """Convert all conflicts into a form that doesn't depend on trans_id"""
 
1302
        name_conflicts = {}
1597
1303
        self.cooked_conflicts.extend(transform.cook_conflicts(
1598
1304
                fs_conflicts, self.tt))
1599
1305
        fp = transform.FinalPaths(self.tt)
1600
1306
        for conflict in self._raw_conflicts:
1601
1307
            conflict_type = conflict[0]
1602
 
            if conflict_type == 'path conflict':
1603
 
                (trans_id, file_id,
1604
 
                this_parent, this_name,
1605
 
                other_parent, other_name) = conflict[1:]
1606
 
                if this_parent is None or this_name is None:
1607
 
                    this_path = '<deleted>'
1608
 
                else:
1609
 
                    parent_path =  fp.get_path(
1610
 
                        self.tt.trans_id_file_id(this_parent))
1611
 
                    this_path = osutils.pathjoin(parent_path, this_name)
1612
 
                if other_parent is None or other_name is None:
1613
 
                    other_path = '<deleted>'
1614
 
                else:
1615
 
                    parent_path =  fp.get_path(
1616
 
                        self.tt.trans_id_file_id(other_parent))
1617
 
                    other_path = osutils.pathjoin(parent_path, other_name)
1618
 
                c = _mod_conflicts.Conflict.factory(
1619
 
                    'path conflict', path=this_path,
1620
 
                    conflict_path=other_path,
1621
 
                    file_id=file_id)
1622
 
            elif conflict_type == 'contents conflict':
 
1308
            if conflict_type in ('name conflict', 'parent conflict'):
 
1309
                trans_id = conflict[1]
 
1310
                conflict_args = conflict[2:]
 
1311
                if trans_id not in name_conflicts:
 
1312
                    name_conflicts[trans_id] = {}
 
1313
                transform.unique_add(name_conflicts[trans_id], conflict_type,
 
1314
                                     conflict_args)
 
1315
            if conflict_type == 'contents conflict':
1623
1316
                for trans_id in conflict[1]:
1624
1317
                    file_id = self.tt.final_file_id(trans_id)
1625
1318
                    if file_id is not None:
1631
1324
                        break
1632
1325
                c = _mod_conflicts.Conflict.factory(conflict_type,
1633
1326
                                                    path=path, file_id=file_id)
1634
 
            elif conflict_type == 'text conflict':
 
1327
                self.cooked_conflicts.append(c)
 
1328
            if conflict_type == 'text conflict':
1635
1329
                trans_id = conflict[1]
1636
1330
                path = fp.get_path(trans_id)
1637
1331
                file_id = self.tt.final_file_id(trans_id)
1638
1332
                c = _mod_conflicts.Conflict.factory(conflict_type,
1639
1333
                                                    path=path, file_id=file_id)
 
1334
                self.cooked_conflicts.append(c)
 
1335
 
 
1336
        for trans_id, conflicts in name_conflicts.iteritems():
 
1337
            try:
 
1338
                this_parent, other_parent = conflicts['parent conflict']
 
1339
                if this_parent == other_parent:
 
1340
                    raise AssertionError()
 
1341
            except KeyError:
 
1342
                this_parent = other_parent = \
 
1343
                    self.tt.final_file_id(self.tt.final_parent(trans_id))
 
1344
            try:
 
1345
                this_name, other_name = conflicts['name conflict']
 
1346
                if this_name == other_name:
 
1347
                    raise AssertionError()
 
1348
            except KeyError:
 
1349
                this_name = other_name = self.tt.final_name(trans_id)
 
1350
            other_path = fp.get_path(trans_id)
 
1351
            if this_parent is not None and this_name is not None:
 
1352
                this_parent_path = \
 
1353
                    fp.get_path(self.tt.trans_id_file_id(this_parent))
 
1354
                this_path = osutils.pathjoin(this_parent_path, this_name)
1640
1355
            else:
1641
 
                raise AssertionError('bad conflict type: %r' % (conflict,))
 
1356
                this_path = "<deleted>"
 
1357
            file_id = self.tt.final_file_id(trans_id)
 
1358
            c = _mod_conflicts.Conflict.factory('path conflict', path=this_path,
 
1359
                                                conflict_path=other_path,
 
1360
                                                file_id=file_id)
1642
1361
            self.cooked_conflicts.append(c)
1643
1362
        self.cooked_conflicts.sort(key=_mod_conflicts.Conflict.sort_key)
1644
1363
 
1650
1369
    supports_reverse_cherrypick = False
1651
1370
    history_based = True
1652
1371
 
1653
 
    def _generate_merge_plan(self, file_id, base):
1654
 
        return self.this_tree.plan_file_merge(file_id, self.other_tree,
 
1372
    def _merged_lines(self, file_id):
 
1373
        """Generate the merged lines.
 
1374
        There is no distinction between lines that are meant to contain <<<<<<<
 
1375
        and conflicts.
 
1376
        """
 
1377
        if self.cherrypick:
 
1378
            base = self.base_tree
 
1379
        else:
 
1380
            base = None
 
1381
        plan = self.this_tree.plan_file_merge(file_id, self.other_tree,
1655
1382
                                              base=base)
1656
 
 
1657
 
    def _merged_lines(self, file_id):
1658
 
        """Generate the merged lines.
1659
 
        There is no distinction between lines that are meant to contain <<<<<<<
1660
 
        and conflicts.
1661
 
        """
1662
 
        if self.cherrypick:
1663
 
            base = self.base_tree
1664
 
        else:
1665
 
            base = None
1666
 
        plan = self._generate_merge_plan(file_id, base)
1667
1383
        if 'merge' in debug.debug_flags:
1668
1384
            plan = list(plan)
1669
1385
            trans_id = self.tt.trans_id_file_id(file_id)
1670
1386
            name = self.tt.final_name(trans_id) + '.plan'
1671
 
            contents = ('%11s|%s' % l for l in plan)
 
1387
            contents = ('%10s|%s' % l for l in plan)
1672
1388
            self.tt.new_file(name, self.tt.final_parent(trans_id), contents)
1673
1389
        textmerge = versionedfile.PlanWeaveMerge(plan, '<<<<<<< TREE\n',
1674
1390
                                                 '>>>>>>> MERGE-SOURCE\n')
1675
 
        lines, conflicts = textmerge.merge_lines(self.reprocess)
1676
 
        if conflicts:
1677
 
            base_lines = textmerge.base_from_plan()
1678
 
        else:
1679
 
            base_lines = None
1680
 
        return lines, base_lines
 
1391
        return textmerge.merge_lines(self.reprocess)
1681
1392
 
1682
1393
    def text_merge(self, file_id, trans_id):
1683
1394
        """Perform a (weave) text merge for a given file and file-id.
1684
1395
        If conflicts are encountered, .THIS and .OTHER files will be emitted,
1685
1396
        and a conflict will be noted.
1686
1397
        """
1687
 
        lines, base_lines = self._merged_lines(file_id)
 
1398
        lines, conflicts = self._merged_lines(file_id)
1688
1399
        lines = list(lines)
1689
1400
        # Note we're checking whether the OUTPUT is binary in this case,
1690
1401
        # because we don't want to get into weave merge guts.
1691
1402
        textfile.check_text_lines(lines)
1692
1403
        self.tt.create_file(lines, trans_id)
1693
 
        if base_lines is not None:
1694
 
            # Conflict
 
1404
        if conflicts:
1695
1405
            self._raw_conflicts.append(('text conflict', trans_id))
1696
1406
            name = self.tt.final_name(trans_id)
1697
1407
            parent_id = self.tt.final_parent(trans_id)
1698
1408
            file_group = self._dump_conflicts(name, parent_id, file_id,
1699
 
                                              no_base=False,
1700
 
                                              base_lines=base_lines)
 
1409
                                              no_base=True)
1701
1410
            file_group.append(trans_id)
1702
1411
 
1703
1412
 
1704
1413
class LCAMerger(WeaveMerger):
1705
1414
 
1706
 
    def _generate_merge_plan(self, file_id, base):
1707
 
        return self.this_tree.plan_file_lca_merge(file_id, self.other_tree,
 
1415
    def _merged_lines(self, file_id):
 
1416
        """Generate the merged lines.
 
1417
        There is no distinction between lines that are meant to contain <<<<<<<
 
1418
        and conflicts.
 
1419
        """
 
1420
        if self.cherrypick:
 
1421
            base = self.base_tree
 
1422
        else:
 
1423
            base = None
 
1424
        plan = self.this_tree.plan_file_lca_merge(file_id, self.other_tree,
1708
1425
                                                  base=base)
 
1426
        if 'merge' in debug.debug_flags:
 
1427
            plan = list(plan)
 
1428
            trans_id = self.tt.trans_id_file_id(file_id)
 
1429
            name = self.tt.final_name(trans_id) + '.plan'
 
1430
            contents = ('%10s|%s' % l for l in plan)
 
1431
            self.tt.new_file(name, self.tt.final_parent(trans_id), contents)
 
1432
        textmerge = versionedfile.PlanWeaveMerge(plan, '<<<<<<< TREE\n',
 
1433
                                                 '>>>>>>> MERGE-SOURCE\n')
 
1434
        return textmerge.merge_lines(self.reprocess)
 
1435
 
1709
1436
 
1710
1437
class Diff3Merger(Merge3Merger):
1711
1438
    """Three-way merger using external diff3 for text merging"""
1759
1486
                other_rev_id=None,
1760
1487
                interesting_files=None,
1761
1488
                this_tree=None,
1762
 
                pb=None,
 
1489
                pb=progress.DummyProgress(),
1763
1490
                change_reporter=None):
1764
1491
    """Primary interface for merging.
1765
1492