~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/merge.py

  • Committer: John Arbash Meinel
  • Date: 2009-06-19 17:40:59 UTC
  • mto: This revision was merged to the branch mainline in revision 4466.
  • Revision ID: john@arbash-meinel.com-20090619174059-jzowjv0d86vzjg4m
Update the python code to do the same checking around known_parent_gdfos being present.

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
 
from bzrlib.lazy_import import lazy_import
20
 
lazy_import(globals(), """
21
23
from bzrlib import (
22
 
    branch as _mod_branch,
23
 
    conflicts as _mod_conflicts,
24
24
    debug,
25
 
    generate_ids,
 
25
    errors,
26
26
    graph as _mod_graph,
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
 
    workingtree,
39
 
    )
40
 
from bzrlib.cleanup import OperationWithCleanups
41
 
""")
42
 
from bzrlib import (
43
 
    decorators,
44
 
    errors,
45
 
    hooks,
46
 
    )
47
 
from bzrlib.symbol_versioning import (
48
 
    deprecated_in,
49
 
    deprecated_method,
50
 
    )
 
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
 
51
63
# TODO: Report back as changes are merged in
52
64
 
53
65
 
54
66
def transform_tree(from_tree, to_tree, interesting_ids=None):
55
 
    from_tree.lock_tree_write()
56
 
    operation = OperationWithCleanups(merge_inner)
57
 
    operation.add_cleanup(from_tree.unlock)
58
 
    operation.run_simple(from_tree.branch, to_tree, from_tree,
59
 
        ignore_zero=True, interesting_ids=interesting_ids, this_tree=from_tree)
60
 
 
61
 
 
62
 
class MergeHooks(hooks.Hooks):
63
 
 
64
 
    def __init__(self):
65
 
        hooks.Hooks.__init__(self)
66
 
        self.create_hook(hooks.HookPoint('merge_file_content',
67
 
            "Called with a bzrlib.merge.Merger object to create a per file "
68
 
            "merge object when starting a merge. "
69
 
            "Should return either None or a subclass of "
70
 
            "``bzrlib.merge.AbstractPerFileMerger``. "
71
 
            "Such objects will then be called per file "
72
 
            "that needs to be merged (including when one "
73
 
            "side has deleted the file and the other has changed it). "
74
 
            "See the AbstractPerFileMerger API docs for details on how it is "
75
 
            "used by merge.",
76
 
            (2, 1), None))
77
 
 
78
 
 
79
 
class AbstractPerFileMerger(object):
80
 
    """PerFileMerger objects are used by plugins extending merge for bzrlib.
81
 
 
82
 
    See ``bzrlib.plugins.news_merge.news_merge`` for an example concrete class.
83
 
    
84
 
    :ivar merger: The Merge3Merger performing the merge.
85
 
    """
86
 
 
87
 
    def __init__(self, merger):
88
 
        """Create a PerFileMerger for use with merger."""
89
 
        self.merger = merger
90
 
 
91
 
    def merge_contents(self, merge_params):
92
 
        """Attempt to merge the contents of a single file.
93
 
        
94
 
        :param merge_params: A bzrlib.merge.MergeHookParams
95
 
        :return : A tuple of (status, chunks), where status is one of
96
 
            'not_applicable', 'success', 'conflicted', or 'delete'.  If status
97
 
            is 'success' or 'conflicted', then chunks should be an iterable of
98
 
            strings for the new file contents.
99
 
        """
100
 
        return ('not applicable', None)
101
 
 
102
 
 
103
 
class PerFileMerger(AbstractPerFileMerger):
104
 
    """Merge individual files when self.file_matches returns True.
105
 
 
106
 
    This class is intended to be subclassed.  The file_matches and
107
 
    merge_matching methods should be overridden with concrete implementations.
108
 
    """
109
 
 
110
 
    def file_matches(self, params):
111
 
        """Return True if merge_matching should be called on this file.
112
 
 
113
 
        Only called with merges of plain files with no clear winner.
114
 
 
115
 
        Subclasses must override this.
116
 
        """
117
 
        raise NotImplementedError(self.file_matches)
118
 
 
119
 
    def get_filename(self, params, tree):
120
 
        """Lookup the filename (i.e. basename, not path), given a Tree (e.g.
121
 
        self.merger.this_tree) and a MergeHookParams.
122
 
        """
123
 
        return osutils.basename(tree.id2path(params.file_id))
124
 
 
125
 
    def get_filepath(self, params, tree):
126
 
        """Calculate the path to the file in a tree.
127
 
 
128
 
        :param params: A MergeHookParams describing the file to merge
129
 
        :param tree: a Tree, e.g. self.merger.this_tree.
130
 
        """
131
 
        return tree.id2path(params.file_id)
132
 
 
133
 
    def merge_contents(self, params):
134
 
        """Merge the contents of a single file."""
135
 
        # Check whether this custom merge logic should be used.
136
 
        if (
137
 
            # OTHER is a straight winner, rely on default merge.
138
 
            params.winner == 'other' or
139
 
            # THIS and OTHER aren't both files.
140
 
            not params.is_file_merge() or
141
 
            # The filename doesn't match *.xml
142
 
            not self.file_matches(params)):
143
 
            return 'not_applicable', None
144
 
        return self.merge_matching(params)
145
 
 
146
 
    def merge_matching(self, params):
147
 
        """Merge the contents of a single file that has matched the criteria
148
 
        in PerFileMerger.merge_contents (is a conflict, is a file,
149
 
        self.file_matches is True).
150
 
 
151
 
        Subclasses must override this.
152
 
        """
153
 
        raise NotImplementedError(self.merge_matching)
154
 
 
155
 
 
156
 
class ConfigurableFileMerger(PerFileMerger):
157
 
    """Merge individual files when configured via a .conf file.
158
 
 
159
 
    This is a base class for concrete custom file merging logic. Concrete
160
 
    classes should implement ``merge_text``.
161
 
 
162
 
    See ``bzrlib.plugins.news_merge.news_merge`` for an example concrete class.
163
 
    
164
 
    :ivar affected_files: The configured file paths to merge.
165
 
 
166
 
    :cvar name_prefix: The prefix to use when looking up configuration
167
 
        details. <name_prefix>_merge_files describes the files targeted by the
168
 
        hook for example.
169
 
        
170
 
    :cvar default_files: The default file paths to merge when no configuration
171
 
        is present.
172
 
    """
173
 
 
174
 
    name_prefix = None
175
 
    default_files = None
176
 
 
177
 
    def __init__(self, merger):
178
 
        super(ConfigurableFileMerger, self).__init__(merger)
179
 
        self.affected_files = None
180
 
        self.default_files = self.__class__.default_files or []
181
 
        self.name_prefix = self.__class__.name_prefix
182
 
        if self.name_prefix is None:
183
 
            raise ValueError("name_prefix must be set.")
184
 
 
185
 
    def file_matches(self, params):
186
 
        """Check whether the file should call the merge hook.
187
 
 
188
 
        <name_prefix>_merge_files configuration variable is a list of files
189
 
        that should use the hook.
190
 
        """
191
 
        affected_files = self.affected_files
192
 
        if affected_files is None:
193
 
            config = self.merger.this_branch.get_config()
194
 
            # Until bzr provides a better policy for caching the config, we
195
 
            # just add the part we're interested in to the params to avoid
196
 
            # reading the config files repeatedly (bazaar.conf, location.conf,
197
 
            # branch.conf).
198
 
            config_key = self.name_prefix + '_merge_files'
199
 
            affected_files = config.get_user_option_as_list(config_key)
200
 
            if affected_files is None:
201
 
                # If nothing was specified in the config, use the default.
202
 
                affected_files = self.default_files
203
 
            self.affected_files = affected_files
204
 
        if affected_files:
205
 
            filepath = self.get_filepath(params, self.merger.this_tree)
206
 
            if filepath in affected_files:
207
 
                return True
208
 
        return False
209
 
 
210
 
    def merge_matching(self, params):
211
 
        return self.merge_text(params)
212
 
 
213
 
    def merge_text(self, params):
214
 
        """Merge the byte contents of a single file.
215
 
 
216
 
        This is called after checking that the merge should be performed in
217
 
        merge_contents, and it should behave as per
218
 
        ``bzrlib.merge.AbstractPerFileMerger.merge_contents``.
219
 
        """
220
 
        raise NotImplementedError(self.merge_text)
221
 
 
222
 
 
223
 
class MergeHookParams(object):
224
 
    """Object holding parameters passed to merge_file_content hooks.
225
 
 
226
 
    There are some fields hooks can access:
227
 
 
228
 
    :ivar file_id: the file ID of the file being merged
229
 
    :ivar trans_id: the transform ID for the merge of this file
230
 
    :ivar this_kind: kind of file_id in 'this' tree
231
 
    :ivar other_kind: kind of file_id in 'other' tree
232
 
    :ivar winner: one of 'this', 'other', 'conflict'
233
 
    """
234
 
 
235
 
    def __init__(self, merger, file_id, trans_id, this_kind, other_kind,
236
 
            winner):
237
 
        self._merger = merger
238
 
        self.file_id = file_id
239
 
        self.trans_id = trans_id
240
 
        self.this_kind = this_kind
241
 
        self.other_kind = other_kind
242
 
        self.winner = winner
243
 
 
244
 
    def is_file_merge(self):
245
 
        """True if this_kind and other_kind are both 'file'."""
246
 
        return self.this_kind == 'file' and self.other_kind == 'file'
247
 
 
248
 
    @decorators.cachedproperty
249
 
    def base_lines(self):
250
 
        """The lines of the 'base' version of the file."""
251
 
        return self._merger.get_lines(self._merger.base_tree, self.file_id)
252
 
 
253
 
    @decorators.cachedproperty
254
 
    def this_lines(self):
255
 
        """The lines of the 'this' version of the file."""
256
 
        return self._merger.get_lines(self._merger.this_tree, self.file_id)
257
 
 
258
 
    @decorators.cachedproperty
259
 
    def other_lines(self):
260
 
        """The lines of the 'other' version of the file."""
261
 
        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)
262
69
 
263
70
 
264
71
class Merger(object):
265
 
 
266
 
    hooks = MergeHooks()
267
 
 
268
72
    def __init__(self, this_branch, other_tree=None, base_tree=None,
269
73
                 this_tree=None, pb=None, change_reporter=None,
270
74
                 recurse='down', revision_graph=None):
285
89
        self.interesting_files = None
286
90
        self.show_base = False
287
91
        self.reprocess = False
288
 
        if pb is not None:
289
 
            warnings.warn("pb parameter to Merger() is deprecated and ignored")
 
92
        if pb is None:
 
93
            pb = DummyProgress()
 
94
        self._pb = pb
290
95
        self.pp = None
291
96
        self.recurse = recurse
292
97
        self.change_reporter = change_reporter
297
102
        self._is_criss_cross = None
298
103
        self._lca_trees = None
299
104
 
300
 
    def cache_trees_with_revision_ids(self, trees):
301
 
        """Cache any tree in trees if it has a revision_id."""
302
 
        for maybe_tree in trees:
303
 
            if maybe_tree is None:
304
 
                continue
305
 
            try:
306
 
                rev_id = maybe_tree.get_revision_id()
307
 
            except AttributeError:
308
 
                continue
309
 
            self._cached_trees[rev_id] = maybe_tree
310
 
 
311
105
    @property
312
106
    def revision_graph(self):
313
107
        if self._revision_graph is None:
375
169
                base_revision_id, tree.branch.last_revision())):
376
170
                base_revision_id = None
377
171
            else:
378
 
                trace.warning('Performing cherrypick')
 
172
                warning('Performing cherrypick')
379
173
        merger = klass.from_revision_ids(pb, tree, other_revision_id,
380
174
                                         base_revision_id, revision_graph=
381
175
                                         revision_graph)
428
222
        return self._cached_trees[revision_id]
429
223
 
430
224
    def _get_tree(self, treespec, possible_transports=None):
 
225
        from bzrlib import workingtree
431
226
        location, revno = treespec
432
227
        if revno is None:
433
228
            tree = workingtree.WorkingTree.open_containing(location)[0]
434
229
            return tree.branch, tree
435
 
        branch = _mod_branch.Branch.open_containing(
436
 
            location, possible_transports)[0]
 
230
        branch = Branch.open_containing(location, possible_transports)[0]
437
231
        if revno == -1:
438
232
            revision_id = branch.last_revision()
439
233
        else:
440
234
            revision_id = branch.get_rev_id(revno)
441
 
        revision_id = _mod_revision.ensure_null(revision_id)
 
235
        revision_id = ensure_null(revision_id)
442
236
        return branch, self.revision_tree(revision_id, branch)
443
237
 
444
 
    @deprecated_method(deprecated_in((2, 1, 0)))
445
238
    def ensure_revision_trees(self):
446
239
        if self.this_revision_tree is None:
447
240
            self.this_basis_tree = self.revision_tree(self.this_basis)
450
243
 
451
244
        if self.other_rev_id is None:
452
245
            other_basis_tree = self.revision_tree(self.other_basis)
453
 
            if other_basis_tree.has_changes(self.other_tree):
454
 
                raise errors.WorkingTreeNotRevision(self.this_tree)
 
246
            changes = other_basis_tree.changes_from(self.other_tree)
 
247
            if changes.has_changed():
 
248
                raise WorkingTreeNotRevision(self.this_tree)
455
249
            other_rev_id = self.other_basis
456
250
            self.other_tree = other_basis_tree
457
251
 
458
 
    @deprecated_method(deprecated_in((2, 1, 0)))
459
252
    def file_revisions(self, file_id):
460
253
        self.ensure_revision_trees()
461
254
        def get_id(tree, file_id):
464
257
        if self.this_rev_id is None:
465
258
            if self.this_basis_tree.get_file_sha1(file_id) != \
466
259
                self.this_tree.get_file_sha1(file_id):
467
 
                raise errors.WorkingTreeNotRevision(self.this_tree)
 
260
                raise WorkingTreeNotRevision(self.this_tree)
468
261
 
469
262
        trees = (self.this_basis_tree, self.other_tree)
470
263
        return [get_id(tree, file_id) for tree in trees]
471
264
 
472
 
    @deprecated_method(deprecated_in((2, 1, 0)))
473
265
    def check_basis(self, check_clean, require_commits=True):
474
266
        if self.this_basis is None and require_commits is True:
475
 
            raise errors.BzrCommandError(
476
 
                "This branch has no commits."
477
 
                " (perhaps you would prefer 'bzr pull')")
 
267
            raise BzrCommandError("This branch has no commits."
 
268
                                  " (perhaps you would prefer 'bzr pull')")
478
269
        if check_clean:
479
270
            self.compare_basis()
480
271
            if self.this_basis != self.this_rev_id:
481
272
                raise errors.UncommittedChanges(self.this_tree)
482
273
 
483
 
    @deprecated_method(deprecated_in((2, 1, 0)))
484
274
    def compare_basis(self):
485
275
        try:
486
276
            basis_tree = self.revision_tree(self.this_tree.last_revision())
487
277
        except errors.NoSuchRevision:
488
278
            basis_tree = self.this_tree.basis_tree()
489
 
        if not self.this_tree.has_changes(basis_tree):
 
279
        changes = self.this_tree.changes_from(basis_tree)
 
280
        if not changes.has_changed():
490
281
            self.this_rev_id = self.this_basis
491
282
 
492
283
    def set_interesting_files(self, file_list):
493
284
        self.interesting_files = file_list
494
285
 
495
286
    def set_pending(self):
496
 
        if (not self.base_is_ancestor or not self.base_is_other_ancestor
497
 
            or self.other_rev_id is None):
 
287
        if not self.base_is_ancestor or not self.base_is_other_ancestor or self.other_rev_id is None:
498
288
            return
499
289
        self._add_parent()
500
290
 
501
291
    def _add_parent(self):
502
292
        new_parents = self.this_tree.get_parent_ids() + [self.other_rev_id]
503
293
        new_parent_trees = []
504
 
        operation = OperationWithCleanups(self.this_tree.set_parent_trees)
505
294
        for revision_id in new_parents:
506
295
            try:
507
296
                tree = self.revision_tree(revision_id)
509
298
                tree = None
510
299
            else:
511
300
                tree.lock_read()
512
 
                operation.add_cleanup(tree.unlock)
513
301
            new_parent_trees.append((revision_id, tree))
514
 
        operation.run_simple(new_parent_trees, allow_leftmost_as_ghost=True)
 
302
        try:
 
303
            self.this_tree.set_parent_trees(new_parent_trees,
 
304
                                            allow_leftmost_as_ghost=True)
 
305
        finally:
 
306
            for _revision_id, tree in new_parent_trees:
 
307
                if tree is not None:
 
308
                    tree.unlock()
515
309
 
516
310
    def set_other(self, other_revision, possible_transports=None):
517
311
        """Set the revision and tree to merge from.
526
320
            self.other_rev_id = _mod_revision.ensure_null(
527
321
                self.other_branch.last_revision())
528
322
            if _mod_revision.is_null(self.other_rev_id):
529
 
                raise errors.NoCommits(self.other_branch)
 
323
                raise NoCommits(self.other_branch)
530
324
            self.other_basis = self.other_rev_id
531
325
        elif other_revision[1] is not None:
532
326
            self.other_rev_id = self.other_branch.get_rev_id(other_revision[1])
535
329
            self.other_rev_id = None
536
330
            self.other_basis = self.other_branch.last_revision()
537
331
            if self.other_basis is None:
538
 
                raise errors.NoCommits(self.other_branch)
 
332
                raise NoCommits(self.other_branch)
539
333
        if self.other_rev_id is not None:
540
334
            self._cached_trees[self.other_rev_id] = self.other_tree
541
335
        self._maybe_fetch(self.other_branch,self.this_branch, self.other_basis)
568
362
            target.fetch(source, revision_id)
569
363
 
570
364
    def find_base(self):
571
 
        revisions = [_mod_revision.ensure_null(self.this_basis),
572
 
                     _mod_revision.ensure_null(self.other_basis)]
573
 
        if _mod_revision.NULL_REVISION in revisions:
574
 
            self.base_rev_id = _mod_revision.NULL_REVISION
 
365
        revisions = [ensure_null(self.this_basis),
 
366
                     ensure_null(self.other_basis)]
 
367
        if NULL_REVISION in revisions:
 
368
            self.base_rev_id = NULL_REVISION
575
369
            self.base_tree = self.revision_tree(self.base_rev_id)
576
370
            self._is_criss_cross = False
577
371
        else:
578
372
            lcas = self.revision_graph.find_lca(revisions[0], revisions[1])
579
373
            self._is_criss_cross = False
580
374
            if len(lcas) == 0:
581
 
                self.base_rev_id = _mod_revision.NULL_REVISION
 
375
                self.base_rev_id = NULL_REVISION
582
376
            elif len(lcas) == 1:
583
377
                self.base_rev_id = list(lcas)[0]
584
378
            else: # len(lcas) > 1
593
387
                    self.base_rev_id = self.revision_graph.find_unique_lca(
594
388
                                            *lcas)
595
389
                self._is_criss_cross = True
596
 
            if self.base_rev_id == _mod_revision.NULL_REVISION:
597
 
                raise errors.UnrelatedBranches()
 
390
            if self.base_rev_id == NULL_REVISION:
 
391
                raise UnrelatedBranches()
598
392
            if self._is_criss_cross:
599
 
                trace.warning('Warning: criss-cross merge encountered.  See bzr'
600
 
                              ' help criss-cross.')
601
 
                trace.mutter('Criss-cross lcas: %r' % lcas)
 
393
                warning('Warning: criss-cross merge encountered.  See bzr'
 
394
                        ' help criss-cross.')
 
395
                mutter('Criss-cross lcas: %r' % lcas)
602
396
                interesting_revision_ids = [self.base_rev_id]
603
397
                interesting_revision_ids.extend(lcas)
604
398
                interesting_trees = dict((t.get_revision_id(), t)
614
408
                self.base_tree = self.revision_tree(self.base_rev_id)
615
409
        self.base_is_ancestor = True
616
410
        self.base_is_other_ancestor = True
617
 
        trace.mutter('Base revid: %r' % self.base_rev_id)
 
411
        mutter('Base revid: %r' % self.base_rev_id)
618
412
 
619
413
    def set_base(self, base_revision):
620
414
        """Set the base revision to use for the merge.
621
415
 
622
416
        :param base_revision: A 2-list containing a path and revision number.
623
417
        """
624
 
        trace.mutter("doing merge() with no base_revision specified")
 
418
        mutter("doing merge() with no base_revision specified")
625
419
        if base_revision == [None, None]:
626
420
            self.find_base()
627
421
        else:
640
434
                  'other_tree': self.other_tree,
641
435
                  'interesting_ids': self.interesting_ids,
642
436
                  'interesting_files': self.interesting_files,
643
 
                  'this_branch': self.this_branch,
 
437
                  'pp': self.pp,
644
438
                  'do_merge': False}
645
439
        if self.merge_type.requires_base:
646
440
            kwargs['base_tree'] = self.base_tree
647
441
        if self.merge_type.supports_reprocess:
648
442
            kwargs['reprocess'] = self.reprocess
649
443
        elif self.reprocess:
650
 
            raise errors.BzrError(
651
 
                "Conflict reduction is not supported for merge"
652
 
                " type %s." % self.merge_type)
 
444
            raise BzrError("Conflict reduction is not supported for merge"
 
445
                                  " type %s." % self.merge_type)
653
446
        if self.merge_type.supports_show_base:
654
447
            kwargs['show_base'] = self.show_base
655
448
        elif self.show_base:
656
 
            raise errors.BzrError("Showing base is not supported for this"
657
 
                                  " merge type. %s" % self.merge_type)
 
449
            raise BzrError("Showing base is not supported for this"
 
450
                           " merge type. %s" % self.merge_type)
658
451
        if (not getattr(self.merge_type, 'supports_reverse_cherrypick', True)
659
452
            and not self.base_is_other_ancestor):
660
453
            raise errors.CannotReverseCherrypick()
664
457
        if self._is_criss_cross and getattr(self.merge_type,
665
458
                                            'supports_lca_trees', False):
666
459
            kwargs['lca_trees'] = self._lca_trees
667
 
        return self.merge_type(pb=None,
 
460
        return self.merge_type(pb=self._pb,
668
461
                               change_reporter=self.change_reporter,
669
462
                               **kwargs)
670
463
 
671
 
    def _do_merge_to(self):
672
 
        merge = self.make_merger()
 
464
    def _do_merge_to(self, merge):
673
465
        if self.other_branch is not None:
674
466
            self.other_branch.update_references(self.this_branch)
675
467
        merge.do_merge()
689
481
                    sub_tree.branch.repository.revision_tree(base_revision)
690
482
                sub_merge.base_rev_id = base_revision
691
483
                sub_merge.do_merge()
692
 
        return merge
693
484
 
694
485
    def do_merge(self):
695
 
        operation = OperationWithCleanups(self._do_merge_to)
696
486
        self.this_tree.lock_tree_write()
697
 
        operation.add_cleanup(self.this_tree.unlock)
698
 
        if self.base_tree is not None:
699
 
            self.base_tree.lock_read()
700
 
            operation.add_cleanup(self.base_tree.unlock)
701
 
        if self.other_tree is not None:
702
 
            self.other_tree.lock_read()
703
 
            operation.add_cleanup(self.other_tree.unlock)
704
 
        merge = operation.run_simple()
 
487
        try:
 
488
            if self.base_tree is not None:
 
489
                self.base_tree.lock_read()
 
490
            try:
 
491
                if self.other_tree is not None:
 
492
                    self.other_tree.lock_read()
 
493
                try:
 
494
                    merge = self.make_merger()
 
495
                    self._do_merge_to(merge)
 
496
                finally:
 
497
                    if self.other_tree is not None:
 
498
                        self.other_tree.unlock()
 
499
            finally:
 
500
                if self.base_tree is not None:
 
501
                    self.base_tree.unlock()
 
502
        finally:
 
503
            self.this_tree.unlock()
705
504
        if len(merge.cooked_conflicts) == 0:
706
 
            if not self.ignore_zero and not trace.is_quiet():
707
 
                trace.note("All changes applied successfully.")
 
505
            if not self.ignore_zero and not is_quiet():
 
506
                note("All changes applied successfully.")
708
507
        else:
709
 
            trace.note("%d conflicts encountered."
710
 
                       % len(merge.cooked_conflicts))
 
508
            note("%d conflicts encountered." % len(merge.cooked_conflicts))
711
509
 
712
510
        return len(merge.cooked_conflicts)
713
511
 
742
540
 
743
541
    def __init__(self, working_tree, this_tree, base_tree, other_tree,
744
542
                 interesting_ids=None, reprocess=False, show_base=False,
745
 
                 pb=None, pp=None, change_reporter=None,
 
543
                 pb=DummyProgress(), pp=None, change_reporter=None,
746
544
                 interesting_files=None, do_merge=True,
747
 
                 cherrypick=False, lca_trees=None, this_branch=None):
 
545
                 cherrypick=False, lca_trees=None):
748
546
        """Initialize the merger object and perform the merge.
749
547
 
750
548
        :param working_tree: The working tree to apply the merge to
751
549
        :param this_tree: The local tree in the merge operation
752
550
        :param base_tree: The common tree in the merge operation
753
551
        :param other_tree: The other tree to merge changes from
754
 
        :param this_branch: The branch associated with this_tree.  Defaults to
755
 
            this_tree.branch if not supplied.
756
552
        :param interesting_ids: The file_ids of files that should be
757
553
            participate in the merge.  May not be combined with
758
554
            interesting_files.
759
555
        :param: reprocess If True, perform conflict-reduction processing.
760
556
        :param show_base: If True, show the base revision in text conflicts.
761
557
            (incompatible with reprocess)
762
 
        :param pb: ignored
 
558
        :param pb: A Progress bar
763
559
        :param pp: A ProgressPhase object
764
560
        :param change_reporter: An object that should report changes made
765
561
        :param interesting_files: The tree-relative paths of files that should
776
572
        if interesting_files is not None and interesting_ids is not None:
777
573
            raise ValueError(
778
574
                'specify either interesting_ids or interesting_files')
779
 
        if this_branch is None:
780
 
            this_branch = this_tree.branch
781
575
        self.interesting_ids = interesting_ids
782
576
        self.interesting_files = interesting_files
783
577
        self.this_tree = working_tree
784
578
        self.base_tree = base_tree
785
579
        self.other_tree = other_tree
786
 
        self.this_branch = this_branch
787
580
        self._raw_conflicts = []
788
581
        self.cooked_conflicts = []
789
582
        self.reprocess = reprocess
794
587
        # making sure we haven't missed any corner cases.
795
588
        # if lca_trees is None:
796
589
        #     self._lca_trees = [self.base_tree]
 
590
        self.pb = pb
 
591
        self.pp = pp
797
592
        self.change_reporter = change_reporter
798
593
        self.cherrypick = cherrypick
 
594
        if self.pp is None:
 
595
            self.pp = ProgressPhase("Merge phase", 3, self.pb)
799
596
        if do_merge:
800
597
            self.do_merge()
801
 
        if pp is not None:
802
 
            warnings.warn("pp argument to Merge3Merger is deprecated")
803
 
        if pb is not None:
804
 
            warnings.warn("pb argument to Merge3Merger is deprecated")
805
598
 
806
599
    def do_merge(self):
807
 
        operation = OperationWithCleanups(self._do_merge)
808
600
        self.this_tree.lock_tree_write()
809
 
        operation.add_cleanup(self.this_tree.unlock)
810
601
        self.base_tree.lock_read()
811
 
        operation.add_cleanup(self.base_tree.unlock)
812
602
        self.other_tree.lock_read()
813
 
        operation.add_cleanup(self.other_tree.unlock)
814
 
        operation.run()
815
 
 
816
 
    def _do_merge(self, operation):
817
 
        self.tt = transform.TreeTransform(self.this_tree, None)
818
 
        operation.add_cleanup(self.tt.finalize)
819
 
        self._compute_transform()
820
 
        results = self.tt.apply(no_conflicts=True)
821
 
        self.write_modified(results)
 
603
        self.tt = TreeTransform(self.this_tree, self.pb)
822
604
        try:
823
 
            self.this_tree.add_conflicts(self.cooked_conflicts)
824
 
        except errors.UnsupportedOperation:
825
 
            pass
 
605
            self.pp.next_phase()
 
606
            self._compute_transform()
 
607
            self.pp.next_phase()
 
608
            results = self.tt.apply(no_conflicts=True)
 
609
            self.write_modified(results)
 
610
            try:
 
611
                self.this_tree.add_conflicts(self.cooked_conflicts)
 
612
            except UnsupportedOperation:
 
613
                pass
 
614
        finally:
 
615
            self.tt.finalize()
 
616
            self.other_tree.unlock()
 
617
            self.base_tree.unlock()
 
618
            self.this_tree.unlock()
 
619
            self.pb.clear()
826
620
 
827
621
    def make_preview_transform(self):
828
 
        operation = OperationWithCleanups(self._make_preview_transform)
829
622
        self.base_tree.lock_read()
830
 
        operation.add_cleanup(self.base_tree.unlock)
831
623
        self.other_tree.lock_read()
832
 
        operation.add_cleanup(self.other_tree.unlock)
833
 
        return operation.run_simple()
834
 
 
835
 
    def _make_preview_transform(self):
836
 
        self.tt = transform.TransformPreview(self.this_tree)
837
 
        self._compute_transform()
 
624
        self.tt = TransformPreview(self.this_tree)
 
625
        try:
 
626
            self.pp.next_phase()
 
627
            self._compute_transform()
 
628
            self.pp.next_phase()
 
629
        finally:
 
630
            self.other_tree.unlock()
 
631
            self.base_tree.unlock()
 
632
            self.pb.clear()
838
633
        return self.tt
839
634
 
840
635
    def _compute_transform(self):
846
641
            resolver = self._lca_multi_way
847
642
        child_pb = ui.ui_factory.nested_progress_bar()
848
643
        try:
849
 
            factories = Merger.hooks['merge_file_content']
850
 
            hooks = [factory(self) for factory in factories] + [self]
851
 
            self.active_hooks = [hook for hook in hooks if hook is not None]
852
644
            for num, (file_id, changed, parents3, names3,
853
645
                      executable3) in enumerate(entries):
854
646
                child_pb.update('Preparing file merge', num, len(entries))
855
647
                self._merge_names(file_id, parents3, names3, resolver=resolver)
856
648
                if changed:
857
 
                    file_status = self._do_merge_contents(file_id)
 
649
                    file_status = self.merge_contents(file_id)
858
650
                else:
859
651
                    file_status = 'unmodified'
860
652
                self._merge_executable(file_id,
862
654
        finally:
863
655
            child_pb.finished()
864
656
        self.fix_root()
865
 
        self._finish_computing_transform()
866
 
 
867
 
    def _finish_computing_transform(self):
868
 
        """Finalize the transform and report the changes.
869
 
 
870
 
        This is the second half of _compute_transform.
871
 
        """
 
657
        self.pp.next_phase()
872
658
        child_pb = ui.ui_factory.nested_progress_bar()
873
659
        try:
874
 
            fs_conflicts = transform.resolve_conflicts(self.tt, child_pb,
875
 
                lambda t, c: transform.conflict_pass(t, c, self.other_tree))
 
660
            fs_conflicts = resolve_conflicts(self.tt, child_pb,
 
661
                lambda t, c: conflict_pass(t, c, self.other_tree))
876
662
        finally:
877
663
            child_pb.finished()
878
664
        if self.change_reporter is not None:
881
667
                self.tt.iter_changes(), self.change_reporter)
882
668
        self.cook_conflicts(fs_conflicts)
883
669
        for conflict in self.cooked_conflicts:
884
 
            trace.warning(conflict)
 
670
            warning(conflict)
885
671
 
886
672
    def _entries3(self):
887
673
        """Gather data about files modified between three trees.
1072
858
                        continue
1073
859
                else:
1074
860
                    raise AssertionError('unhandled kind: %s' % other_ie.kind)
 
861
                # XXX: We need to handle kind == 'symlink'
1075
862
 
1076
863
            # If we have gotten this far, that means something has changed
1077
864
            result.append((file_id, content_changed,
1084
871
                          ))
1085
872
        return result
1086
873
 
 
874
 
1087
875
    def fix_root(self):
1088
 
        if self.tt.final_kind(self.tt.root) is None:
 
876
        try:
 
877
            self.tt.final_kind(self.tt.root)
 
878
        except NoSuchFile:
1089
879
            self.tt.cancel_deletion(self.tt.root)
1090
880
        if self.tt.final_file_id(self.tt.root) is None:
1091
881
            self.tt.version_file(self.tt.tree_file_id(self.tt.root),
1096
886
        other_root = self.tt.trans_id_file_id(other_root_file_id)
1097
887
        if other_root == self.tt.root:
1098
888
            return
 
889
        try:
 
890
            self.tt.final_kind(other_root)
 
891
        except NoSuchFile:
 
892
            return
1099
893
        if self.other_tree.inventory.root.file_id in self.this_tree.inventory:
1100
 
            # the other tree's root is a non-root in the current tree (as when
1101
 
            # a previously unrelated branch is merged into another)
 
894
            # the other tree's root is a non-root in the current tree
1102
895
            return
1103
 
        if self.tt.final_kind(other_root) is not None:
1104
 
            other_root_is_present = True
1105
 
        else:
1106
 
            # other_root doesn't have a physical representation. We still need
1107
 
            # to move any references to the actual root of the tree.
1108
 
            other_root_is_present = False
1109
 
        # 'other_tree.inventory.root' is not present in this tree. We are
1110
 
        # calling adjust_path for children which *want* to be present with a
1111
 
        # correct place to go.
1112
 
        for thing, child in self.other_tree.inventory.root.children.iteritems():
 
896
        self.reparent_children(self.other_tree.inventory.root, self.tt.root)
 
897
        self.tt.cancel_creation(other_root)
 
898
        self.tt.cancel_versioning(other_root)
 
899
 
 
900
    def reparent_children(self, ie, target):
 
901
        for thing, child in ie.children.iteritems():
1113
902
            trans_id = self.tt.trans_id_file_id(child.file_id)
1114
 
            if not other_root_is_present:
1115
 
                if self.tt.final_kind(trans_id) is not None:
1116
 
                    # The item exist in the final tree and has a defined place
1117
 
                    # to go already.
1118
 
                    continue
1119
 
            # Move the item into the root
1120
 
            self.tt.adjust_path(self.tt.final_name(trans_id),
1121
 
                                self.tt.root, trans_id)
1122
 
        if other_root_is_present:
1123
 
            self.tt.cancel_creation(other_root)
1124
 
            self.tt.cancel_versioning(other_root)
 
903
            self.tt.adjust_path(self.tt.final_name(trans_id), target, trans_id)
1125
904
 
1126
905
    def write_modified(self, results):
1127
906
        modified_hashes = {}
1159
938
    @staticmethod
1160
939
    def executable(tree, file_id):
1161
940
        """Determine the executability of a file-id (used as a key method)."""
1162
 
        if not tree.has_id(file_id):
 
941
        if file_id not in tree:
1163
942
            return None
1164
943
        if tree.kind(file_id) != "file":
1165
944
            return False
1168
947
    @staticmethod
1169
948
    def kind(tree, file_id):
1170
949
        """Determine the kind of a file-id (used as a key method)."""
1171
 
        if not tree.has_id(file_id):
 
950
        if file_id not in tree:
1172
951
            return None
1173
952
        return tree.kind(file_id)
1174
953
 
1175
954
    @staticmethod
1176
955
    def _three_way(base, other, this):
 
956
        #if base == other, either they all agree, or only THIS has changed.
1177
957
        if base == other:
1178
 
            # if 'base == other', either they all agree, or only 'this' has
1179
 
            # changed.
1180
958
            return 'this'
1181
959
        elif this not in (base, other):
1182
 
            # 'this' is neither 'base' nor 'other', so both sides changed
1183
960
            return 'conflict'
 
961
        # "Ambiguous clean merge" -- both sides have made the same change.
1184
962
        elif this == other:
1185
 
            # "Ambiguous clean merge" -- both sides have made the same change.
1186
963
            return "this"
 
964
        # this == base: only other has changed.
1187
965
        else:
1188
 
            # this == base: only other has changed.
1189
966
            return "other"
1190
967
 
1191
968
    @staticmethod
1235
1012
                # only has an lca value
1236
1013
                return 'other'
1237
1014
 
1238
 
        # At this point, the lcas disagree, and the tip disagree
 
1015
        # At this point, the lcas disagree, and the tips disagree
1239
1016
        return 'conflict'
1240
1017
 
1241
1018
    @staticmethod
1242
 
    @deprecated_method(deprecated_in((2, 2, 0)))
1243
1019
    def scalar_three_way(this_tree, base_tree, other_tree, file_id, key):
1244
1020
        """Do a three-way test on a scalar.
1245
1021
        Return "this", "other" or "conflict", depending whether a value wins.
1260
1036
 
1261
1037
    def merge_names(self, file_id):
1262
1038
        def get_entry(tree):
1263
 
            if tree.has_id(file_id):
 
1039
            if file_id in tree.inventory:
1264
1040
                return tree.inventory[file_id]
1265
1041
            else:
1266
1042
                return None
1295
1071
                parent_id_winner = "other"
1296
1072
        if name_winner == "this" and parent_id_winner == "this":
1297
1073
            return
1298
 
        if name_winner == 'conflict' or parent_id_winner == 'conflict':
1299
 
            # Creating helpers (.OTHER or .THIS) here cause problems down the
1300
 
            # road if a ContentConflict needs to be created so we should not do
1301
 
            # that
1302
 
            trans_id = self.tt.trans_id_file_id(file_id)
1303
 
            self._raw_conflicts.append(('path conflict', trans_id, file_id,
1304
 
                                        this_parent, this_name,
1305
 
                                        other_parent, other_name))
 
1074
        if name_winner == "conflict":
 
1075
            trans_id = self.tt.trans_id_file_id(file_id)
 
1076
            self._raw_conflicts.append(('name conflict', trans_id,
 
1077
                                        this_name, other_name))
 
1078
        if parent_id_winner == "conflict":
 
1079
            trans_id = self.tt.trans_id_file_id(file_id)
 
1080
            self._raw_conflicts.append(('parent conflict', trans_id,
 
1081
                                        this_parent, other_parent))
1306
1082
        if other_name is None:
1307
1083
            # it doesn't matter whether the result was 'other' or
1308
1084
            # 'conflict'-- if there's no 'other', we leave it alone.
1309
1085
            return
 
1086
        # if we get here, name_winner and parent_winner are set to safe values.
 
1087
        trans_id = self.tt.trans_id_file_id(file_id)
1310
1088
        parent_id = parents[self.winner_idx[parent_id_winner]]
1311
1089
        if parent_id is not None:
1312
 
            # if we get here, name_winner and parent_winner are set to safe
1313
 
            # values.
 
1090
            parent_trans_id = self.tt.trans_id_file_id(parent_id)
1314
1091
            self.tt.adjust_path(names[self.winner_idx[name_winner]],
1315
 
                                self.tt.trans_id_file_id(parent_id),
1316
 
                                self.tt.trans_id_file_id(file_id))
 
1092
                                parent_trans_id, trans_id)
1317
1093
 
1318
 
    def _do_merge_contents(self, file_id):
 
1094
    def merge_contents(self, file_id):
1319
1095
        """Performs a merge on file_id contents."""
1320
1096
        def contents_pair(tree):
1321
1097
            if file_id not in tree:
1329
1105
                contents = None
1330
1106
            return kind, contents
1331
1107
 
 
1108
        def contents_conflict():
 
1109
            trans_id = self.tt.trans_id_file_id(file_id)
 
1110
            name = self.tt.final_name(trans_id)
 
1111
            parent_id = self.tt.final_parent(trans_id)
 
1112
            if file_id in self.this_tree.inventory:
 
1113
                self.tt.unversion_file(trans_id)
 
1114
                if file_id in self.this_tree:
 
1115
                    self.tt.delete_contents(trans_id)
 
1116
            file_group = self._dump_conflicts(name, parent_id, file_id,
 
1117
                                              set_version=True)
 
1118
            self._raw_conflicts.append(('contents conflict', file_group))
 
1119
 
1332
1120
        # See SPOT run.  run, SPOT, run.
1333
1121
        # So we're not QUITE repeating ourselves; we do tricky things with
1334
1122
        # file kind...
1350
1138
        if winner == 'this':
1351
1139
            # No interesting changes introduced by OTHER
1352
1140
            return "unmodified"
1353
 
        # We have a hypothetical conflict, but if we have files, then we
1354
 
        # can try to merge the content
1355
1141
        trans_id = self.tt.trans_id_file_id(file_id)
1356
 
        params = MergeHookParams(self, file_id, trans_id, this_pair[0],
1357
 
            other_pair[0], winner)
1358
 
        hooks = self.active_hooks
1359
 
        hook_status = 'not_applicable'
1360
 
        for hook in hooks:
1361
 
            hook_status, lines = hook.merge_contents(params)
1362
 
            if hook_status != 'not_applicable':
1363
 
                # Don't try any more hooks, this one applies.
1364
 
                break
1365
 
        result = "modified"
1366
 
        if hook_status == 'not_applicable':
1367
 
            # This is a contents conflict, because none of the available
1368
 
            # functions could merge it.
1369
 
            result = None
1370
 
            name = self.tt.final_name(trans_id)
1371
 
            parent_id = self.tt.final_parent(trans_id)
1372
 
            if self.this_tree.has_id(file_id):
1373
 
                self.tt.unversion_file(trans_id)
1374
 
            file_group = self._dump_conflicts(name, parent_id, file_id,
1375
 
                                              set_version=True)
1376
 
            self._raw_conflicts.append(('contents conflict', file_group))
1377
 
        elif hook_status == 'success':
1378
 
            self.tt.create_file(lines, trans_id)
1379
 
        elif hook_status == 'conflicted':
1380
 
            # XXX: perhaps the hook should be able to provide
1381
 
            # the BASE/THIS/OTHER files?
1382
 
            self.tt.create_file(lines, trans_id)
1383
 
            self._raw_conflicts.append(('text conflict', trans_id))
1384
 
            name = self.tt.final_name(trans_id)
1385
 
            parent_id = self.tt.final_parent(trans_id)
1386
 
            self._dump_conflicts(name, parent_id, file_id)
1387
 
        elif hook_status == 'delete':
1388
 
            self.tt.unversion_file(trans_id)
1389
 
            result = "deleted"
1390
 
        elif hook_status == 'done':
1391
 
            # The hook function did whatever it needs to do directly, no
1392
 
            # further action needed here.
1393
 
            pass
1394
 
        else:
1395
 
            raise AssertionError('unknown hook_status: %r' % (hook_status,))
1396
 
        if not self.this_tree.has_id(file_id) and result == "modified":
1397
 
            self.tt.version_file(file_id, trans_id)
1398
 
        # The merge has been performed, so the old contents should not be
1399
 
        # retained.
1400
 
        self.tt.delete_contents(trans_id)
1401
 
        return result
1402
 
 
1403
 
    def _default_other_winner_merge(self, merge_hook_params):
1404
 
        """Replace this contents with other."""
1405
 
        file_id = merge_hook_params.file_id
1406
 
        trans_id = merge_hook_params.trans_id
1407
 
        file_in_this = self.this_tree.has_id(file_id)
1408
 
        if self.other_tree.has_id(file_id):
1409
 
            # OTHER changed the file
1410
 
            wt = self.this_tree
1411
 
            if wt.supports_content_filtering():
1412
 
                # We get the path from the working tree if it exists.
1413
 
                # That fails though when OTHER is adding a file, so
1414
 
                # we fall back to the other tree to find the path if
1415
 
                # it doesn't exist locally.
1416
 
                try:
1417
 
                    filter_tree_path = wt.id2path(file_id)
1418
 
                except errors.NoSuchId:
1419
 
                    filter_tree_path = self.other_tree.id2path(file_id)
1420
 
            else:
1421
 
                # Skip the id2path lookup for older formats
1422
 
                filter_tree_path = None
1423
 
            transform.create_from_tree(self.tt, trans_id,
1424
 
                             self.other_tree, file_id,
1425
 
                             filter_tree_path=filter_tree_path)
1426
 
            return 'done', None
1427
 
        elif file_in_this:
1428
 
            # OTHER deleted the file
1429
 
            return 'delete', None
1430
 
        else:
1431
 
            raise AssertionError(
1432
 
                'winner is OTHER, but file_id %r not in THIS or OTHER tree'
1433
 
                % (file_id,))
1434
 
 
1435
 
    def merge_contents(self, merge_hook_params):
1436
 
        """Fallback merge logic after user installed hooks."""
1437
 
        # This function is used in merge hooks as the fallback instance.
1438
 
        # Perhaps making this function and the functions it calls be a 
1439
 
        # a separate class would be better.
1440
 
        if merge_hook_params.winner == 'other':
 
1142
        if winner == 'other':
1441
1143
            # OTHER is a straight winner, so replace this contents with other
1442
 
            return self._default_other_winner_merge(merge_hook_params)
1443
 
        elif merge_hook_params.is_file_merge():
1444
 
            # THIS and OTHER are both files, so text merge.  Either
1445
 
            # BASE is a file, or both converted to files, so at least we
1446
 
            # have agreement that output should be a file.
1447
 
            try:
1448
 
                self.text_merge(merge_hook_params.file_id,
1449
 
                    merge_hook_params.trans_id)
1450
 
            except errors.BinaryFile:
1451
 
                return 'not_applicable', None
1452
 
            return 'done', None
 
1144
            file_in_this = file_id in self.this_tree
 
1145
            if file_in_this:
 
1146
                # Remove any existing contents
 
1147
                self.tt.delete_contents(trans_id)
 
1148
            if file_id in self.other_tree:
 
1149
                # OTHER changed the file
 
1150
                create_from_tree(self.tt, trans_id,
 
1151
                                 self.other_tree, file_id)
 
1152
                if not file_in_this:
 
1153
                    self.tt.version_file(file_id, trans_id)
 
1154
                return "modified"
 
1155
            elif file_in_this:
 
1156
                # OTHER deleted the file
 
1157
                self.tt.unversion_file(trans_id)
 
1158
                return "deleted"
1453
1159
        else:
1454
 
            return 'not_applicable', None
 
1160
            # We have a hypothetical conflict, but if we have files, then we
 
1161
            # can try to merge the content
 
1162
            if this_pair[0] == 'file' and other_pair[0] == 'file':
 
1163
                # THIS and OTHER are both files, so text merge.  Either
 
1164
                # BASE is a file, or both converted to files, so at least we
 
1165
                # have agreement that output should be a file.
 
1166
                try:
 
1167
                    self.text_merge(file_id, trans_id)
 
1168
                except BinaryFile:
 
1169
                    return contents_conflict()
 
1170
                if file_id not in self.this_tree:
 
1171
                    self.tt.version_file(file_id, trans_id)
 
1172
                try:
 
1173
                    self.tt.tree_kind(trans_id)
 
1174
                    self.tt.delete_contents(trans_id)
 
1175
                except NoSuchFile:
 
1176
                    pass
 
1177
                return "modified"
 
1178
            else:
 
1179
                return contents_conflict()
1455
1180
 
1456
1181
    def get_lines(self, tree, file_id):
1457
1182
        """Return the lines in a file, or an empty list."""
1458
 
        if tree.has_id(file_id):
1459
 
            return tree.get_file_lines(file_id)
 
1183
        if file_id in tree:
 
1184
            return tree.get_file(file_id).readlines()
1460
1185
        else:
1461
1186
            return []
1462
1187
 
1464
1189
        """Perform a three-way text merge on a file_id"""
1465
1190
        # it's possible that we got here with base as a different type.
1466
1191
        # if so, we just want two-way text conflicts.
1467
 
        if self.base_tree.has_id(file_id) and \
 
1192
        if file_id in self.base_tree and \
1468
1193
            self.base_tree.kind(file_id) == "file":
1469
1194
            base_lines = self.get_lines(self.base_tree, file_id)
1470
1195
        else:
1471
1196
            base_lines = []
1472
1197
        other_lines = self.get_lines(self.other_tree, file_id)
1473
1198
        this_lines = self.get_lines(self.this_tree, file_id)
1474
 
        m3 = merge3.Merge3(base_lines, this_lines, other_lines,
1475
 
                           is_cherrypick=self.cherrypick)
 
1199
        m3 = Merge3(base_lines, this_lines, other_lines,
 
1200
                    is_cherrypick=self.cherrypick)
1476
1201
        start_marker = "!START OF MERGE CONFLICT!" + "I HOPE THIS IS UNIQUE"
1477
1202
        if self.show_base is True:
1478
1203
            base_marker = '|' * 7
1516
1241
                ('THIS', self.this_tree, this_lines)]
1517
1242
        if not no_base:
1518
1243
            data.append(('BASE', self.base_tree, base_lines))
1519
 
 
1520
 
        # We need to use the actual path in the working tree of the file here,
1521
 
        # ignoring the conflict suffixes
1522
 
        wt = self.this_tree
1523
 
        if wt.supports_content_filtering():
1524
 
            try:
1525
 
                filter_tree_path = wt.id2path(file_id)
1526
 
            except errors.NoSuchId:
1527
 
                # file has been deleted
1528
 
                filter_tree_path = None
1529
 
        else:
1530
 
            # Skip the id2path lookup for older formats
1531
 
            filter_tree_path = None
1532
 
 
1533
1244
        versioned = False
1534
1245
        file_group = []
1535
1246
        for suffix, tree, lines in data:
1536
 
            if tree.has_id(file_id):
 
1247
            if file_id in tree:
1537
1248
                trans_id = self._conflict_file(name, parent_id, tree, file_id,
1538
 
                                               suffix, lines, filter_tree_path)
 
1249
                                               suffix, lines)
1539
1250
                file_group.append(trans_id)
1540
1251
                if set_version and not versioned:
1541
1252
                    self.tt.version_file(file_id, trans_id)
1543
1254
        return file_group
1544
1255
 
1545
1256
    def _conflict_file(self, name, parent_id, tree, file_id, suffix,
1546
 
                       lines=None, filter_tree_path=None):
 
1257
                       lines=None):
1547
1258
        """Emit a single conflict file."""
1548
1259
        name = name + '.' + suffix
1549
1260
        trans_id = self.tt.create_path(name, parent_id)
1550
 
        transform.create_from_tree(self.tt, trans_id, tree, file_id, lines,
1551
 
            filter_tree_path)
 
1261
        create_from_tree(self.tt, trans_id, tree, file_id, lines)
1552
1262
        return trans_id
1553
1263
 
1554
1264
    def merge_executable(self, file_id, file_status):
1575
1285
        if winner == 'this' and file_status != "modified":
1576
1286
            return
1577
1287
        trans_id = self.tt.trans_id_file_id(file_id)
1578
 
        if self.tt.final_kind(trans_id) != "file":
 
1288
        try:
 
1289
            if self.tt.final_kind(trans_id) != "file":
 
1290
                return
 
1291
        except NoSuchFile:
1579
1292
            return
1580
1293
        if winner == "this":
1581
1294
            executability = this_executable
1582
1295
        else:
1583
 
            if self.other_tree.has_id(file_id):
 
1296
            if file_id in self.other_tree:
1584
1297
                executability = other_executable
1585
 
            elif self.this_tree.has_id(file_id):
 
1298
            elif file_id in self.this_tree:
1586
1299
                executability = this_executable
1587
 
            elif self.base_tree_has_id(file_id):
 
1300
            elif file_id in self.base_tree:
1588
1301
                executability = base_executable
1589
1302
        if executability is not None:
1590
1303
            trans_id = self.tt.trans_id_file_id(file_id)
1592
1305
 
1593
1306
    def cook_conflicts(self, fs_conflicts):
1594
1307
        """Convert all conflicts into a form that doesn't depend on trans_id"""
1595
 
        self.cooked_conflicts.extend(transform.cook_conflicts(
1596
 
                fs_conflicts, self.tt))
1597
 
        fp = transform.FinalPaths(self.tt)
 
1308
        from conflicts import Conflict
 
1309
        name_conflicts = {}
 
1310
        self.cooked_conflicts.extend(cook_conflicts(fs_conflicts, self.tt))
 
1311
        fp = FinalPaths(self.tt)
1598
1312
        for conflict in self._raw_conflicts:
1599
1313
            conflict_type = conflict[0]
1600
 
            if conflict_type == 'path conflict':
1601
 
                (trans_id, file_id,
1602
 
                this_parent, this_name,
1603
 
                other_parent, other_name) = conflict[1:]
1604
 
                if this_parent is None or this_name is None:
1605
 
                    this_path = '<deleted>'
1606
 
                else:
1607
 
                    parent_path =  fp.get_path(
1608
 
                        self.tt.trans_id_file_id(this_parent))
1609
 
                    this_path = osutils.pathjoin(parent_path, this_name)
1610
 
                if other_parent is None or other_name is None:
1611
 
                    other_path = '<deleted>'
1612
 
                else:
1613
 
                    parent_path =  fp.get_path(
1614
 
                        self.tt.trans_id_file_id(other_parent))
1615
 
                    other_path = osutils.pathjoin(parent_path, other_name)
1616
 
                c = _mod_conflicts.Conflict.factory(
1617
 
                    'path conflict', path=this_path,
1618
 
                    conflict_path=other_path,
1619
 
                    file_id=file_id)
1620
 
            elif conflict_type == 'contents conflict':
 
1314
            if conflict_type in ('name conflict', 'parent conflict'):
 
1315
                trans_id = conflict[1]
 
1316
                conflict_args = conflict[2:]
 
1317
                if trans_id not in name_conflicts:
 
1318
                    name_conflicts[trans_id] = {}
 
1319
                unique_add(name_conflicts[trans_id], conflict_type,
 
1320
                           conflict_args)
 
1321
            if conflict_type == 'contents conflict':
1621
1322
                for trans_id in conflict[1]:
1622
1323
                    file_id = self.tt.final_file_id(trans_id)
1623
1324
                    if file_id is not None:
1627
1328
                    if path.endswith(suffix):
1628
1329
                        path = path[:-len(suffix)]
1629
1330
                        break
1630
 
                c = _mod_conflicts.Conflict.factory(conflict_type,
1631
 
                                                    path=path, file_id=file_id)
1632
 
            elif conflict_type == 'text conflict':
 
1331
                c = Conflict.factory(conflict_type, path=path, file_id=file_id)
 
1332
                self.cooked_conflicts.append(c)
 
1333
            if conflict_type == 'text conflict':
1633
1334
                trans_id = conflict[1]
1634
1335
                path = fp.get_path(trans_id)
1635
1336
                file_id = self.tt.final_file_id(trans_id)
1636
 
                c = _mod_conflicts.Conflict.factory(conflict_type,
1637
 
                                                    path=path, file_id=file_id)
 
1337
                c = Conflict.factory(conflict_type, path=path, file_id=file_id)
 
1338
                self.cooked_conflicts.append(c)
 
1339
 
 
1340
        for trans_id, conflicts in name_conflicts.iteritems():
 
1341
            try:
 
1342
                this_parent, other_parent = conflicts['parent conflict']
 
1343
                if this_parent == other_parent:
 
1344
                    raise AssertionError()
 
1345
            except KeyError:
 
1346
                this_parent = other_parent = \
 
1347
                    self.tt.final_file_id(self.tt.final_parent(trans_id))
 
1348
            try:
 
1349
                this_name, other_name = conflicts['name conflict']
 
1350
                if this_name == other_name:
 
1351
                    raise AssertionError()
 
1352
            except KeyError:
 
1353
                this_name = other_name = self.tt.final_name(trans_id)
 
1354
            other_path = fp.get_path(trans_id)
 
1355
            if this_parent is not None and this_name is not None:
 
1356
                this_parent_path = \
 
1357
                    fp.get_path(self.tt.trans_id_file_id(this_parent))
 
1358
                this_path = pathjoin(this_parent_path, this_name)
1638
1359
            else:
1639
 
                raise AssertionError('bad conflict type: %r' % (conflict,))
 
1360
                this_path = "<deleted>"
 
1361
            file_id = self.tt.final_file_id(trans_id)
 
1362
            c = Conflict.factory('path conflict', path=this_path,
 
1363
                                 conflict_path=other_path, file_id=file_id)
1640
1364
            self.cooked_conflicts.append(c)
1641
 
        self.cooked_conflicts.sort(key=_mod_conflicts.Conflict.sort_key)
 
1365
        self.cooked_conflicts.sort(key=Conflict.sort_key)
1642
1366
 
1643
1367
 
1644
1368
class WeaveMerger(Merge3Merger):
1648
1372
    supports_reverse_cherrypick = False
1649
1373
    history_based = True
1650
1374
 
1651
 
    def _generate_merge_plan(self, file_id, base):
1652
 
        return self.this_tree.plan_file_merge(file_id, self.other_tree,
 
1375
    def _merged_lines(self, file_id):
 
1376
        """Generate the merged lines.
 
1377
        There is no distinction between lines that are meant to contain <<<<<<<
 
1378
        and conflicts.
 
1379
        """
 
1380
        if self.cherrypick:
 
1381
            base = self.base_tree
 
1382
        else:
 
1383
            base = None
 
1384
        plan = self.this_tree.plan_file_merge(file_id, self.other_tree,
1653
1385
                                              base=base)
1654
 
 
1655
 
    def _merged_lines(self, file_id):
1656
 
        """Generate the merged lines.
1657
 
        There is no distinction between lines that are meant to contain <<<<<<<
1658
 
        and conflicts.
1659
 
        """
1660
 
        if self.cherrypick:
1661
 
            base = self.base_tree
1662
 
        else:
1663
 
            base = None
1664
 
        plan = self._generate_merge_plan(file_id, base)
1665
1386
        if 'merge' in debug.debug_flags:
1666
1387
            plan = list(plan)
1667
1388
            trans_id = self.tt.trans_id_file_id(file_id)
1668
1389
            name = self.tt.final_name(trans_id) + '.plan'
1669
 
            contents = ('%11s|%s' % l for l in plan)
 
1390
            contents = ('%10s|%s' % l for l in plan)
1670
1391
            self.tt.new_file(name, self.tt.final_parent(trans_id), contents)
1671
 
        textmerge = versionedfile.PlanWeaveMerge(plan, '<<<<<<< TREE\n',
1672
 
                                                 '>>>>>>> MERGE-SOURCE\n')
1673
 
        lines, conflicts = textmerge.merge_lines(self.reprocess)
1674
 
        if conflicts:
1675
 
            base_lines = textmerge.base_from_plan()
1676
 
        else:
1677
 
            base_lines = None
1678
 
        return lines, base_lines
 
1392
        textmerge = PlanWeaveMerge(plan, '<<<<<<< TREE\n',
 
1393
            '>>>>>>> MERGE-SOURCE\n')
 
1394
        return textmerge.merge_lines(self.reprocess)
1679
1395
 
1680
1396
    def text_merge(self, file_id, trans_id):
1681
1397
        """Perform a (weave) text merge for a given file and file-id.
1682
1398
        If conflicts are encountered, .THIS and .OTHER files will be emitted,
1683
1399
        and a conflict will be noted.
1684
1400
        """
1685
 
        lines, base_lines = self._merged_lines(file_id)
 
1401
        lines, conflicts = self._merged_lines(file_id)
1686
1402
        lines = list(lines)
1687
1403
        # Note we're checking whether the OUTPUT is binary in this case,
1688
1404
        # because we don't want to get into weave merge guts.
1689
 
        textfile.check_text_lines(lines)
 
1405
        check_text_lines(lines)
1690
1406
        self.tt.create_file(lines, trans_id)
1691
 
        if base_lines is not None:
1692
 
            # Conflict
 
1407
        if conflicts:
1693
1408
            self._raw_conflicts.append(('text conflict', trans_id))
1694
1409
            name = self.tt.final_name(trans_id)
1695
1410
            parent_id = self.tt.final_parent(trans_id)
1696
1411
            file_group = self._dump_conflicts(name, parent_id, file_id,
1697
 
                                              no_base=False,
1698
 
                                              base_lines=base_lines)
 
1412
                                              no_base=True)
1699
1413
            file_group.append(trans_id)
1700
1414
 
1701
1415
 
1702
1416
class LCAMerger(WeaveMerger):
1703
1417
 
1704
 
    def _generate_merge_plan(self, file_id, base):
1705
 
        return self.this_tree.plan_file_lca_merge(file_id, self.other_tree,
 
1418
    def _merged_lines(self, file_id):
 
1419
        """Generate the merged lines.
 
1420
        There is no distinction between lines that are meant to contain <<<<<<<
 
1421
        and conflicts.
 
1422
        """
 
1423
        if self.cherrypick:
 
1424
            base = self.base_tree
 
1425
        else:
 
1426
            base = None
 
1427
        plan = self.this_tree.plan_file_lca_merge(file_id, self.other_tree,
1706
1428
                                                  base=base)
 
1429
        if 'merge' in debug.debug_flags:
 
1430
            plan = list(plan)
 
1431
            trans_id = self.tt.trans_id_file_id(file_id)
 
1432
            name = self.tt.final_name(trans_id) + '.plan'
 
1433
            contents = ('%10s|%s' % l for l in plan)
 
1434
            self.tt.new_file(name, self.tt.final_parent(trans_id), contents)
 
1435
        textmerge = PlanWeaveMerge(plan, '<<<<<<< TREE\n',
 
1436
            '>>>>>>> MERGE-SOURCE\n')
 
1437
        return textmerge.merge_lines(self.reprocess)
 
1438
 
1707
1439
 
1708
1440
class Diff3Merger(Merge3Merger):
1709
1441
    """Three-way merger using external diff3 for text merging"""
1710
1442
 
1711
1443
    def dump_file(self, temp_dir, name, tree, file_id):
1712
 
        out_path = osutils.pathjoin(temp_dir, name)
 
1444
        out_path = pathjoin(temp_dir, name)
1713
1445
        out_file = open(out_path, "wb")
1714
1446
        try:
1715
1447
            in_file = tree.get_file(file_id)
1727
1459
        import bzrlib.patch
1728
1460
        temp_dir = osutils.mkdtemp(prefix="bzr-")
1729
1461
        try:
1730
 
            new_file = osutils.pathjoin(temp_dir, "new")
 
1462
            new_file = pathjoin(temp_dir, "new")
1731
1463
            this = self.dump_file(temp_dir, "this", self.this_tree, file_id)
1732
1464
            base = self.dump_file(temp_dir, "base", self.base_tree, file_id)
1733
1465
            other = self.dump_file(temp_dir, "other", self.other_tree, file_id)
1734
1466
            status = bzrlib.patch.diff3(new_file, this, base, other)
1735
1467
            if status not in (0, 1):
1736
 
                raise errors.BzrError("Unhandled diff3 exit code")
 
1468
                raise BzrError("Unhandled diff3 exit code")
1737
1469
            f = open(new_file, 'rb')
1738
1470
            try:
1739
1471
                self.tt.create_file(f, trans_id)
1748
1480
            osutils.rmtree(temp_dir)
1749
1481
 
1750
1482
 
1751
 
class PathNotInTree(errors.BzrError):
1752
 
 
1753
 
    _fmt = """Merge-into failed because %(tree)s does not contain %(path)s."""
1754
 
 
1755
 
    def __init__(self, path, tree):
1756
 
        errors.BzrError.__init__(self, path=path, tree=tree)
1757
 
 
1758
 
 
1759
 
class MergeIntoMerger(Merger):
1760
 
    """Merger that understands other_tree will be merged into a subdir.
1761
 
 
1762
 
    This also changes the Merger api so that it uses real Branch, revision_id,
1763
 
    and RevisonTree objects, rather than using revision specs.
1764
 
    """
1765
 
 
1766
 
    def __init__(self, this_tree, other_branch, other_tree, target_subdir,
1767
 
            source_subpath, other_rev_id=None):
1768
 
        """Create a new MergeIntoMerger object.
1769
 
 
1770
 
        source_subpath in other_tree will be effectively copied to
1771
 
        target_subdir in this_tree.
1772
 
 
1773
 
        :param this_tree: The tree that we will be merging into.
1774
 
        :param other_branch: The Branch we will be merging from.
1775
 
        :param other_tree: The RevisionTree object we want to merge.
1776
 
        :param target_subdir: The relative path where we want to merge
1777
 
            other_tree into this_tree
1778
 
        :param source_subpath: The relative path specifying the subtree of
1779
 
            other_tree to merge into this_tree.
1780
 
        """
1781
 
        # It is assumed that we are merging a tree that is not in our current
1782
 
        # ancestry, which means we are using the "EmptyTree" as our basis.
1783
 
        null_ancestor_tree = this_tree.branch.repository.revision_tree(
1784
 
                                _mod_revision.NULL_REVISION)
1785
 
        super(MergeIntoMerger, self).__init__(
1786
 
            this_branch=this_tree.branch,
1787
 
            this_tree=this_tree,
1788
 
            other_tree=other_tree,
1789
 
            base_tree=null_ancestor_tree,
1790
 
            )
1791
 
        self._target_subdir = target_subdir
1792
 
        self._source_subpath = source_subpath
1793
 
        self.other_branch = other_branch
1794
 
        if other_rev_id is None:
1795
 
            other_rev_id = other_tree.get_revision_id()
1796
 
        self.other_rev_id = self.other_basis = other_rev_id
1797
 
        self.base_is_ancestor = True
1798
 
        self.backup_files = True
1799
 
        self.merge_type = Merge3Merger
1800
 
        self.show_base = False
1801
 
        self.reprocess = False
1802
 
        self.interesting_ids = None
1803
 
        self.merge_type = _MergeTypeParameterizer(MergeIntoMergeType,
1804
 
              target_subdir=self._target_subdir,
1805
 
              source_subpath=self._source_subpath)
1806
 
        if self._source_subpath != '':
1807
 
            # If this isn't a partial merge make sure the revisions will be
1808
 
            # present.
1809
 
            self._maybe_fetch(self.other_branch, self.this_branch,
1810
 
                self.other_basis)
1811
 
 
1812
 
    def set_pending(self):
1813
 
        if self._source_subpath != '':
1814
 
            return
1815
 
        Merger.set_pending(self)
1816
 
 
1817
 
 
1818
 
class _MergeTypeParameterizer(object):
1819
 
    """Wrap a merge-type class to provide extra parameters.
1820
 
    
1821
 
    This is hack used by MergeIntoMerger to pass some extra parameters to its
1822
 
    merge_type.  Merger.do_merge() sets up its own set of parameters to pass to
1823
 
    the 'merge_type' member.  It is difficult override do_merge without
1824
 
    re-writing the whole thing, so instead we create a wrapper which will pass
1825
 
    the extra parameters.
1826
 
    """
1827
 
 
1828
 
    def __init__(self, merge_type, **kwargs):
1829
 
        self._extra_kwargs = kwargs
1830
 
        self._merge_type = merge_type
1831
 
 
1832
 
    def __call__(self, *args, **kwargs):
1833
 
        kwargs.update(self._extra_kwargs)
1834
 
        return self._merge_type(*args, **kwargs)
1835
 
 
1836
 
    def __getattr__(self, name):
1837
 
        return getattr(self._merge_type, name)
1838
 
 
1839
 
 
1840
 
class MergeIntoMergeType(Merge3Merger):
1841
 
    """Merger that incorporates a tree (or part of a tree) into another."""
1842
 
 
1843
 
    def __init__(self, *args, **kwargs):
1844
 
        """Initialize the merger object.
1845
 
 
1846
 
        :param args: See Merge3Merger.__init__'s args.
1847
 
        :param kwargs: See Merge3Merger.__init__'s keyword args, except for
1848
 
            source_subpath and target_subdir.
1849
 
        :keyword source_subpath: The relative path specifying the subtree of
1850
 
            other_tree to merge into this_tree.
1851
 
        :keyword target_subdir: The relative path where we want to merge
1852
 
            other_tree into this_tree
1853
 
        """
1854
 
        # All of the interesting work happens during Merge3Merger.__init__(),
1855
 
        # so we have have to hack in to get our extra parameters set.
1856
 
        self._source_subpath = kwargs.pop('source_subpath')
1857
 
        self._target_subdir = kwargs.pop('target_subdir')
1858
 
        super(MergeIntoMergeType, self).__init__(*args, **kwargs)
1859
 
 
1860
 
    def _compute_transform(self):
1861
 
        child_pb = ui.ui_factory.nested_progress_bar()
1862
 
        try:
1863
 
            entries = self._entries_to_incorporate()
1864
 
            entries = list(entries)
1865
 
            for num, (entry, parent_id) in enumerate(entries):
1866
 
                child_pb.update('Preparing file merge', num, len(entries))
1867
 
                parent_trans_id = self.tt.trans_id_file_id(parent_id)
1868
 
                trans_id = transform.new_by_entry(self.tt, entry,
1869
 
                    parent_trans_id, self.other_tree)
1870
 
        finally:
1871
 
            child_pb.finished()
1872
 
        self._finish_computing_transform()
1873
 
 
1874
 
    def _entries_to_incorporate(self):
1875
 
        """Yields pairs of (inventory_entry, new_parent)."""
1876
 
        other_inv = self.other_tree.inventory
1877
 
        subdir_id = other_inv.path2id(self._source_subpath)
1878
 
        if subdir_id is None:
1879
 
            # XXX: The error would be clearer if it gave the URL of the source
1880
 
            # branch, but we don't have a reference to that here.
1881
 
            raise PathNotInTree(self._source_subpath, "Source tree")
1882
 
        subdir = other_inv[subdir_id]
1883
 
        parent_in_target = osutils.dirname(self._target_subdir)
1884
 
        target_id = self.this_tree.inventory.path2id(parent_in_target)
1885
 
        if target_id is None:
1886
 
            raise PathNotInTree(self._target_subdir, "Target tree")
1887
 
        name_in_target = osutils.basename(self._target_subdir)
1888
 
        merge_into_root = subdir.copy()
1889
 
        merge_into_root.name = name_in_target
1890
 
        if merge_into_root.file_id in self.this_tree.inventory:
1891
 
            # Give the root a new file-id.
1892
 
            # This can happen fairly easily if the directory we are
1893
 
            # incorporating is the root, and both trees have 'TREE_ROOT' as
1894
 
            # their root_id.  Users will expect this to Just Work, so we
1895
 
            # change the file-id here.
1896
 
            # Non-root file-ids could potentially conflict too.  That's really
1897
 
            # an edge case, so we don't do anything special for those.  We let
1898
 
            # them cause conflicts.
1899
 
            merge_into_root.file_id = generate_ids.gen_file_id(name_in_target)
1900
 
        yield (merge_into_root, target_id)
1901
 
        if subdir.kind != 'directory':
1902
 
            # No children, so we are done.
1903
 
            return
1904
 
        for ignored_path, entry in other_inv.iter_entries_by_dir(subdir_id):
1905
 
            parent_id = entry.parent_id
1906
 
            if parent_id == subdir.file_id:
1907
 
                # The root's parent ID has changed, so make sure children of
1908
 
                # the root refer to the new ID.
1909
 
                parent_id = merge_into_root.file_id
1910
 
            yield (entry, parent_id)
1911
 
 
1912
 
 
1913
1483
def merge_inner(this_branch, other_tree, base_tree, ignore_zero=False,
1914
1484
                backup_files=False,
1915
1485
                merge_type=Merge3Merger,
1919
1489
                other_rev_id=None,
1920
1490
                interesting_files=None,
1921
1491
                this_tree=None,
1922
 
                pb=None,
 
1492
                pb=DummyProgress(),
1923
1493
                change_reporter=None):
1924
1494
    """Primary interface for merging.
1925
1495
 
1926
 
    Typical use is probably::
1927
 
 
1928
 
        merge_inner(branch, branch.get_revision_tree(other_revision),
1929
 
                    branch.get_revision_tree(base_revision))
1930
 
    """
 
1496
        typical use is probably
 
1497
        'merge_inner(branch, branch.get_revision_tree(other_revision),
 
1498
                     branch.get_revision_tree(base_revision))'
 
1499
        """
1931
1500
    if this_tree is None:
1932
 
        raise errors.BzrError("bzrlib.merge.merge_inner requires a this_tree "
1933
 
                              "parameter as of bzrlib version 0.8.")
 
1501
        raise BzrError("bzrlib.merge.merge_inner requires a this_tree "
 
1502
            "parameter as of bzrlib version 0.8.")
1934
1503
    merger = Merger(this_branch, other_tree, base_tree, this_tree=this_tree,
1935
1504
                    pb=pb, change_reporter=change_reporter)
1936
1505
    merger.backup_files = backup_files
1949
1518
    get_revision_id = getattr(base_tree, 'get_revision_id', None)
1950
1519
    if get_revision_id is None:
1951
1520
        get_revision_id = base_tree.last_revision
1952
 
    merger.cache_trees_with_revision_ids([other_tree, base_tree, this_tree])
1953
1521
    merger.set_base_revision(get_revision_id(), this_branch)
1954
1522
    return merger.do_merge()
1955
1523
 
2154
1722
        super(_PlanMerge, self).__init__(a_rev, b_rev, vf, key_prefix)
2155
1723
        self.a_key = self._key_prefix + (self.a_rev,)
2156
1724
        self.b_key = self._key_prefix + (self.b_rev,)
2157
 
        self.graph = _mod_graph.Graph(self.vf)
 
1725
        self.graph = Graph(self.vf)
2158
1726
        heads = self.graph.heads((self.a_key, self.b_key))
2159
1727
        if len(heads) == 1:
2160
1728
            # one side dominates, so we can just return its values, yay for
2165
1733
                other = b_rev
2166
1734
            else:
2167
1735
                other = a_rev
2168
 
            trace.mutter('found dominating revision for %s\n%s > %s', self.vf,
2169
 
                         self._head_key[-1], other)
 
1736
            mutter('found dominating revision for %s\n%s > %s', self.vf,
 
1737
                   self._head_key[-1], other)
2170
1738
            self._weave = None
2171
1739
        else:
2172
1740
            self._head_key = None
2186
1754
        while True:
2187
1755
            next_lcas = self.graph.find_lca(*cur_ancestors)
2188
1756
            # Map a plain NULL_REVISION to a simple no-ancestors
2189
 
            if next_lcas == set([_mod_revision.NULL_REVISION]):
 
1757
            if next_lcas == set([NULL_REVISION]):
2190
1758
                next_lcas = ()
2191
1759
            # Order the lca's based on when they were merged into the tip
2192
1760
            # While the actual merge portion of weave merge uses a set() of
2204
1772
            elif len(next_lcas) > 2:
2205
1773
                # More than 2 lca's, fall back to grabbing all nodes between
2206
1774
                # this and the unique lca.
2207
 
                trace.mutter('More than 2 LCAs, falling back to all nodes for:'
2208
 
                             ' %s, %s\n=> %s',
2209
 
                             self.a_key, self.b_key, cur_ancestors)
 
1775
                mutter('More than 2 LCAs, falling back to all nodes for:'
 
1776
                       ' %s, %s\n=> %s', self.a_key, self.b_key, cur_ancestors)
2210
1777
                cur_lcas = next_lcas
2211
1778
                while len(cur_lcas) > 1:
2212
1779
                    cur_lcas = self.graph.find_lca(*cur_lcas)
2215
1782
                    unique_lca = None
2216
1783
                else:
2217
1784
                    unique_lca = list(cur_lcas)[0]
2218
 
                    if unique_lca == _mod_revision.NULL_REVISION:
 
1785
                    if unique_lca == NULL_REVISION:
2219
1786
                        # find_lca will return a plain 'NULL_REVISION' rather
2220
1787
                        # than a key tuple when there is no common ancestor, we
2221
1788
                        # prefer to just use None, because it doesn't confuse
2244
1811
            # We remove NULL_REVISION because it isn't a proper tuple key, and
2245
1812
            # thus confuses things like _get_interesting_texts, and our logic
2246
1813
            # to add the texts into the memory weave.
2247
 
            if _mod_revision.NULL_REVISION in parent_map:
2248
 
                parent_map.pop(_mod_revision.NULL_REVISION)
 
1814
            if NULL_REVISION in parent_map:
 
1815
                parent_map.pop(NULL_REVISION)
2249
1816
        else:
2250
1817
            interesting = set()
2251
1818
            for tip in tip_keys:
2403
1970
        lcas = graph.find_lca(key_prefix + (a_rev,), key_prefix + (b_rev,))
2404
1971
        self.lcas = set()
2405
1972
        for lca in lcas:
2406
 
            if lca == _mod_revision.NULL_REVISION:
 
1973
            if lca == NULL_REVISION:
2407
1974
                self.lcas.add(lca)
2408
1975
            else:
2409
1976
                self.lcas.add(lca[-1])