~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/merge.py

  • Committer: Tarmac
  • Author(s): Vincent Ladeuil
  • Date: 2017-01-30 14:42:05 UTC
  • mfrom: (6620.1.1 trunk)
  • Revision ID: tarmac-20170130144205-r8fh2xpmiuxyozpv
Merge  2.7 into trunk including fix for bug #1657238 [r=vila]

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2005, 2006, 2008 Canonical Ltd
 
1
# Copyright (C) 2005-2011 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
12
12
#
13
13
# You should have received a copy of the GNU General Public License
14
14
# along with this program; if not, write to the Free Software
15
 
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
16
 
 
17
 
 
18
 
import errno
19
 
from itertools import chain
20
 
import os
 
15
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
 
16
 
 
17
from __future__ import absolute_import
 
18
 
21
19
import warnings
22
20
 
 
21
from bzrlib.lazy_import import lazy_import
 
22
lazy_import(globals(), """
23
23
from bzrlib import (
 
24
    branch as _mod_branch,
 
25
    cleanup,
 
26
    conflicts as _mod_conflicts,
24
27
    debug,
25
 
    errors,
 
28
    generate_ids,
26
29
    graph as _mod_graph,
 
30
    merge3,
27
31
    osutils,
28
32
    patiencediff,
29
 
    registry,
30
33
    revision as _mod_revision,
 
34
    textfile,
 
35
    trace,
 
36
    transform,
31
37
    tree as _mod_tree,
32
38
    tsort,
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
 
 
 
39
    ui,
 
40
    versionedfile,
 
41
    workingtree,
 
42
    )
 
43
from bzrlib.i18n import gettext
 
44
""")
 
45
from bzrlib import (
 
46
    decorators,
 
47
    errors,
 
48
    hooks,
 
49
    registry,
 
50
    )
 
51
from bzrlib.symbol_versioning import (
 
52
    deprecated_in,
 
53
    deprecated_method,
 
54
    )
63
55
# TODO: Report back as changes are merged in
64
56
 
65
57
 
66
58
def transform_tree(from_tree, to_tree, interesting_ids=None):
67
 
    merge_inner(from_tree.branch, to_tree, from_tree, ignore_zero=True,
68
 
                interesting_ids=interesting_ids, this_tree=from_tree)
 
59
    from_tree.lock_tree_write()
 
60
    operation = cleanup.OperationWithCleanups(merge_inner)
 
61
    operation.add_cleanup(from_tree.unlock)
 
62
    operation.run_simple(from_tree.branch, to_tree, from_tree,
 
63
        ignore_zero=True, interesting_ids=interesting_ids, this_tree=from_tree)
 
64
 
 
65
 
 
66
class MergeHooks(hooks.Hooks):
 
67
 
 
68
    def __init__(self):
 
69
        hooks.Hooks.__init__(self, "bzrlib.merge", "Merger.hooks")
 
70
        self.add_hook('merge_file_content',
 
71
            "Called with a bzrlib.merge.Merger object to create a per file "
 
72
            "merge object when starting a merge. "
 
73
            "Should return either None or a subclass of "
 
74
            "``bzrlib.merge.AbstractPerFileMerger``. "
 
75
            "Such objects will then be called per file "
 
76
            "that needs to be merged (including when one "
 
77
            "side has deleted the file and the other has changed it). "
 
78
            "See the AbstractPerFileMerger API docs for details on how it is "
 
79
            "used by merge.",
 
80
            (2, 1))
 
81
        self.add_hook('pre_merge',
 
82
            'Called before a merge. '
 
83
            'Receives a Merger object as the single argument.',
 
84
            (2, 5))
 
85
        self.add_hook('post_merge',
 
86
            'Called after a merge. '
 
87
            'Receives a Merger object as the single argument. '
 
88
            'The return value is ignored.',
 
89
            (2, 5))
 
90
 
 
91
 
 
92
class AbstractPerFileMerger(object):
 
93
    """PerFileMerger objects are used by plugins extending merge for bzrlib.
 
94
 
 
95
    See ``bzrlib.plugins.news_merge.news_merge`` for an example concrete class.
 
96
    
 
97
    :ivar merger: The Merge3Merger performing the merge.
 
98
    """
 
99
 
 
100
    def __init__(self, merger):
 
101
        """Create a PerFileMerger for use with merger."""
 
102
        self.merger = merger
 
103
 
 
104
    def merge_contents(self, merge_params):
 
105
        """Attempt to merge the contents of a single file.
 
106
        
 
107
        :param merge_params: A bzrlib.merge.MergeFileHookParams
 
108
        :return: A tuple of (status, chunks), where status is one of
 
109
            'not_applicable', 'success', 'conflicted', or 'delete'.  If status
 
110
            is 'success' or 'conflicted', then chunks should be an iterable of
 
111
            strings for the new file contents.
 
112
        """
 
113
        return ('not applicable', None)
 
114
 
 
115
 
 
116
class PerFileMerger(AbstractPerFileMerger):
 
117
    """Merge individual files when self.file_matches returns True.
 
118
 
 
119
    This class is intended to be subclassed.  The file_matches and
 
120
    merge_matching methods should be overridden with concrete implementations.
 
121
    """
 
122
 
 
123
    def file_matches(self, params):
 
124
        """Return True if merge_matching should be called on this file.
 
125
 
 
126
        Only called with merges of plain files with no clear winner.
 
127
 
 
128
        Subclasses must override this.
 
129
        """
 
130
        raise NotImplementedError(self.file_matches)
 
131
 
 
132
    def get_filename(self, params, tree):
 
133
        """Lookup the filename (i.e. basename, not path), given a Tree (e.g.
 
134
        self.merger.this_tree) and a MergeFileHookParams.
 
135
        """
 
136
        return osutils.basename(tree.id2path(params.file_id))
 
137
 
 
138
    def get_filepath(self, params, tree):
 
139
        """Calculate the path to the file in a tree.
 
140
 
 
141
        :param params: A MergeFileHookParams describing the file to merge
 
142
        :param tree: a Tree, e.g. self.merger.this_tree.
 
143
        """
 
144
        return tree.id2path(params.file_id)
 
145
 
 
146
    def merge_contents(self, params):
 
147
        """Merge the contents of a single file."""
 
148
        # Check whether this custom merge logic should be used.
 
149
        if (
 
150
            # OTHER is a straight winner, rely on default merge.
 
151
            params.winner == 'other' or
 
152
            # THIS and OTHER aren't both files.
 
153
            not params.is_file_merge() or
 
154
            # The filename doesn't match
 
155
            not self.file_matches(params)):
 
156
            return 'not_applicable', None
 
157
        return self.merge_matching(params)
 
158
 
 
159
    def merge_matching(self, params):
 
160
        """Merge the contents of a single file that has matched the criteria
 
161
        in PerFileMerger.merge_contents (is a conflict, is a file,
 
162
        self.file_matches is True).
 
163
 
 
164
        Subclasses must override this.
 
165
        """
 
166
        raise NotImplementedError(self.merge_matching)
 
167
 
 
168
 
 
169
class ConfigurableFileMerger(PerFileMerger):
 
170
    """Merge individual files when configured via a .conf file.
 
171
 
 
172
    This is a base class for concrete custom file merging logic. Concrete
 
173
    classes should implement ``merge_text``.
 
174
 
 
175
    See ``bzrlib.plugins.news_merge.news_merge`` for an example concrete class.
 
176
    
 
177
    :ivar affected_files: The configured file paths to merge.
 
178
 
 
179
    :cvar name_prefix: The prefix to use when looking up configuration
 
180
        details. <name_prefix>_merge_files describes the files targeted by the
 
181
        hook for example.
 
182
        
 
183
    :cvar default_files: The default file paths to merge when no configuration
 
184
        is present.
 
185
    """
 
186
 
 
187
    name_prefix = None
 
188
    default_files = None
 
189
 
 
190
    def __init__(self, merger):
 
191
        super(ConfigurableFileMerger, self).__init__(merger)
 
192
        self.affected_files = None
 
193
        self.default_files = self.__class__.default_files or []
 
194
        self.name_prefix = self.__class__.name_prefix
 
195
        if self.name_prefix is None:
 
196
            raise ValueError("name_prefix must be set.")
 
197
 
 
198
    def file_matches(self, params):
 
199
        """Check whether the file should call the merge hook.
 
200
 
 
201
        <name_prefix>_merge_files configuration variable is a list of files
 
202
        that should use the hook.
 
203
        """
 
204
        affected_files = self.affected_files
 
205
        if affected_files is None:
 
206
            config = self.merger.this_branch.get_config()
 
207
            # Until bzr provides a better policy for caching the config, we
 
208
            # just add the part we're interested in to the params to avoid
 
209
            # reading the config files repeatedly (bazaar.conf, location.conf,
 
210
            # branch.conf).
 
211
            config_key = self.name_prefix + '_merge_files'
 
212
            affected_files = config.get_user_option_as_list(config_key)
 
213
            if affected_files is None:
 
214
                # If nothing was specified in the config, use the default.
 
215
                affected_files = self.default_files
 
216
            self.affected_files = affected_files
 
217
        if affected_files:
 
218
            filepath = self.get_filepath(params, self.merger.this_tree)
 
219
            if filepath in affected_files:
 
220
                return True
 
221
        return False
 
222
 
 
223
    def merge_matching(self, params):
 
224
        return self.merge_text(params)
 
225
 
 
226
    def merge_text(self, params):
 
227
        """Merge the byte contents of a single file.
 
228
 
 
229
        This is called after checking that the merge should be performed in
 
230
        merge_contents, and it should behave as per
 
231
        ``bzrlib.merge.AbstractPerFileMerger.merge_contents``.
 
232
        """
 
233
        raise NotImplementedError(self.merge_text)
 
234
 
 
235
 
 
236
class MergeFileHookParams(object):
 
237
    """Object holding parameters passed to merge_file_content hooks.
 
238
 
 
239
    There are some fields hooks can access:
 
240
 
 
241
    :ivar file_id: the file ID of the file being merged
 
242
    :ivar trans_id: the transform ID for the merge of this file
 
243
    :ivar this_kind: kind of file_id in 'this' tree
 
244
    :ivar other_kind: kind of file_id in 'other' tree
 
245
    :ivar winner: one of 'this', 'other', 'conflict'
 
246
    """
 
247
 
 
248
    def __init__(self, merger, file_id, trans_id, this_kind, other_kind,
 
249
            winner):
 
250
        self._merger = merger
 
251
        self.file_id = file_id
 
252
        self.trans_id = trans_id
 
253
        self.this_kind = this_kind
 
254
        self.other_kind = other_kind
 
255
        self.winner = winner
 
256
 
 
257
    def is_file_merge(self):
 
258
        """True if this_kind and other_kind are both 'file'."""
 
259
        return self.this_kind == 'file' and self.other_kind == 'file'
 
260
 
 
261
    @decorators.cachedproperty
 
262
    def base_lines(self):
 
263
        """The lines of the 'base' version of the file."""
 
264
        return self._merger.get_lines(self._merger.base_tree, self.file_id)
 
265
 
 
266
    @decorators.cachedproperty
 
267
    def this_lines(self):
 
268
        """The lines of the 'this' version of the file."""
 
269
        return self._merger.get_lines(self._merger.this_tree, self.file_id)
 
270
 
 
271
    @decorators.cachedproperty
 
272
    def other_lines(self):
 
273
        """The lines of the 'other' version of the file."""
 
274
        return self._merger.get_lines(self._merger.other_tree, self.file_id)
69
275
 
70
276
 
71
277
class Merger(object):
 
278
 
 
279
    hooks = MergeHooks()
 
280
 
72
281
    def __init__(self, this_branch, other_tree=None, base_tree=None,
73
 
                 this_tree=None, pb=DummyProgress(), change_reporter=None,
 
282
                 this_tree=None, pb=None, change_reporter=None,
74
283
                 recurse='down', revision_graph=None):
75
284
        object.__init__(self)
76
285
        self.this_branch = this_branch
89
298
        self.interesting_files = None
90
299
        self.show_base = False
91
300
        self.reprocess = False
92
 
        self._pb = pb
 
301
        if pb is not None:
 
302
            warnings.warn("pb parameter to Merger() is deprecated and ignored")
93
303
        self.pp = None
94
304
        self.recurse = recurse
95
305
        self.change_reporter = change_reporter
100
310
        self._is_criss_cross = None
101
311
        self._lca_trees = None
102
312
 
 
313
    def cache_trees_with_revision_ids(self, trees):
 
314
        """Cache any tree in trees if it has a revision_id."""
 
315
        for maybe_tree in trees:
 
316
            if maybe_tree is None:
 
317
                continue
 
318
            try:
 
319
                rev_id = maybe_tree.get_revision_id()
 
320
            except AttributeError:
 
321
                continue
 
322
            self._cached_trees[rev_id] = maybe_tree
 
323
 
103
324
    @property
104
325
    def revision_graph(self):
105
326
        if self._revision_graph is None:
132
353
                                      _set_base_is_other_ancestor)
133
354
 
134
355
    @staticmethod
135
 
    def from_uncommitted(tree, other_tree, pb, base_tree=None):
 
356
    def from_uncommitted(tree, other_tree, pb=None, base_tree=None):
136
357
        """Return a Merger for uncommitted changes in other_tree.
137
358
 
138
359
        :param tree: The tree to merge into
167
388
                base_revision_id, tree.branch.last_revision())):
168
389
                base_revision_id = None
169
390
            else:
170
 
                warning('Performing cherrypick')
 
391
                trace.warning('Performing cherrypick')
171
392
        merger = klass.from_revision_ids(pb, tree, other_revision_id,
172
393
                                         base_revision_id, revision_graph=
173
394
                                         revision_graph)
220
441
        return self._cached_trees[revision_id]
221
442
 
222
443
    def _get_tree(self, treespec, possible_transports=None):
223
 
        from bzrlib import workingtree
224
444
        location, revno = treespec
225
445
        if revno is None:
226
446
            tree = workingtree.WorkingTree.open_containing(location)[0]
227
447
            return tree.branch, tree
228
 
        branch = Branch.open_containing(location, possible_transports)[0]
 
448
        branch = _mod_branch.Branch.open_containing(
 
449
            location, possible_transports)[0]
229
450
        if revno == -1:
230
451
            revision_id = branch.last_revision()
231
452
        else:
232
453
            revision_id = branch.get_rev_id(revno)
233
 
        revision_id = ensure_null(revision_id)
 
454
        revision_id = _mod_revision.ensure_null(revision_id)
234
455
        return branch, self.revision_tree(revision_id, branch)
235
456
 
236
 
    def ensure_revision_trees(self):
237
 
        if self.this_revision_tree is None:
238
 
            self.this_basis_tree = self.revision_tree(self.this_basis)
239
 
            if self.this_basis == self.this_rev_id:
240
 
                self.this_revision_tree = self.this_basis_tree
241
 
 
242
 
        if self.other_rev_id is None:
243
 
            other_basis_tree = self.revision_tree(self.other_basis)
244
 
            changes = other_basis_tree.changes_from(self.other_tree)
245
 
            if changes.has_changed():
246
 
                raise WorkingTreeNotRevision(self.this_tree)
247
 
            other_rev_id = self.other_basis
248
 
            self.other_tree = other_basis_tree
249
 
 
250
 
    def file_revisions(self, file_id):
251
 
        self.ensure_revision_trees()
252
 
        def get_id(tree, file_id):
253
 
            revision_id = tree.inventory[file_id].revision
254
 
            return revision_id
255
 
        if self.this_rev_id is None:
256
 
            if self.this_basis_tree.get_file_sha1(file_id) != \
257
 
                self.this_tree.get_file_sha1(file_id):
258
 
                raise WorkingTreeNotRevision(self.this_tree)
259
 
 
260
 
        trees = (self.this_basis_tree, self.other_tree)
261
 
        return [get_id(tree, file_id) for tree in trees]
262
 
 
263
 
    def check_basis(self, check_clean, require_commits=True):
264
 
        if self.this_basis is None and require_commits is True:
265
 
            raise BzrCommandError("This branch has no commits."
266
 
                                  " (perhaps you would prefer 'bzr pull')")
267
 
        if check_clean:
268
 
            self.compare_basis()
269
 
            if self.this_basis != self.this_rev_id:
270
 
                raise errors.UncommittedChanges(self.this_tree)
271
 
 
272
 
    def compare_basis(self):
273
 
        try:
274
 
            basis_tree = self.revision_tree(self.this_tree.last_revision())
275
 
        except errors.NoSuchRevision:
276
 
            basis_tree = self.this_tree.basis_tree()
277
 
        changes = self.this_tree.changes_from(basis_tree)
278
 
        if not changes.has_changed():
279
 
            self.this_rev_id = self.this_basis
280
 
 
281
457
    def set_interesting_files(self, file_list):
282
458
        self.interesting_files = file_list
283
459
 
284
460
    def set_pending(self):
285
 
        if not self.base_is_ancestor or not self.base_is_other_ancestor or self.other_rev_id is None:
 
461
        if (not self.base_is_ancestor or not self.base_is_other_ancestor
 
462
            or self.other_rev_id is None):
286
463
            return
287
464
        self._add_parent()
288
465
 
289
466
    def _add_parent(self):
290
467
        new_parents = self.this_tree.get_parent_ids() + [self.other_rev_id]
291
468
        new_parent_trees = []
 
469
        operation = cleanup.OperationWithCleanups(
 
470
            self.this_tree.set_parent_trees)
292
471
        for revision_id in new_parents:
293
472
            try:
294
473
                tree = self.revision_tree(revision_id)
296
475
                tree = None
297
476
            else:
298
477
                tree.lock_read()
 
478
                operation.add_cleanup(tree.unlock)
299
479
            new_parent_trees.append((revision_id, tree))
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()
 
480
        operation.run_simple(new_parent_trees, allow_leftmost_as_ghost=True)
307
481
 
308
482
    def set_other(self, other_revision, possible_transports=None):
309
483
        """Set the revision and tree to merge from.
318
492
            self.other_rev_id = _mod_revision.ensure_null(
319
493
                self.other_branch.last_revision())
320
494
            if _mod_revision.is_null(self.other_rev_id):
321
 
                raise NoCommits(self.other_branch)
 
495
                raise errors.NoCommits(self.other_branch)
322
496
            self.other_basis = self.other_rev_id
323
497
        elif other_revision[1] is not None:
324
498
            self.other_rev_id = self.other_branch.get_rev_id(other_revision[1])
327
501
            self.other_rev_id = None
328
502
            self.other_basis = self.other_branch.last_revision()
329
503
            if self.other_basis is None:
330
 
                raise NoCommits(self.other_branch)
 
504
                raise errors.NoCommits(self.other_branch)
331
505
        if self.other_rev_id is not None:
332
506
            self._cached_trees[self.other_rev_id] = self.other_tree
333
 
        self._maybe_fetch(self.other_branch,self.this_branch, self.other_basis)
 
507
        self._maybe_fetch(self.other_branch, self.this_branch, self.other_basis)
334
508
 
335
509
    def set_other_revision(self, revision_id, other_branch):
336
510
        """Set 'other' based on a branch and revision id
360
534
            target.fetch(source, revision_id)
361
535
 
362
536
    def find_base(self):
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
 
537
        revisions = [_mod_revision.ensure_null(self.this_basis),
 
538
                     _mod_revision.ensure_null(self.other_basis)]
 
539
        if _mod_revision.NULL_REVISION in revisions:
 
540
            self.base_rev_id = _mod_revision.NULL_REVISION
367
541
            self.base_tree = self.revision_tree(self.base_rev_id)
368
542
            self._is_criss_cross = False
369
543
        else:
370
544
            lcas = self.revision_graph.find_lca(revisions[0], revisions[1])
371
545
            self._is_criss_cross = False
372
546
            if len(lcas) == 0:
373
 
                self.base_rev_id = NULL_REVISION
 
547
                self.base_rev_id = _mod_revision.NULL_REVISION
374
548
            elif len(lcas) == 1:
375
549
                self.base_rev_id = list(lcas)[0]
376
550
            else: # len(lcas) > 1
 
551
                self._is_criss_cross = True
377
552
                if len(lcas) > 2:
378
553
                    # find_unique_lca can only handle 2 nodes, so we have to
379
554
                    # start back at the beginning. It is a shame to traverse
384
559
                else:
385
560
                    self.base_rev_id = self.revision_graph.find_unique_lca(
386
561
                                            *lcas)
387
 
                self._is_criss_cross = True
388
 
            if self.base_rev_id == NULL_REVISION:
389
 
                raise UnrelatedBranches()
 
562
                sorted_lca_keys = self.revision_graph.find_merge_order(
 
563
                    revisions[0], lcas)
 
564
                if self.base_rev_id == _mod_revision.NULL_REVISION:
 
565
                    self.base_rev_id = sorted_lca_keys[0]
 
566
 
 
567
            if self.base_rev_id == _mod_revision.NULL_REVISION:
 
568
                raise errors.UnrelatedBranches()
390
569
            if self._is_criss_cross:
391
 
                warning('Warning: criss-cross merge encountered.  See bzr'
392
 
                        ' help criss-cross.')
393
 
                interesting_revision_ids = [self.base_rev_id]
394
 
                interesting_revision_ids.extend(lcas)
 
570
                trace.warning('Warning: criss-cross merge encountered.  See bzr'
 
571
                              ' help criss-cross.')
 
572
                trace.mutter('Criss-cross lcas: %r' % lcas)
 
573
                if self.base_rev_id in lcas:
 
574
                    trace.mutter('Unable to find unique lca. '
 
575
                                 'Fallback %r as best option.'
 
576
                                 % self.base_rev_id)
 
577
                interesting_revision_ids = set(lcas)
 
578
                interesting_revision_ids.add(self.base_rev_id)
395
579
                interesting_trees = dict((t.get_revision_id(), t)
396
580
                    for t in self.this_branch.repository.revision_trees(
397
581
                        interesting_revision_ids))
398
582
                self._cached_trees.update(interesting_trees)
399
 
                self.base_tree = interesting_trees.pop(self.base_rev_id)
400
 
                sorted_lca_keys = self.revision_graph.find_merge_order(
401
 
                    revisions[0], lcas)
 
583
                if self.base_rev_id in lcas:
 
584
                    self.base_tree = interesting_trees[self.base_rev_id]
 
585
                else:
 
586
                    self.base_tree = interesting_trees.pop(self.base_rev_id)
402
587
                self._lca_trees = [interesting_trees[key]
403
588
                                   for key in sorted_lca_keys]
404
589
            else:
405
590
                self.base_tree = self.revision_tree(self.base_rev_id)
406
591
        self.base_is_ancestor = True
407
592
        self.base_is_other_ancestor = True
 
593
        trace.mutter('Base revid: %r' % self.base_rev_id)
408
594
 
409
595
    def set_base(self, base_revision):
410
596
        """Set the base revision to use for the merge.
411
597
 
412
598
        :param base_revision: A 2-list containing a path and revision number.
413
599
        """
414
 
        mutter("doing merge() with no base_revision specified")
 
600
        trace.mutter("doing merge() with no base_revision specified")
415
601
        if base_revision == [None, None]:
416
602
            self.find_base()
417
603
        else:
426
612
            self._maybe_fetch(base_branch, self.this_branch, self.base_rev_id)
427
613
 
428
614
    def make_merger(self):
429
 
        kwargs = {'working_tree':self.this_tree, 'this_tree': self.this_tree,
 
615
        kwargs = {'working_tree': self.this_tree, 'this_tree': self.this_tree,
430
616
                  'other_tree': self.other_tree,
431
617
                  'interesting_ids': self.interesting_ids,
432
618
                  'interesting_files': self.interesting_files,
433
 
                  'pp': self.pp,
 
619
                  'this_branch': self.this_branch,
 
620
                  'other_branch': self.other_branch,
434
621
                  'do_merge': False}
435
622
        if self.merge_type.requires_base:
436
623
            kwargs['base_tree'] = self.base_tree
437
624
        if self.merge_type.supports_reprocess:
438
625
            kwargs['reprocess'] = self.reprocess
439
626
        elif self.reprocess:
440
 
            raise BzrError("Conflict reduction is not supported for merge"
441
 
                                  " type %s." % self.merge_type)
 
627
            raise errors.BzrError(
 
628
                "Conflict reduction is not supported for merge"
 
629
                " type %s." % self.merge_type)
442
630
        if self.merge_type.supports_show_base:
443
631
            kwargs['show_base'] = self.show_base
444
632
        elif self.show_base:
445
 
            raise BzrError("Showing base is not supported for this"
446
 
                           " merge type. %s" % self.merge_type)
 
633
            raise errors.BzrError("Showing base is not supported for this"
 
634
                                  " merge type. %s" % self.merge_type)
447
635
        if (not getattr(self.merge_type, 'supports_reverse_cherrypick', True)
448
636
            and not self.base_is_other_ancestor):
449
637
            raise errors.CannotReverseCherrypick()
453
641
        if self._is_criss_cross and getattr(self.merge_type,
454
642
                                            'supports_lca_trees', False):
455
643
            kwargs['lca_trees'] = self._lca_trees
456
 
        return self.merge_type(pb=self._pb,
 
644
        return self.merge_type(pb=None,
457
645
                               change_reporter=self.change_reporter,
458
646
                               **kwargs)
459
647
 
460
 
    def _do_merge_to(self, merge):
 
648
    def _do_merge_to(self):
 
649
        merge = self.make_merger()
 
650
        if self.other_branch is not None:
 
651
            self.other_branch.update_references(self.this_branch)
 
652
        for hook in Merger.hooks['pre_merge']:
 
653
            hook(merge)
461
654
        merge.do_merge()
 
655
        for hook in Merger.hooks['post_merge']:
 
656
            hook(merge)
462
657
        if self.recurse == 'down':
463
658
            for relpath, file_id in self.this_tree.iter_references():
464
659
                sub_tree = self.this_tree.get_nested_tree(file_id, relpath)
468
663
                    continue
469
664
                sub_merge = Merger(sub_tree.branch, this_tree=sub_tree)
470
665
                sub_merge.merge_type = self.merge_type
471
 
                other_branch = self.other_branch.reference_parent(file_id, relpath)
 
666
                other_branch = self.other_branch.reference_parent(file_id,
 
667
                                                                  relpath)
472
668
                sub_merge.set_other_revision(other_revision, other_branch)
473
669
                base_revision = self.base_tree.get_reference_revision(file_id)
474
670
                sub_merge.base_tree = \
475
671
                    sub_tree.branch.repository.revision_tree(base_revision)
476
672
                sub_merge.base_rev_id = base_revision
477
673
                sub_merge.do_merge()
478
 
        
 
674
        return merge
 
675
 
479
676
    def do_merge(self):
 
677
        operation = cleanup.OperationWithCleanups(self._do_merge_to)
480
678
        self.this_tree.lock_tree_write()
481
 
        try:
482
 
            if self.base_tree is not None:
483
 
                self.base_tree.lock_read()
484
 
            try:
485
 
                if self.other_tree is not None:
486
 
                    self.other_tree.lock_read()
487
 
                try:
488
 
                    merge = self.make_merger()
489
 
                    self._do_merge_to(merge)
490
 
                finally:
491
 
                    if self.other_tree is not None:
492
 
                        self.other_tree.unlock()
493
 
            finally:
494
 
                if self.base_tree is not None:
495
 
                    self.base_tree.unlock()
496
 
        finally:
497
 
            self.this_tree.unlock()
 
679
        operation.add_cleanup(self.this_tree.unlock)
 
680
        if self.base_tree is not None:
 
681
            self.base_tree.lock_read()
 
682
            operation.add_cleanup(self.base_tree.unlock)
 
683
        if self.other_tree is not None:
 
684
            self.other_tree.lock_read()
 
685
            operation.add_cleanup(self.other_tree.unlock)
 
686
        merge = operation.run_simple()
498
687
        if len(merge.cooked_conflicts) == 0:
499
 
            if not self.ignore_zero and not is_quiet():
500
 
                note("All changes applied successfully.")
 
688
            if not self.ignore_zero and not trace.is_quiet():
 
689
                trace.note(gettext("All changes applied successfully."))
501
690
        else:
502
 
            note("%d conflicts encountered." % len(merge.cooked_conflicts))
 
691
            trace.note(gettext("%d conflicts encountered.")
 
692
                       % len(merge.cooked_conflicts))
503
693
 
504
694
        return len(merge.cooked_conflicts)
505
695
 
532
722
    winner_idx = {"this": 2, "other": 1, "conflict": 1}
533
723
    supports_lca_trees = True
534
724
 
535
 
    def __init__(self, working_tree, this_tree, base_tree, other_tree, 
 
725
    def __init__(self, working_tree, this_tree, base_tree, other_tree,
536
726
                 interesting_ids=None, reprocess=False, show_base=False,
537
 
                 pb=DummyProgress(), pp=None, change_reporter=None,
 
727
                 pb=None, pp=None, change_reporter=None,
538
728
                 interesting_files=None, do_merge=True,
539
 
                 cherrypick=False, lca_trees=None):
 
729
                 cherrypick=False, lca_trees=None, this_branch=None,
 
730
                 other_branch=None):
540
731
        """Initialize the merger object and perform the merge.
541
732
 
542
733
        :param working_tree: The working tree to apply the merge to
543
734
        :param this_tree: The local tree in the merge operation
544
735
        :param base_tree: The common tree in the merge operation
545
 
        :param other_tree: The other other tree to merge changes from
 
736
        :param other_tree: The other tree to merge changes from
 
737
        :param this_branch: The branch associated with this_tree.  Defaults to
 
738
            this_tree.branch if not supplied.
 
739
        :param other_branch: The branch associated with other_tree, if any.
546
740
        :param interesting_ids: The file_ids of files that should be
547
741
            participate in the merge.  May not be combined with
548
742
            interesting_files.
549
743
        :param: reprocess If True, perform conflict-reduction processing.
550
744
        :param show_base: If True, show the base revision in text conflicts.
551
745
            (incompatible with reprocess)
552
 
        :param pb: A Progress bar
 
746
        :param pb: ignored
553
747
        :param pp: A ProgressPhase object
554
748
        :param change_reporter: An object that should report changes made
555
749
        :param interesting_files: The tree-relative paths of files that should
566
760
        if interesting_files is not None and interesting_ids is not None:
567
761
            raise ValueError(
568
762
                'specify either interesting_ids or interesting_files')
 
763
        if this_branch is None:
 
764
            this_branch = this_tree.branch
569
765
        self.interesting_ids = interesting_ids
570
766
        self.interesting_files = interesting_files
571
 
        self.this_tree = working_tree
 
767
        self.working_tree = working_tree
 
768
        self.this_tree = this_tree
572
769
        self.base_tree = base_tree
573
770
        self.other_tree = other_tree
 
771
        self.this_branch = this_branch
 
772
        self.other_branch = other_branch
574
773
        self._raw_conflicts = []
575
774
        self.cooked_conflicts = []
576
775
        self.reprocess = reprocess
581
780
        # making sure we haven't missed any corner cases.
582
781
        # if lca_trees is None:
583
782
        #     self._lca_trees = [self.base_tree]
584
 
        self.pb = pb
585
 
        self.pp = pp
586
783
        self.change_reporter = change_reporter
587
784
        self.cherrypick = cherrypick
588
 
        if self.pp is None:
589
 
            self.pp = ProgressPhase("Merge phase", 3, self.pb)
590
785
        if do_merge:
591
786
            self.do_merge()
 
787
        if pp is not None:
 
788
            warnings.warn("pp argument to Merge3Merger is deprecated")
 
789
        if pb is not None:
 
790
            warnings.warn("pb argument to Merge3Merger is deprecated")
592
791
 
593
792
    def do_merge(self):
594
 
        self.this_tree.lock_tree_write()
 
793
        operation = cleanup.OperationWithCleanups(self._do_merge)
 
794
        self.working_tree.lock_tree_write()
 
795
        operation.add_cleanup(self.working_tree.unlock)
 
796
        self.this_tree.lock_read()
 
797
        operation.add_cleanup(self.this_tree.unlock)
595
798
        self.base_tree.lock_read()
 
799
        operation.add_cleanup(self.base_tree.unlock)
596
800
        self.other_tree.lock_read()
597
 
        self.tt = TreeTransform(self.this_tree, self.pb)
 
801
        operation.add_cleanup(self.other_tree.unlock)
 
802
        operation.run()
 
803
 
 
804
    def _do_merge(self, operation):
 
805
        self.tt = transform.TreeTransform(self.working_tree, None)
 
806
        operation.add_cleanup(self.tt.finalize)
 
807
        self._compute_transform()
 
808
        results = self.tt.apply(no_conflicts=True)
 
809
        self.write_modified(results)
598
810
        try:
599
 
            self.pp.next_phase()
600
 
            self._compute_transform()
601
 
            self.pp.next_phase()
602
 
            results = self.tt.apply(no_conflicts=True)
603
 
            self.write_modified(results)
604
 
            try:
605
 
                self.this_tree.add_conflicts(self.cooked_conflicts)
606
 
            except UnsupportedOperation:
607
 
                pass
608
 
        finally:
609
 
            self.tt.finalize()
610
 
            self.other_tree.unlock()
611
 
            self.base_tree.unlock()
612
 
            self.this_tree.unlock()
613
 
            self.pb.clear()
 
811
            self.working_tree.add_conflicts(self.cooked_conflicts)
 
812
        except errors.UnsupportedOperation:
 
813
            pass
614
814
 
615
815
    def make_preview_transform(self):
 
816
        operation = cleanup.OperationWithCleanups(self._make_preview_transform)
616
817
        self.base_tree.lock_read()
 
818
        operation.add_cleanup(self.base_tree.unlock)
617
819
        self.other_tree.lock_read()
618
 
        self.tt = TransformPreview(self.this_tree)
619
 
        try:
620
 
            self.pp.next_phase()
621
 
            self._compute_transform()
622
 
            self.pp.next_phase()
623
 
        finally:
624
 
            self.other_tree.unlock()
625
 
            self.base_tree.unlock()
626
 
            self.pb.clear()
 
820
        operation.add_cleanup(self.other_tree.unlock)
 
821
        return operation.run_simple()
 
822
 
 
823
    def _make_preview_transform(self):
 
824
        self.tt = transform.TransformPreview(self.working_tree)
 
825
        self._compute_transform()
627
826
        return self.tt
628
827
 
629
828
    def _compute_transform(self):
633
832
        else:
634
833
            entries = self._entries_lca()
635
834
            resolver = self._lca_multi_way
 
835
        # Prepare merge hooks
 
836
        factories = Merger.hooks['merge_file_content']
 
837
        # One hook for each registered one plus our default merger
 
838
        hooks = [factory(self) for factory in factories] + [self]
 
839
        self.active_hooks = [hook for hook in hooks if hook is not None]
636
840
        child_pb = ui.ui_factory.nested_progress_bar()
637
841
        try:
638
842
            for num, (file_id, changed, parents3, names3,
639
843
                      executable3) in enumerate(entries):
640
 
                child_pb.update('Preparing file merge', num, len(entries))
 
844
                # Try merging each entry
 
845
                child_pb.update(gettext('Preparing file merge'),
 
846
                                num, len(entries))
641
847
                self._merge_names(file_id, parents3, names3, resolver=resolver)
642
848
                if changed:
643
 
                    file_status = self.merge_contents(file_id)
 
849
                    file_status = self._do_merge_contents(file_id)
644
850
                else:
645
851
                    file_status = 'unmodified'
646
852
                self._merge_executable(file_id,
647
853
                    executable3, file_status, resolver=resolver)
648
854
        finally:
649
855
            child_pb.finished()
650
 
        self.fix_root()
651
 
        self.pp.next_phase()
 
856
        self.tt.fixup_new_roots()
 
857
        self._finish_computing_transform()
 
858
 
 
859
    def _finish_computing_transform(self):
 
860
        """Finalize the transform and report the changes.
 
861
 
 
862
        This is the second half of _compute_transform.
 
863
        """
652
864
        child_pb = ui.ui_factory.nested_progress_bar()
653
865
        try:
654
 
            fs_conflicts = resolve_conflicts(self.tt, child_pb,
655
 
                lambda t, c: conflict_pass(t, c, self.other_tree))
 
866
            fs_conflicts = transform.resolve_conflicts(self.tt, child_pb,
 
867
                lambda t, c: transform.conflict_pass(t, c, self.other_tree))
656
868
        finally:
657
869
            child_pb.finished()
658
870
        if self.change_reporter is not None:
661
873
                self.tt.iter_changes(), self.change_reporter)
662
874
        self.cook_conflicts(fs_conflicts)
663
875
        for conflict in self.cooked_conflicts:
664
 
            warning(conflict)
 
876
            trace.warning(unicode(conflict))
665
877
 
666
878
    def _entries3(self):
667
879
        """Gather data about files modified between three trees.
674
886
        """
675
887
        result = []
676
888
        iterator = self.other_tree.iter_changes(self.base_tree,
677
 
                include_unchanged=True, specific_files=self.interesting_files,
 
889
                specific_files=self.interesting_files,
678
890
                extra_trees=[self.this_tree])
679
891
        this_entries = dict((e.file_id, e) for p, e in
680
892
                            self.this_tree.iter_entries_by_dir(
706
918
        it then compares with THIS and BASE.
707
919
 
708
920
        For the multi-valued entries, the format will be (BASE, [lca1, lca2])
709
 
        :return: [(file_id, changed, parents, names, executable)]
710
 
            file_id     Simple file_id of the entry
711
 
            changed     Boolean, True if the kind or contents changed
712
 
                        else False
713
 
            parents     ((base, [parent_id, in, lcas]), parent_id_other,
714
 
                         parent_id_this)
715
 
            names       ((base, [name, in, lcas]), name_in_other, name_in_this)
716
 
            executable  ((base, [exec, in, lcas]), exec_in_other, exec_in_this)
 
921
 
 
922
        :return: [(file_id, changed, parents, names, executable)], where:
 
923
 
 
924
            * file_id: Simple file_id of the entry
 
925
            * changed: Boolean, True if the kind or contents changed else False
 
926
            * parents: ((base, [parent_id, in, lcas]), parent_id_other,
 
927
                        parent_id_this)
 
928
            * names:   ((base, [name, in, lcas]), name_in_other, name_in_this)
 
929
            * executable: ((base, [exec, in, lcas]), exec_in_other,
 
930
                        exec_in_this)
717
931
        """
718
932
        if self.interesting_files is not None:
719
933
            lookup_trees = [self.this_tree, self.base_tree]
726
940
        result = []
727
941
        walker = _mod_tree.MultiWalker(self.other_tree, self._lca_trees)
728
942
 
729
 
        base_inventory = self.base_tree.inventory
730
 
        this_inventory = self.this_tree.inventory
 
943
        base_inventory = self.base_tree.root_inventory
 
944
        this_inventory = self.this_tree.root_inventory
731
945
        for path, file_id, other_ie, lca_values in walker.iter_all():
732
946
            # Is this modified at all from any of the other trees?
733
947
            if other_ie is None:
761
975
                else:
762
976
                    lca_entries.append(lca_ie)
763
977
 
764
 
            if file_id in base_inventory:
 
978
            if base_inventory.has_id(file_id):
765
979
                base_ie = base_inventory[file_id]
766
980
            else:
767
981
                base_ie = _none_entry
768
982
 
769
 
            if file_id in this_inventory:
 
983
            if this_inventory.has_id(file_id):
770
984
                this_ie = this_inventory[file_id]
771
985
            else:
772
986
                this_ie = _none_entry
794
1008
            content_changed = True
795
1009
            if kind_winner == 'this':
796
1010
                # No kind change in OTHER, see if there are *any* changes
797
 
                if other_ie.kind == None:
798
 
                    # No content and 'this' wins the kind, so skip this?
799
 
                    # continue
800
 
                    pass
801
 
                elif other_ie.kind == 'directory':
 
1011
                if other_ie.kind == 'directory':
802
1012
                    if parent_id_winner == 'this' and name_winner == 'this':
803
1013
                        # No change for this directory in OTHER, skip
804
1014
                        continue
805
1015
                    content_changed = False
806
 
                elif other_ie.kind == 'file':
 
1016
                elif other_ie.kind is None or other_ie.kind == 'file':
807
1017
                    def get_sha1(ie, tree):
808
1018
                        if ie.kind != 'file':
809
1019
                            return None
856
1066
                        continue
857
1067
                else:
858
1068
                    raise AssertionError('unhandled kind: %s' % other_ie.kind)
859
 
                # XXX: We need to handle kind == 'symlink'
860
1069
 
861
1070
            # If we have gotten this far, that means something has changed
862
1071
            result.append((file_id, content_changed,
869
1078
                          ))
870
1079
        return result
871
1080
 
872
 
 
873
 
    def fix_root(self):
874
 
        try:
875
 
            self.tt.final_kind(self.tt.root)
876
 
        except NoSuchFile:
877
 
            self.tt.cancel_deletion(self.tt.root)
878
 
        if self.tt.final_file_id(self.tt.root) is None:
879
 
            self.tt.version_file(self.tt.tree_file_id(self.tt.root), 
880
 
                                 self.tt.root)
881
 
        other_root_file_id = self.other_tree.get_root_id()
882
 
        if other_root_file_id is None:
883
 
            return
884
 
        other_root = self.tt.trans_id_file_id(other_root_file_id)
885
 
        if other_root == self.tt.root:
886
 
            return
887
 
        try:
888
 
            self.tt.final_kind(other_root)
889
 
        except NoSuchFile:
890
 
            return
891
 
        if self.other_tree.inventory.root.file_id in self.this_tree.inventory:
892
 
            # the other tree's root is a non-root in the current tree
893
 
            return
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():
900
 
            trans_id = self.tt.trans_id_file_id(child.file_id)
901
 
            self.tt.adjust_path(self.tt.final_name(trans_id), target, trans_id)
902
 
 
903
1081
    def write_modified(self, results):
904
1082
        modified_hashes = {}
905
1083
        for path in results.modified_paths:
906
 
            file_id = self.this_tree.path2id(self.this_tree.relpath(path))
 
1084
            file_id = self.working_tree.path2id(self.working_tree.relpath(path))
907
1085
            if file_id is None:
908
1086
                continue
909
 
            hash = self.this_tree.get_file_sha1(file_id)
 
1087
            hash = self.working_tree.get_file_sha1(file_id)
910
1088
            if hash is None:
911
1089
                continue
912
1090
            modified_hashes[file_id] = hash
913
 
        self.this_tree.set_merge_modified(modified_hashes)
 
1091
        self.working_tree.set_merge_modified(modified_hashes)
914
1092
 
915
1093
    @staticmethod
916
1094
    def parent(entry, file_id):
925
1103
        if entry is None:
926
1104
            return None
927
1105
        return entry.name
928
 
    
 
1106
 
929
1107
    @staticmethod
930
1108
    def contents_sha1(tree, file_id):
931
1109
        """Determine the sha1 of the file contents (used as a key method)."""
932
 
        if file_id not in tree:
 
1110
        if not tree.has_id(file_id):
933
1111
            return None
934
1112
        return tree.get_file_sha1(file_id)
935
1113
 
936
1114
    @staticmethod
937
1115
    def executable(tree, file_id):
938
1116
        """Determine the executability of a file-id (used as a key method)."""
939
 
        if file_id not in tree:
 
1117
        if not tree.has_id(file_id):
940
1118
            return None
941
1119
        if tree.kind(file_id) != "file":
942
1120
            return False
945
1123
    @staticmethod
946
1124
    def kind(tree, file_id):
947
1125
        """Determine the kind of a file-id (used as a key method)."""
948
 
        if file_id not in tree:
 
1126
        if not tree.has_id(file_id):
949
1127
            return None
950
1128
        return tree.kind(file_id)
951
1129
 
952
1130
    @staticmethod
953
1131
    def _three_way(base, other, this):
954
 
        #if base == other, either they all agree, or only THIS has changed.
955
1132
        if base == other:
 
1133
            # if 'base == other', either they all agree, or only 'this' has
 
1134
            # changed.
956
1135
            return 'this'
957
1136
        elif this not in (base, other):
 
1137
            # 'this' is neither 'base' nor 'other', so both sides changed
958
1138
            return 'conflict'
959
 
        # "Ambiguous clean merge" -- both sides have made the same change.
960
1139
        elif this == other:
 
1140
            # "Ambiguous clean merge" -- both sides have made the same change.
961
1141
            return "this"
962
 
        # this == base: only other has changed.
963
1142
        else:
 
1143
            # this == base: only other has changed.
964
1144
            return "other"
965
1145
 
966
1146
    @staticmethod
979
1159
        :return: 'this', 'other', or 'conflict' depending on whether an entry
980
1160
            changed or not.
981
1161
        """
982
 
        # See doc/developers/lca_merge_resolution.txt for details about this
 
1162
        # See doc/developers/lca_tree_merging.txt for details about this
983
1163
        # algorithm.
984
1164
        if other == this:
985
1165
            # Either Ambiguously clean, or nothing was actually changed. We
1010
1190
                # only has an lca value
1011
1191
                return 'other'
1012
1192
 
1013
 
        # At this point, the lcas disagree, and the tips disagree
 
1193
        # At this point, the lcas disagree, and the tip disagree
1014
1194
        return 'conflict'
1015
1195
 
1016
 
    @staticmethod
1017
 
    def scalar_three_way(this_tree, base_tree, other_tree, file_id, key):
1018
 
        """Do a three-way test on a scalar.
1019
 
        Return "this", "other" or "conflict", depending whether a value wins.
1020
 
        """
1021
 
        key_base = key(base_tree, file_id)
1022
 
        key_other = key(other_tree, file_id)
1023
 
        #if base == other, either they all agree, or only THIS has changed.
1024
 
        if key_base == key_other:
1025
 
            return "this"
1026
 
        key_this = key(this_tree, file_id)
1027
 
        # "Ambiguous clean merge"
1028
 
        if key_this == key_other:
1029
 
            return "this"
1030
 
        elif key_this == key_base:
1031
 
            return "other"
1032
 
        else:
1033
 
            return "conflict"
1034
 
 
1035
1196
    def merge_names(self, file_id):
1036
1197
        def get_entry(tree):
1037
 
            if file_id in tree.inventory:
1038
 
                return tree.inventory[file_id]
1039
 
            else:
 
1198
            try:
 
1199
                return tree.root_inventory[file_id]
 
1200
            except errors.NoSuchId:
1040
1201
                return None
1041
1202
        this_entry = get_entry(self.this_tree)
1042
1203
        other_entry = get_entry(self.other_tree)
1069
1230
                parent_id_winner = "other"
1070
1231
        if name_winner == "this" and parent_id_winner == "this":
1071
1232
            return
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))
1080
 
        if other_name is None:
1081
 
            # it doesn't matter whether the result was 'other' or 
1082
 
            # 'conflict'-- if there's no 'other', we leave it alone.
 
1233
        if name_winner == 'conflict' or parent_id_winner == 'conflict':
 
1234
            # Creating helpers (.OTHER or .THIS) here cause problems down the
 
1235
            # road if a ContentConflict needs to be created so we should not do
 
1236
            # that
 
1237
            trans_id = self.tt.trans_id_file_id(file_id)
 
1238
            self._raw_conflicts.append(('path conflict', trans_id, file_id,
 
1239
                                        this_parent, this_name,
 
1240
                                        other_parent, other_name))
 
1241
        if not self.other_tree.has_id(file_id):
 
1242
            # it doesn't matter whether the result was 'other' or
 
1243
            # 'conflict'-- if it has no file id, we leave it alone.
1083
1244
            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)
1086
1245
        parent_id = parents[self.winner_idx[parent_id_winner]]
1087
 
        if parent_id is not None:
1088
 
            parent_trans_id = self.tt.trans_id_file_id(parent_id)
1089
 
            self.tt.adjust_path(names[self.winner_idx[name_winner]],
1090
 
                                parent_trans_id, trans_id)
 
1246
        name = names[self.winner_idx[name_winner]]
 
1247
        if parent_id is not None or name is not None:
 
1248
            # if we get here, name_winner and parent_winner are set to safe
 
1249
            # values.
 
1250
            if parent_id is None and name is not None:
 
1251
                # if parent_id is None and name is non-None, current file is
 
1252
                # the tree root.
 
1253
                if names[self.winner_idx[parent_id_winner]] != '':
 
1254
                    raise AssertionError(
 
1255
                        'File looks like a root, but named %s' %
 
1256
                        names[self.winner_idx[parent_id_winner]])
 
1257
                parent_trans_id = transform.ROOT_PARENT
 
1258
            else:
 
1259
                parent_trans_id = self.tt.trans_id_file_id(parent_id)
 
1260
            self.tt.adjust_path(name, parent_trans_id,
 
1261
                                self.tt.trans_id_file_id(file_id))
1091
1262
 
1092
 
    def merge_contents(self, file_id):
1093
 
        """Performa a merge on file_id contents."""
 
1263
    def _do_merge_contents(self, file_id):
 
1264
        """Performs a merge on file_id contents."""
1094
1265
        def contents_pair(tree):
1095
 
            if file_id not in tree:
 
1266
            if not tree.has_id(file_id):
1096
1267
                return (None, None)
1097
1268
            kind = tree.kind(file_id)
1098
1269
            if kind == "file":
1103
1274
                contents = None
1104
1275
            return kind, contents
1105
1276
 
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
 
 
1118
1277
        # See SPOT run.  run, SPOT, run.
1119
1278
        # So we're not QUITE repeating ourselves; we do tricky things with
1120
1279
        # file kind...
1121
1280
        base_pair = contents_pair(self.base_tree)
1122
1281
        other_pair = contents_pair(self.other_tree)
1123
 
        if base_pair == other_pair:
1124
 
            # OTHER introduced no changes
1125
 
            return "unmodified"
1126
 
        this_pair = contents_pair(self.this_tree)
1127
 
        if this_pair == other_pair:
1128
 
            # THIS and OTHER introduced the same changes
1129
 
            return "unmodified"
 
1282
        if self._lca_trees:
 
1283
            this_pair = contents_pair(self.this_tree)
 
1284
            lca_pairs = [contents_pair(tree) for tree in self._lca_trees]
 
1285
            winner = self._lca_multi_way((base_pair, lca_pairs), other_pair,
 
1286
                                         this_pair, allow_overriding_lca=False)
1130
1287
        else:
1131
 
            trans_id = self.tt.trans_id_file_id(file_id)
1132
 
            if this_pair == base_pair:
1133
 
                # only OTHER introduced changes
1134
 
                if file_id in self.this_tree:
1135
 
                    # Remove any existing contents
1136
 
                    self.tt.delete_contents(trans_id)
1137
 
                if file_id in self.other_tree:
1138
 
                    # OTHER changed the file
1139
 
                    create_from_tree(self.tt, trans_id,
1140
 
                                     self.other_tree, file_id)
1141
 
                    if file_id not in self.this_tree:
1142
 
                        self.tt.version_file(file_id, trans_id)
1143
 
                    return "modified"
1144
 
                elif file_id in self.this_tree.inventory:
1145
 
                    # OTHER deleted the file
 
1288
            if base_pair == other_pair:
 
1289
                winner = 'this'
 
1290
            else:
 
1291
                # We delayed evaluating this_pair as long as we can to avoid
 
1292
                # unnecessary sha1 calculation
 
1293
                this_pair = contents_pair(self.this_tree)
 
1294
                winner = self._three_way(base_pair, other_pair, this_pair)
 
1295
        if winner == 'this':
 
1296
            # No interesting changes introduced by OTHER
 
1297
            return "unmodified"
 
1298
        # We have a hypothetical conflict, but if we have files, then we
 
1299
        # can try to merge the content
 
1300
        trans_id = self.tt.trans_id_file_id(file_id)
 
1301
        params = MergeFileHookParams(self, file_id, trans_id, this_pair[0],
 
1302
            other_pair[0], winner)
 
1303
        hooks = self.active_hooks
 
1304
        hook_status = 'not_applicable'
 
1305
        for hook in hooks:
 
1306
            hook_status, lines = hook.merge_contents(params)
 
1307
            if hook_status != 'not_applicable':
 
1308
                # Don't try any more hooks, this one applies.
 
1309
                break
 
1310
        # If the merge ends up replacing the content of the file, we get rid of
 
1311
        # it at the end of this method (this variable is used to track the
 
1312
        # exceptions to this rule).
 
1313
        keep_this = False
 
1314
        result = "modified"
 
1315
        if hook_status == 'not_applicable':
 
1316
            # No merge hook was able to resolve the situation. Two cases exist:
 
1317
            # a content conflict or a duplicate one.
 
1318
            result = None
 
1319
            name = self.tt.final_name(trans_id)
 
1320
            parent_id = self.tt.final_parent(trans_id)
 
1321
            duplicate = False
 
1322
            inhibit_content_conflict = False
 
1323
            if params.this_kind is None: # file_id is not in THIS
 
1324
                # Is the name used for a different file_id ?
 
1325
                dupe_path = self.other_tree.id2path(file_id)
 
1326
                this_id = self.this_tree.path2id(dupe_path)
 
1327
                if this_id is not None:
 
1328
                    # Two entries for the same path
 
1329
                    keep_this = True
 
1330
                    # versioning the merged file will trigger a duplicate
 
1331
                    # conflict
 
1332
                    self.tt.version_file(file_id, trans_id)
 
1333
                    transform.create_from_tree(
 
1334
                        self.tt, trans_id, self.other_tree, file_id,
 
1335
                        filter_tree_path=self._get_filter_tree_path(file_id))
 
1336
                    inhibit_content_conflict = True
 
1337
            elif params.other_kind is None: # file_id is not in OTHER
 
1338
                # Is the name used for a different file_id ?
 
1339
                dupe_path = self.this_tree.id2path(file_id)
 
1340
                other_id = self.other_tree.path2id(dupe_path)
 
1341
                if other_id is not None:
 
1342
                    # Two entries for the same path again, but here, the other
 
1343
                    # entry will also be merged.  We simply inhibit the
 
1344
                    # 'content' conflict creation because we know OTHER will
 
1345
                    # create (or has already created depending on ordering) an
 
1346
                    # entry at the same path. This will trigger a 'duplicate'
 
1347
                    # conflict later.
 
1348
                    keep_this = True
 
1349
                    inhibit_content_conflict = True
 
1350
            if not inhibit_content_conflict:
 
1351
                if params.this_kind is not None:
1146
1352
                    self.tt.unversion_file(trans_id)
1147
 
                    return "deleted"
1148
 
            #BOTH THIS and OTHER introduced changes; scalar conflict
1149
 
            elif this_pair[0] == "file" and other_pair[0] == "file":
1150
 
                # THIS and OTHER are both files, so text merge.  Either
1151
 
                # BASE is a file, or both converted to files, so at least we
1152
 
                # have agreement that output should be a file.
1153
 
                try:
1154
 
                    self.text_merge(file_id, trans_id)
1155
 
                except BinaryFile:
1156
 
                    return contents_conflict()
1157
 
                if file_id not in self.this_tree:
1158
 
                    self.tt.version_file(file_id, trans_id)
1159
 
                try:
1160
 
                    self.tt.tree_kind(trans_id)
1161
 
                    self.tt.delete_contents(trans_id)
1162
 
                except NoSuchFile:
1163
 
                    pass
1164
 
                return "modified"
1165
 
            else:
1166
 
                # Scalar conflict, can't text merge.  Dump conflicts
1167
 
                return contents_conflict()
 
1353
                # This is a contents conflict, because none of the available
 
1354
                # functions could merge it.
 
1355
                file_group = self._dump_conflicts(name, parent_id, file_id,
 
1356
                                                  set_version=True)
 
1357
                self._raw_conflicts.append(('contents conflict', file_group))
 
1358
        elif hook_status == 'success':
 
1359
            self.tt.create_file(lines, trans_id)
 
1360
        elif hook_status == 'conflicted':
 
1361
            # XXX: perhaps the hook should be able to provide
 
1362
            # the BASE/THIS/OTHER files?
 
1363
            self.tt.create_file(lines, trans_id)
 
1364
            self._raw_conflicts.append(('text conflict', trans_id))
 
1365
            name = self.tt.final_name(trans_id)
 
1366
            parent_id = self.tt.final_parent(trans_id)
 
1367
            self._dump_conflicts(name, parent_id, file_id)
 
1368
        elif hook_status == 'delete':
 
1369
            self.tt.unversion_file(trans_id)
 
1370
            result = "deleted"
 
1371
        elif hook_status == 'done':
 
1372
            # The hook function did whatever it needs to do directly, no
 
1373
            # further action needed here.
 
1374
            pass
 
1375
        else:
 
1376
            raise AssertionError('unknown hook_status: %r' % (hook_status,))
 
1377
        if not self.this_tree.has_id(file_id) and result == "modified":
 
1378
            self.tt.version_file(file_id, trans_id)
 
1379
        if not keep_this:
 
1380
            # The merge has been performed and produced a new content, so the
 
1381
            # old contents should not be retained.
 
1382
            self.tt.delete_contents(trans_id)
 
1383
        return result
 
1384
 
 
1385
    def _default_other_winner_merge(self, merge_hook_params):
 
1386
        """Replace this contents with other."""
 
1387
        file_id = merge_hook_params.file_id
 
1388
        trans_id = merge_hook_params.trans_id
 
1389
        if self.other_tree.has_id(file_id):
 
1390
            # OTHER changed the file
 
1391
            transform.create_from_tree(
 
1392
                self.tt, trans_id, self.other_tree, file_id,
 
1393
                filter_tree_path=self._get_filter_tree_path(file_id))
 
1394
            return 'done', None
 
1395
        elif self.this_tree.has_id(file_id):
 
1396
            # OTHER deleted the file
 
1397
            return 'delete', None
 
1398
        else:
 
1399
            raise AssertionError(
 
1400
                'winner is OTHER, but file_id %r not in THIS or OTHER tree'
 
1401
                % (file_id,))
 
1402
 
 
1403
    def merge_contents(self, merge_hook_params):
 
1404
        """Fallback merge logic after user installed hooks."""
 
1405
        # This function is used in merge hooks as the fallback instance.
 
1406
        # Perhaps making this function and the functions it calls be a 
 
1407
        # a separate class would be better.
 
1408
        if merge_hook_params.winner == 'other':
 
1409
            # OTHER is a straight winner, so replace this contents with other
 
1410
            return self._default_other_winner_merge(merge_hook_params)
 
1411
        elif merge_hook_params.is_file_merge():
 
1412
            # THIS and OTHER are both files, so text merge.  Either
 
1413
            # BASE is a file, or both converted to files, so at least we
 
1414
            # have agreement that output should be a file.
 
1415
            try:
 
1416
                self.text_merge(merge_hook_params.file_id,
 
1417
                    merge_hook_params.trans_id)
 
1418
            except errors.BinaryFile:
 
1419
                return 'not_applicable', None
 
1420
            return 'done', None
 
1421
        else:
 
1422
            return 'not_applicable', None
1168
1423
 
1169
1424
    def get_lines(self, tree, file_id):
1170
1425
        """Return the lines in a file, or an empty list."""
1171
 
        if file_id in tree:
1172
 
            return tree.get_file(file_id).readlines()
 
1426
        if tree.has_id(file_id):
 
1427
            return tree.get_file_lines(file_id)
1173
1428
        else:
1174
1429
            return []
1175
1430
 
1177
1432
        """Perform a three-way text merge on a file_id"""
1178
1433
        # it's possible that we got here with base as a different type.
1179
1434
        # if so, we just want two-way text conflicts.
1180
 
        if file_id in self.base_tree and \
 
1435
        if self.base_tree.has_id(file_id) and \
1181
1436
            self.base_tree.kind(file_id) == "file":
1182
1437
            base_lines = self.get_lines(self.base_tree, file_id)
1183
1438
        else:
1184
1439
            base_lines = []
1185
1440
        other_lines = self.get_lines(self.other_tree, file_id)
1186
1441
        this_lines = self.get_lines(self.this_tree, file_id)
1187
 
        m3 = Merge3(base_lines, this_lines, other_lines,
1188
 
                    is_cherrypick=self.cherrypick)
 
1442
        m3 = merge3.Merge3(base_lines, this_lines, other_lines,
 
1443
                           is_cherrypick=self.cherrypick)
1189
1444
        start_marker = "!START OF MERGE CONFLICT!" + "I HOPE THIS IS UNIQUE"
1190
1445
        if self.show_base is True:
1191
1446
            base_marker = '|' * 7
1194
1449
 
1195
1450
        def iter_merge3(retval):
1196
1451
            retval["text_conflicts"] = False
1197
 
            for line in m3.merge_lines(name_a = "TREE", 
1198
 
                                       name_b = "MERGE-SOURCE", 
 
1452
            for line in m3.merge_lines(name_a = "TREE",
 
1453
                                       name_b = "MERGE-SOURCE",
1199
1454
                                       name_base = "BASE-REVISION",
1200
 
                                       start_marker=start_marker, 
 
1455
                                       start_marker=start_marker,
1201
1456
                                       base_marker=base_marker,
1202
1457
                                       reprocess=self.reprocess):
1203
1458
                if line.startswith(start_marker):
1212
1467
            self._raw_conflicts.append(('text conflict', trans_id))
1213
1468
            name = self.tt.final_name(trans_id)
1214
1469
            parent_id = self.tt.final_parent(trans_id)
1215
 
            file_group = self._dump_conflicts(name, parent_id, file_id, 
 
1470
            file_group = self._dump_conflicts(name, parent_id, file_id,
1216
1471
                                              this_lines, base_lines,
1217
1472
                                              other_lines)
1218
1473
            file_group.append(trans_id)
1219
1474
 
1220
 
    def _dump_conflicts(self, name, parent_id, file_id, this_lines=None, 
 
1475
 
 
1476
    def _get_filter_tree_path(self, file_id):
 
1477
        if self.this_tree.supports_content_filtering():
 
1478
            # We get the path from the working tree if it exists.
 
1479
            # That fails though when OTHER is adding a file, so
 
1480
            # we fall back to the other tree to find the path if
 
1481
            # it doesn't exist locally.
 
1482
            try:
 
1483
                return self.this_tree.id2path(file_id)
 
1484
            except errors.NoSuchId:
 
1485
                return self.other_tree.id2path(file_id)
 
1486
        # Skip the id2path lookup for older formats
 
1487
        return None
 
1488
 
 
1489
    def _dump_conflicts(self, name, parent_id, file_id, this_lines=None,
1221
1490
                        base_lines=None, other_lines=None, set_version=False,
1222
1491
                        no_base=False):
1223
1492
        """Emit conflict files.
1225
1494
        determined automatically.  If set_version is true, the .OTHER, .THIS
1226
1495
        or .BASE (in that order) will be created as versioned files.
1227
1496
        """
1228
 
        data = [('OTHER', self.other_tree, other_lines), 
 
1497
        data = [('OTHER', self.other_tree, other_lines),
1229
1498
                ('THIS', self.this_tree, this_lines)]
1230
1499
        if not no_base:
1231
1500
            data.append(('BASE', self.base_tree, base_lines))
 
1501
 
 
1502
        # We need to use the actual path in the working tree of the file here,
 
1503
        # ignoring the conflict suffixes
 
1504
        wt = self.this_tree
 
1505
        if wt.supports_content_filtering():
 
1506
            try:
 
1507
                filter_tree_path = wt.id2path(file_id)
 
1508
            except errors.NoSuchId:
 
1509
                # file has been deleted
 
1510
                filter_tree_path = None
 
1511
        else:
 
1512
            # Skip the id2path lookup for older formats
 
1513
            filter_tree_path = None
 
1514
 
1232
1515
        versioned = False
1233
1516
        file_group = []
1234
1517
        for suffix, tree, lines in data:
1235
 
            if file_id in tree:
 
1518
            if tree.has_id(file_id):
1236
1519
                trans_id = self._conflict_file(name, parent_id, tree, file_id,
1237
 
                                               suffix, lines)
 
1520
                                               suffix, lines, filter_tree_path)
1238
1521
                file_group.append(trans_id)
1239
1522
                if set_version and not versioned:
1240
1523
                    self.tt.version_file(file_id, trans_id)
1241
1524
                    versioned = True
1242
1525
        return file_group
1243
 
           
 
1526
 
1244
1527
    def _conflict_file(self, name, parent_id, tree, file_id, suffix,
1245
 
                       lines=None):
 
1528
                       lines=None, filter_tree_path=None):
1246
1529
        """Emit a single conflict file."""
1247
1530
        name = name + '.' + suffix
1248
1531
        trans_id = self.tt.create_path(name, parent_id)
1249
 
        create_from_tree(self.tt, trans_id, tree, file_id, lines)
 
1532
        transform.create_from_tree(self.tt, trans_id, tree, file_id, lines,
 
1533
            filter_tree_path)
1250
1534
        return trans_id
1251
1535
 
1252
1536
    def merge_executable(self, file_id, file_status):
1273
1557
        if winner == 'this' and file_status != "modified":
1274
1558
            return
1275
1559
        trans_id = self.tt.trans_id_file_id(file_id)
1276
 
        try:
1277
 
            if self.tt.final_kind(trans_id) != "file":
1278
 
                return
1279
 
        except NoSuchFile:
 
1560
        if self.tt.final_kind(trans_id) != "file":
1280
1561
            return
1281
1562
        if winner == "this":
1282
1563
            executability = this_executable
1283
1564
        else:
1284
 
            if file_id in self.other_tree:
 
1565
            if self.other_tree.has_id(file_id):
1285
1566
                executability = other_executable
1286
 
            elif file_id in self.this_tree:
 
1567
            elif self.this_tree.has_id(file_id):
1287
1568
                executability = this_executable
1288
 
            elif file_id in self.base_tree:
 
1569
            elif self.base_tree_has_id(file_id):
1289
1570
                executability = base_executable
1290
1571
        if executability is not None:
1291
1572
            trans_id = self.tt.trans_id_file_id(file_id)
1293
1574
 
1294
1575
    def cook_conflicts(self, fs_conflicts):
1295
1576
        """Convert all conflicts into a form that doesn't depend on trans_id"""
1296
 
        from conflicts import Conflict
1297
 
        name_conflicts = {}
1298
 
        self.cooked_conflicts.extend(cook_conflicts(fs_conflicts, self.tt))
1299
 
        fp = FinalPaths(self.tt)
 
1577
        content_conflict_file_ids = set()
 
1578
        cooked_conflicts = transform.cook_conflicts(fs_conflicts, self.tt)
 
1579
        fp = transform.FinalPaths(self.tt)
1300
1580
        for conflict in self._raw_conflicts:
1301
1581
            conflict_type = conflict[0]
1302
 
            if conflict_type in ('name conflict', 'parent conflict'):
1303
 
                trans_id = conflict[1]
1304
 
                conflict_args = conflict[2:]
1305
 
                if trans_id not in name_conflicts:
1306
 
                    name_conflicts[trans_id] = {}
1307
 
                unique_add(name_conflicts[trans_id], conflict_type, 
1308
 
                           conflict_args)
1309
 
            if conflict_type == 'contents conflict':
 
1582
            if conflict_type == 'path conflict':
 
1583
                (trans_id, file_id,
 
1584
                this_parent, this_name,
 
1585
                other_parent, other_name) = conflict[1:]
 
1586
                if this_parent is None or this_name is None:
 
1587
                    this_path = '<deleted>'
 
1588
                else:
 
1589
                    parent_path =  fp.get_path(
 
1590
                        self.tt.trans_id_file_id(this_parent))
 
1591
                    this_path = osutils.pathjoin(parent_path, this_name)
 
1592
                if other_parent is None or other_name is None:
 
1593
                    other_path = '<deleted>'
 
1594
                else:
 
1595
                    if other_parent == self.other_tree.get_root_id():
 
1596
                        # The tree transform doesn't know about the other root,
 
1597
                        # so we special case here to avoid a NoFinalPath
 
1598
                        # exception
 
1599
                        parent_path = ''
 
1600
                    else:
 
1601
                        parent_path =  fp.get_path(
 
1602
                            self.tt.trans_id_file_id(other_parent))
 
1603
                    other_path = osutils.pathjoin(parent_path, other_name)
 
1604
                c = _mod_conflicts.Conflict.factory(
 
1605
                    'path conflict', path=this_path,
 
1606
                    conflict_path=other_path,
 
1607
                    file_id=file_id)
 
1608
            elif conflict_type == 'contents conflict':
1310
1609
                for trans_id in conflict[1]:
1311
1610
                    file_id = self.tt.final_file_id(trans_id)
1312
1611
                    if file_id is not None:
 
1612
                        # Ok we found the relevant file-id
1313
1613
                        break
1314
1614
                path = fp.get_path(trans_id)
1315
1615
                for suffix in ('.BASE', '.THIS', '.OTHER'):
1316
1616
                    if path.endswith(suffix):
 
1617
                        # Here is the raw path
1317
1618
                        path = path[:-len(suffix)]
1318
1619
                        break
1319
 
                c = Conflict.factory(conflict_type, path=path, file_id=file_id)
1320
 
                self.cooked_conflicts.append(c)
1321
 
            if conflict_type == 'text conflict':
 
1620
                c = _mod_conflicts.Conflict.factory(conflict_type,
 
1621
                                                    path=path, file_id=file_id)
 
1622
                content_conflict_file_ids.add(file_id)
 
1623
            elif conflict_type == 'text conflict':
1322
1624
                trans_id = conflict[1]
1323
1625
                path = fp.get_path(trans_id)
1324
1626
                file_id = self.tt.final_file_id(trans_id)
1325
 
                c = Conflict.factory(conflict_type, path=path, file_id=file_id)
1326
 
                self.cooked_conflicts.append(c)
 
1627
                c = _mod_conflicts.Conflict.factory(conflict_type,
 
1628
                                                    path=path, file_id=file_id)
 
1629
            else:
 
1630
                raise AssertionError('bad conflict type: %r' % (conflict,))
 
1631
            cooked_conflicts.append(c)
1327
1632
 
1328
 
        for trans_id, conflicts in name_conflicts.iteritems():
1329
 
            try:
1330
 
                this_parent, other_parent = conflicts['parent conflict']
1331
 
                if this_parent == other_parent:
1332
 
                    raise AssertionError()
1333
 
            except KeyError:
1334
 
                this_parent = other_parent = \
1335
 
                    self.tt.final_file_id(self.tt.final_parent(trans_id))
1336
 
            try:
1337
 
                this_name, other_name = conflicts['name conflict']
1338
 
                if this_name == other_name:
1339
 
                    raise AssertionError()
1340
 
            except KeyError:
1341
 
                this_name = other_name = self.tt.final_name(trans_id)
1342
 
            other_path = fp.get_path(trans_id)
1343
 
            if this_parent is not None and this_name is not None:
1344
 
                this_parent_path = \
1345
 
                    fp.get_path(self.tt.trans_id_file_id(this_parent))
1346
 
                this_path = pathjoin(this_parent_path, this_name)
1347
 
            else:
1348
 
                this_path = "<deleted>"
1349
 
            file_id = self.tt.final_file_id(trans_id)
1350
 
            c = Conflict.factory('path conflict', path=this_path,
1351
 
                                 conflict_path=other_path, file_id=file_id)
 
1633
        self.cooked_conflicts = []
 
1634
        # We want to get rid of path conflicts when a corresponding contents
 
1635
        # conflict exists. This can occur when one branch deletes a file while
 
1636
        # the other renames *and* modifies it. In this case, the content
 
1637
        # conflict is enough.
 
1638
        for c in cooked_conflicts:
 
1639
            if (c.typestring == 'path conflict'
 
1640
                and c.file_id in content_conflict_file_ids):
 
1641
                continue
1352
1642
            self.cooked_conflicts.append(c)
1353
 
        self.cooked_conflicts.sort(key=Conflict.sort_key)
 
1643
        self.cooked_conflicts.sort(key=_mod_conflicts.Conflict.sort_key)
1354
1644
 
1355
1645
 
1356
1646
class WeaveMerger(Merge3Merger):
1360
1650
    supports_reverse_cherrypick = False
1361
1651
    history_based = True
1362
1652
 
1363
 
    def _merged_lines(self, file_id):
1364
 
        """Generate the merged lines.
1365
 
        There is no distinction between lines that are meant to contain <<<<<<<
1366
 
        and conflicts.
1367
 
        """
1368
 
        if self.cherrypick:
1369
 
            base = self.base_tree
1370
 
        else:
1371
 
            base = None
1372
 
        plan = self.this_tree.plan_file_merge(file_id, self.other_tree,
 
1653
    def _generate_merge_plan(self, file_id, base):
 
1654
        return self.this_tree.plan_file_merge(file_id, self.other_tree,
1373
1655
                                              base=base)
 
1656
 
 
1657
    def _merged_lines(self, file_id):
 
1658
        """Generate the merged lines.
 
1659
        There is no distinction between lines that are meant to contain <<<<<<<
 
1660
        and conflicts.
 
1661
        """
 
1662
        if self.cherrypick:
 
1663
            base = self.base_tree
 
1664
        else:
 
1665
            base = None
 
1666
        plan = self._generate_merge_plan(file_id, base)
1374
1667
        if 'merge' in debug.debug_flags:
1375
1668
            plan = list(plan)
1376
1669
            trans_id = self.tt.trans_id_file_id(file_id)
1377
1670
            name = self.tt.final_name(trans_id) + '.plan'
1378
 
            contents = ('%10s|%s' % l for l in plan)
 
1671
            contents = ('%11s|%s' % l for l in plan)
1379
1672
            self.tt.new_file(name, self.tt.final_parent(trans_id), contents)
1380
 
        textmerge = PlanWeaveMerge(plan, '<<<<<<< TREE\n',
1381
 
            '>>>>>>> MERGE-SOURCE\n')
1382
 
        return textmerge.merge_lines(self.reprocess)
 
1673
        textmerge = versionedfile.PlanWeaveMerge(plan, '<<<<<<< TREE\n',
 
1674
                                                 '>>>>>>> MERGE-SOURCE\n')
 
1675
        lines, conflicts = textmerge.merge_lines(self.reprocess)
 
1676
        if conflicts:
 
1677
            base_lines = textmerge.base_from_plan()
 
1678
        else:
 
1679
            base_lines = None
 
1680
        return lines, base_lines
1383
1681
 
1384
1682
    def text_merge(self, file_id, trans_id):
1385
1683
        """Perform a (weave) text merge for a given file and file-id.
1386
1684
        If conflicts are encountered, .THIS and .OTHER files will be emitted,
1387
1685
        and a conflict will be noted.
1388
1686
        """
1389
 
        lines, conflicts = self._merged_lines(file_id)
 
1687
        lines, base_lines = self._merged_lines(file_id)
1390
1688
        lines = list(lines)
1391
 
        # Note we're checking whether the OUTPUT is binary in this case, 
 
1689
        # Note we're checking whether the OUTPUT is binary in this case,
1392
1690
        # because we don't want to get into weave merge guts.
1393
 
        check_text_lines(lines)
 
1691
        textfile.check_text_lines(lines)
1394
1692
        self.tt.create_file(lines, trans_id)
1395
 
        if conflicts:
 
1693
        if base_lines is not None:
 
1694
            # Conflict
1396
1695
            self._raw_conflicts.append(('text conflict', trans_id))
1397
1696
            name = self.tt.final_name(trans_id)
1398
1697
            parent_id = self.tt.final_parent(trans_id)
1399
 
            file_group = self._dump_conflicts(name, parent_id, file_id, 
1400
 
                                              no_base=True)
 
1698
            file_group = self._dump_conflicts(name, parent_id, file_id,
 
1699
                                              no_base=False,
 
1700
                                              base_lines=base_lines)
1401
1701
            file_group.append(trans_id)
1402
1702
 
1403
1703
 
1404
1704
class LCAMerger(WeaveMerger):
1405
1705
 
1406
 
    def _merged_lines(self, file_id):
1407
 
        """Generate the merged lines.
1408
 
        There is no distinction between lines that are meant to contain <<<<<<<
1409
 
        and conflicts.
1410
 
        """
1411
 
        if self.cherrypick:
1412
 
            base = self.base_tree
1413
 
        else:
1414
 
            base = None
1415
 
        plan = self.this_tree.plan_file_lca_merge(file_id, self.other_tree,
 
1706
    def _generate_merge_plan(self, file_id, base):
 
1707
        return self.this_tree.plan_file_lca_merge(file_id, self.other_tree,
1416
1708
                                                  base=base)
1417
 
        if 'merge' in debug.debug_flags:
1418
 
            plan = list(plan)
1419
 
            trans_id = self.tt.trans_id_file_id(file_id)
1420
 
            name = self.tt.final_name(trans_id) + '.plan'
1421
 
            contents = ('%10s|%s' % l for l in plan)
1422
 
            self.tt.new_file(name, self.tt.final_parent(trans_id), contents)
1423
 
        textmerge = PlanWeaveMerge(plan, '<<<<<<< TREE\n',
1424
 
            '>>>>>>> MERGE-SOURCE\n')
1425
 
        return textmerge.merge_lines(self.reprocess)
1426
 
 
1427
1709
 
1428
1710
class Diff3Merger(Merge3Merger):
1429
1711
    """Three-way merger using external diff3 for text merging"""
1430
1712
 
1431
1713
    def dump_file(self, temp_dir, name, tree, file_id):
1432
 
        out_path = pathjoin(temp_dir, name)
 
1714
        out_path = osutils.pathjoin(temp_dir, name)
1433
1715
        out_file = open(out_path, "wb")
1434
1716
        try:
1435
1717
            in_file = tree.get_file(file_id)
1447
1729
        import bzrlib.patch
1448
1730
        temp_dir = osutils.mkdtemp(prefix="bzr-")
1449
1731
        try:
1450
 
            new_file = pathjoin(temp_dir, "new")
 
1732
            new_file = osutils.pathjoin(temp_dir, "new")
1451
1733
            this = self.dump_file(temp_dir, "this", self.this_tree, file_id)
1452
1734
            base = self.dump_file(temp_dir, "base", self.base_tree, file_id)
1453
1735
            other = self.dump_file(temp_dir, "other", self.other_tree, file_id)
1454
1736
            status = bzrlib.patch.diff3(new_file, this, base, other)
1455
1737
            if status not in (0, 1):
1456
 
                raise BzrError("Unhandled diff3 exit code")
 
1738
                raise errors.BzrError("Unhandled diff3 exit code")
1457
1739
            f = open(new_file, 'rb')
1458
1740
            try:
1459
1741
                self.tt.create_file(f, trans_id)
1468
1750
            osutils.rmtree(temp_dir)
1469
1751
 
1470
1752
 
 
1753
class PathNotInTree(errors.BzrError):
 
1754
 
 
1755
    _fmt = """Merge-into failed because %(tree)s does not contain %(path)s."""
 
1756
 
 
1757
    def __init__(self, path, tree):
 
1758
        errors.BzrError.__init__(self, path=path, tree=tree)
 
1759
 
 
1760
 
 
1761
class MergeIntoMerger(Merger):
 
1762
    """Merger that understands other_tree will be merged into a subdir.
 
1763
 
 
1764
    This also changes the Merger api so that it uses real Branch, revision_id,
 
1765
    and RevisonTree objects, rather than using revision specs.
 
1766
    """
 
1767
 
 
1768
    def __init__(self, this_tree, other_branch, other_tree, target_subdir,
 
1769
            source_subpath, other_rev_id=None):
 
1770
        """Create a new MergeIntoMerger object.
 
1771
 
 
1772
        source_subpath in other_tree will be effectively copied to
 
1773
        target_subdir in this_tree.
 
1774
 
 
1775
        :param this_tree: The tree that we will be merging into.
 
1776
        :param other_branch: The Branch we will be merging from.
 
1777
        :param other_tree: The RevisionTree object we want to merge.
 
1778
        :param target_subdir: The relative path where we want to merge
 
1779
            other_tree into this_tree
 
1780
        :param source_subpath: The relative path specifying the subtree of
 
1781
            other_tree to merge into this_tree.
 
1782
        """
 
1783
        # It is assumed that we are merging a tree that is not in our current
 
1784
        # ancestry, which means we are using the "EmptyTree" as our basis.
 
1785
        null_ancestor_tree = this_tree.branch.repository.revision_tree(
 
1786
                                _mod_revision.NULL_REVISION)
 
1787
        super(MergeIntoMerger, self).__init__(
 
1788
            this_branch=this_tree.branch,
 
1789
            this_tree=this_tree,
 
1790
            other_tree=other_tree,
 
1791
            base_tree=null_ancestor_tree,
 
1792
            )
 
1793
        self._target_subdir = target_subdir
 
1794
        self._source_subpath = source_subpath
 
1795
        self.other_branch = other_branch
 
1796
        if other_rev_id is None:
 
1797
            other_rev_id = other_tree.get_revision_id()
 
1798
        self.other_rev_id = self.other_basis = other_rev_id
 
1799
        self.base_is_ancestor = True
 
1800
        self.backup_files = True
 
1801
        self.merge_type = Merge3Merger
 
1802
        self.show_base = False
 
1803
        self.reprocess = False
 
1804
        self.interesting_ids = None
 
1805
        self.merge_type = _MergeTypeParameterizer(MergeIntoMergeType,
 
1806
              target_subdir=self._target_subdir,
 
1807
              source_subpath=self._source_subpath)
 
1808
        if self._source_subpath != '':
 
1809
            # If this isn't a partial merge make sure the revisions will be
 
1810
            # present.
 
1811
            self._maybe_fetch(self.other_branch, self.this_branch,
 
1812
                self.other_basis)
 
1813
 
 
1814
    def set_pending(self):
 
1815
        if self._source_subpath != '':
 
1816
            return
 
1817
        Merger.set_pending(self)
 
1818
 
 
1819
 
 
1820
class _MergeTypeParameterizer(object):
 
1821
    """Wrap a merge-type class to provide extra parameters.
 
1822
    
 
1823
    This is hack used by MergeIntoMerger to pass some extra parameters to its
 
1824
    merge_type.  Merger.do_merge() sets up its own set of parameters to pass to
 
1825
    the 'merge_type' member.  It is difficult override do_merge without
 
1826
    re-writing the whole thing, so instead we create a wrapper which will pass
 
1827
    the extra parameters.
 
1828
    """
 
1829
 
 
1830
    def __init__(self, merge_type, **kwargs):
 
1831
        self._extra_kwargs = kwargs
 
1832
        self._merge_type = merge_type
 
1833
 
 
1834
    def __call__(self, *args, **kwargs):
 
1835
        kwargs.update(self._extra_kwargs)
 
1836
        return self._merge_type(*args, **kwargs)
 
1837
 
 
1838
    def __getattr__(self, name):
 
1839
        return getattr(self._merge_type, name)
 
1840
 
 
1841
 
 
1842
class MergeIntoMergeType(Merge3Merger):
 
1843
    """Merger that incorporates a tree (or part of a tree) into another."""
 
1844
 
 
1845
    def __init__(self, *args, **kwargs):
 
1846
        """Initialize the merger object.
 
1847
 
 
1848
        :param args: See Merge3Merger.__init__'s args.
 
1849
        :param kwargs: See Merge3Merger.__init__'s keyword args, except for
 
1850
            source_subpath and target_subdir.
 
1851
        :keyword source_subpath: The relative path specifying the subtree of
 
1852
            other_tree to merge into this_tree.
 
1853
        :keyword target_subdir: The relative path where we want to merge
 
1854
            other_tree into this_tree
 
1855
        """
 
1856
        # All of the interesting work happens during Merge3Merger.__init__(),
 
1857
        # so we have have to hack in to get our extra parameters set.
 
1858
        self._source_subpath = kwargs.pop('source_subpath')
 
1859
        self._target_subdir = kwargs.pop('target_subdir')
 
1860
        super(MergeIntoMergeType, self).__init__(*args, **kwargs)
 
1861
 
 
1862
    def _compute_transform(self):
 
1863
        child_pb = ui.ui_factory.nested_progress_bar()
 
1864
        try:
 
1865
            entries = self._entries_to_incorporate()
 
1866
            entries = list(entries)
 
1867
            for num, (entry, parent_id) in enumerate(entries):
 
1868
                child_pb.update(gettext('Preparing file merge'), num, len(entries))
 
1869
                parent_trans_id = self.tt.trans_id_file_id(parent_id)
 
1870
                trans_id = transform.new_by_entry(self.tt, entry,
 
1871
                    parent_trans_id, self.other_tree)
 
1872
        finally:
 
1873
            child_pb.finished()
 
1874
        self._finish_computing_transform()
 
1875
 
 
1876
    def _entries_to_incorporate(self):
 
1877
        """Yields pairs of (inventory_entry, new_parent)."""
 
1878
        other_inv = self.other_tree.root_inventory
 
1879
        subdir_id = other_inv.path2id(self._source_subpath)
 
1880
        if subdir_id is None:
 
1881
            # XXX: The error would be clearer if it gave the URL of the source
 
1882
            # branch, but we don't have a reference to that here.
 
1883
            raise PathNotInTree(self._source_subpath, "Source tree")
 
1884
        subdir = other_inv[subdir_id]
 
1885
        parent_in_target = osutils.dirname(self._target_subdir)
 
1886
        target_id = self.this_tree.path2id(parent_in_target)
 
1887
        if target_id is None:
 
1888
            raise PathNotInTree(self._target_subdir, "Target tree")
 
1889
        name_in_target = osutils.basename(self._target_subdir)
 
1890
        merge_into_root = subdir.copy()
 
1891
        merge_into_root.name = name_in_target
 
1892
        if self.this_tree.has_id(merge_into_root.file_id):
 
1893
            # Give the root a new file-id.
 
1894
            # This can happen fairly easily if the directory we are
 
1895
            # incorporating is the root, and both trees have 'TREE_ROOT' as
 
1896
            # their root_id.  Users will expect this to Just Work, so we
 
1897
            # change the file-id here.
 
1898
            # Non-root file-ids could potentially conflict too.  That's really
 
1899
            # an edge case, so we don't do anything special for those.  We let
 
1900
            # them cause conflicts.
 
1901
            merge_into_root.file_id = generate_ids.gen_file_id(name_in_target)
 
1902
        yield (merge_into_root, target_id)
 
1903
        if subdir.kind != 'directory':
 
1904
            # No children, so we are done.
 
1905
            return
 
1906
        for ignored_path, entry in other_inv.iter_entries_by_dir(subdir_id):
 
1907
            parent_id = entry.parent_id
 
1908
            if parent_id == subdir.file_id:
 
1909
                # The root's parent ID has changed, so make sure children of
 
1910
                # the root refer to the new ID.
 
1911
                parent_id = merge_into_root.file_id
 
1912
            yield (entry, parent_id)
 
1913
 
 
1914
 
1471
1915
def merge_inner(this_branch, other_tree, base_tree, ignore_zero=False,
1472
1916
                backup_files=False,
1473
1917
                merge_type=Merge3Merger,
1477
1921
                other_rev_id=None,
1478
1922
                interesting_files=None,
1479
1923
                this_tree=None,
1480
 
                pb=DummyProgress(),
 
1924
                pb=None,
1481
1925
                change_reporter=None):
1482
 
    """Primary interface for merging. 
1483
 
 
1484
 
        typical use is probably 
1485
 
        'merge_inner(branch, branch.get_revision_tree(other_revision),
1486
 
                     branch.get_revision_tree(base_revision))'
1487
 
        """
 
1926
    """Primary interface for merging.
 
1927
 
 
1928
    Typical use is probably::
 
1929
 
 
1930
        merge_inner(branch, branch.get_revision_tree(other_revision),
 
1931
                    branch.get_revision_tree(base_revision))
 
1932
    """
1488
1933
    if this_tree is None:
1489
 
        raise BzrError("bzrlib.merge.merge_inner requires a this_tree "
1490
 
            "parameter as of bzrlib version 0.8.")
 
1934
        raise errors.BzrError("bzrlib.merge.merge_inner requires a this_tree "
 
1935
                              "parameter")
1491
1936
    merger = Merger(this_branch, other_tree, base_tree, this_tree=this_tree,
1492
1937
                    pb=pb, change_reporter=change_reporter)
1493
1938
    merger.backup_files = backup_files
1506
1951
    get_revision_id = getattr(base_tree, 'get_revision_id', None)
1507
1952
    if get_revision_id is None:
1508
1953
        get_revision_id = base_tree.last_revision
 
1954
    merger.cache_trees_with_revision_ids([other_tree, base_tree, this_tree])
1509
1955
    merger.set_base_revision(get_revision_id(), this_branch)
1510
1956
    return merger.do_merge()
1511
1957
 
 
1958
 
 
1959
merge_type_registry = registry.Registry()
 
1960
merge_type_registry.register('diff3', Diff3Merger,
 
1961
                             "Merge using external diff3.")
 
1962
merge_type_registry.register('lca', LCAMerger,
 
1963
                             "LCA-newness merge.")
 
1964
merge_type_registry.register('merge3', Merge3Merger,
 
1965
                             "Native diff3-style merge.")
 
1966
merge_type_registry.register('weave', WeaveMerger,
 
1967
                             "Weave-based merge.")
 
1968
 
 
1969
 
1512
1970
def get_merge_type_registry():
1513
 
    """Merge type registry is in bzrlib.option to avoid circular imports.
 
1971
    """Merge type registry was previously in bzrlib.option
1514
1972
 
1515
 
    This method provides a sanctioned way to retrieve it.
 
1973
    This method provides a backwards compatible way to retrieve it.
1516
1974
    """
1517
 
    from bzrlib import option
1518
 
    return option._merge_type_registry
 
1975
    return merge_type_registry
1519
1976
 
1520
1977
 
1521
1978
def _plan_annotate_merge(annotated_a, annotated_b, ancestors_a, ancestors_b):
1579
2036
 
1580
2037
    def get_lines(self, revisions):
1581
2038
        """Get lines for revisions from the backing VersionedFiles.
1582
 
        
 
2039
 
1583
2040
        :raises RevisionNotPresent: on absent texts.
1584
2041
        """
1585
2042
        keys = [(self._key_prefix + (rev,)) for rev in revisions]
1587
2044
        for record in self.vf.get_record_stream(keys, 'unordered', True):
1588
2045
            if record.storage_kind == 'absent':
1589
2046
                raise errors.RevisionNotPresent(record.key, self.vf)
1590
 
            result[record.key[-1]] = osutils.split_lines(
1591
 
                record.get_bytes_as('fulltext'))
 
2047
            result[record.key[-1]] = osutils.chunks_to_lines(
 
2048
                record.get_bytes_as('chunked'))
1592
2049
        return result
1593
2050
 
1594
2051
    def plan_merge(self):
1710
2167
        super(_PlanMerge, self).__init__(a_rev, b_rev, vf, key_prefix)
1711
2168
        self.a_key = self._key_prefix + (self.a_rev,)
1712
2169
        self.b_key = self._key_prefix + (self.b_rev,)
1713
 
        self.graph = Graph(self.vf)
 
2170
        self.graph = _mod_graph.Graph(self.vf)
1714
2171
        heads = self.graph.heads((self.a_key, self.b_key))
1715
2172
        if len(heads) == 1:
1716
2173
            # one side dominates, so we can just return its values, yay for
1721
2178
                other = b_rev
1722
2179
            else:
1723
2180
                other = a_rev
1724
 
            mutter('found dominating revision for %s\n%s > %s', self.vf,
1725
 
                   self._head_key[-1], other)
 
2181
            trace.mutter('found dominating revision for %s\n%s > %s', self.vf,
 
2182
                         self._head_key[-1], other)
1726
2183
            self._weave = None
1727
2184
        else:
1728
2185
            self._head_key = None
1742
2199
        while True:
1743
2200
            next_lcas = self.graph.find_lca(*cur_ancestors)
1744
2201
            # Map a plain NULL_REVISION to a simple no-ancestors
1745
 
            if next_lcas == set([NULL_REVISION]):
 
2202
            if next_lcas == set([_mod_revision.NULL_REVISION]):
1746
2203
                next_lcas = ()
1747
2204
            # Order the lca's based on when they were merged into the tip
1748
2205
            # While the actual merge portion of weave merge uses a set() of
1760
2217
            elif len(next_lcas) > 2:
1761
2218
                # More than 2 lca's, fall back to grabbing all nodes between
1762
2219
                # this and the unique lca.
1763
 
                mutter('More than 2 LCAs, falling back to all nodes for:'
1764
 
                       ' %s, %s\n=> %s', self.a_key, self.b_key, cur_ancestors)
 
2220
                trace.mutter('More than 2 LCAs, falling back to all nodes for:'
 
2221
                             ' %s, %s\n=> %s',
 
2222
                             self.a_key, self.b_key, cur_ancestors)
1765
2223
                cur_lcas = next_lcas
1766
2224
                while len(cur_lcas) > 1:
1767
2225
                    cur_lcas = self.graph.find_lca(*cur_lcas)
1770
2228
                    unique_lca = None
1771
2229
                else:
1772
2230
                    unique_lca = list(cur_lcas)[0]
1773
 
                    if unique_lca == NULL_REVISION:
 
2231
                    if unique_lca == _mod_revision.NULL_REVISION:
1774
2232
                        # find_lca will return a plain 'NULL_REVISION' rather
1775
2233
                        # than a key tuple when there is no common ancestor, we
1776
2234
                        # prefer to just use None, because it doesn't confuse
1784
2242
 
1785
2243
    def _find_unique_parents(self, tip_keys, base_key):
1786
2244
        """Find ancestors of tip that aren't ancestors of base.
1787
 
        
 
2245
 
1788
2246
        :param tip_keys: Nodes that are interesting
1789
2247
        :param base_key: Cull all ancestors of this node
1790
2248
        :return: The parent map for all revisions between tip_keys and
1799
2257
            # We remove NULL_REVISION because it isn't a proper tuple key, and
1800
2258
            # thus confuses things like _get_interesting_texts, and our logic
1801
2259
            # to add the texts into the memory weave.
1802
 
            if NULL_REVISION in parent_map:
1803
 
                parent_map.pop(NULL_REVISION)
 
2260
            if _mod_revision.NULL_REVISION in parent_map:
 
2261
                parent_map.pop(_mod_revision.NULL_REVISION)
1804
2262
        else:
1805
2263
            interesting = set()
1806
2264
            for tip in tip_keys:
1850
2308
    @staticmethod
1851
2309
    def _prune_tails(parent_map, child_map, tails_to_remove):
1852
2310
        """Remove tails from the parent map.
1853
 
        
 
2311
 
1854
2312
        This will remove the supplied revisions until no more children have 0
1855
2313
        parents.
1856
2314
 
1945
2403
class _PlanLCAMerge(_PlanMergeBase):
1946
2404
    """
1947
2405
    This merge algorithm differs from _PlanMerge in that:
 
2406
 
1948
2407
    1. comparisons are done against LCAs only
1949
2408
    2. cases where a contested line is new versus one LCA but old versus
1950
2409
       another are marked as conflicts, by emitting the line as conflicted-a
1958
2417
        lcas = graph.find_lca(key_prefix + (a_rev,), key_prefix + (b_rev,))
1959
2418
        self.lcas = set()
1960
2419
        for lca in lcas:
1961
 
            if lca == NULL_REVISION:
 
2420
            if lca == _mod_revision.NULL_REVISION:
1962
2421
                self.lcas.add(lca)
1963
2422
            else:
1964
2423
                self.lcas.add(lca[-1])
1991
2450
 
1992
2451
        If a line is killed and new, this indicates that the two merge
1993
2452
        revisions contain differing conflict resolutions.
 
2453
 
1994
2454
        :param revision_id: The id of the revision in which the lines are
1995
2455
            unique
1996
2456
        :param unique_line_numbers: The line numbers of unique lines.
1997
 
        :return a tuple of (new_this, killed_other):
 
2457
        :return: a tuple of (new_this, killed_other)
1998
2458
        """
1999
2459
        new = set()
2000
2460
        killed = set()