~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/merge.py

  • Committer: Matthew Fuller
  • Date: 2009-08-18 08:10:44 UTC
  • mto: (4772.1.1 integration)
  • mto: This revision was merged to the branch mainline in revision 4773.
  • Revision ID: fullermd@over-yonder.net-20090818081044-2due6ius01c4pwjl
Fix up some doctests to handle things ending up as RevisionSpec_dwim's
instead of RS_revno, and ending up as _dwim's (which may error
eventually, but won't until we try to evaluate them) instead of
insta-errors.

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
 
 
18
import errno
 
19
from itertools import chain
 
20
import os
17
21
import warnings
18
22
 
19
23
from bzrlib import (
20
 
    branch as _mod_branch,
21
 
    conflicts as _mod_conflicts,
22
24
    debug,
23
 
    decorators,
24
25
    errors,
25
26
    graph as _mod_graph,
26
 
    hooks,
27
 
    merge3,
28
27
    osutils,
29
28
    patiencediff,
 
29
    registry,
30
30
    revision as _mod_revision,
31
 
    textfile,
32
 
    trace,
33
 
    transform,
34
31
    tree as _mod_tree,
35
32
    tsort,
36
 
    ui,
37
 
    versionedfile
38
 
    )
39
 
from bzrlib.cleanup import OperationWithCleanups
40
 
from bzrlib.symbol_versioning import (
41
 
    deprecated_in,
42
 
    deprecated_method,
43
 
    )
 
33
    )
 
34
from bzrlib.branch import Branch
 
35
from bzrlib.conflicts import ConflictList, Conflict
 
36
from bzrlib.errors import (BzrCommandError,
 
37
                           BzrError,
 
38
                           NoCommonAncestor,
 
39
                           NoCommits,
 
40
                           NoSuchRevision,
 
41
                           NoSuchFile,
 
42
                           NotBranchError,
 
43
                           NotVersionedError,
 
44
                           UnrelatedBranches,
 
45
                           UnsupportedOperation,
 
46
                           WorkingTreeNotRevision,
 
47
                           BinaryFile,
 
48
                           )
 
49
from bzrlib.graph import Graph
 
50
from bzrlib.merge3 import Merge3
 
51
from bzrlib.osutils import rename, pathjoin
 
52
from progress import DummyProgress, ProgressPhase
 
53
from bzrlib.revision import (NULL_REVISION, ensure_null)
 
54
from bzrlib.textfile import check_text_lines
 
55
from bzrlib.trace import mutter, warning, note, is_quiet
 
56
from bzrlib.transform import (TransformPreview, TreeTransform,
 
57
                              resolve_conflicts, cook_conflicts,
 
58
                              conflict_pass, FinalPaths, create_from_tree,
 
59
                              unique_add, ROOT_PARENT)
 
60
from bzrlib.versionedfile import PlanWeaveMerge
 
61
from bzrlib import ui
 
62
 
44
63
# TODO: Report back as changes are merged in
45
64
 
46
65
 
47
66
def transform_tree(from_tree, to_tree, interesting_ids=None):
48
 
    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 ConfigurableFileMerger(AbstractPerFileMerger):
97
 
    """Merge individual files when configured via a .conf file.
98
 
 
99
 
    This is a base class for concrete custom file merging logic. Concrete
100
 
    classes should implement ``merge_text``.
101
 
 
102
 
    See ``bzrlib.plugins.news_merge.news_merge`` for an example concrete class.
103
 
    
104
 
    :ivar affected_files: The configured file paths to merge.
105
 
 
106
 
    :cvar name_prefix: The prefix to use when looking up configuration
107
 
        details. <name_prefix>_merge_files describes the files targeted by the
108
 
        hook for example.
109
 
        
110
 
    :cvar default_files: The default file paths to merge when no configuration
111
 
        is present.
112
 
    """
113
 
 
114
 
    name_prefix = None
115
 
    default_files = None
116
 
 
117
 
    def __init__(self, merger):
118
 
        super(ConfigurableFileMerger, self).__init__(merger)
119
 
        self.affected_files = None
120
 
        self.default_files = self.__class__.default_files or []
121
 
        self.name_prefix = self.__class__.name_prefix
122
 
        if self.name_prefix is None:
123
 
            raise ValueError("name_prefix must be set.")
124
 
 
125
 
    def filename_matches_config(self, params):
126
 
        """Check whether the file should call the merge hook.
127
 
 
128
 
        <name_prefix>_merge_files configuration variable is a list of files
129
 
        that should use the hook.
130
 
        """
131
 
        affected_files = self.affected_files
132
 
        if affected_files is None:
133
 
            config = self.merger.this_branch.get_config()
134
 
            # Until bzr provides a better policy for caching the config, we
135
 
            # just add the part we're interested in to the params to avoid
136
 
            # reading the config files repeatedly (bazaar.conf, location.conf,
137
 
            # branch.conf).
138
 
            config_key = self.name_prefix + '_merge_files'
139
 
            affected_files = config.get_user_option_as_list(config_key)
140
 
            if affected_files is None:
141
 
                # If nothing was specified in the config, use the default.
142
 
                affected_files = self.default_files
143
 
            self.affected_files = affected_files
144
 
        if affected_files:
145
 
            filename = self.merger.this_tree.id2path(params.file_id)
146
 
            if filename in affected_files:
147
 
                return True
148
 
        return False
149
 
 
150
 
    def merge_contents(self, params):
151
 
        """Merge the contents of a single file."""
152
 
        # First, check whether this custom merge logic should be used.  We
153
 
        # expect most files should not be merged by this handler.
154
 
        if (
155
 
            # OTHER is a straight winner, rely on default merge.
156
 
            params.winner == 'other' or
157
 
            # THIS and OTHER aren't both files.
158
 
            not params.is_file_merge() or
159
 
            # The filename isn't listed in the 'NAME_merge_files' config
160
 
            # option.
161
 
            not self.filename_matches_config(params)):
162
 
            return 'not_applicable', None
163
 
        return self.merge_text(params)
164
 
 
165
 
    def merge_text(self, params):
166
 
        """Merge the byte contents of a single file.
167
 
 
168
 
        This is called after checking that the merge should be performed in
169
 
        merge_contents, and it should behave as per
170
 
        ``bzrlib.merge.AbstractPerFileMerger.merge_contents``.
171
 
        """
172
 
        raise NotImplementedError(self.merge_text)
173
 
 
174
 
 
175
 
class MergeHookParams(object):
176
 
    """Object holding parameters passed to merge_file_content hooks.
177
 
 
178
 
    There are some fields hooks can access:
179
 
 
180
 
    :ivar file_id: the file ID of the file being merged
181
 
    :ivar trans_id: the transform ID for the merge of this file
182
 
    :ivar this_kind: kind of file_id in 'this' tree
183
 
    :ivar other_kind: kind of file_id in 'other' tree
184
 
    :ivar winner: one of 'this', 'other', 'conflict'
185
 
    """
186
 
 
187
 
    def __init__(self, merger, file_id, trans_id, this_kind, other_kind,
188
 
            winner):
189
 
        self._merger = merger
190
 
        self.file_id = file_id
191
 
        self.trans_id = trans_id
192
 
        self.this_kind = this_kind
193
 
        self.other_kind = other_kind
194
 
        self.winner = winner
195
 
 
196
 
    def is_file_merge(self):
197
 
        """True if this_kind and other_kind are both 'file'."""
198
 
        return self.this_kind == 'file' and self.other_kind == 'file'
199
 
 
200
 
    @decorators.cachedproperty
201
 
    def base_lines(self):
202
 
        """The lines of the 'base' version of the file."""
203
 
        return self._merger.get_lines(self._merger.base_tree, self.file_id)
204
 
 
205
 
    @decorators.cachedproperty
206
 
    def this_lines(self):
207
 
        """The lines of the 'this' version of the file."""
208
 
        return self._merger.get_lines(self._merger.this_tree, self.file_id)
209
 
 
210
 
    @decorators.cachedproperty
211
 
    def other_lines(self):
212
 
        """The lines of the 'other' version of the file."""
213
 
        return self._merger.get_lines(self._merger.other_tree, self.file_id)
 
67
    merge_inner(from_tree.branch, to_tree, from_tree, ignore_zero=True,
 
68
                interesting_ids=interesting_ids, this_tree=from_tree)
214
69
 
215
70
 
216
71
class Merger(object):
217
 
 
218
 
    hooks = MergeHooks()
219
 
 
220
72
    def __init__(self, this_branch, other_tree=None, base_tree=None,
221
73
                 this_tree=None, pb=None, change_reporter=None,
222
74
                 recurse='down', revision_graph=None):
237
89
        self.interesting_files = None
238
90
        self.show_base = False
239
91
        self.reprocess = False
240
 
        if pb is not None:
241
 
            warnings.warn("pb parameter to Merger() is deprecated and ignored")
 
92
        if pb is None:
 
93
            pb = DummyProgress()
 
94
        self._pb = pb
242
95
        self.pp = None
243
96
        self.recurse = recurse
244
97
        self.change_reporter = change_reporter
249
102
        self._is_criss_cross = None
250
103
        self._lca_trees = None
251
104
 
252
 
    def cache_trees_with_revision_ids(self, trees):
253
 
        """Cache any tree in trees if it has a revision_id."""
254
 
        for maybe_tree in trees:
255
 
            if maybe_tree is None:
256
 
                continue
257
 
            try:
258
 
                rev_id = maybe_tree.get_revision_id()
259
 
            except AttributeError:
260
 
                continue
261
 
            self._cached_trees[rev_id] = maybe_tree
262
 
 
263
105
    @property
264
106
    def revision_graph(self):
265
107
        if self._revision_graph is None:
327
169
                base_revision_id, tree.branch.last_revision())):
328
170
                base_revision_id = None
329
171
            else:
330
 
                trace.warning('Performing cherrypick')
 
172
                warning('Performing cherrypick')
331
173
        merger = klass.from_revision_ids(pb, tree, other_revision_id,
332
174
                                         base_revision_id, revision_graph=
333
175
                                         revision_graph)
385
227
        if revno is None:
386
228
            tree = workingtree.WorkingTree.open_containing(location)[0]
387
229
            return tree.branch, tree
388
 
        branch = _mod_branch.Branch.open_containing(
389
 
            location, possible_transports)[0]
 
230
        branch = Branch.open_containing(location, possible_transports)[0]
390
231
        if revno == -1:
391
232
            revision_id = branch.last_revision()
392
233
        else:
393
234
            revision_id = branch.get_rev_id(revno)
394
 
        revision_id = _mod_revision.ensure_null(revision_id)
 
235
        revision_id = ensure_null(revision_id)
395
236
        return branch, self.revision_tree(revision_id, branch)
396
237
 
397
 
    @deprecated_method(deprecated_in((2, 1, 0)))
398
238
    def ensure_revision_trees(self):
399
239
        if self.this_revision_tree is None:
400
240
            self.this_basis_tree = self.revision_tree(self.this_basis)
404
244
        if self.other_rev_id is None:
405
245
            other_basis_tree = self.revision_tree(self.other_basis)
406
246
            if other_basis_tree.has_changes(self.other_tree):
407
 
                raise errors.WorkingTreeNotRevision(self.this_tree)
 
247
                raise WorkingTreeNotRevision(self.this_tree)
408
248
            other_rev_id = self.other_basis
409
249
            self.other_tree = other_basis_tree
410
250
 
411
 
    @deprecated_method(deprecated_in((2, 1, 0)))
412
251
    def file_revisions(self, file_id):
413
252
        self.ensure_revision_trees()
414
253
        def get_id(tree, file_id):
417
256
        if self.this_rev_id is None:
418
257
            if self.this_basis_tree.get_file_sha1(file_id) != \
419
258
                self.this_tree.get_file_sha1(file_id):
420
 
                raise errors.WorkingTreeNotRevision(self.this_tree)
 
259
                raise WorkingTreeNotRevision(self.this_tree)
421
260
 
422
261
        trees = (self.this_basis_tree, self.other_tree)
423
262
        return [get_id(tree, file_id) for tree in trees]
424
263
 
425
 
    @deprecated_method(deprecated_in((2, 1, 0)))
426
264
    def check_basis(self, check_clean, require_commits=True):
427
265
        if self.this_basis is None and require_commits is True:
428
 
            raise errors.BzrCommandError(
429
 
                "This branch has no commits."
430
 
                " (perhaps you would prefer 'bzr pull')")
 
266
            raise BzrCommandError("This branch has no commits."
 
267
                                  " (perhaps you would prefer 'bzr pull')")
431
268
        if check_clean:
432
269
            self.compare_basis()
433
270
            if self.this_basis != self.this_rev_id:
434
271
                raise errors.UncommittedChanges(self.this_tree)
435
272
 
436
 
    @deprecated_method(deprecated_in((2, 1, 0)))
437
273
    def compare_basis(self):
438
274
        try:
439
275
            basis_tree = self.revision_tree(self.this_tree.last_revision())
446
282
        self.interesting_files = file_list
447
283
 
448
284
    def set_pending(self):
449
 
        if (not self.base_is_ancestor or not self.base_is_other_ancestor
450
 
            or self.other_rev_id is None):
 
285
        if not self.base_is_ancestor or not self.base_is_other_ancestor or self.other_rev_id is None:
451
286
            return
452
287
        self._add_parent()
453
288
 
454
289
    def _add_parent(self):
455
290
        new_parents = self.this_tree.get_parent_ids() + [self.other_rev_id]
456
291
        new_parent_trees = []
457
 
        operation = OperationWithCleanups(self.this_tree.set_parent_trees)
458
292
        for revision_id in new_parents:
459
293
            try:
460
294
                tree = self.revision_tree(revision_id)
462
296
                tree = None
463
297
            else:
464
298
                tree.lock_read()
465
 
                operation.add_cleanup(tree.unlock)
466
299
            new_parent_trees.append((revision_id, tree))
467
 
        operation.run_simple(new_parent_trees, allow_leftmost_as_ghost=True)
 
300
        try:
 
301
            self.this_tree.set_parent_trees(new_parent_trees,
 
302
                                            allow_leftmost_as_ghost=True)
 
303
        finally:
 
304
            for _revision_id, tree in new_parent_trees:
 
305
                if tree is not None:
 
306
                    tree.unlock()
468
307
 
469
308
    def set_other(self, other_revision, possible_transports=None):
470
309
        """Set the revision and tree to merge from.
479
318
            self.other_rev_id = _mod_revision.ensure_null(
480
319
                self.other_branch.last_revision())
481
320
            if _mod_revision.is_null(self.other_rev_id):
482
 
                raise errors.NoCommits(self.other_branch)
 
321
                raise NoCommits(self.other_branch)
483
322
            self.other_basis = self.other_rev_id
484
323
        elif other_revision[1] is not None:
485
324
            self.other_rev_id = self.other_branch.get_rev_id(other_revision[1])
488
327
            self.other_rev_id = None
489
328
            self.other_basis = self.other_branch.last_revision()
490
329
            if self.other_basis is None:
491
 
                raise errors.NoCommits(self.other_branch)
 
330
                raise NoCommits(self.other_branch)
492
331
        if self.other_rev_id is not None:
493
332
            self._cached_trees[self.other_rev_id] = self.other_tree
494
333
        self._maybe_fetch(self.other_branch,self.this_branch, self.other_basis)
521
360
            target.fetch(source, revision_id)
522
361
 
523
362
    def find_base(self):
524
 
        revisions = [_mod_revision.ensure_null(self.this_basis),
525
 
                     _mod_revision.ensure_null(self.other_basis)]
526
 
        if _mod_revision.NULL_REVISION in revisions:
527
 
            self.base_rev_id = _mod_revision.NULL_REVISION
 
363
        revisions = [ensure_null(self.this_basis),
 
364
                     ensure_null(self.other_basis)]
 
365
        if NULL_REVISION in revisions:
 
366
            self.base_rev_id = NULL_REVISION
528
367
            self.base_tree = self.revision_tree(self.base_rev_id)
529
368
            self._is_criss_cross = False
530
369
        else:
531
370
            lcas = self.revision_graph.find_lca(revisions[0], revisions[1])
532
371
            self._is_criss_cross = False
533
372
            if len(lcas) == 0:
534
 
                self.base_rev_id = _mod_revision.NULL_REVISION
 
373
                self.base_rev_id = NULL_REVISION
535
374
            elif len(lcas) == 1:
536
375
                self.base_rev_id = list(lcas)[0]
537
376
            else: # len(lcas) > 1
546
385
                    self.base_rev_id = self.revision_graph.find_unique_lca(
547
386
                                            *lcas)
548
387
                self._is_criss_cross = True
549
 
            if self.base_rev_id == _mod_revision.NULL_REVISION:
550
 
                raise errors.UnrelatedBranches()
 
388
            if self.base_rev_id == NULL_REVISION:
 
389
                raise UnrelatedBranches()
551
390
            if self._is_criss_cross:
552
 
                trace.warning('Warning: criss-cross merge encountered.  See bzr'
553
 
                              ' help criss-cross.')
554
 
                trace.mutter('Criss-cross lcas: %r' % lcas)
 
391
                warning('Warning: criss-cross merge encountered.  See bzr'
 
392
                        ' help criss-cross.')
 
393
                mutter('Criss-cross lcas: %r' % lcas)
555
394
                interesting_revision_ids = [self.base_rev_id]
556
395
                interesting_revision_ids.extend(lcas)
557
396
                interesting_trees = dict((t.get_revision_id(), t)
567
406
                self.base_tree = self.revision_tree(self.base_rev_id)
568
407
        self.base_is_ancestor = True
569
408
        self.base_is_other_ancestor = True
570
 
        trace.mutter('Base revid: %r' % self.base_rev_id)
 
409
        mutter('Base revid: %r' % self.base_rev_id)
571
410
 
572
411
    def set_base(self, base_revision):
573
412
        """Set the base revision to use for the merge.
574
413
 
575
414
        :param base_revision: A 2-list containing a path and revision number.
576
415
        """
577
 
        trace.mutter("doing merge() with no base_revision specified")
 
416
        mutter("doing merge() with no base_revision specified")
578
417
        if base_revision == [None, None]:
579
418
            self.find_base()
580
419
        else:
593
432
                  'other_tree': self.other_tree,
594
433
                  'interesting_ids': self.interesting_ids,
595
434
                  'interesting_files': self.interesting_files,
596
 
                  'this_branch': self.this_branch,
 
435
                  'pp': self.pp,
597
436
                  'do_merge': False}
598
437
        if self.merge_type.requires_base:
599
438
            kwargs['base_tree'] = self.base_tree
600
439
        if self.merge_type.supports_reprocess:
601
440
            kwargs['reprocess'] = self.reprocess
602
441
        elif self.reprocess:
603
 
            raise errors.BzrError(
604
 
                "Conflict reduction is not supported for merge"
605
 
                " type %s." % self.merge_type)
 
442
            raise BzrError("Conflict reduction is not supported for merge"
 
443
                                  " type %s." % self.merge_type)
606
444
        if self.merge_type.supports_show_base:
607
445
            kwargs['show_base'] = self.show_base
608
446
        elif self.show_base:
609
 
            raise errors.BzrError("Showing base is not supported for this"
610
 
                                  " merge type. %s" % self.merge_type)
 
447
            raise BzrError("Showing base is not supported for this"
 
448
                           " merge type. %s" % self.merge_type)
611
449
        if (not getattr(self.merge_type, 'supports_reverse_cherrypick', True)
612
450
            and not self.base_is_other_ancestor):
613
451
            raise errors.CannotReverseCherrypick()
617
455
        if self._is_criss_cross and getattr(self.merge_type,
618
456
                                            'supports_lca_trees', False):
619
457
            kwargs['lca_trees'] = self._lca_trees
620
 
        return self.merge_type(pb=None,
 
458
        return self.merge_type(pb=self._pb,
621
459
                               change_reporter=self.change_reporter,
622
460
                               **kwargs)
623
461
 
624
 
    def _do_merge_to(self):
625
 
        merge = self.make_merger()
 
462
    def _do_merge_to(self, merge):
626
463
        if self.other_branch is not None:
627
464
            self.other_branch.update_references(self.this_branch)
628
465
        merge.do_merge()
642
479
                    sub_tree.branch.repository.revision_tree(base_revision)
643
480
                sub_merge.base_rev_id = base_revision
644
481
                sub_merge.do_merge()
645
 
        return merge
646
482
 
647
483
    def do_merge(self):
648
 
        operation = OperationWithCleanups(self._do_merge_to)
649
484
        self.this_tree.lock_tree_write()
650
 
        operation.add_cleanup(self.this_tree.unlock)
651
 
        if self.base_tree is not None:
652
 
            self.base_tree.lock_read()
653
 
            operation.add_cleanup(self.base_tree.unlock)
654
 
        if self.other_tree is not None:
655
 
            self.other_tree.lock_read()
656
 
            operation.add_cleanup(self.other_tree.unlock)
657
 
        merge = operation.run_simple()
 
485
        try:
 
486
            if self.base_tree is not None:
 
487
                self.base_tree.lock_read()
 
488
            try:
 
489
                if self.other_tree is not None:
 
490
                    self.other_tree.lock_read()
 
491
                try:
 
492
                    merge = self.make_merger()
 
493
                    self._do_merge_to(merge)
 
494
                finally:
 
495
                    if self.other_tree is not None:
 
496
                        self.other_tree.unlock()
 
497
            finally:
 
498
                if self.base_tree is not None:
 
499
                    self.base_tree.unlock()
 
500
        finally:
 
501
            self.this_tree.unlock()
658
502
        if len(merge.cooked_conflicts) == 0:
659
 
            if not self.ignore_zero and not trace.is_quiet():
660
 
                trace.note("All changes applied successfully.")
 
503
            if not self.ignore_zero and not is_quiet():
 
504
                note("All changes applied successfully.")
661
505
        else:
662
 
            trace.note("%d conflicts encountered."
663
 
                       % len(merge.cooked_conflicts))
 
506
            note("%d conflicts encountered." % len(merge.cooked_conflicts))
664
507
 
665
508
        return len(merge.cooked_conflicts)
666
509
 
695
538
 
696
539
    def __init__(self, working_tree, this_tree, base_tree, other_tree,
697
540
                 interesting_ids=None, reprocess=False, show_base=False,
698
 
                 pb=None, pp=None, change_reporter=None,
 
541
                 pb=DummyProgress(), pp=None, change_reporter=None,
699
542
                 interesting_files=None, do_merge=True,
700
 
                 cherrypick=False, lca_trees=None, this_branch=None):
 
543
                 cherrypick=False, lca_trees=None):
701
544
        """Initialize the merger object and perform the merge.
702
545
 
703
546
        :param working_tree: The working tree to apply the merge to
704
547
        :param this_tree: The local tree in the merge operation
705
548
        :param base_tree: The common tree in the merge operation
706
549
        :param other_tree: The other tree to merge changes from
707
 
        :param this_branch: The branch associated with this_tree
708
550
        :param interesting_ids: The file_ids of files that should be
709
551
            participate in the merge.  May not be combined with
710
552
            interesting_files.
711
553
        :param: reprocess If True, perform conflict-reduction processing.
712
554
        :param show_base: If True, show the base revision in text conflicts.
713
555
            (incompatible with reprocess)
714
 
        :param pb: ignored
 
556
        :param pb: A Progress bar
715
557
        :param pp: A ProgressPhase object
716
558
        :param change_reporter: An object that should report changes made
717
559
        :param interesting_files: The tree-relative paths of files that should
733
575
        self.this_tree = working_tree
734
576
        self.base_tree = base_tree
735
577
        self.other_tree = other_tree
736
 
        self.this_branch = this_branch
737
578
        self._raw_conflicts = []
738
579
        self.cooked_conflicts = []
739
580
        self.reprocess = reprocess
744
585
        # making sure we haven't missed any corner cases.
745
586
        # if lca_trees is None:
746
587
        #     self._lca_trees = [self.base_tree]
 
588
        self.pb = pb
 
589
        self.pp = pp
747
590
        self.change_reporter = change_reporter
748
591
        self.cherrypick = cherrypick
 
592
        if self.pp is None:
 
593
            self.pp = ProgressPhase("Merge phase", 3, self.pb)
749
594
        if do_merge:
750
595
            self.do_merge()
751
 
        if pp is not None:
752
 
            warnings.warn("pp argument to Merge3Merger is deprecated")
753
 
        if pb is not None:
754
 
            warnings.warn("pb argument to Merge3Merger is deprecated")
755
596
 
756
597
    def do_merge(self):
757
 
        operation = OperationWithCleanups(self._do_merge)
758
598
        self.this_tree.lock_tree_write()
759
 
        operation.add_cleanup(self.this_tree.unlock)
760
599
        self.base_tree.lock_read()
761
 
        operation.add_cleanup(self.base_tree.unlock)
762
600
        self.other_tree.lock_read()
763
 
        operation.add_cleanup(self.other_tree.unlock)
764
 
        operation.run()
765
 
 
766
 
    def _do_merge(self, operation):
767
 
        self.tt = transform.TreeTransform(self.this_tree, None)
768
 
        operation.add_cleanup(self.tt.finalize)
769
 
        self._compute_transform()
770
 
        results = self.tt.apply(no_conflicts=True)
771
 
        self.write_modified(results)
 
601
        self.tt = TreeTransform(self.this_tree, self.pb)
772
602
        try:
773
 
            self.this_tree.add_conflicts(self.cooked_conflicts)
774
 
        except errors.UnsupportedOperation:
775
 
            pass
 
603
            self.pp.next_phase()
 
604
            self._compute_transform()
 
605
            self.pp.next_phase()
 
606
            results = self.tt.apply(no_conflicts=True)
 
607
            self.write_modified(results)
 
608
            try:
 
609
                self.this_tree.add_conflicts(self.cooked_conflicts)
 
610
            except UnsupportedOperation:
 
611
                pass
 
612
        finally:
 
613
            self.tt.finalize()
 
614
            self.other_tree.unlock()
 
615
            self.base_tree.unlock()
 
616
            self.this_tree.unlock()
 
617
            self.pb.clear()
776
618
 
777
619
    def make_preview_transform(self):
778
 
        operation = OperationWithCleanups(self._make_preview_transform)
779
620
        self.base_tree.lock_read()
780
 
        operation.add_cleanup(self.base_tree.unlock)
781
621
        self.other_tree.lock_read()
782
 
        operation.add_cleanup(self.other_tree.unlock)
783
 
        return operation.run_simple()
784
 
 
785
 
    def _make_preview_transform(self):
786
 
        self.tt = transform.TransformPreview(self.this_tree)
787
 
        self._compute_transform()
 
622
        self.tt = TransformPreview(self.this_tree)
 
623
        try:
 
624
            self.pp.next_phase()
 
625
            self._compute_transform()
 
626
            self.pp.next_phase()
 
627
        finally:
 
628
            self.other_tree.unlock()
 
629
            self.base_tree.unlock()
 
630
            self.pb.clear()
788
631
        return self.tt
789
632
 
790
633
    def _compute_transform(self):
796
639
            resolver = self._lca_multi_way
797
640
        child_pb = ui.ui_factory.nested_progress_bar()
798
641
        try:
799
 
            factories = Merger.hooks['merge_file_content']
800
 
            hooks = [factory(self) for factory in factories] + [self]
801
 
            self.active_hooks = [hook for hook in hooks if hook is not None]
802
642
            for num, (file_id, changed, parents3, names3,
803
643
                      executable3) in enumerate(entries):
804
644
                child_pb.update('Preparing file merge', num, len(entries))
805
645
                self._merge_names(file_id, parents3, names3, resolver=resolver)
806
646
                if changed:
807
 
                    file_status = self._do_merge_contents(file_id)
 
647
                    file_status = self.merge_contents(file_id)
808
648
                else:
809
649
                    file_status = 'unmodified'
810
650
                self._merge_executable(file_id,
812
652
        finally:
813
653
            child_pb.finished()
814
654
        self.fix_root()
 
655
        self.pp.next_phase()
815
656
        child_pb = ui.ui_factory.nested_progress_bar()
816
657
        try:
817
 
            fs_conflicts = transform.resolve_conflicts(self.tt, child_pb,
818
 
                lambda t, c: transform.conflict_pass(t, c, self.other_tree))
 
658
            fs_conflicts = resolve_conflicts(self.tt, child_pb,
 
659
                lambda t, c: conflict_pass(t, c, self.other_tree))
819
660
        finally:
820
661
            child_pb.finished()
821
662
        if self.change_reporter is not None:
824
665
                self.tt.iter_changes(), self.change_reporter)
825
666
        self.cook_conflicts(fs_conflicts)
826
667
        for conflict in self.cooked_conflicts:
827
 
            trace.warning(conflict)
 
668
            warning(conflict)
828
669
 
829
670
    def _entries3(self):
830
671
        """Gather data about files modified between three trees.
1032
873
    def fix_root(self):
1033
874
        try:
1034
875
            self.tt.final_kind(self.tt.root)
1035
 
        except errors.NoSuchFile:
 
876
        except NoSuchFile:
1036
877
            self.tt.cancel_deletion(self.tt.root)
1037
878
        if self.tt.final_file_id(self.tt.root) is None:
1038
879
            self.tt.version_file(self.tt.tree_file_id(self.tt.root),
1045
886
            return
1046
887
        try:
1047
888
            self.tt.final_kind(other_root)
1048
 
        except errors.NoSuchFile:
 
889
        except NoSuchFile:
1049
890
            return
1050
 
        if self.this_tree.has_id(self.other_tree.inventory.root.file_id):
 
891
        if self.other_tree.inventory.root.file_id in self.this_tree.inventory:
1051
892
            # the other tree's root is a non-root in the current tree
1052
893
            return
1053
894
        self.reparent_children(self.other_tree.inventory.root, self.tt.root)
1095
936
    @staticmethod
1096
937
    def executable(tree, file_id):
1097
938
        """Determine the executability of a file-id (used as a key method)."""
1098
 
        if not tree.has_id(file_id):
 
939
        if file_id not in tree:
1099
940
            return None
1100
941
        if tree.kind(file_id) != "file":
1101
942
            return False
1104
945
    @staticmethod
1105
946
    def kind(tree, file_id):
1106
947
        """Determine the kind of a file-id (used as a key method)."""
1107
 
        if not tree.has_id(file_id):
 
948
        if file_id not in tree:
1108
949
            return None
1109
950
        return tree.kind(file_id)
1110
951
 
1193
1034
 
1194
1035
    def merge_names(self, file_id):
1195
1036
        def get_entry(tree):
1196
 
            if tree.has_id(file_id):
 
1037
            if file_id in tree.inventory:
1197
1038
                return tree.inventory[file_id]
1198
1039
            else:
1199
1040
                return None
1248
1089
            self.tt.adjust_path(names[self.winner_idx[name_winner]],
1249
1090
                                parent_trans_id, trans_id)
1250
1091
 
1251
 
    def _do_merge_contents(self, file_id):
 
1092
    def merge_contents(self, file_id):
1252
1093
        """Performs a merge on file_id contents."""
1253
1094
        def contents_pair(tree):
1254
1095
            if file_id not in tree:
1262
1103
                contents = None
1263
1104
            return kind, contents
1264
1105
 
 
1106
        def contents_conflict():
 
1107
            trans_id = self.tt.trans_id_file_id(file_id)
 
1108
            name = self.tt.final_name(trans_id)
 
1109
            parent_id = self.tt.final_parent(trans_id)
 
1110
            if file_id in self.this_tree.inventory:
 
1111
                self.tt.unversion_file(trans_id)
 
1112
                if file_id in self.this_tree:
 
1113
                    self.tt.delete_contents(trans_id)
 
1114
            file_group = self._dump_conflicts(name, parent_id, file_id,
 
1115
                                              set_version=True)
 
1116
            self._raw_conflicts.append(('contents conflict', file_group))
 
1117
 
1265
1118
        # See SPOT run.  run, SPOT, run.
1266
1119
        # So we're not QUITE repeating ourselves; we do tricky things with
1267
1120
        # file kind...
1283
1136
        if winner == 'this':
1284
1137
            # No interesting changes introduced by OTHER
1285
1138
            return "unmodified"
1286
 
        # We have a hypothetical conflict, but if we have files, then we
1287
 
        # can try to merge the content
1288
1139
        trans_id = self.tt.trans_id_file_id(file_id)
1289
 
        params = MergeHookParams(self, file_id, trans_id, this_pair[0],
1290
 
            other_pair[0], winner)
1291
 
        hooks = self.active_hooks
1292
 
        hook_status = 'not_applicable'
1293
 
        for hook in hooks:
1294
 
            hook_status, lines = hook.merge_contents(params)
1295
 
            if hook_status != 'not_applicable':
1296
 
                # Don't try any more hooks, this one applies.
1297
 
                break
1298
 
        result = "modified"
1299
 
        if hook_status == 'not_applicable':
1300
 
            # This is a contents conflict, because none of the available
1301
 
            # functions could merge it.
1302
 
            result = None
1303
 
            name = self.tt.final_name(trans_id)
1304
 
            parent_id = self.tt.final_parent(trans_id)
1305
 
            if self.this_tree.has_id(file_id):
1306
 
                self.tt.unversion_file(trans_id)
1307
 
            file_group = self._dump_conflicts(name, parent_id, file_id,
1308
 
                                              set_version=True)
1309
 
            self._raw_conflicts.append(('contents conflict', file_group))
1310
 
        elif hook_status == 'success':
1311
 
            self.tt.create_file(lines, trans_id)
1312
 
        elif hook_status == 'conflicted':
1313
 
            # XXX: perhaps the hook should be able to provide
1314
 
            # the BASE/THIS/OTHER files?
1315
 
            self.tt.create_file(lines, trans_id)
1316
 
            self._raw_conflicts.append(('text conflict', trans_id))
1317
 
            name = self.tt.final_name(trans_id)
1318
 
            parent_id = self.tt.final_parent(trans_id)
1319
 
            self._dump_conflicts(name, parent_id, file_id)
1320
 
        elif hook_status == 'delete':
1321
 
            self.tt.unversion_file(trans_id)
1322
 
            result = "deleted"
1323
 
        elif hook_status == 'done':
1324
 
            # The hook function did whatever it needs to do directly, no
1325
 
            # further action needed here.
1326
 
            pass
1327
 
        else:
1328
 
            raise AssertionError('unknown hook_status: %r' % (hook_status,))
1329
 
        if not self.this_tree.has_id(file_id) and result == "modified":
1330
 
            self.tt.version_file(file_id, trans_id)
1331
 
        # The merge has been performed, so the old contents should not be
1332
 
        # retained.
1333
 
        try:
1334
 
            self.tt.delete_contents(trans_id)
1335
 
        except errors.NoSuchFile:
1336
 
            pass
1337
 
        return result
1338
 
 
1339
 
    def _default_other_winner_merge(self, merge_hook_params):
1340
 
        """Replace this contents with other."""
1341
 
        file_id = merge_hook_params.file_id
1342
 
        trans_id = merge_hook_params.trans_id
1343
 
        file_in_this = self.this_tree.has_id(file_id)
1344
 
        if self.other_tree.has_id(file_id):
1345
 
            # OTHER changed the file
1346
 
            wt = self.this_tree
1347
 
            if wt.supports_content_filtering():
1348
 
                # We get the path from the working tree if it exists.
1349
 
                # That fails though when OTHER is adding a file, so
1350
 
                # we fall back to the other tree to find the path if
1351
 
                # it doesn't exist locally.
1352
 
                try:
1353
 
                    filter_tree_path = wt.id2path(file_id)
1354
 
                except errors.NoSuchId:
1355
 
                    filter_tree_path = self.other_tree.id2path(file_id)
1356
 
            else:
1357
 
                # Skip the id2path lookup for older formats
1358
 
                filter_tree_path = None
1359
 
            transform.create_from_tree(self.tt, trans_id,
1360
 
                             self.other_tree, file_id,
1361
 
                             filter_tree_path=filter_tree_path)
1362
 
            return 'done', None
1363
 
        elif file_in_this:
1364
 
            # OTHER deleted the file
1365
 
            return 'delete', None
1366
 
        else:
1367
 
            raise AssertionError(
1368
 
                'winner is OTHER, but file_id %r not in THIS or OTHER tree'
1369
 
                % (file_id,))
1370
 
 
1371
 
    def merge_contents(self, merge_hook_params):
1372
 
        """Fallback merge logic after user installed hooks."""
1373
 
        # This function is used in merge hooks as the fallback instance.
1374
 
        # Perhaps making this function and the functions it calls be a 
1375
 
        # a separate class would be better.
1376
 
        if merge_hook_params.winner == 'other':
 
1140
        if winner == 'other':
1377
1141
            # OTHER is a straight winner, so replace this contents with other
1378
 
            return self._default_other_winner_merge(merge_hook_params)
1379
 
        elif merge_hook_params.is_file_merge():
1380
 
            # THIS and OTHER are both files, so text merge.  Either
1381
 
            # BASE is a file, or both converted to files, so at least we
1382
 
            # have agreement that output should be a file.
1383
 
            try:
1384
 
                self.text_merge(merge_hook_params.file_id,
1385
 
                    merge_hook_params.trans_id)
1386
 
            except errors.BinaryFile:
1387
 
                return 'not_applicable', None
1388
 
            return 'done', None
 
1142
            file_in_this = file_id in self.this_tree
 
1143
            if file_in_this:
 
1144
                # Remove any existing contents
 
1145
                self.tt.delete_contents(trans_id)
 
1146
            if file_id in self.other_tree:
 
1147
                # OTHER changed the file
 
1148
                create_from_tree(self.tt, trans_id,
 
1149
                                 self.other_tree, file_id)
 
1150
                if not file_in_this:
 
1151
                    self.tt.version_file(file_id, trans_id)
 
1152
                return "modified"
 
1153
            elif file_in_this:
 
1154
                # OTHER deleted the file
 
1155
                self.tt.unversion_file(trans_id)
 
1156
                return "deleted"
1389
1157
        else:
1390
 
            return 'not_applicable', None
 
1158
            # We have a hypothetical conflict, but if we have files, then we
 
1159
            # can try to merge the content
 
1160
            if this_pair[0] == 'file' and other_pair[0] == 'file':
 
1161
                # THIS and OTHER are both files, so text merge.  Either
 
1162
                # BASE is a file, or both converted to files, so at least we
 
1163
                # have agreement that output should be a file.
 
1164
                try:
 
1165
                    self.text_merge(file_id, trans_id)
 
1166
                except BinaryFile:
 
1167
                    return contents_conflict()
 
1168
                if file_id not in self.this_tree:
 
1169
                    self.tt.version_file(file_id, trans_id)
 
1170
                try:
 
1171
                    self.tt.tree_kind(trans_id)
 
1172
                    self.tt.delete_contents(trans_id)
 
1173
                except NoSuchFile:
 
1174
                    pass
 
1175
                return "modified"
 
1176
            else:
 
1177
                return contents_conflict()
1391
1178
 
1392
1179
    def get_lines(self, tree, file_id):
1393
1180
        """Return the lines in a file, or an empty list."""
1394
 
        if tree.has_id(file_id):
 
1181
        if file_id in tree:
1395
1182
            return tree.get_file(file_id).readlines()
1396
1183
        else:
1397
1184
            return []
1400
1187
        """Perform a three-way text merge on a file_id"""
1401
1188
        # it's possible that we got here with base as a different type.
1402
1189
        # if so, we just want two-way text conflicts.
1403
 
        if self.base_tree.has_id(file_id) and \
 
1190
        if file_id in self.base_tree and \
1404
1191
            self.base_tree.kind(file_id) == "file":
1405
1192
            base_lines = self.get_lines(self.base_tree, file_id)
1406
1193
        else:
1407
1194
            base_lines = []
1408
1195
        other_lines = self.get_lines(self.other_tree, file_id)
1409
1196
        this_lines = self.get_lines(self.this_tree, file_id)
1410
 
        m3 = merge3.Merge3(base_lines, this_lines, other_lines,
1411
 
                           is_cherrypick=self.cherrypick)
 
1197
        m3 = Merge3(base_lines, this_lines, other_lines,
 
1198
                    is_cherrypick=self.cherrypick)
1412
1199
        start_marker = "!START OF MERGE CONFLICT!" + "I HOPE THIS IS UNIQUE"
1413
1200
        if self.show_base is True:
1414
1201
            base_marker = '|' * 7
1452
1239
                ('THIS', self.this_tree, this_lines)]
1453
1240
        if not no_base:
1454
1241
            data.append(('BASE', self.base_tree, base_lines))
1455
 
 
1456
 
        # We need to use the actual path in the working tree of the file here,
1457
 
        # ignoring the conflict suffixes
1458
 
        wt = self.this_tree
1459
 
        if wt.supports_content_filtering():
1460
 
            try:
1461
 
                filter_tree_path = wt.id2path(file_id)
1462
 
            except errors.NoSuchId:
1463
 
                # file has been deleted
1464
 
                filter_tree_path = None
1465
 
        else:
1466
 
            # Skip the id2path lookup for older formats
1467
 
            filter_tree_path = None
1468
 
 
1469
1242
        versioned = False
1470
1243
        file_group = []
1471
1244
        for suffix, tree, lines in data:
1472
 
            if tree.has_id(file_id):
 
1245
            if file_id in tree:
1473
1246
                trans_id = self._conflict_file(name, parent_id, tree, file_id,
1474
 
                                               suffix, lines, filter_tree_path)
 
1247
                                               suffix, lines)
1475
1248
                file_group.append(trans_id)
1476
1249
                if set_version and not versioned:
1477
1250
                    self.tt.version_file(file_id, trans_id)
1479
1252
        return file_group
1480
1253
 
1481
1254
    def _conflict_file(self, name, parent_id, tree, file_id, suffix,
1482
 
                       lines=None, filter_tree_path=None):
 
1255
                       lines=None):
1483
1256
        """Emit a single conflict file."""
1484
1257
        name = name + '.' + suffix
1485
1258
        trans_id = self.tt.create_path(name, parent_id)
1486
 
        transform.create_from_tree(self.tt, trans_id, tree, file_id, lines,
1487
 
            filter_tree_path)
 
1259
        create_from_tree(self.tt, trans_id, tree, file_id, lines)
1488
1260
        return trans_id
1489
1261
 
1490
1262
    def merge_executable(self, file_id, file_status):
1514
1286
        try:
1515
1287
            if self.tt.final_kind(trans_id) != "file":
1516
1288
                return
1517
 
        except errors.NoSuchFile:
 
1289
        except NoSuchFile:
1518
1290
            return
1519
1291
        if winner == "this":
1520
1292
            executability = this_executable
1521
1293
        else:
1522
 
            if self.other_tree.has_id(file_id):
 
1294
            if file_id in self.other_tree:
1523
1295
                executability = other_executable
1524
 
            elif self.this_tree.has_id(file_id):
 
1296
            elif file_id in self.this_tree:
1525
1297
                executability = this_executable
1526
 
            elif self.base_tree_has_id(file_id):
 
1298
            elif file_id in self.base_tree:
1527
1299
                executability = base_executable
1528
1300
        if executability is not None:
1529
1301
            trans_id = self.tt.trans_id_file_id(file_id)
1531
1303
 
1532
1304
    def cook_conflicts(self, fs_conflicts):
1533
1305
        """Convert all conflicts into a form that doesn't depend on trans_id"""
 
1306
        from conflicts import Conflict
1534
1307
        name_conflicts = {}
1535
 
        self.cooked_conflicts.extend(transform.cook_conflicts(
1536
 
                fs_conflicts, self.tt))
1537
 
        fp = transform.FinalPaths(self.tt)
 
1308
        self.cooked_conflicts.extend(cook_conflicts(fs_conflicts, self.tt))
 
1309
        fp = FinalPaths(self.tt)
1538
1310
        for conflict in self._raw_conflicts:
1539
1311
            conflict_type = conflict[0]
1540
1312
            if conflict_type in ('name conflict', 'parent conflict'):
1542
1314
                conflict_args = conflict[2:]
1543
1315
                if trans_id not in name_conflicts:
1544
1316
                    name_conflicts[trans_id] = {}
1545
 
                transform.unique_add(name_conflicts[trans_id], conflict_type,
1546
 
                                     conflict_args)
 
1317
                unique_add(name_conflicts[trans_id], conflict_type,
 
1318
                           conflict_args)
1547
1319
            if conflict_type == 'contents conflict':
1548
1320
                for trans_id in conflict[1]:
1549
1321
                    file_id = self.tt.final_file_id(trans_id)
1554
1326
                    if path.endswith(suffix):
1555
1327
                        path = path[:-len(suffix)]
1556
1328
                        break
1557
 
                c = _mod_conflicts.Conflict.factory(conflict_type,
1558
 
                                                    path=path, file_id=file_id)
 
1329
                c = Conflict.factory(conflict_type, path=path, file_id=file_id)
1559
1330
                self.cooked_conflicts.append(c)
1560
1331
            if conflict_type == 'text conflict':
1561
1332
                trans_id = conflict[1]
1562
1333
                path = fp.get_path(trans_id)
1563
1334
                file_id = self.tt.final_file_id(trans_id)
1564
 
                c = _mod_conflicts.Conflict.factory(conflict_type,
1565
 
                                                    path=path, file_id=file_id)
 
1335
                c = Conflict.factory(conflict_type, path=path, file_id=file_id)
1566
1336
                self.cooked_conflicts.append(c)
1567
1337
 
1568
1338
        for trans_id, conflicts in name_conflicts.iteritems():
1583
1353
            if this_parent is not None and this_name is not None:
1584
1354
                this_parent_path = \
1585
1355
                    fp.get_path(self.tt.trans_id_file_id(this_parent))
1586
 
                this_path = osutils.pathjoin(this_parent_path, this_name)
 
1356
                this_path = pathjoin(this_parent_path, this_name)
1587
1357
            else:
1588
1358
                this_path = "<deleted>"
1589
1359
            file_id = self.tt.final_file_id(trans_id)
1590
 
            c = _mod_conflicts.Conflict.factory('path conflict', path=this_path,
1591
 
                                                conflict_path=other_path,
1592
 
                                                file_id=file_id)
 
1360
            c = Conflict.factory('path conflict', path=this_path,
 
1361
                                 conflict_path=other_path, file_id=file_id)
1593
1362
            self.cooked_conflicts.append(c)
1594
 
        self.cooked_conflicts.sort(key=_mod_conflicts.Conflict.sort_key)
 
1363
        self.cooked_conflicts.sort(key=Conflict.sort_key)
1595
1364
 
1596
1365
 
1597
1366
class WeaveMerger(Merge3Merger):
1601
1370
    supports_reverse_cherrypick = False
1602
1371
    history_based = True
1603
1372
 
1604
 
    def _generate_merge_plan(self, file_id, base):
1605
 
        return self.this_tree.plan_file_merge(file_id, self.other_tree,
 
1373
    def _merged_lines(self, file_id):
 
1374
        """Generate the merged lines.
 
1375
        There is no distinction between lines that are meant to contain <<<<<<<
 
1376
        and conflicts.
 
1377
        """
 
1378
        if self.cherrypick:
 
1379
            base = self.base_tree
 
1380
        else:
 
1381
            base = None
 
1382
        plan = self.this_tree.plan_file_merge(file_id, self.other_tree,
1606
1383
                                              base=base)
1607
 
 
1608
 
    def _merged_lines(self, file_id):
1609
 
        """Generate the merged lines.
1610
 
        There is no distinction between lines that are meant to contain <<<<<<<
1611
 
        and conflicts.
1612
 
        """
1613
 
        if self.cherrypick:
1614
 
            base = self.base_tree
1615
 
        else:
1616
 
            base = None
1617
 
        plan = self._generate_merge_plan(file_id, base)
1618
1384
        if 'merge' in debug.debug_flags:
1619
1385
            plan = list(plan)
1620
1386
            trans_id = self.tt.trans_id_file_id(file_id)
1621
1387
            name = self.tt.final_name(trans_id) + '.plan'
1622
 
            contents = ('%11s|%s' % l for l in plan)
 
1388
            contents = ('%10s|%s' % l for l in plan)
1623
1389
            self.tt.new_file(name, self.tt.final_parent(trans_id), contents)
1624
 
        textmerge = versionedfile.PlanWeaveMerge(plan, '<<<<<<< TREE\n',
1625
 
                                                 '>>>>>>> MERGE-SOURCE\n')
1626
 
        lines, conflicts = textmerge.merge_lines(self.reprocess)
1627
 
        if conflicts:
1628
 
            base_lines = textmerge.base_from_plan()
1629
 
        else:
1630
 
            base_lines = None
1631
 
        return lines, base_lines
 
1390
        textmerge = PlanWeaveMerge(plan, '<<<<<<< TREE\n',
 
1391
            '>>>>>>> MERGE-SOURCE\n')
 
1392
        return textmerge.merge_lines(self.reprocess)
1632
1393
 
1633
1394
    def text_merge(self, file_id, trans_id):
1634
1395
        """Perform a (weave) text merge for a given file and file-id.
1635
1396
        If conflicts are encountered, .THIS and .OTHER files will be emitted,
1636
1397
        and a conflict will be noted.
1637
1398
        """
1638
 
        lines, base_lines = self._merged_lines(file_id)
 
1399
        lines, conflicts = self._merged_lines(file_id)
1639
1400
        lines = list(lines)
1640
1401
        # Note we're checking whether the OUTPUT is binary in this case,
1641
1402
        # because we don't want to get into weave merge guts.
1642
 
        textfile.check_text_lines(lines)
 
1403
        check_text_lines(lines)
1643
1404
        self.tt.create_file(lines, trans_id)
1644
 
        if base_lines is not None:
1645
 
            # Conflict
 
1405
        if conflicts:
1646
1406
            self._raw_conflicts.append(('text conflict', trans_id))
1647
1407
            name = self.tt.final_name(trans_id)
1648
1408
            parent_id = self.tt.final_parent(trans_id)
1649
1409
            file_group = self._dump_conflicts(name, parent_id, file_id,
1650
 
                                              no_base=False,
1651
 
                                              base_lines=base_lines)
 
1410
                                              no_base=True)
1652
1411
            file_group.append(trans_id)
1653
1412
 
1654
1413
 
1655
1414
class LCAMerger(WeaveMerger):
1656
1415
 
1657
 
    def _generate_merge_plan(self, file_id, base):
1658
 
        return self.this_tree.plan_file_lca_merge(file_id, self.other_tree,
 
1416
    def _merged_lines(self, file_id):
 
1417
        """Generate the merged lines.
 
1418
        There is no distinction between lines that are meant to contain <<<<<<<
 
1419
        and conflicts.
 
1420
        """
 
1421
        if self.cherrypick:
 
1422
            base = self.base_tree
 
1423
        else:
 
1424
            base = None
 
1425
        plan = self.this_tree.plan_file_lca_merge(file_id, self.other_tree,
1659
1426
                                                  base=base)
 
1427
        if 'merge' in debug.debug_flags:
 
1428
            plan = list(plan)
 
1429
            trans_id = self.tt.trans_id_file_id(file_id)
 
1430
            name = self.tt.final_name(trans_id) + '.plan'
 
1431
            contents = ('%10s|%s' % l for l in plan)
 
1432
            self.tt.new_file(name, self.tt.final_parent(trans_id), contents)
 
1433
        textmerge = PlanWeaveMerge(plan, '<<<<<<< TREE\n',
 
1434
            '>>>>>>> MERGE-SOURCE\n')
 
1435
        return textmerge.merge_lines(self.reprocess)
 
1436
 
1660
1437
 
1661
1438
class Diff3Merger(Merge3Merger):
1662
1439
    """Three-way merger using external diff3 for text merging"""
1663
1440
 
1664
1441
    def dump_file(self, temp_dir, name, tree, file_id):
1665
 
        out_path = osutils.pathjoin(temp_dir, name)
 
1442
        out_path = pathjoin(temp_dir, name)
1666
1443
        out_file = open(out_path, "wb")
1667
1444
        try:
1668
1445
            in_file = tree.get_file(file_id)
1680
1457
        import bzrlib.patch
1681
1458
        temp_dir = osutils.mkdtemp(prefix="bzr-")
1682
1459
        try:
1683
 
            new_file = osutils.pathjoin(temp_dir, "new")
 
1460
            new_file = pathjoin(temp_dir, "new")
1684
1461
            this = self.dump_file(temp_dir, "this", self.this_tree, file_id)
1685
1462
            base = self.dump_file(temp_dir, "base", self.base_tree, file_id)
1686
1463
            other = self.dump_file(temp_dir, "other", self.other_tree, file_id)
1687
1464
            status = bzrlib.patch.diff3(new_file, this, base, other)
1688
1465
            if status not in (0, 1):
1689
 
                raise errors.BzrError("Unhandled diff3 exit code")
 
1466
                raise BzrError("Unhandled diff3 exit code")
1690
1467
            f = open(new_file, 'rb')
1691
1468
            try:
1692
1469
                self.tt.create_file(f, trans_id)
1710
1487
                other_rev_id=None,
1711
1488
                interesting_files=None,
1712
1489
                this_tree=None,
1713
 
                pb=None,
 
1490
                pb=DummyProgress(),
1714
1491
                change_reporter=None):
1715
1492
    """Primary interface for merging.
1716
1493
 
1719
1496
                     branch.get_revision_tree(base_revision))'
1720
1497
        """
1721
1498
    if this_tree is None:
1722
 
        raise errors.BzrError("bzrlib.merge.merge_inner requires a this_tree "
1723
 
                              "parameter as of bzrlib version 0.8.")
 
1499
        raise BzrError("bzrlib.merge.merge_inner requires a this_tree "
 
1500
            "parameter as of bzrlib version 0.8.")
1724
1501
    merger = Merger(this_branch, other_tree, base_tree, this_tree=this_tree,
1725
1502
                    pb=pb, change_reporter=change_reporter)
1726
1503
    merger.backup_files = backup_files
1739
1516
    get_revision_id = getattr(base_tree, 'get_revision_id', None)
1740
1517
    if get_revision_id is None:
1741
1518
        get_revision_id = base_tree.last_revision
1742
 
    merger.cache_trees_with_revision_ids([other_tree, base_tree, this_tree])
1743
1519
    merger.set_base_revision(get_revision_id(), this_branch)
1744
1520
    return merger.do_merge()
1745
1521
 
1944
1720
        super(_PlanMerge, self).__init__(a_rev, b_rev, vf, key_prefix)
1945
1721
        self.a_key = self._key_prefix + (self.a_rev,)
1946
1722
        self.b_key = self._key_prefix + (self.b_rev,)
1947
 
        self.graph = _mod_graph.Graph(self.vf)
 
1723
        self.graph = Graph(self.vf)
1948
1724
        heads = self.graph.heads((self.a_key, self.b_key))
1949
1725
        if len(heads) == 1:
1950
1726
            # one side dominates, so we can just return its values, yay for
1955
1731
                other = b_rev
1956
1732
            else:
1957
1733
                other = a_rev
1958
 
            trace.mutter('found dominating revision for %s\n%s > %s', self.vf,
1959
 
                         self._head_key[-1], other)
 
1734
            mutter('found dominating revision for %s\n%s > %s', self.vf,
 
1735
                   self._head_key[-1], other)
1960
1736
            self._weave = None
1961
1737
        else:
1962
1738
            self._head_key = None
1976
1752
        while True:
1977
1753
            next_lcas = self.graph.find_lca(*cur_ancestors)
1978
1754
            # Map a plain NULL_REVISION to a simple no-ancestors
1979
 
            if next_lcas == set([_mod_revision.NULL_REVISION]):
 
1755
            if next_lcas == set([NULL_REVISION]):
1980
1756
                next_lcas = ()
1981
1757
            # Order the lca's based on when they were merged into the tip
1982
1758
            # While the actual merge portion of weave merge uses a set() of
1994
1770
            elif len(next_lcas) > 2:
1995
1771
                # More than 2 lca's, fall back to grabbing all nodes between
1996
1772
                # this and the unique lca.
1997
 
                trace.mutter('More than 2 LCAs, falling back to all nodes for:'
1998
 
                             ' %s, %s\n=> %s',
1999
 
                             self.a_key, self.b_key, cur_ancestors)
 
1773
                mutter('More than 2 LCAs, falling back to all nodes for:'
 
1774
                       ' %s, %s\n=> %s', self.a_key, self.b_key, cur_ancestors)
2000
1775
                cur_lcas = next_lcas
2001
1776
                while len(cur_lcas) > 1:
2002
1777
                    cur_lcas = self.graph.find_lca(*cur_lcas)
2005
1780
                    unique_lca = None
2006
1781
                else:
2007
1782
                    unique_lca = list(cur_lcas)[0]
2008
 
                    if unique_lca == _mod_revision.NULL_REVISION:
 
1783
                    if unique_lca == NULL_REVISION:
2009
1784
                        # find_lca will return a plain 'NULL_REVISION' rather
2010
1785
                        # than a key tuple when there is no common ancestor, we
2011
1786
                        # prefer to just use None, because it doesn't confuse
2034
1809
            # We remove NULL_REVISION because it isn't a proper tuple key, and
2035
1810
            # thus confuses things like _get_interesting_texts, and our logic
2036
1811
            # to add the texts into the memory weave.
2037
 
            if _mod_revision.NULL_REVISION in parent_map:
2038
 
                parent_map.pop(_mod_revision.NULL_REVISION)
 
1812
            if NULL_REVISION in parent_map:
 
1813
                parent_map.pop(NULL_REVISION)
2039
1814
        else:
2040
1815
            interesting = set()
2041
1816
            for tip in tip_keys:
2193
1968
        lcas = graph.find_lca(key_prefix + (a_rev,), key_prefix + (b_rev,))
2194
1969
        self.lcas = set()
2195
1970
        for lca in lcas:
2196
 
            if lca == _mod_revision.NULL_REVISION:
 
1971
            if lca == NULL_REVISION:
2197
1972
                self.lcas.add(lca)
2198
1973
            else:
2199
1974
                self.lcas.add(lca[-1])