~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/merge.py

  • Committer: Canonical.com Patch Queue Manager
  • Date: 2009-07-22 18:18:34 UTC
  • mfrom: (4537.2.1 1.18-absent-content)
  • Revision ID: pqm@pqm.ubuntu.com-20090722181834-2geyfaa06s9himqg
(jam) Add AbsentContentFactory.get_bytes_as,
        which just raises a better error.

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)
451
244
        if self.other_rev_id is None:
452
245
            other_basis_tree = self.revision_tree(self.other_basis)
453
246
            if other_basis_tree.has_changes(self.other_tree):
454
 
                raise errors.WorkingTreeNotRevision(self.this_tree)
 
247
                raise WorkingTreeNotRevision(self.this_tree)
455
248
            other_rev_id = self.other_basis
456
249
            self.other_tree = other_basis_tree
457
250
 
458
 
    @deprecated_method(deprecated_in((2, 1, 0)))
459
251
    def file_revisions(self, file_id):
460
252
        self.ensure_revision_trees()
461
253
        def get_id(tree, file_id):
464
256
        if self.this_rev_id is None:
465
257
            if self.this_basis_tree.get_file_sha1(file_id) != \
466
258
                self.this_tree.get_file_sha1(file_id):
467
 
                raise errors.WorkingTreeNotRevision(self.this_tree)
 
259
                raise WorkingTreeNotRevision(self.this_tree)
468
260
 
469
261
        trees = (self.this_basis_tree, self.other_tree)
470
262
        return [get_id(tree, file_id) for tree in trees]
471
263
 
472
 
    @deprecated_method(deprecated_in((2, 1, 0)))
473
264
    def check_basis(self, check_clean, require_commits=True):
474
265
        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')")
 
266
            raise BzrCommandError("This branch has no commits."
 
267
                                  " (perhaps you would prefer 'bzr pull')")
478
268
        if check_clean:
479
269
            self.compare_basis()
480
270
            if self.this_basis != self.this_rev_id:
481
271
                raise errors.UncommittedChanges(self.this_tree)
482
272
 
483
 
    @deprecated_method(deprecated_in((2, 1, 0)))
484
273
    def compare_basis(self):
485
274
        try:
486
275
            basis_tree = self.revision_tree(self.this_tree.last_revision())
493
282
        self.interesting_files = file_list
494
283
 
495
284
    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):
 
285
        if not self.base_is_ancestor or not self.base_is_other_ancestor or self.other_rev_id is None:
498
286
            return
499
287
        self._add_parent()
500
288
 
501
289
    def _add_parent(self):
502
290
        new_parents = self.this_tree.get_parent_ids() + [self.other_rev_id]
503
291
        new_parent_trees = []
504
 
        operation = OperationWithCleanups(self.this_tree.set_parent_trees)
505
292
        for revision_id in new_parents:
506
293
            try:
507
294
                tree = self.revision_tree(revision_id)
509
296
                tree = None
510
297
            else:
511
298
                tree.lock_read()
512
 
                operation.add_cleanup(tree.unlock)
513
299
            new_parent_trees.append((revision_id, tree))
514
 
        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()
515
307
 
516
308
    def set_other(self, other_revision, possible_transports=None):
517
309
        """Set the revision and tree to merge from.
526
318
            self.other_rev_id = _mod_revision.ensure_null(
527
319
                self.other_branch.last_revision())
528
320
            if _mod_revision.is_null(self.other_rev_id):
529
 
                raise errors.NoCommits(self.other_branch)
 
321
                raise NoCommits(self.other_branch)
530
322
            self.other_basis = self.other_rev_id
531
323
        elif other_revision[1] is not None:
532
324
            self.other_rev_id = self.other_branch.get_rev_id(other_revision[1])
535
327
            self.other_rev_id = None
536
328
            self.other_basis = self.other_branch.last_revision()
537
329
            if self.other_basis is None:
538
 
                raise errors.NoCommits(self.other_branch)
 
330
                raise NoCommits(self.other_branch)
539
331
        if self.other_rev_id is not None:
540
332
            self._cached_trees[self.other_rev_id] = self.other_tree
541
333
        self._maybe_fetch(self.other_branch,self.this_branch, self.other_basis)
568
360
            target.fetch(source, revision_id)
569
361
 
570
362
    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
 
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
575
367
            self.base_tree = self.revision_tree(self.base_rev_id)
576
368
            self._is_criss_cross = False
577
369
        else:
578
370
            lcas = self.revision_graph.find_lca(revisions[0], revisions[1])
579
371
            self._is_criss_cross = False
580
372
            if len(lcas) == 0:
581
 
                self.base_rev_id = _mod_revision.NULL_REVISION
 
373
                self.base_rev_id = NULL_REVISION
582
374
            elif len(lcas) == 1:
583
375
                self.base_rev_id = list(lcas)[0]
584
376
            else: # len(lcas) > 1
593
385
                    self.base_rev_id = self.revision_graph.find_unique_lca(
594
386
                                            *lcas)
595
387
                self._is_criss_cross = True
596
 
            if self.base_rev_id == _mod_revision.NULL_REVISION:
597
 
                raise errors.UnrelatedBranches()
 
388
            if self.base_rev_id == NULL_REVISION:
 
389
                raise UnrelatedBranches()
598
390
            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)
 
391
                warning('Warning: criss-cross merge encountered.  See bzr'
 
392
                        ' help criss-cross.')
 
393
                mutter('Criss-cross lcas: %r' % lcas)
602
394
                interesting_revision_ids = [self.base_rev_id]
603
395
                interesting_revision_ids.extend(lcas)
604
396
                interesting_trees = dict((t.get_revision_id(), t)
614
406
                self.base_tree = self.revision_tree(self.base_rev_id)
615
407
        self.base_is_ancestor = True
616
408
        self.base_is_other_ancestor = True
617
 
        trace.mutter('Base revid: %r' % self.base_rev_id)
 
409
        mutter('Base revid: %r' % self.base_rev_id)
618
410
 
619
411
    def set_base(self, base_revision):
620
412
        """Set the base revision to use for the merge.
621
413
 
622
414
        :param base_revision: A 2-list containing a path and revision number.
623
415
        """
624
 
        trace.mutter("doing merge() with no base_revision specified")
 
416
        mutter("doing merge() with no base_revision specified")
625
417
        if base_revision == [None, None]:
626
418
            self.find_base()
627
419
        else:
640
432
                  'other_tree': self.other_tree,
641
433
                  'interesting_ids': self.interesting_ids,
642
434
                  'interesting_files': self.interesting_files,
643
 
                  'this_branch': self.this_branch,
 
435
                  'pp': self.pp,
644
436
                  'do_merge': False}
645
437
        if self.merge_type.requires_base:
646
438
            kwargs['base_tree'] = self.base_tree
647
439
        if self.merge_type.supports_reprocess:
648
440
            kwargs['reprocess'] = self.reprocess
649
441
        elif self.reprocess:
650
 
            raise errors.BzrError(
651
 
                "Conflict reduction is not supported for merge"
652
 
                " type %s." % self.merge_type)
 
442
            raise BzrError("Conflict reduction is not supported for merge"
 
443
                                  " type %s." % self.merge_type)
653
444
        if self.merge_type.supports_show_base:
654
445
            kwargs['show_base'] = self.show_base
655
446
        elif self.show_base:
656
 
            raise errors.BzrError("Showing base is not supported for this"
657
 
                                  " merge type. %s" % self.merge_type)
 
447
            raise BzrError("Showing base is not supported for this"
 
448
                           " merge type. %s" % self.merge_type)
658
449
        if (not getattr(self.merge_type, 'supports_reverse_cherrypick', True)
659
450
            and not self.base_is_other_ancestor):
660
451
            raise errors.CannotReverseCherrypick()
664
455
        if self._is_criss_cross and getattr(self.merge_type,
665
456
                                            'supports_lca_trees', False):
666
457
            kwargs['lca_trees'] = self._lca_trees
667
 
        return self.merge_type(pb=None,
 
458
        return self.merge_type(pb=self._pb,
668
459
                               change_reporter=self.change_reporter,
669
460
                               **kwargs)
670
461
 
671
 
    def _do_merge_to(self):
672
 
        merge = self.make_merger()
 
462
    def _do_merge_to(self, merge):
673
463
        if self.other_branch is not None:
674
464
            self.other_branch.update_references(self.this_branch)
675
465
        merge.do_merge()
689
479
                    sub_tree.branch.repository.revision_tree(base_revision)
690
480
                sub_merge.base_rev_id = base_revision
691
481
                sub_merge.do_merge()
692
 
        return merge
693
482
 
694
483
    def do_merge(self):
695
 
        operation = OperationWithCleanups(self._do_merge_to)
696
484
        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()
 
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()
705
502
        if len(merge.cooked_conflicts) == 0:
706
 
            if not self.ignore_zero and not trace.is_quiet():
707
 
                trace.note("All changes applied successfully.")
 
503
            if not self.ignore_zero and not is_quiet():
 
504
                note("All changes applied successfully.")
708
505
        else:
709
 
            trace.note("%d conflicts encountered."
710
 
                       % len(merge.cooked_conflicts))
 
506
            note("%d conflicts encountered." % len(merge.cooked_conflicts))
711
507
 
712
508
        return len(merge.cooked_conflicts)
713
509
 
742
538
 
743
539
    def __init__(self, working_tree, this_tree, base_tree, other_tree,
744
540
                 interesting_ids=None, reprocess=False, show_base=False,
745
 
                 pb=None, pp=None, change_reporter=None,
 
541
                 pb=DummyProgress(), pp=None, change_reporter=None,
746
542
                 interesting_files=None, do_merge=True,
747
 
                 cherrypick=False, lca_trees=None, this_branch=None):
 
543
                 cherrypick=False, lca_trees=None):
748
544
        """Initialize the merger object and perform the merge.
749
545
 
750
546
        :param working_tree: The working tree to apply the merge to
751
547
        :param this_tree: The local tree in the merge operation
752
548
        :param base_tree: The common tree in the merge operation
753
549
        :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
550
        :param interesting_ids: The file_ids of files that should be
757
551
            participate in the merge.  May not be combined with
758
552
            interesting_files.
759
553
        :param: reprocess If True, perform conflict-reduction processing.
760
554
        :param show_base: If True, show the base revision in text conflicts.
761
555
            (incompatible with reprocess)
762
 
        :param pb: ignored
 
556
        :param pb: A Progress bar
763
557
        :param pp: A ProgressPhase object
764
558
        :param change_reporter: An object that should report changes made
765
559
        :param interesting_files: The tree-relative paths of files that should
776
570
        if interesting_files is not None and interesting_ids is not None:
777
571
            raise ValueError(
778
572
                'specify either interesting_ids or interesting_files')
779
 
        if this_branch is None:
780
 
            this_branch = this_tree.branch
781
573
        self.interesting_ids = interesting_ids
782
574
        self.interesting_files = interesting_files
783
575
        self.this_tree = working_tree
784
576
        self.base_tree = base_tree
785
577
        self.other_tree = other_tree
786
 
        self.this_branch = this_branch
787
578
        self._raw_conflicts = []
788
579
        self.cooked_conflicts = []
789
580
        self.reprocess = reprocess
794
585
        # making sure we haven't missed any corner cases.
795
586
        # if lca_trees is None:
796
587
        #     self._lca_trees = [self.base_tree]
 
588
        self.pb = pb
 
589
        self.pp = pp
797
590
        self.change_reporter = change_reporter
798
591
        self.cherrypick = cherrypick
 
592
        if self.pp is None:
 
593
            self.pp = ProgressPhase("Merge phase", 3, self.pb)
799
594
        if do_merge:
800
595
            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
596
 
806
597
    def do_merge(self):
807
 
        operation = OperationWithCleanups(self._do_merge)
808
598
        self.this_tree.lock_tree_write()
809
 
        operation.add_cleanup(self.this_tree.unlock)
810
599
        self.base_tree.lock_read()
811
 
        operation.add_cleanup(self.base_tree.unlock)
812
600
        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)
 
601
        self.tt = TreeTransform(self.this_tree, self.pb)
822
602
        try:
823
 
            self.this_tree.add_conflicts(self.cooked_conflicts)
824
 
        except errors.UnsupportedOperation:
825
 
            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()
826
618
 
827
619
    def make_preview_transform(self):
828
 
        operation = OperationWithCleanups(self._make_preview_transform)
829
620
        self.base_tree.lock_read()
830
 
        operation.add_cleanup(self.base_tree.unlock)
831
621
        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()
 
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()
838
631
        return self.tt
839
632
 
840
633
    def _compute_transform(self):
846
639
            resolver = self._lca_multi_way
847
640
        child_pb = ui.ui_factory.nested_progress_bar()
848
641
        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
642
            for num, (file_id, changed, parents3, names3,
853
643
                      executable3) in enumerate(entries):
854
644
                child_pb.update('Preparing file merge', num, len(entries))
855
645
                self._merge_names(file_id, parents3, names3, resolver=resolver)
856
646
                if changed:
857
 
                    file_status = self._do_merge_contents(file_id)
 
647
                    file_status = self.merge_contents(file_id)
858
648
                else:
859
649
                    file_status = 'unmodified'
860
650
                self._merge_executable(file_id,
862
652
        finally:
863
653
            child_pb.finished()
864
654
        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
 
        """
 
655
        self.pp.next_phase()
872
656
        child_pb = ui.ui_factory.nested_progress_bar()
873
657
        try:
874
 
            fs_conflicts = transform.resolve_conflicts(self.tt, child_pb,
875
 
                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))
876
660
        finally:
877
661
            child_pb.finished()
878
662
        if self.change_reporter is not None:
881
665
                self.tt.iter_changes(), self.change_reporter)
882
666
        self.cook_conflicts(fs_conflicts)
883
667
        for conflict in self.cooked_conflicts:
884
 
            trace.warning(conflict)
 
668
            warning(conflict)
885
669
 
886
670
    def _entries3(self):
887
671
        """Gather data about files modified between three trees.
1072
856
                        continue
1073
857
                else:
1074
858
                    raise AssertionError('unhandled kind: %s' % other_ie.kind)
 
859
                # XXX: We need to handle kind == 'symlink'
1075
860
 
1076
861
            # If we have gotten this far, that means something has changed
1077
862
            result.append((file_id, content_changed,
1084
869
                          ))
1085
870
        return result
1086
871
 
 
872
 
1087
873
    def fix_root(self):
1088
874
        try:
1089
875
            self.tt.final_kind(self.tt.root)
1090
 
        except errors.NoSuchFile:
 
876
        except NoSuchFile:
1091
877
            self.tt.cancel_deletion(self.tt.root)
1092
878
        if self.tt.final_file_id(self.tt.root) is None:
1093
879
            self.tt.version_file(self.tt.tree_file_id(self.tt.root),
1098
884
        other_root = self.tt.trans_id_file_id(other_root_file_id)
1099
885
        if other_root == self.tt.root:
1100
886
            return
 
887
        try:
 
888
            self.tt.final_kind(other_root)
 
889
        except NoSuchFile:
 
890
            return
1101
891
        if self.other_tree.inventory.root.file_id in self.this_tree.inventory:
1102
 
            # the other tree's root is a non-root in the current tree (as when
1103
 
            # a previously unrelated branch is merged into another)
 
892
            # the other tree's root is a non-root in the current tree
1104
893
            return
1105
 
        try:
1106
 
            self.tt.final_kind(other_root)
1107
 
            other_root_is_present = True
1108
 
        except errors.NoSuchFile:
1109
 
            # other_root doesn't have a physical representation. We still need
1110
 
            # to move any references to the actual root of the tree.
1111
 
            other_root_is_present = False
1112
 
        # 'other_tree.inventory.root' is not present in this tree. We are
1113
 
        # calling adjust_path for children which *want* to be present with a
1114
 
        # correct place to go.
1115
 
        for thing, child in self.other_tree.inventory.root.children.iteritems():
 
894
        self.reparent_children(self.other_tree.inventory.root, self.tt.root)
 
895
        self.tt.cancel_creation(other_root)
 
896
        self.tt.cancel_versioning(other_root)
 
897
 
 
898
    def reparent_children(self, ie, target):
 
899
        for thing, child in ie.children.iteritems():
1116
900
            trans_id = self.tt.trans_id_file_id(child.file_id)
1117
 
            if not other_root_is_present:
1118
 
                # FIXME: Make final_kind returns None instead of raising
1119
 
                # NoSuchFile to avoid the ugly construct below -- vila 20100402
1120
 
                try:
1121
 
                    self.tt.final_kind(trans_id)
1122
 
                    # The item exist in the final tree and has a defined place
1123
 
                    # to go already.
1124
 
                    continue
1125
 
                except errors.NoSuchFile, e:
1126
 
                    pass
1127
 
            # Move the item into the root
1128
 
            self.tt.adjust_path(self.tt.final_name(trans_id),
1129
 
                                self.tt.root, trans_id)
1130
 
        if other_root_is_present:
1131
 
            self.tt.cancel_creation(other_root)
1132
 
            self.tt.cancel_versioning(other_root)
 
901
            self.tt.adjust_path(self.tt.final_name(trans_id), target, trans_id)
1133
902
 
1134
903
    def write_modified(self, results):
1135
904
        modified_hashes = {}
1167
936
    @staticmethod
1168
937
    def executable(tree, file_id):
1169
938
        """Determine the executability of a file-id (used as a key method)."""
1170
 
        if not tree.has_id(file_id):
 
939
        if file_id not in tree:
1171
940
            return None
1172
941
        if tree.kind(file_id) != "file":
1173
942
            return False
1176
945
    @staticmethod
1177
946
    def kind(tree, file_id):
1178
947
        """Determine the kind of a file-id (used as a key method)."""
1179
 
        if not tree.has_id(file_id):
 
948
        if file_id not in tree:
1180
949
            return None
1181
950
        return tree.kind(file_id)
1182
951
 
1183
952
    @staticmethod
1184
953
    def _three_way(base, other, this):
 
954
        #if base == other, either they all agree, or only THIS has changed.
1185
955
        if base == other:
1186
 
            # if 'base == other', either they all agree, or only 'this' has
1187
 
            # changed.
1188
956
            return 'this'
1189
957
        elif this not in (base, other):
1190
 
            # 'this' is neither 'base' nor 'other', so both sides changed
1191
958
            return 'conflict'
 
959
        # "Ambiguous clean merge" -- both sides have made the same change.
1192
960
        elif this == other:
1193
 
            # "Ambiguous clean merge" -- both sides have made the same change.
1194
961
            return "this"
 
962
        # this == base: only other has changed.
1195
963
        else:
1196
 
            # this == base: only other has changed.
1197
964
            return "other"
1198
965
 
1199
966
    @staticmethod
1243
1010
                # only has an lca value
1244
1011
                return 'other'
1245
1012
 
1246
 
        # At this point, the lcas disagree, and the tip disagree
 
1013
        # At this point, the lcas disagree, and the tips disagree
1247
1014
        return 'conflict'
1248
1015
 
1249
1016
    @staticmethod
1250
 
    @deprecated_method(deprecated_in((2, 2, 0)))
1251
1017
    def scalar_three_way(this_tree, base_tree, other_tree, file_id, key):
1252
1018
        """Do a three-way test on a scalar.
1253
1019
        Return "this", "other" or "conflict", depending whether a value wins.
1268
1034
 
1269
1035
    def merge_names(self, file_id):
1270
1036
        def get_entry(tree):
1271
 
            if tree.has_id(file_id):
 
1037
            if file_id in tree.inventory:
1272
1038
                return tree.inventory[file_id]
1273
1039
            else:
1274
1040
                return None
1303
1069
                parent_id_winner = "other"
1304
1070
        if name_winner == "this" and parent_id_winner == "this":
1305
1071
            return
1306
 
        if name_winner == 'conflict' or parent_id_winner == 'conflict':
1307
 
            # Creating helpers (.OTHER or .THIS) here cause problems down the
1308
 
            # road if a ContentConflict needs to be created so we should not do
1309
 
            # that
1310
 
            trans_id = self.tt.trans_id_file_id(file_id)
1311
 
            self._raw_conflicts.append(('path conflict', trans_id, file_id,
1312
 
                                        this_parent, this_name,
1313
 
                                        other_parent, other_name))
 
1072
        if name_winner == "conflict":
 
1073
            trans_id = self.tt.trans_id_file_id(file_id)
 
1074
            self._raw_conflicts.append(('name conflict', trans_id,
 
1075
                                        this_name, other_name))
 
1076
        if parent_id_winner == "conflict":
 
1077
            trans_id = self.tt.trans_id_file_id(file_id)
 
1078
            self._raw_conflicts.append(('parent conflict', trans_id,
 
1079
                                        this_parent, other_parent))
1314
1080
        if other_name is None:
1315
1081
            # it doesn't matter whether the result was 'other' or
1316
1082
            # 'conflict'-- if there's no 'other', we leave it alone.
1317
1083
            return
 
1084
        # if we get here, name_winner and parent_winner are set to safe values.
 
1085
        trans_id = self.tt.trans_id_file_id(file_id)
1318
1086
        parent_id = parents[self.winner_idx[parent_id_winner]]
1319
1087
        if parent_id is not None:
1320
 
            # if we get here, name_winner and parent_winner are set to safe
1321
 
            # values.
 
1088
            parent_trans_id = self.tt.trans_id_file_id(parent_id)
1322
1089
            self.tt.adjust_path(names[self.winner_idx[name_winner]],
1323
 
                                self.tt.trans_id_file_id(parent_id),
1324
 
                                self.tt.trans_id_file_id(file_id))
 
1090
                                parent_trans_id, trans_id)
1325
1091
 
1326
 
    def _do_merge_contents(self, file_id):
 
1092
    def merge_contents(self, file_id):
1327
1093
        """Performs a merge on file_id contents."""
1328
1094
        def contents_pair(tree):
1329
1095
            if file_id not in tree:
1337
1103
                contents = None
1338
1104
            return kind, contents
1339
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
 
1340
1118
        # See SPOT run.  run, SPOT, run.
1341
1119
        # So we're not QUITE repeating ourselves; we do tricky things with
1342
1120
        # file kind...
1358
1136
        if winner == 'this':
1359
1137
            # No interesting changes introduced by OTHER
1360
1138
            return "unmodified"
1361
 
        # We have a hypothetical conflict, but if we have files, then we
1362
 
        # can try to merge the content
1363
1139
        trans_id = self.tt.trans_id_file_id(file_id)
1364
 
        params = MergeHookParams(self, file_id, trans_id, this_pair[0],
1365
 
            other_pair[0], winner)
1366
 
        hooks = self.active_hooks
1367
 
        hook_status = 'not_applicable'
1368
 
        for hook in hooks:
1369
 
            hook_status, lines = hook.merge_contents(params)
1370
 
            if hook_status != 'not_applicable':
1371
 
                # Don't try any more hooks, this one applies.
1372
 
                break
1373
 
        result = "modified"
1374
 
        if hook_status == 'not_applicable':
1375
 
            # This is a contents conflict, because none of the available
1376
 
            # functions could merge it.
1377
 
            result = None
1378
 
            name = self.tt.final_name(trans_id)
1379
 
            parent_id = self.tt.final_parent(trans_id)
1380
 
            if self.this_tree.has_id(file_id):
1381
 
                self.tt.unversion_file(trans_id)
1382
 
            file_group = self._dump_conflicts(name, parent_id, file_id,
1383
 
                                              set_version=True)
1384
 
            self._raw_conflicts.append(('contents conflict', file_group))
1385
 
        elif hook_status == 'success':
1386
 
            self.tt.create_file(lines, trans_id)
1387
 
        elif hook_status == 'conflicted':
1388
 
            # XXX: perhaps the hook should be able to provide
1389
 
            # the BASE/THIS/OTHER files?
1390
 
            self.tt.create_file(lines, trans_id)
1391
 
            self._raw_conflicts.append(('text conflict', trans_id))
1392
 
            name = self.tt.final_name(trans_id)
1393
 
            parent_id = self.tt.final_parent(trans_id)
1394
 
            self._dump_conflicts(name, parent_id, file_id)
1395
 
        elif hook_status == 'delete':
1396
 
            self.tt.unversion_file(trans_id)
1397
 
            result = "deleted"
1398
 
        elif hook_status == 'done':
1399
 
            # The hook function did whatever it needs to do directly, no
1400
 
            # further action needed here.
1401
 
            pass
1402
 
        else:
1403
 
            raise AssertionError('unknown hook_status: %r' % (hook_status,))
1404
 
        if not self.this_tree.has_id(file_id) and result == "modified":
1405
 
            self.tt.version_file(file_id, trans_id)
1406
 
        # The merge has been performed, so the old contents should not be
1407
 
        # retained.
1408
 
        try:
1409
 
            self.tt.delete_contents(trans_id)
1410
 
        except errors.NoSuchFile:
1411
 
            pass
1412
 
        return result
1413
 
 
1414
 
    def _default_other_winner_merge(self, merge_hook_params):
1415
 
        """Replace this contents with other."""
1416
 
        file_id = merge_hook_params.file_id
1417
 
        trans_id = merge_hook_params.trans_id
1418
 
        file_in_this = self.this_tree.has_id(file_id)
1419
 
        if self.other_tree.has_id(file_id):
1420
 
            # OTHER changed the file
1421
 
            wt = self.this_tree
1422
 
            if wt.supports_content_filtering():
1423
 
                # We get the path from the working tree if it exists.
1424
 
                # That fails though when OTHER is adding a file, so
1425
 
                # we fall back to the other tree to find the path if
1426
 
                # it doesn't exist locally.
1427
 
                try:
1428
 
                    filter_tree_path = wt.id2path(file_id)
1429
 
                except errors.NoSuchId:
1430
 
                    filter_tree_path = self.other_tree.id2path(file_id)
1431
 
            else:
1432
 
                # Skip the id2path lookup for older formats
1433
 
                filter_tree_path = None
1434
 
            transform.create_from_tree(self.tt, trans_id,
1435
 
                             self.other_tree, file_id,
1436
 
                             filter_tree_path=filter_tree_path)
1437
 
            return 'done', None
1438
 
        elif file_in_this:
1439
 
            # OTHER deleted the file
1440
 
            return 'delete', None
1441
 
        else:
1442
 
            raise AssertionError(
1443
 
                'winner is OTHER, but file_id %r not in THIS or OTHER tree'
1444
 
                % (file_id,))
1445
 
 
1446
 
    def merge_contents(self, merge_hook_params):
1447
 
        """Fallback merge logic after user installed hooks."""
1448
 
        # This function is used in merge hooks as the fallback instance.
1449
 
        # Perhaps making this function and the functions it calls be a 
1450
 
        # a separate class would be better.
1451
 
        if merge_hook_params.winner == 'other':
 
1140
        if winner == 'other':
1452
1141
            # OTHER is a straight winner, so replace this contents with other
1453
 
            return self._default_other_winner_merge(merge_hook_params)
1454
 
        elif merge_hook_params.is_file_merge():
1455
 
            # THIS and OTHER are both files, so text merge.  Either
1456
 
            # BASE is a file, or both converted to files, so at least we
1457
 
            # have agreement that output should be a file.
1458
 
            try:
1459
 
                self.text_merge(merge_hook_params.file_id,
1460
 
                    merge_hook_params.trans_id)
1461
 
            except errors.BinaryFile:
1462
 
                return 'not_applicable', None
1463
 
            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"
1464
1157
        else:
1465
 
            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()
1466
1178
 
1467
1179
    def get_lines(self, tree, file_id):
1468
1180
        """Return the lines in a file, or an empty list."""
1469
 
        if tree.has_id(file_id):
1470
 
            return tree.get_file_lines(file_id)
 
1181
        if file_id in tree:
 
1182
            return tree.get_file(file_id).readlines()
1471
1183
        else:
1472
1184
            return []
1473
1185
 
1475
1187
        """Perform a three-way text merge on a file_id"""
1476
1188
        # it's possible that we got here with base as a different type.
1477
1189
        # if so, we just want two-way text conflicts.
1478
 
        if self.base_tree.has_id(file_id) and \
 
1190
        if file_id in self.base_tree and \
1479
1191
            self.base_tree.kind(file_id) == "file":
1480
1192
            base_lines = self.get_lines(self.base_tree, file_id)
1481
1193
        else:
1482
1194
            base_lines = []
1483
1195
        other_lines = self.get_lines(self.other_tree, file_id)
1484
1196
        this_lines = self.get_lines(self.this_tree, file_id)
1485
 
        m3 = merge3.Merge3(base_lines, this_lines, other_lines,
1486
 
                           is_cherrypick=self.cherrypick)
 
1197
        m3 = Merge3(base_lines, this_lines, other_lines,
 
1198
                    is_cherrypick=self.cherrypick)
1487
1199
        start_marker = "!START OF MERGE CONFLICT!" + "I HOPE THIS IS UNIQUE"
1488
1200
        if self.show_base is True:
1489
1201
            base_marker = '|' * 7
1527
1239
                ('THIS', self.this_tree, this_lines)]
1528
1240
        if not no_base:
1529
1241
            data.append(('BASE', self.base_tree, base_lines))
1530
 
 
1531
 
        # We need to use the actual path in the working tree of the file here,
1532
 
        # ignoring the conflict suffixes
1533
 
        wt = self.this_tree
1534
 
        if wt.supports_content_filtering():
1535
 
            try:
1536
 
                filter_tree_path = wt.id2path(file_id)
1537
 
            except errors.NoSuchId:
1538
 
                # file has been deleted
1539
 
                filter_tree_path = None
1540
 
        else:
1541
 
            # Skip the id2path lookup for older formats
1542
 
            filter_tree_path = None
1543
 
 
1544
1242
        versioned = False
1545
1243
        file_group = []
1546
1244
        for suffix, tree, lines in data:
1547
 
            if tree.has_id(file_id):
 
1245
            if file_id in tree:
1548
1246
                trans_id = self._conflict_file(name, parent_id, tree, file_id,
1549
 
                                               suffix, lines, filter_tree_path)
 
1247
                                               suffix, lines)
1550
1248
                file_group.append(trans_id)
1551
1249
                if set_version and not versioned:
1552
1250
                    self.tt.version_file(file_id, trans_id)
1554
1252
        return file_group
1555
1253
 
1556
1254
    def _conflict_file(self, name, parent_id, tree, file_id, suffix,
1557
 
                       lines=None, filter_tree_path=None):
 
1255
                       lines=None):
1558
1256
        """Emit a single conflict file."""
1559
1257
        name = name + '.' + suffix
1560
1258
        trans_id = self.tt.create_path(name, parent_id)
1561
 
        transform.create_from_tree(self.tt, trans_id, tree, file_id, lines,
1562
 
            filter_tree_path)
 
1259
        create_from_tree(self.tt, trans_id, tree, file_id, lines)
1563
1260
        return trans_id
1564
1261
 
1565
1262
    def merge_executable(self, file_id, file_status):
1589
1286
        try:
1590
1287
            if self.tt.final_kind(trans_id) != "file":
1591
1288
                return
1592
 
        except errors.NoSuchFile:
 
1289
        except NoSuchFile:
1593
1290
            return
1594
1291
        if winner == "this":
1595
1292
            executability = this_executable
1596
1293
        else:
1597
 
            if self.other_tree.has_id(file_id):
 
1294
            if file_id in self.other_tree:
1598
1295
                executability = other_executable
1599
 
            elif self.this_tree.has_id(file_id):
 
1296
            elif file_id in self.this_tree:
1600
1297
                executability = this_executable
1601
 
            elif self.base_tree_has_id(file_id):
 
1298
            elif file_id in self.base_tree:
1602
1299
                executability = base_executable
1603
1300
        if executability is not None:
1604
1301
            trans_id = self.tt.trans_id_file_id(file_id)
1606
1303
 
1607
1304
    def cook_conflicts(self, fs_conflicts):
1608
1305
        """Convert all conflicts into a form that doesn't depend on trans_id"""
1609
 
        self.cooked_conflicts.extend(transform.cook_conflicts(
1610
 
                fs_conflicts, self.tt))
1611
 
        fp = transform.FinalPaths(self.tt)
 
1306
        from conflicts import Conflict
 
1307
        name_conflicts = {}
 
1308
        self.cooked_conflicts.extend(cook_conflicts(fs_conflicts, self.tt))
 
1309
        fp = FinalPaths(self.tt)
1612
1310
        for conflict in self._raw_conflicts:
1613
1311
            conflict_type = conflict[0]
1614
 
            if conflict_type == 'path conflict':
1615
 
                (trans_id, file_id,
1616
 
                this_parent, this_name,
1617
 
                other_parent, other_name) = conflict[1:]
1618
 
                if this_parent is None or this_name is None:
1619
 
                    this_path = '<deleted>'
1620
 
                else:
1621
 
                    parent_path =  fp.get_path(
1622
 
                        self.tt.trans_id_file_id(this_parent))
1623
 
                    this_path = osutils.pathjoin(parent_path, this_name)
1624
 
                if other_parent is None or other_name is None:
1625
 
                    other_path = '<deleted>'
1626
 
                else:
1627
 
                    parent_path =  fp.get_path(
1628
 
                        self.tt.trans_id_file_id(other_parent))
1629
 
                    other_path = osutils.pathjoin(parent_path, other_name)
1630
 
                c = _mod_conflicts.Conflict.factory(
1631
 
                    'path conflict', path=this_path,
1632
 
                    conflict_path=other_path,
1633
 
                    file_id=file_id)
1634
 
            elif conflict_type == 'contents conflict':
 
1312
            if conflict_type in ('name conflict', 'parent conflict'):
 
1313
                trans_id = conflict[1]
 
1314
                conflict_args = conflict[2:]
 
1315
                if trans_id not in name_conflicts:
 
1316
                    name_conflicts[trans_id] = {}
 
1317
                unique_add(name_conflicts[trans_id], conflict_type,
 
1318
                           conflict_args)
 
1319
            if conflict_type == 'contents conflict':
1635
1320
                for trans_id in conflict[1]:
1636
1321
                    file_id = self.tt.final_file_id(trans_id)
1637
1322
                    if file_id is not None:
1641
1326
                    if path.endswith(suffix):
1642
1327
                        path = path[:-len(suffix)]
1643
1328
                        break
1644
 
                c = _mod_conflicts.Conflict.factory(conflict_type,
1645
 
                                                    path=path, file_id=file_id)
1646
 
            elif conflict_type == 'text conflict':
 
1329
                c = Conflict.factory(conflict_type, path=path, file_id=file_id)
 
1330
                self.cooked_conflicts.append(c)
 
1331
            if conflict_type == 'text conflict':
1647
1332
                trans_id = conflict[1]
1648
1333
                path = fp.get_path(trans_id)
1649
1334
                file_id = self.tt.final_file_id(trans_id)
1650
 
                c = _mod_conflicts.Conflict.factory(conflict_type,
1651
 
                                                    path=path, file_id=file_id)
 
1335
                c = Conflict.factory(conflict_type, path=path, file_id=file_id)
 
1336
                self.cooked_conflicts.append(c)
 
1337
 
 
1338
        for trans_id, conflicts in name_conflicts.iteritems():
 
1339
            try:
 
1340
                this_parent, other_parent = conflicts['parent conflict']
 
1341
                if this_parent == other_parent:
 
1342
                    raise AssertionError()
 
1343
            except KeyError:
 
1344
                this_parent = other_parent = \
 
1345
                    self.tt.final_file_id(self.tt.final_parent(trans_id))
 
1346
            try:
 
1347
                this_name, other_name = conflicts['name conflict']
 
1348
                if this_name == other_name:
 
1349
                    raise AssertionError()
 
1350
            except KeyError:
 
1351
                this_name = other_name = self.tt.final_name(trans_id)
 
1352
            other_path = fp.get_path(trans_id)
 
1353
            if this_parent is not None and this_name is not None:
 
1354
                this_parent_path = \
 
1355
                    fp.get_path(self.tt.trans_id_file_id(this_parent))
 
1356
                this_path = pathjoin(this_parent_path, this_name)
1652
1357
            else:
1653
 
                raise AssertionError('bad conflict type: %r' % (conflict,))
 
1358
                this_path = "<deleted>"
 
1359
            file_id = self.tt.final_file_id(trans_id)
 
1360
            c = Conflict.factory('path conflict', path=this_path,
 
1361
                                 conflict_path=other_path, file_id=file_id)
1654
1362
            self.cooked_conflicts.append(c)
1655
 
        self.cooked_conflicts.sort(key=_mod_conflicts.Conflict.sort_key)
 
1363
        self.cooked_conflicts.sort(key=Conflict.sort_key)
1656
1364
 
1657
1365
 
1658
1366
class WeaveMerger(Merge3Merger):
1662
1370
    supports_reverse_cherrypick = False
1663
1371
    history_based = True
1664
1372
 
1665
 
    def _generate_merge_plan(self, file_id, base):
1666
 
        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,
1667
1383
                                              base=base)
1668
 
 
1669
 
    def _merged_lines(self, file_id):
1670
 
        """Generate the merged lines.
1671
 
        There is no distinction between lines that are meant to contain <<<<<<<
1672
 
        and conflicts.
1673
 
        """
1674
 
        if self.cherrypick:
1675
 
            base = self.base_tree
1676
 
        else:
1677
 
            base = None
1678
 
        plan = self._generate_merge_plan(file_id, base)
1679
1384
        if 'merge' in debug.debug_flags:
1680
1385
            plan = list(plan)
1681
1386
            trans_id = self.tt.trans_id_file_id(file_id)
1682
1387
            name = self.tt.final_name(trans_id) + '.plan'
1683
 
            contents = ('%11s|%s' % l for l in plan)
 
1388
            contents = ('%10s|%s' % l for l in plan)
1684
1389
            self.tt.new_file(name, self.tt.final_parent(trans_id), contents)
1685
 
        textmerge = versionedfile.PlanWeaveMerge(plan, '<<<<<<< TREE\n',
1686
 
                                                 '>>>>>>> MERGE-SOURCE\n')
1687
 
        lines, conflicts = textmerge.merge_lines(self.reprocess)
1688
 
        if conflicts:
1689
 
            base_lines = textmerge.base_from_plan()
1690
 
        else:
1691
 
            base_lines = None
1692
 
        return lines, base_lines
 
1390
        textmerge = PlanWeaveMerge(plan, '<<<<<<< TREE\n',
 
1391
            '>>>>>>> MERGE-SOURCE\n')
 
1392
        return textmerge.merge_lines(self.reprocess)
1693
1393
 
1694
1394
    def text_merge(self, file_id, trans_id):
1695
1395
        """Perform a (weave) text merge for a given file and file-id.
1696
1396
        If conflicts are encountered, .THIS and .OTHER files will be emitted,
1697
1397
        and a conflict will be noted.
1698
1398
        """
1699
 
        lines, base_lines = self._merged_lines(file_id)
 
1399
        lines, conflicts = self._merged_lines(file_id)
1700
1400
        lines = list(lines)
1701
1401
        # Note we're checking whether the OUTPUT is binary in this case,
1702
1402
        # because we don't want to get into weave merge guts.
1703
 
        textfile.check_text_lines(lines)
 
1403
        check_text_lines(lines)
1704
1404
        self.tt.create_file(lines, trans_id)
1705
 
        if base_lines is not None:
1706
 
            # Conflict
 
1405
        if conflicts:
1707
1406
            self._raw_conflicts.append(('text conflict', trans_id))
1708
1407
            name = self.tt.final_name(trans_id)
1709
1408
            parent_id = self.tt.final_parent(trans_id)
1710
1409
            file_group = self._dump_conflicts(name, parent_id, file_id,
1711
 
                                              no_base=False,
1712
 
                                              base_lines=base_lines)
 
1410
                                              no_base=True)
1713
1411
            file_group.append(trans_id)
1714
1412
 
1715
1413
 
1716
1414
class LCAMerger(WeaveMerger):
1717
1415
 
1718
 
    def _generate_merge_plan(self, file_id, base):
1719
 
        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,
1720
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
 
1721
1437
 
1722
1438
class Diff3Merger(Merge3Merger):
1723
1439
    """Three-way merger using external diff3 for text merging"""
1724
1440
 
1725
1441
    def dump_file(self, temp_dir, name, tree, file_id):
1726
 
        out_path = osutils.pathjoin(temp_dir, name)
 
1442
        out_path = pathjoin(temp_dir, name)
1727
1443
        out_file = open(out_path, "wb")
1728
1444
        try:
1729
1445
            in_file = tree.get_file(file_id)
1741
1457
        import bzrlib.patch
1742
1458
        temp_dir = osutils.mkdtemp(prefix="bzr-")
1743
1459
        try:
1744
 
            new_file = osutils.pathjoin(temp_dir, "new")
 
1460
            new_file = pathjoin(temp_dir, "new")
1745
1461
            this = self.dump_file(temp_dir, "this", self.this_tree, file_id)
1746
1462
            base = self.dump_file(temp_dir, "base", self.base_tree, file_id)
1747
1463
            other = self.dump_file(temp_dir, "other", self.other_tree, file_id)
1748
1464
            status = bzrlib.patch.diff3(new_file, this, base, other)
1749
1465
            if status not in (0, 1):
1750
 
                raise errors.BzrError("Unhandled diff3 exit code")
 
1466
                raise BzrError("Unhandled diff3 exit code")
1751
1467
            f = open(new_file, 'rb')
1752
1468
            try:
1753
1469
                self.tt.create_file(f, trans_id)
1762
1478
            osutils.rmtree(temp_dir)
1763
1479
 
1764
1480
 
1765
 
class PathNotInTree(errors.BzrError):
1766
 
 
1767
 
    _fmt = """Merge-into failed because %(tree)s does not contain %(path)s."""
1768
 
 
1769
 
    def __init__(self, path, tree):
1770
 
        errors.BzrError.__init__(self, path=path, tree=tree)
1771
 
 
1772
 
 
1773
 
class MergeIntoMerger(Merger):
1774
 
    """Merger that understands other_tree will be merged into a subdir.
1775
 
 
1776
 
    This also changes the Merger api so that it uses real Branch, revision_id,
1777
 
    and RevisonTree objects, rather than using revision specs.
1778
 
    """
1779
 
 
1780
 
    def __init__(self, this_tree, other_branch, other_tree, target_subdir,
1781
 
            source_subpath, other_rev_id=None):
1782
 
        """Create a new MergeIntoMerger object.
1783
 
 
1784
 
        source_subpath in other_tree will be effectively copied to
1785
 
        target_subdir in this_tree.
1786
 
 
1787
 
        :param this_tree: The tree that we will be merging into.
1788
 
        :param other_branch: The Branch we will be merging from.
1789
 
        :param other_tree: The RevisionTree object we want to merge.
1790
 
        :param target_subdir: The relative path where we want to merge
1791
 
            other_tree into this_tree
1792
 
        :param source_subpath: The relative path specifying the subtree of
1793
 
            other_tree to merge into this_tree.
1794
 
        """
1795
 
        # It is assumed that we are merging a tree that is not in our current
1796
 
        # ancestry, which means we are using the "EmptyTree" as our basis.
1797
 
        null_ancestor_tree = this_tree.branch.repository.revision_tree(
1798
 
                                _mod_revision.NULL_REVISION)
1799
 
        super(MergeIntoMerger, self).__init__(
1800
 
            this_branch=this_tree.branch,
1801
 
            this_tree=this_tree,
1802
 
            other_tree=other_tree,
1803
 
            base_tree=null_ancestor_tree,
1804
 
            )
1805
 
        self._target_subdir = target_subdir
1806
 
        self._source_subpath = source_subpath
1807
 
        self.other_branch = other_branch
1808
 
        if other_rev_id is None:
1809
 
            other_rev_id = other_tree.get_revision_id()
1810
 
        self.other_rev_id = self.other_basis = other_rev_id
1811
 
        self.base_is_ancestor = True
1812
 
        self.backup_files = True
1813
 
        self.merge_type = Merge3Merger
1814
 
        self.show_base = False
1815
 
        self.reprocess = False
1816
 
        self.interesting_ids = None
1817
 
        self.merge_type = _MergeTypeParameterizer(MergeIntoMergeType,
1818
 
              target_subdir=self._target_subdir,
1819
 
              source_subpath=self._source_subpath)
1820
 
        if self._source_subpath != '':
1821
 
            # If this isn't a partial merge make sure the revisions will be
1822
 
            # present.
1823
 
            self._maybe_fetch(self.other_branch, self.this_branch,
1824
 
                self.other_basis)
1825
 
 
1826
 
    def set_pending(self):
1827
 
        if self._source_subpath != '':
1828
 
            return
1829
 
        Merger.set_pending(self)
1830
 
 
1831
 
 
1832
 
class _MergeTypeParameterizer(object):
1833
 
    """Wrap a merge-type class to provide extra parameters.
1834
 
    
1835
 
    This is hack used by MergeIntoMerger to pass some extra parameters to its
1836
 
    merge_type.  Merger.do_merge() sets up its own set of parameters to pass to
1837
 
    the 'merge_type' member.  It is difficult override do_merge without
1838
 
    re-writing the whole thing, so instead we create a wrapper which will pass
1839
 
    the extra parameters.
1840
 
    """
1841
 
 
1842
 
    def __init__(self, merge_type, **kwargs):
1843
 
        self._extra_kwargs = kwargs
1844
 
        self._merge_type = merge_type
1845
 
 
1846
 
    def __call__(self, *args, **kwargs):
1847
 
        kwargs.update(self._extra_kwargs)
1848
 
        return self._merge_type(*args, **kwargs)
1849
 
 
1850
 
    def __getattr__(self, name):
1851
 
        return getattr(self._merge_type, name)
1852
 
 
1853
 
 
1854
 
class MergeIntoMergeType(Merge3Merger):
1855
 
    """Merger that incorporates a tree (or part of a tree) into another."""
1856
 
 
1857
 
    def __init__(self, *args, **kwargs):
1858
 
        """Initialize the merger object.
1859
 
 
1860
 
        :param args: See Merge3Merger.__init__'s args.
1861
 
        :param kwargs: See Merge3Merger.__init__'s keyword args, except for
1862
 
            source_subpath and target_subdir.
1863
 
        :keyword source_subpath: The relative path specifying the subtree of
1864
 
            other_tree to merge into this_tree.
1865
 
        :keyword target_subdir: The relative path where we want to merge
1866
 
            other_tree into this_tree
1867
 
        """
1868
 
        # All of the interesting work happens during Merge3Merger.__init__(),
1869
 
        # so we have have to hack in to get our extra parameters set.
1870
 
        self._source_subpath = kwargs.pop('source_subpath')
1871
 
        self._target_subdir = kwargs.pop('target_subdir')
1872
 
        super(MergeIntoMergeType, self).__init__(*args, **kwargs)
1873
 
 
1874
 
    def _compute_transform(self):
1875
 
        child_pb = ui.ui_factory.nested_progress_bar()
1876
 
        try:
1877
 
            entries = self._entries_to_incorporate()
1878
 
            entries = list(entries)
1879
 
            for num, (entry, parent_id) in enumerate(entries):
1880
 
                child_pb.update('Preparing file merge', num, len(entries))
1881
 
                parent_trans_id = self.tt.trans_id_file_id(parent_id)
1882
 
                trans_id = transform.new_by_entry(self.tt, entry,
1883
 
                    parent_trans_id, self.other_tree)
1884
 
        finally:
1885
 
            child_pb.finished()
1886
 
        self._finish_computing_transform()
1887
 
 
1888
 
    def _entries_to_incorporate(self):
1889
 
        """Yields pairs of (inventory_entry, new_parent)."""
1890
 
        other_inv = self.other_tree.inventory
1891
 
        subdir_id = other_inv.path2id(self._source_subpath)
1892
 
        if subdir_id is None:
1893
 
            # XXX: The error would be clearer if it gave the URL of the source
1894
 
            # branch, but we don't have a reference to that here.
1895
 
            raise PathNotInTree(self._source_subpath, "Source tree")
1896
 
        subdir = other_inv[subdir_id]
1897
 
        parent_in_target = osutils.dirname(self._target_subdir)
1898
 
        target_id = self.this_tree.inventory.path2id(parent_in_target)
1899
 
        if target_id is None:
1900
 
            raise PathNotInTree(self._target_subdir, "Target tree")
1901
 
        name_in_target = osutils.basename(self._target_subdir)
1902
 
        merge_into_root = subdir.copy()
1903
 
        merge_into_root.name = name_in_target
1904
 
        if merge_into_root.file_id in self.this_tree.inventory:
1905
 
            # Give the root a new file-id.
1906
 
            # This can happen fairly easily if the directory we are
1907
 
            # incorporating is the root, and both trees have 'TREE_ROOT' as
1908
 
            # their root_id.  Users will expect this to Just Work, so we
1909
 
            # change the file-id here.
1910
 
            # Non-root file-ids could potentially conflict too.  That's really
1911
 
            # an edge case, so we don't do anything special for those.  We let
1912
 
            # them cause conflicts.
1913
 
            merge_into_root.file_id = generate_ids.gen_file_id(name_in_target)
1914
 
        yield (merge_into_root, target_id)
1915
 
        if subdir.kind != 'directory':
1916
 
            # No children, so we are done.
1917
 
            return
1918
 
        for ignored_path, entry in other_inv.iter_entries_by_dir(subdir_id):
1919
 
            parent_id = entry.parent_id
1920
 
            if parent_id == subdir.file_id:
1921
 
                # The root's parent ID has changed, so make sure children of
1922
 
                # the root refer to the new ID.
1923
 
                parent_id = merge_into_root.file_id
1924
 
            yield (entry, parent_id)
1925
 
 
1926
 
 
1927
1481
def merge_inner(this_branch, other_tree, base_tree, ignore_zero=False,
1928
1482
                backup_files=False,
1929
1483
                merge_type=Merge3Merger,
1933
1487
                other_rev_id=None,
1934
1488
                interesting_files=None,
1935
1489
                this_tree=None,
1936
 
                pb=None,
 
1490
                pb=DummyProgress(),
1937
1491
                change_reporter=None):
1938
1492
    """Primary interface for merging.
1939
1493
 
1940
 
    Typical use is probably::
1941
 
 
1942
 
        merge_inner(branch, branch.get_revision_tree(other_revision),
1943
 
                    branch.get_revision_tree(base_revision))
1944
 
    """
 
1494
        typical use is probably
 
1495
        'merge_inner(branch, branch.get_revision_tree(other_revision),
 
1496
                     branch.get_revision_tree(base_revision))'
 
1497
        """
1945
1498
    if this_tree is None:
1946
 
        raise errors.BzrError("bzrlib.merge.merge_inner requires a this_tree "
1947
 
                              "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.")
1948
1501
    merger = Merger(this_branch, other_tree, base_tree, this_tree=this_tree,
1949
1502
                    pb=pb, change_reporter=change_reporter)
1950
1503
    merger.backup_files = backup_files
1963
1516
    get_revision_id = getattr(base_tree, 'get_revision_id', None)
1964
1517
    if get_revision_id is None:
1965
1518
        get_revision_id = base_tree.last_revision
1966
 
    merger.cache_trees_with_revision_ids([other_tree, base_tree, this_tree])
1967
1519
    merger.set_base_revision(get_revision_id(), this_branch)
1968
1520
    return merger.do_merge()
1969
1521
 
2168
1720
        super(_PlanMerge, self).__init__(a_rev, b_rev, vf, key_prefix)
2169
1721
        self.a_key = self._key_prefix + (self.a_rev,)
2170
1722
        self.b_key = self._key_prefix + (self.b_rev,)
2171
 
        self.graph = _mod_graph.Graph(self.vf)
 
1723
        self.graph = Graph(self.vf)
2172
1724
        heads = self.graph.heads((self.a_key, self.b_key))
2173
1725
        if len(heads) == 1:
2174
1726
            # one side dominates, so we can just return its values, yay for
2179
1731
                other = b_rev
2180
1732
            else:
2181
1733
                other = a_rev
2182
 
            trace.mutter('found dominating revision for %s\n%s > %s', self.vf,
2183
 
                         self._head_key[-1], other)
 
1734
            mutter('found dominating revision for %s\n%s > %s', self.vf,
 
1735
                   self._head_key[-1], other)
2184
1736
            self._weave = None
2185
1737
        else:
2186
1738
            self._head_key = None
2200
1752
        while True:
2201
1753
            next_lcas = self.graph.find_lca(*cur_ancestors)
2202
1754
            # Map a plain NULL_REVISION to a simple no-ancestors
2203
 
            if next_lcas == set([_mod_revision.NULL_REVISION]):
 
1755
            if next_lcas == set([NULL_REVISION]):
2204
1756
                next_lcas = ()
2205
1757
            # Order the lca's based on when they were merged into the tip
2206
1758
            # While the actual merge portion of weave merge uses a set() of
2218
1770
            elif len(next_lcas) > 2:
2219
1771
                # More than 2 lca's, fall back to grabbing all nodes between
2220
1772
                # this and the unique lca.
2221
 
                trace.mutter('More than 2 LCAs, falling back to all nodes for:'
2222
 
                             ' %s, %s\n=> %s',
2223
 
                             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)
2224
1775
                cur_lcas = next_lcas
2225
1776
                while len(cur_lcas) > 1:
2226
1777
                    cur_lcas = self.graph.find_lca(*cur_lcas)
2229
1780
                    unique_lca = None
2230
1781
                else:
2231
1782
                    unique_lca = list(cur_lcas)[0]
2232
 
                    if unique_lca == _mod_revision.NULL_REVISION:
 
1783
                    if unique_lca == NULL_REVISION:
2233
1784
                        # find_lca will return a plain 'NULL_REVISION' rather
2234
1785
                        # than a key tuple when there is no common ancestor, we
2235
1786
                        # prefer to just use None, because it doesn't confuse
2258
1809
            # We remove NULL_REVISION because it isn't a proper tuple key, and
2259
1810
            # thus confuses things like _get_interesting_texts, and our logic
2260
1811
            # to add the texts into the memory weave.
2261
 
            if _mod_revision.NULL_REVISION in parent_map:
2262
 
                parent_map.pop(_mod_revision.NULL_REVISION)
 
1812
            if NULL_REVISION in parent_map:
 
1813
                parent_map.pop(NULL_REVISION)
2263
1814
        else:
2264
1815
            interesting = set()
2265
1816
            for tip in tip_keys:
2417
1968
        lcas = graph.find_lca(key_prefix + (a_rev,), key_prefix + (b_rev,))
2418
1969
        self.lcas = set()
2419
1970
        for lca in lcas:
2420
 
            if lca == _mod_revision.NULL_REVISION:
 
1971
            if lca == NULL_REVISION:
2421
1972
                self.lcas.add(lca)
2422
1973
            else:
2423
1974
                self.lcas.add(lca[-1])