/brz/remove-bazaar

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

« back to all changes in this revision

Viewing changes to bzrlib/merge.py

  • Committer: Robert Collins
  • Date: 2010-05-06 11:08:10 UTC
  • mto: This revision was merged to the branch mainline in revision 5223.
  • Revision ID: robertc@robertcollins.net-20100506110810-h3j07fh5gmw54s25
Cleaner matcher matching revised unlocking protocol.

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2005-2011 Canonical Ltd
 
1
# Copyright (C) 2005-2010 Canonical Ltd
2
2
#
3
3
# This program is free software; you can redistribute it and/or modify
4
4
# it under the terms of the GNU General Public License as published by
14
14
# along with this program; if not, write to the Free Software
15
15
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
16
16
 
17
 
from __future__ import absolute_import
 
17
import warnings
18
18
 
19
 
from .lazy_import import lazy_import
20
 
lazy_import(globals(), """
21
 
from breezy import (
 
19
from bzrlib import (
22
20
    branch as _mod_branch,
23
 
    cleanup,
24
21
    conflicts as _mod_conflicts,
25
22
    debug,
26
 
    generate_ids,
 
23
    decorators,
 
24
    errors,
27
25
    graph as _mod_graph,
 
26
    hooks,
28
27
    merge3,
29
28
    osutils,
30
29
    patiencediff,
35
34
    tree as _mod_tree,
36
35
    tsort,
37
36
    ui,
38
 
    workingtree,
39
 
    )
40
 
from breezy.bzr import (
41
 
    versionedfile,
42
 
    )
43
 
from breezy.i18n import gettext
44
 
""")
45
 
from . import (
46
 
    decorators,
47
 
    errors,
48
 
    hooks,
49
 
    registry,
50
 
    )
51
 
from .sixish import (
52
 
    viewitems,
 
37
    versionedfile
 
38
    )
 
39
from bzrlib.cleanup import OperationWithCleanups
 
40
from bzrlib.symbol_versioning import (
 
41
    deprecated_in,
 
42
    deprecated_method,
53
43
    )
54
44
# TODO: Report back as changes are merged in
55
45
 
56
46
 
57
 
def transform_tree(from_tree, to_tree, interesting_files=None):
58
 
    with from_tree.lock_tree_write():
59
 
        merge_inner(from_tree.branch, to_tree, from_tree,
60
 
                    ignore_zero=True, this_tree=from_tree,
61
 
                    interesting_files=interesting_files)
 
47
def transform_tree(from_tree, to_tree, interesting_ids=None):
 
48
    from_tree.lock_tree_write()
 
49
    operation = OperationWithCleanups(merge_inner)
 
50
    operation.add_cleanup(from_tree.unlock)
 
51
    operation.run_simple(from_tree.branch, to_tree, from_tree,
 
52
        ignore_zero=True, interesting_ids=interesting_ids, this_tree=from_tree)
62
53
 
63
54
 
64
55
class MergeHooks(hooks.Hooks):
65
56
 
66
57
    def __init__(self):
67
 
        hooks.Hooks.__init__(self, "breezy.merge", "Merger.hooks")
68
 
        self.add_hook('merge_file_content',
69
 
                      "Called with a breezy.merge.Merger object to create a per file "
70
 
                      "merge object when starting a merge. "
71
 
                      "Should return either None or a subclass of "
72
 
                      "``breezy.merge.AbstractPerFileMerger``. "
73
 
                      "Such objects will then be called per file "
74
 
                      "that needs to be merged (including when one "
75
 
                      "side has deleted the file and the other has changed it). "
76
 
                      "See the AbstractPerFileMerger API docs for details on how it is "
77
 
                      "used by merge.",
78
 
                      (2, 1))
79
 
        self.add_hook('pre_merge',
80
 
                      'Called before a merge. '
81
 
                      'Receives a Merger object as the single argument.',
82
 
                      (2, 5))
83
 
        self.add_hook('post_merge',
84
 
                      'Called after a merge. '
85
 
                      'Receives a Merger object as the single argument. '
86
 
                      'The return value is ignored.',
87
 
                      (2, 5))
 
58
        hooks.Hooks.__init__(self)
 
59
        self.create_hook(hooks.HookPoint('merge_file_content',
 
60
            "Called with a bzrlib.merge.Merger object to create a per file "
 
61
            "merge object when starting a merge. "
 
62
            "Should return either None or a subclass of "
 
63
            "``bzrlib.merge.AbstractPerFileMerger``. "
 
64
            "Such objects will then be called per file "
 
65
            "that needs to be merged (including when one "
 
66
            "side has deleted the file and the other has changed it). "
 
67
            "See the AbstractPerFileMerger API docs for details on how it is "
 
68
            "used by merge.",
 
69
            (2, 1), None))
88
70
 
89
71
 
90
72
class AbstractPerFileMerger(object):
91
 
    """PerFileMerger objects are used by plugins extending merge for breezy.
92
 
 
93
 
    See ``breezy.plugins.news_merge.news_merge`` for an example concrete class.
94
 
 
 
73
    """PerFileMerger objects are used by plugins extending merge for bzrlib.
 
74
 
 
75
    See ``bzrlib.plugins.news_merge.news_merge`` for an example concrete class.
 
76
    
95
77
    :ivar merger: The Merge3Merger performing the merge.
96
78
    """
97
79
 
101
83
 
102
84
    def merge_contents(self, merge_params):
103
85
        """Attempt to merge the contents of a single file.
104
 
 
105
 
        :param merge_params: A breezy.merge.MergeFileHookParams
106
 
        :return: A tuple of (status, chunks), where status is one of
 
86
        
 
87
        :param merge_params: A bzrlib.merge.MergeHookParams
 
88
        :return : A tuple of (status, chunks), where status is one of
107
89
            'not_applicable', 'success', 'conflicted', or 'delete'.  If status
108
90
            is 'success' or 'conflicted', then chunks should be an iterable of
109
91
            strings for the new file contents.
127
109
        """
128
110
        raise NotImplementedError(self.file_matches)
129
111
 
 
112
    def get_filename(self, params, tree):
 
113
        """Lookup the filename (i.e. basename, not path), given a Tree (e.g.
 
114
        self.merger.this_tree) and a MergeHookParams.
 
115
        """
 
116
        return osutils.basename(tree.id2path(params.file_id))
 
117
 
 
118
    def get_filepath(self, params, tree):
 
119
        """Calculate the path to the file in a tree.
 
120
 
 
121
        :param params: A MergeHookParams describing the file to merge
 
122
        :param tree: a Tree, e.g. self.merger.this_tree.
 
123
        """
 
124
        return tree.id2path(params.file_id)
 
125
 
130
126
    def merge_contents(self, params):
131
127
        """Merge the contents of a single file."""
132
128
        # Check whether this custom merge logic should be used.
135
131
            params.winner == 'other' or
136
132
            # THIS and OTHER aren't both files.
137
133
            not params.is_file_merge() or
138
 
            # The filename doesn't match
139
 
                not self.file_matches(params)):
 
134
            # The filename doesn't match *.xml
 
135
            not self.file_matches(params)):
140
136
            return 'not_applicable', None
141
137
        return self.merge_matching(params)
142
138
 
156
152
    This is a base class for concrete custom file merging logic. Concrete
157
153
    classes should implement ``merge_text``.
158
154
 
159
 
    See ``breezy.plugins.news_merge.news_merge`` for an example concrete class.
160
 
 
 
155
    See ``bzrlib.plugins.news_merge.news_merge`` for an example concrete class.
 
156
    
161
157
    :ivar affected_files: The configured file paths to merge.
162
158
 
163
159
    :cvar name_prefix: The prefix to use when looking up configuration
164
160
        details. <name_prefix>_merge_files describes the files targeted by the
165
161
        hook for example.
166
 
 
 
162
        
167
163
    :cvar default_files: The default file paths to merge when no configuration
168
164
        is present.
169
165
    """
190
186
            config = self.merger.this_branch.get_config()
191
187
            # Until bzr provides a better policy for caching the config, we
192
188
            # just add the part we're interested in to the params to avoid
193
 
            # reading the config files repeatedly (breezy.conf, location.conf,
 
189
            # reading the config files repeatedly (bazaar.conf, location.conf,
194
190
            # branch.conf).
195
191
            config_key = self.name_prefix + '_merge_files'
196
192
            affected_files = config.get_user_option_as_list(config_key)
199
195
                affected_files = self.default_files
200
196
            self.affected_files = affected_files
201
197
        if affected_files:
202
 
            filepath = params.this_path
 
198
            filepath = self.get_filepath(params, self.merger.this_tree)
203
199
            if filepath in affected_files:
204
200
                return True
205
201
        return False
212
208
 
213
209
        This is called after checking that the merge should be performed in
214
210
        merge_contents, and it should behave as per
215
 
        ``breezy.merge.AbstractPerFileMerger.merge_contents``.
 
211
        ``bzrlib.merge.AbstractPerFileMerger.merge_contents``.
216
212
        """
217
213
        raise NotImplementedError(self.merge_text)
218
214
 
219
215
 
220
 
class MergeFileHookParams(object):
 
216
class MergeHookParams(object):
221
217
    """Object holding parameters passed to merge_file_content hooks.
222
218
 
223
219
    There are some fields hooks can access:
224
220
 
225
221
    :ivar file_id: the file ID of the file being merged
226
 
    :ivar base_path: Path in base tree
227
 
    :ivar other_path: Path in other tree
228
 
    :ivar this_path: Path in this tree
229
222
    :ivar trans_id: the transform ID for the merge of this file
230
223
    :ivar this_kind: kind of file_id in 'this' tree
231
224
    :ivar other_kind: kind of file_id in 'other' tree
232
225
    :ivar winner: one of 'this', 'other', 'conflict'
233
226
    """
234
227
 
235
 
    def __init__(self, merger, file_id, paths, trans_id, this_kind, other_kind,
236
 
                 winner):
 
228
    def __init__(self, merger, file_id, trans_id, this_kind, other_kind,
 
229
            winner):
237
230
        self._merger = merger
238
231
        self.file_id = file_id
239
 
        self.paths = paths
240
 
        self.base_path, self.other_path, self.this_path = paths
241
232
        self.trans_id = trans_id
242
233
        self.this_kind = this_kind
243
234
        self.other_kind = other_kind
250
241
    @decorators.cachedproperty
251
242
    def base_lines(self):
252
243
        """The lines of the 'base' version of the file."""
253
 
        return self._merger.get_lines(self._merger.base_tree, self.base_path)
 
244
        return self._merger.get_lines(self._merger.base_tree, self.file_id)
254
245
 
255
246
    @decorators.cachedproperty
256
247
    def this_lines(self):
257
248
        """The lines of the 'this' version of the file."""
258
 
        return self._merger.get_lines(self._merger.this_tree, self.this_path)
 
249
        return self._merger.get_lines(self._merger.this_tree, self.file_id)
259
250
 
260
251
    @decorators.cachedproperty
261
252
    def other_lines(self):
262
253
        """The lines of the 'other' version of the file."""
263
 
        return self._merger.get_lines(self._merger.other_tree, self.other_path)
 
254
        return self._merger.get_lines(self._merger.other_tree, self.file_id)
264
255
 
265
256
 
266
257
class Merger(object):
268
259
    hooks = MergeHooks()
269
260
 
270
261
    def __init__(self, this_branch, other_tree=None, base_tree=None,
271
 
                 this_tree=None, change_reporter=None,
 
262
                 this_tree=None, pb=None, change_reporter=None,
272
263
                 recurse='down', revision_graph=None):
273
264
        object.__init__(self)
274
265
        self.this_branch = this_branch
283
274
        self.base_tree = base_tree
284
275
        self.ignore_zero = False
285
276
        self.backup_files = False
 
277
        self.interesting_ids = None
286
278
        self.interesting_files = None
287
279
        self.show_base = False
288
280
        self.reprocess = False
 
281
        if pb is not None:
 
282
            warnings.warn("pb parameter to Merger() is deprecated and ignored")
289
283
        self.pp = None
290
284
        self.recurse = recurse
291
285
        self.change_reporter = change_reporter
339
333
                                      _set_base_is_other_ancestor)
340
334
 
341
335
    @staticmethod
342
 
    def from_uncommitted(tree, other_tree, base_tree=None):
 
336
    def from_uncommitted(tree, other_tree, pb=None, base_tree=None):
343
337
        """Return a Merger for uncommitted changes in other_tree.
344
338
 
345
339
        :param tree: The tree to merge into
346
340
        :param other_tree: The tree to get uncommitted changes from
 
341
        :param pb: A progress indicator
347
342
        :param base_tree: The basis to use for the merge.  If unspecified,
348
343
            other_tree.basis_tree() will be used.
349
344
        """
350
345
        if base_tree is None:
351
346
            base_tree = other_tree.basis_tree()
352
 
        merger = Merger(tree.branch, other_tree, base_tree, tree)
 
347
        merger = Merger(tree.branch, other_tree, base_tree, tree, pb)
353
348
        merger.base_rev_id = merger.base_tree.get_revision_id()
354
349
        merger.other_rev_id = None
355
350
        merger.other_basis = merger.base_rev_id
356
351
        return merger
357
352
 
358
353
    @classmethod
359
 
    def from_mergeable(klass, tree, mergeable):
 
354
    def from_mergeable(klass, tree, mergeable, pb):
360
355
        """Return a Merger for a bundle or merge directive.
361
356
 
362
357
        :param tree: The tree to merge changes into
363
358
        :param mergeable: A merge directive or bundle
 
359
        :param pb: A progress indicator
364
360
        """
365
361
        mergeable.install_revisions(tree.branch.repository)
366
362
        base_revision_id, other_revision_id, verified =\
369
365
        if base_revision_id is not None:
370
366
            if (base_revision_id != _mod_revision.NULL_REVISION and
371
367
                revision_graph.is_ancestor(
372
 
                    base_revision_id, tree.branch.last_revision())):
 
368
                base_revision_id, tree.branch.last_revision())):
373
369
                base_revision_id = None
374
370
            else:
375
371
                trace.warning('Performing cherrypick')
376
 
        merger = klass.from_revision_ids(tree, other_revision_id,
377
 
                                         base_revision_id, revision_graph=revision_graph)
 
372
        merger = klass.from_revision_ids(pb, tree, other_revision_id,
 
373
                                         base_revision_id, revision_graph=
 
374
                                         revision_graph)
378
375
        return merger, verified
379
376
 
380
377
    @staticmethod
381
 
    def from_revision_ids(tree, other, base=None, other_branch=None,
 
378
    def from_revision_ids(pb, tree, other, base=None, other_branch=None,
382
379
                          base_branch=None, revision_graph=None,
383
380
                          tree_branch=None):
384
381
        """Return a Merger for revision-ids.
385
382
 
 
383
        :param pb: A progress indicator
386
384
        :param tree: The tree to merge changes into
387
385
        :param other: The revision-id to use as OTHER
388
386
        :param base: The revision-id to use as BASE.  If not specified, will
398
396
        """
399
397
        if tree_branch is None:
400
398
            tree_branch = tree.branch
401
 
        merger = Merger(tree_branch, this_tree=tree,
 
399
        merger = Merger(tree_branch, this_tree=tree, pb=pb,
402
400
                        revision_graph=revision_graph)
403
401
        if other_branch is None:
404
402
            other_branch = tree.branch
423
421
        return self._cached_trees[revision_id]
424
422
 
425
423
    def _get_tree(self, treespec, possible_transports=None):
 
424
        from bzrlib import workingtree
426
425
        location, revno = treespec
427
426
        if revno is None:
428
427
            tree = workingtree.WorkingTree.open_containing(location)[0]
436
435
        revision_id = _mod_revision.ensure_null(revision_id)
437
436
        return branch, self.revision_tree(revision_id, branch)
438
437
 
 
438
    @deprecated_method(deprecated_in((2, 1, 0)))
 
439
    def ensure_revision_trees(self):
 
440
        if self.this_revision_tree is None:
 
441
            self.this_basis_tree = self.revision_tree(self.this_basis)
 
442
            if self.this_basis == self.this_rev_id:
 
443
                self.this_revision_tree = self.this_basis_tree
 
444
 
 
445
        if self.other_rev_id is None:
 
446
            other_basis_tree = self.revision_tree(self.other_basis)
 
447
            if other_basis_tree.has_changes(self.other_tree):
 
448
                raise errors.WorkingTreeNotRevision(self.this_tree)
 
449
            other_rev_id = self.other_basis
 
450
            self.other_tree = other_basis_tree
 
451
 
 
452
    @deprecated_method(deprecated_in((2, 1, 0)))
 
453
    def file_revisions(self, file_id):
 
454
        self.ensure_revision_trees()
 
455
        def get_id(tree, file_id):
 
456
            revision_id = tree.inventory[file_id].revision
 
457
            return revision_id
 
458
        if self.this_rev_id is None:
 
459
            if self.this_basis_tree.get_file_sha1(file_id) != \
 
460
                self.this_tree.get_file_sha1(file_id):
 
461
                raise errors.WorkingTreeNotRevision(self.this_tree)
 
462
 
 
463
        trees = (self.this_basis_tree, self.other_tree)
 
464
        return [get_id(tree, file_id) for tree in trees]
 
465
 
 
466
    @deprecated_method(deprecated_in((2, 1, 0)))
 
467
    def check_basis(self, check_clean, require_commits=True):
 
468
        if self.this_basis is None and require_commits is True:
 
469
            raise errors.BzrCommandError(
 
470
                "This branch has no commits."
 
471
                " (perhaps you would prefer 'bzr pull')")
 
472
        if check_clean:
 
473
            self.compare_basis()
 
474
            if self.this_basis != self.this_rev_id:
 
475
                raise errors.UncommittedChanges(self.this_tree)
 
476
 
 
477
    @deprecated_method(deprecated_in((2, 1, 0)))
 
478
    def compare_basis(self):
 
479
        try:
 
480
            basis_tree = self.revision_tree(self.this_tree.last_revision())
 
481
        except errors.NoSuchRevision:
 
482
            basis_tree = self.this_tree.basis_tree()
 
483
        if not self.this_tree.has_changes(basis_tree):
 
484
            self.this_rev_id = self.this_basis
 
485
 
439
486
    def set_interesting_files(self, file_list):
440
487
        self.interesting_files = file_list
441
488
 
442
489
    def set_pending(self):
443
490
        if (not self.base_is_ancestor or not self.base_is_other_ancestor
444
 
                or self.other_rev_id is None):
 
491
            or self.other_rev_id is None):
445
492
            return
446
493
        self._add_parent()
447
494
 
448
495
    def _add_parent(self):
449
496
        new_parents = self.this_tree.get_parent_ids() + [self.other_rev_id]
450
497
        new_parent_trees = []
451
 
        operation = cleanup.OperationWithCleanups(
452
 
            self.this_tree.set_parent_trees)
 
498
        operation = OperationWithCleanups(self.this_tree.set_parent_trees)
453
499
        for revision_id in new_parents:
454
500
            try:
455
501
                tree = self.revision_tree(revision_id)
486
532
                raise errors.NoCommits(self.other_branch)
487
533
        if self.other_rev_id is not None:
488
534
            self._cached_trees[self.other_rev_id] = self.other_tree
489
 
        self._maybe_fetch(self.other_branch,
490
 
                          self.this_branch, self.other_basis)
 
535
        self._maybe_fetch(self.other_branch,self.this_branch, self.other_basis)
491
536
 
492
537
    def set_other_revision(self, revision_id, other_branch):
493
538
        """Set 'other' based on a branch and revision id
530
575
                self.base_rev_id = _mod_revision.NULL_REVISION
531
576
            elif len(lcas) == 1:
532
577
                self.base_rev_id = list(lcas)[0]
533
 
            else:  # len(lcas) > 1
534
 
                self._is_criss_cross = True
 
578
            else: # len(lcas) > 1
535
579
                if len(lcas) > 2:
536
580
                    # find_unique_lca can only handle 2 nodes, so we have to
537
581
                    # start back at the beginning. It is a shame to traverse
538
582
                    # the graph again, but better than re-implementing
539
583
                    # find_unique_lca.
540
584
                    self.base_rev_id = self.revision_graph.find_unique_lca(
541
 
                        revisions[0], revisions[1])
 
585
                                            revisions[0], revisions[1])
542
586
                else:
543
587
                    self.base_rev_id = self.revision_graph.find_unique_lca(
544
 
                        *lcas)
545
 
                sorted_lca_keys = self.revision_graph.find_merge_order(
546
 
                    revisions[0], lcas)
547
 
                if self.base_rev_id == _mod_revision.NULL_REVISION:
548
 
                    self.base_rev_id = sorted_lca_keys[0]
549
 
 
 
588
                                            *lcas)
 
589
                self._is_criss_cross = True
550
590
            if self.base_rev_id == _mod_revision.NULL_REVISION:
551
591
                raise errors.UnrelatedBranches()
552
592
            if self._is_criss_cross:
553
593
                trace.warning('Warning: criss-cross merge encountered.  See bzr'
554
594
                              ' help criss-cross.')
555
595
                trace.mutter('Criss-cross lcas: %r' % lcas)
556
 
                if self.base_rev_id in lcas:
557
 
                    trace.mutter('Unable to find unique lca. '
558
 
                                 'Fallback %r as best option.'
559
 
                                 % self.base_rev_id)
560
 
                interesting_revision_ids = set(lcas)
561
 
                interesting_revision_ids.add(self.base_rev_id)
 
596
                interesting_revision_ids = [self.base_rev_id]
 
597
                interesting_revision_ids.extend(lcas)
562
598
                interesting_trees = dict((t.get_revision_id(), t)
563
 
                                         for t in self.this_branch.repository.revision_trees(
564
 
                    interesting_revision_ids))
 
599
                    for t in self.this_branch.repository.revision_trees(
 
600
                        interesting_revision_ids))
565
601
                self._cached_trees.update(interesting_trees)
566
 
                if self.base_rev_id in lcas:
567
 
                    self.base_tree = interesting_trees[self.base_rev_id]
568
 
                else:
569
 
                    self.base_tree = interesting_trees.pop(self.base_rev_id)
 
602
                self.base_tree = interesting_trees.pop(self.base_rev_id)
 
603
                sorted_lca_keys = self.revision_graph.find_merge_order(
 
604
                    revisions[0], lcas)
570
605
                self._lca_trees = [interesting_trees[key]
571
606
                                   for key in sorted_lca_keys]
572
607
            else:
595
630
            self._maybe_fetch(base_branch, self.this_branch, self.base_rev_id)
596
631
 
597
632
    def make_merger(self):
598
 
        kwargs = {'working_tree': self.this_tree, 'this_tree': self.this_tree,
 
633
        kwargs = {'working_tree':self.this_tree, 'this_tree': self.this_tree,
599
634
                  'other_tree': self.other_tree,
 
635
                  'interesting_ids': self.interesting_ids,
600
636
                  'interesting_files': self.interesting_files,
601
637
                  'this_branch': self.this_branch,
602
 
                  'other_branch': self.other_branch,
603
638
                  'do_merge': False}
604
639
        if self.merge_type.requires_base:
605
640
            kwargs['base_tree'] = self.base_tree
615
650
            raise errors.BzrError("Showing base is not supported for this"
616
651
                                  " merge type. %s" % self.merge_type)
617
652
        if (not getattr(self.merge_type, 'supports_reverse_cherrypick', True)
618
 
                and not self.base_is_other_ancestor):
 
653
            and not self.base_is_other_ancestor):
619
654
            raise errors.CannotReverseCherrypick()
620
655
        if self.merge_type.supports_cherrypick:
621
656
            kwargs['cherrypick'] = (not self.base_is_ancestor or
623
658
        if self._is_criss_cross and getattr(self.merge_type,
624
659
                                            'supports_lca_trees', False):
625
660
            kwargs['lca_trees'] = self._lca_trees
626
 
        return self.merge_type(change_reporter=self.change_reporter,
 
661
        return self.merge_type(pb=None,
 
662
                               change_reporter=self.change_reporter,
627
663
                               **kwargs)
628
664
 
629
665
    def _do_merge_to(self):
630
666
        merge = self.make_merger()
631
667
        if self.other_branch is not None:
632
668
            self.other_branch.update_references(self.this_branch)
633
 
        for hook in Merger.hooks['pre_merge']:
634
 
            hook(merge)
635
669
        merge.do_merge()
636
 
        for hook in Merger.hooks['post_merge']:
637
 
            hook(merge)
638
670
        if self.recurse == 'down':
639
671
            for relpath, file_id in self.this_tree.iter_references():
640
 
                sub_tree = self.this_tree.get_nested_tree(relpath)
 
672
                sub_tree = self.this_tree.get_nested_tree(file_id, relpath)
641
673
                other_revision = self.other_tree.get_reference_revision(
642
 
                    relpath)
643
 
                if other_revision == sub_tree.last_revision():
 
674
                    file_id, relpath)
 
675
                if  other_revision == sub_tree.last_revision():
644
676
                    continue
645
677
                sub_merge = Merger(sub_tree.branch, this_tree=sub_tree)
646
678
                sub_merge.merge_type = self.merge_type
647
 
                other_branch = self.other_branch.reference_parent(
648
 
                    relpath, file_id)
 
679
                other_branch = self.other_branch.reference_parent(file_id, relpath)
649
680
                sub_merge.set_other_revision(other_revision, other_branch)
650
 
                base_tree_path = _mod_tree.find_previous_path(
651
 
                    self.this_tree, self.base_tree, relpath)
652
 
                base_revision = self.base_tree.get_reference_revision(
653
 
                    base_tree_path)
 
681
                base_revision = self.base_tree.get_reference_revision(file_id)
654
682
                sub_merge.base_tree = \
655
683
                    sub_tree.branch.repository.revision_tree(base_revision)
656
684
                sub_merge.base_rev_id = base_revision
658
686
        return merge
659
687
 
660
688
    def do_merge(self):
661
 
        operation = cleanup.OperationWithCleanups(self._do_merge_to)
 
689
        operation = OperationWithCleanups(self._do_merge_to)
662
690
        self.this_tree.lock_tree_write()
663
691
        operation.add_cleanup(self.this_tree.unlock)
664
692
        if self.base_tree is not None:
670
698
        merge = operation.run_simple()
671
699
        if len(merge.cooked_conflicts) == 0:
672
700
            if not self.ignore_zero and not trace.is_quiet():
673
 
                trace.note(gettext("All changes applied successfully."))
 
701
                trace.note("All changes applied successfully.")
674
702
        else:
675
 
            trace.note(gettext("%d conflicts encountered.")
 
703
            trace.note("%d conflicts encountered."
676
704
                       % len(merge.cooked_conflicts))
677
705
 
678
706
        return len(merge.cooked_conflicts)
692
720
    symlink_target = None
693
721
    text_sha1 = None
694
722
 
695
 
 
696
723
_none_entry = _InventoryNoneEntry()
697
724
 
698
725
 
706
733
    supports_reverse_cherrypick = True
707
734
    winner_idx = {"this": 2, "other": 1, "conflict": 1}
708
735
    supports_lca_trees = True
709
 
    requires_file_merge_plan = False
710
736
 
711
737
    def __init__(self, working_tree, this_tree, base_tree, other_tree,
712
 
                 reprocess=False, show_base=False,
713
 
                 change_reporter=None, interesting_files=None, do_merge=True,
714
 
                 cherrypick=False, lca_trees=None, this_branch=None,
715
 
                 other_branch=None):
 
738
                 interesting_ids=None, reprocess=False, show_base=False,
 
739
                 pb=None, pp=None, change_reporter=None,
 
740
                 interesting_files=None, do_merge=True,
 
741
                 cherrypick=False, lca_trees=None, this_branch=None):
716
742
        """Initialize the merger object and perform the merge.
717
743
 
718
744
        :param working_tree: The working tree to apply the merge to
721
747
        :param other_tree: The other tree to merge changes from
722
748
        :param this_branch: The branch associated with this_tree.  Defaults to
723
749
            this_tree.branch if not supplied.
724
 
        :param other_branch: The branch associated with other_tree, if any.
 
750
        :param interesting_ids: The file_ids of files that should be
 
751
            participate in the merge.  May not be combined with
 
752
            interesting_files.
725
753
        :param: reprocess If True, perform conflict-reduction processing.
726
754
        :param show_base: If True, show the base revision in text conflicts.
727
755
            (incompatible with reprocess)
 
756
        :param pb: ignored
 
757
        :param pp: A ProgressPhase object
728
758
        :param change_reporter: An object that should report changes made
729
759
        :param interesting_files: The tree-relative paths of files that should
730
760
            participate in the merge.  If these paths refer to directories,
731
 
            the contents of those directories will also be included.  If not
732
 
            specified, all files may participate in the
 
761
            the contents of those directories will also be included.  May not
 
762
            be combined with interesting_ids.  If neither interesting_files nor
 
763
            interesting_ids is specified, all files may participate in the
733
764
            merge.
734
765
        :param lca_trees: Can be set to a dictionary of {revision_id:rev_tree}
735
766
            if the ancestry was found to include a criss-cross merge.
736
767
            Otherwise should be None.
737
768
        """
738
769
        object.__init__(self)
 
770
        if interesting_files is not None and interesting_ids is not None:
 
771
            raise ValueError(
 
772
                'specify either interesting_ids or interesting_files')
739
773
        if this_branch is None:
740
774
            this_branch = this_tree.branch
 
775
        self.interesting_ids = interesting_ids
741
776
        self.interesting_files = interesting_files
742
 
        self.working_tree = working_tree
743
 
        self.this_tree = this_tree
 
777
        self.this_tree = working_tree
744
778
        self.base_tree = base_tree
745
779
        self.other_tree = other_tree
746
780
        self.this_branch = this_branch
747
 
        self.other_branch = other_branch
748
781
        self._raw_conflicts = []
749
782
        self.cooked_conflicts = []
750
783
        self.reprocess = reprocess
759
792
        self.cherrypick = cherrypick
760
793
        if do_merge:
761
794
            self.do_merge()
 
795
        if pp is not None:
 
796
            warnings.warn("pp argument to Merge3Merger is deprecated")
 
797
        if pb is not None:
 
798
            warnings.warn("pb argument to Merge3Merger is deprecated")
762
799
 
763
800
    def do_merge(self):
764
 
        operation = cleanup.OperationWithCleanups(self._do_merge)
765
 
        self.working_tree.lock_tree_write()
766
 
        operation.add_cleanup(self.working_tree.unlock)
767
 
        self.this_tree.lock_read()
 
801
        operation = OperationWithCleanups(self._do_merge)
 
802
        self.this_tree.lock_tree_write()
768
803
        operation.add_cleanup(self.this_tree.unlock)
769
804
        self.base_tree.lock_read()
770
805
        operation.add_cleanup(self.base_tree.unlock)
773
808
        operation.run()
774
809
 
775
810
    def _do_merge(self, operation):
776
 
        self.tt = transform.TreeTransform(self.working_tree, None)
 
811
        self.tt = transform.TreeTransform(self.this_tree, None)
777
812
        operation.add_cleanup(self.tt.finalize)
778
813
        self._compute_transform()
779
814
        results = self.tt.apply(no_conflicts=True)
780
815
        self.write_modified(results)
781
816
        try:
782
 
            self.working_tree.add_conflicts(self.cooked_conflicts)
 
817
            self.this_tree.add_conflicts(self.cooked_conflicts)
783
818
        except errors.UnsupportedOperation:
784
819
            pass
785
820
 
786
821
    def make_preview_transform(self):
787
 
        with self.base_tree.lock_read(), self.other_tree.lock_read():
788
 
            self.tt = transform.TransformPreview(self.working_tree)
789
 
            self._compute_transform()
790
 
            return self.tt
 
822
        operation = OperationWithCleanups(self._make_preview_transform)
 
823
        self.base_tree.lock_read()
 
824
        operation.add_cleanup(self.base_tree.unlock)
 
825
        self.other_tree.lock_read()
 
826
        operation.add_cleanup(self.other_tree.unlock)
 
827
        return operation.run_simple()
 
828
 
 
829
    def _make_preview_transform(self):
 
830
        self.tt = transform.TransformPreview(self.this_tree)
 
831
        self._compute_transform()
 
832
        return self.tt
791
833
 
792
834
    def _compute_transform(self):
793
835
        if self._lca_trees is None:
796
838
        else:
797
839
            entries = self._entries_lca()
798
840
            resolver = self._lca_multi_way
799
 
        # Prepare merge hooks
800
 
        factories = Merger.hooks['merge_file_content']
801
 
        # One hook for each registered one plus our default merger
802
 
        hooks = [factory(self) for factory in factories] + [self]
803
 
        self.active_hooks = [hook for hook in hooks if hook is not None]
804
 
        with ui.ui_factory.nested_progress_bar() as child_pb:
805
 
            for num, (file_id, changed, paths3, parents3, names3,
 
841
        child_pb = ui.ui_factory.nested_progress_bar()
 
842
        try:
 
843
            factories = Merger.hooks['merge_file_content']
 
844
            hooks = [factory(self) for factory in factories] + [self]
 
845
            self.active_hooks = [hook for hook in hooks if hook is not None]
 
846
            for num, (file_id, changed, parents3, names3,
806
847
                      executable3) in enumerate(entries):
807
 
                # Try merging each entry
808
 
                child_pb.update(gettext('Preparing file merge'),
809
 
                                num, len(entries))
810
 
                self._merge_names(file_id, paths3, parents3,
811
 
                                  names3, resolver=resolver)
 
848
                child_pb.update('Preparing file merge', num, len(entries))
 
849
                self._merge_names(file_id, parents3, names3, resolver=resolver)
812
850
                if changed:
813
 
                    file_status = self._do_merge_contents(paths3, file_id)
 
851
                    file_status = self._do_merge_contents(file_id)
814
852
                else:
815
853
                    file_status = 'unmodified'
816
 
                self._merge_executable(paths3, file_id, executable3,
817
 
                                       file_status, resolver=resolver)
818
 
        self.tt.fixup_new_roots()
819
 
        self._finish_computing_transform()
820
 
 
821
 
    def _finish_computing_transform(self):
822
 
        """Finalize the transform and report the changes.
823
 
 
824
 
        This is the second half of _compute_transform.
825
 
        """
826
 
        with ui.ui_factory.nested_progress_bar() as child_pb:
 
854
                self._merge_executable(file_id,
 
855
                    executable3, file_status, resolver=resolver)
 
856
        finally:
 
857
            child_pb.finished()
 
858
        self.fix_root()
 
859
        child_pb = ui.ui_factory.nested_progress_bar()
 
860
        try:
827
861
            fs_conflicts = transform.resolve_conflicts(self.tt, child_pb,
828
 
                                                       lambda t, c: transform.conflict_pass(t, c, self.other_tree))
 
862
                lambda t, c: transform.conflict_pass(t, c, self.other_tree))
 
863
        finally:
 
864
            child_pb.finished()
829
865
        if self.change_reporter is not None:
830
 
            from breezy import delta
 
866
            from bzrlib import delta
831
867
            delta.report_changes(
832
868
                self.tt.iter_changes(), self.change_reporter)
833
869
        self.cook_conflicts(fs_conflicts)
834
870
        for conflict in self.cooked_conflicts:
835
 
            trace.warning('%s', conflict.describe())
 
871
            trace.warning(conflict)
836
872
 
837
873
    def _entries3(self):
838
874
        """Gather data about files modified between three trees.
845
881
        """
846
882
        result = []
847
883
        iterator = self.other_tree.iter_changes(self.base_tree,
848
 
                                                specific_files=self.interesting_files,
849
 
                                                extra_trees=[self.this_tree])
850
 
        this_interesting_files = self.this_tree.find_related_paths_across_trees(
851
 
            self.interesting_files, trees=[self.other_tree])
852
 
        this_entries = dict(self.this_tree.iter_entries_by_dir(
853
 
                            specific_files=this_interesting_files))
 
884
                include_unchanged=True, specific_files=self.interesting_files,
 
885
                extra_trees=[self.this_tree])
 
886
        this_entries = dict((e.file_id, e) for p, e in
 
887
                            self.this_tree.iter_entries_by_dir(
 
888
                            self.interesting_ids))
854
889
        for (file_id, paths, changed, versioned, parents, names, kind,
855
890
             executable) in iterator:
856
 
            if paths[0] is not None:
857
 
                this_path = _mod_tree.find_previous_path(
858
 
                    self.base_tree, self.this_tree, paths[0])
859
 
            else:
860
 
                this_path = _mod_tree.find_previous_path(
861
 
                    self.other_tree, self.this_tree, paths[1])
862
 
            this_entry = this_entries.get(this_path)
863
 
            if this_entry is not None:
864
 
                this_name = this_entry.name
865
 
                this_parent = this_entry.parent_id
866
 
                this_executable = this_entry.executable
 
891
            if (self.interesting_ids is not None and
 
892
                file_id not in self.interesting_ids):
 
893
                continue
 
894
            entry = this_entries.get(file_id)
 
895
            if entry is not None:
 
896
                this_name = entry.name
 
897
                this_parent = entry.parent_id
 
898
                this_executable = entry.executable
867
899
            else:
868
900
                this_name = None
869
901
                this_parent = None
870
902
                this_executable = None
871
903
            parents3 = parents + (this_parent,)
872
904
            names3 = names + (this_name,)
873
 
            paths3 = paths + (this_path, )
874
905
            executable3 = executable + (this_executable,)
875
 
            result.append((file_id, changed, paths3,
876
 
                           parents3, names3, executable3))
 
906
            result.append((file_id, changed, parents3, names3, executable3))
877
907
        return result
878
908
 
879
909
    def _entries_lca(self):
883
913
        it then compares with THIS and BASE.
884
914
 
885
915
        For the multi-valued entries, the format will be (BASE, [lca1, lca2])
886
 
 
887
 
        :return: [(file_id, changed, paths, parents, names, executable)], where:
888
 
 
889
 
            * file_id: Simple file_id of the entry
890
 
            * changed: Boolean, True if the kind or contents changed else False
891
 
            * paths: ((base, [path, in, lcas]), path_other, path_this)
892
 
            * parents: ((base, [parent_id, in, lcas]), parent_id_other,
893
 
                        parent_id_this)
894
 
            * names:   ((base, [name, in, lcas]), name_in_other, name_in_this)
895
 
            * executable: ((base, [exec, in, lcas]), exec_in_other,
896
 
                        exec_in_this)
 
916
        :return: [(file_id, changed, parents, names, executable)]
 
917
            file_id     Simple file_id of the entry
 
918
            changed     Boolean, True if the kind or contents changed
 
919
                        else False
 
920
            parents     ((base, [parent_id, in, lcas]), parent_id_other,
 
921
                         parent_id_this)
 
922
            names       ((base, [name, in, lcas]), name_in_other, name_in_this)
 
923
            executable  ((base, [exec, in, lcas]), exec_in_other, exec_in_this)
897
924
        """
898
925
        if self.interesting_files is not None:
899
926
            lookup_trees = [self.this_tree, self.base_tree]
900
927
            lookup_trees.extend(self._lca_trees)
901
928
            # I think we should include the lca trees as well
902
 
            interesting_files = self.other_tree.find_related_paths_across_trees(
903
 
                self.interesting_files, lookup_trees)
 
929
            interesting_ids = self.other_tree.paths2ids(self.interesting_files,
 
930
                                                        lookup_trees)
904
931
        else:
905
 
            interesting_files = None
 
932
            interesting_ids = self.interesting_ids
906
933
        result = []
907
934
        walker = _mod_tree.MultiWalker(self.other_tree, self._lca_trees)
908
935
 
909
 
        base_inventory = self.base_tree.root_inventory
910
 
        this_inventory = self.this_tree.root_inventory
 
936
        base_inventory = self.base_tree.inventory
 
937
        this_inventory = self.this_tree.inventory
911
938
        for path, file_id, other_ie, lca_values in walker.iter_all():
912
939
            # Is this modified at all from any of the other trees?
913
940
            if other_ie is None:
914
941
                other_ie = _none_entry
915
 
                other_path = None
916
 
            else:
917
 
                other_path = self.other_tree.id2path(file_id)
918
 
            if interesting_files is not None and other_path not in interesting_files:
 
942
            if interesting_ids is not None and file_id not in interesting_ids:
919
943
                continue
920
944
 
921
945
            # If other_revision is found in any of the lcas, that means this
938
962
                    continue
939
963
 
940
964
            lca_entries = []
941
 
            lca_paths = []
942
965
            for lca_path, lca_ie in lca_values:
943
966
                if lca_ie is None:
944
967
                    lca_entries.append(_none_entry)
945
 
                    lca_paths.append(None)
946
968
                else:
947
969
                    lca_entries.append(lca_ie)
948
 
                    lca_paths.append(lca_path)
949
970
 
950
 
            try:
951
 
                base_ie = base_inventory.get_entry(file_id)
952
 
            except errors.NoSuchId:
 
971
            if file_id in base_inventory:
 
972
                base_ie = base_inventory[file_id]
 
973
            else:
953
974
                base_ie = _none_entry
954
 
                base_path = None
 
975
 
 
976
            if file_id in this_inventory:
 
977
                this_ie = this_inventory[file_id]
955
978
            else:
956
 
                base_path = self.base_tree.id2path(file_id)
957
 
 
958
 
            try:
959
 
                this_ie = this_inventory.get_entry(file_id)
960
 
            except errors.NoSuchId:
961
979
                this_ie = _none_entry
962
 
                this_path = None
963
 
            else:
964
 
                this_path = self.this_tree.id2path(file_id)
965
980
 
966
981
            lca_kinds = []
967
982
            lca_parent_ids = []
992
1007
                        continue
993
1008
                    content_changed = False
994
1009
                elif other_ie.kind is None or other_ie.kind == 'file':
995
 
                    def get_sha1(tree, path):
996
 
                        if path is None:
997
 
                            return None
998
 
                        try:
999
 
                            return tree.get_file_sha1(path)
1000
 
                        except errors.NoSuchFile:
1001
 
                            return None
1002
 
                    base_sha1 = get_sha1(self.base_tree, base_path)
1003
 
                    lca_sha1s = [get_sha1(tree, lca_path)
1004
 
                                 for tree, lca_path
1005
 
                                 in zip(self._lca_trees, lca_paths)]
1006
 
                    this_sha1 = get_sha1(self.this_tree, this_path)
1007
 
                    other_sha1 = get_sha1(self.other_tree, other_path)
 
1010
                    def get_sha1(ie, tree):
 
1011
                        if ie.kind != 'file':
 
1012
                            return None
 
1013
                        return tree.get_file_sha1(file_id)
 
1014
                    base_sha1 = get_sha1(base_ie, self.base_tree)
 
1015
                    lca_sha1s = [get_sha1(ie, tree) for ie, tree
 
1016
                                 in zip(lca_entries, self._lca_trees)]
 
1017
                    this_sha1 = get_sha1(this_ie, self.this_tree)
 
1018
                    other_sha1 = get_sha1(other_ie, self.other_tree)
1008
1019
                    sha1_winner = self._lca_multi_way(
1009
1020
                        (base_sha1, lca_sha1s), other_sha1, this_sha1,
1010
1021
                        allow_overriding_lca=False)
1012
1023
                        (base_ie.executable, lca_executable),
1013
1024
                        other_ie.executable, this_ie.executable)
1014
1025
                    if (parent_id_winner == 'this' and name_winner == 'this'
1015
 
                            and sha1_winner == 'this' and exec_winner == 'this'):
 
1026
                        and sha1_winner == 'this' and exec_winner == 'this'):
1016
1027
                        # No kind, parent, name, exec, or content change for
1017
1028
                        # OTHER, so this node is not considered interesting
1018
1029
                        continue
1019
1030
                    if sha1_winner == 'this':
1020
1031
                        content_changed = False
1021
1032
                elif other_ie.kind == 'symlink':
1022
 
                    def get_target(ie, tree, path):
 
1033
                    def get_target(ie, tree):
1023
1034
                        if ie.kind != 'symlink':
1024
1035
                            return None
1025
 
                        return tree.get_symlink_target(path)
1026
 
                    base_target = get_target(base_ie, self.base_tree, base_path)
1027
 
                    lca_targets = [get_target(ie, tree, lca_path) for ie, tree, lca_path
1028
 
                                   in zip(lca_entries, self._lca_trees, lca_paths)]
1029
 
                    this_target = get_target(
1030
 
                        this_ie, self.this_tree, this_path)
1031
 
                    other_target = get_target(
1032
 
                        other_ie, self.other_tree, other_path)
 
1036
                        return tree.get_symlink_target(file_id)
 
1037
                    base_target = get_target(base_ie, self.base_tree)
 
1038
                    lca_targets = [get_target(ie, tree) for ie, tree
 
1039
                                   in zip(lca_entries, self._lca_trees)]
 
1040
                    this_target = get_target(this_ie, self.this_tree)
 
1041
                    other_target = get_target(other_ie, self.other_tree)
1033
1042
                    target_winner = self._lca_multi_way(
1034
1043
                        (base_target, lca_targets),
1035
1044
                        other_target, this_target)
1036
1045
                    if (parent_id_winner == 'this' and name_winner == 'this'
1037
 
                            and target_winner == 'this'):
 
1046
                        and target_winner == 'this'):
1038
1047
                        # No kind, parent, name, or symlink target change
1039
1048
                        # not interesting
1040
1049
                        continue
1053
1062
 
1054
1063
            # If we have gotten this far, that means something has changed
1055
1064
            result.append((file_id, content_changed,
1056
 
                           ((base_path, lca_paths),
1057
 
                            other_path, this_path),
1058
1065
                           ((base_ie.parent_id, lca_parent_ids),
1059
1066
                            other_ie.parent_id, this_ie.parent_id),
1060
1067
                           ((base_ie.name, lca_names),
1061
1068
                            other_ie.name, this_ie.name),
1062
1069
                           ((base_ie.executable, lca_executable),
1063
1070
                            other_ie.executable, this_ie.executable)
1064
 
                           ))
 
1071
                          ))
1065
1072
        return result
1066
1073
 
 
1074
 
 
1075
    def fix_root(self):
 
1076
        try:
 
1077
            self.tt.final_kind(self.tt.root)
 
1078
        except errors.NoSuchFile:
 
1079
            self.tt.cancel_deletion(self.tt.root)
 
1080
        if self.tt.final_file_id(self.tt.root) is None:
 
1081
            self.tt.version_file(self.tt.tree_file_id(self.tt.root),
 
1082
                                 self.tt.root)
 
1083
        other_root_file_id = self.other_tree.get_root_id()
 
1084
        if other_root_file_id is None:
 
1085
            return
 
1086
        other_root = self.tt.trans_id_file_id(other_root_file_id)
 
1087
        if other_root == self.tt.root:
 
1088
            return
 
1089
        if self.other_tree.inventory.root.file_id in self.this_tree.inventory:
 
1090
            # the other tree's root is a non-root in the current tree (as when
 
1091
            # a previously unrelated branch is merged into another)
 
1092
            return
 
1093
        try:
 
1094
            self.tt.final_kind(other_root)
 
1095
            other_root_is_present = True
 
1096
        except errors.NoSuchFile:
 
1097
            # other_root doesn't have a physical representation. We still need
 
1098
            # to move any references to the actual root of the tree.
 
1099
            other_root_is_present = False
 
1100
        # 'other_tree.inventory.root' is not present in this tree. We are
 
1101
        # calling adjust_path for children which *want* to be present with a
 
1102
        # correct place to go.
 
1103
        for thing, child in self.other_tree.inventory.root.children.iteritems():
 
1104
            trans_id = self.tt.trans_id_file_id(child.file_id)
 
1105
            if not other_root_is_present:
 
1106
                # FIXME: Make final_kind returns None instead of raising
 
1107
                # NoSuchFile to avoid the ugly construct below -- vila 20100402
 
1108
                try:
 
1109
                    self.tt.final_kind(trans_id)
 
1110
                    # The item exist in the final tree and has a defined place
 
1111
                    # to go already.
 
1112
                    continue
 
1113
                except errors.NoSuchFile, e:
 
1114
                    pass
 
1115
            # Move the item into the root
 
1116
            self.tt.adjust_path(self.tt.final_name(trans_id),
 
1117
                                self.tt.root, trans_id)
 
1118
        if other_root_is_present:
 
1119
            self.tt.cancel_creation(other_root)
 
1120
            self.tt.cancel_versioning(other_root)
 
1121
 
1067
1122
    def write_modified(self, results):
1068
 
        if not self.working_tree.supports_merge_modified():
1069
 
            return
1070
1123
        modified_hashes = {}
1071
1124
        for path in results.modified_paths:
1072
 
            wt_relpath = self.working_tree.relpath(path)
1073
 
            file_id = self.working_tree.path2id(wt_relpath)
 
1125
            file_id = self.this_tree.path2id(self.this_tree.relpath(path))
1074
1126
            if file_id is None:
1075
1127
                continue
1076
 
            hash = self.working_tree.get_file_sha1(wt_relpath)
 
1128
            hash = self.this_tree.get_file_sha1(file_id)
1077
1129
            if hash is None:
1078
1130
                continue
1079
1131
            modified_hashes[file_id] = hash
1080
 
        self.working_tree.set_merge_modified(modified_hashes)
 
1132
        self.this_tree.set_merge_modified(modified_hashes)
1081
1133
 
1082
1134
    @staticmethod
1083
 
    def parent(entry):
 
1135
    def parent(entry, file_id):
1084
1136
        """Determine the parent for a file_id (used as a key method)"""
1085
1137
        if entry is None:
1086
1138
            return None
1087
1139
        return entry.parent_id
1088
1140
 
1089
1141
    @staticmethod
1090
 
    def name(entry):
 
1142
    def name(entry, file_id):
1091
1143
        """Determine the name for a file_id (used as a key method)"""
1092
1144
        if entry is None:
1093
1145
            return None
1094
1146
        return entry.name
1095
1147
 
1096
1148
    @staticmethod
1097
 
    def contents_sha1(tree, path):
 
1149
    def contents_sha1(tree, file_id):
1098
1150
        """Determine the sha1 of the file contents (used as a key method)."""
1099
 
        try:
1100
 
            return tree.get_file_sha1(path)
1101
 
        except errors.NoSuchFile:
 
1151
        if file_id not in tree:
1102
1152
            return None
 
1153
        return tree.get_file_sha1(file_id)
1103
1154
 
1104
1155
    @staticmethod
1105
 
    def executable(tree, path):
 
1156
    def executable(tree, file_id):
1106
1157
        """Determine the executability of a file-id (used as a key method)."""
1107
 
        try:
1108
 
            if tree.kind(path) != "file":
1109
 
                return False
1110
 
        except errors.NoSuchFile:
 
1158
        if not tree.has_id(file_id):
1111
1159
            return None
1112
 
        return tree.is_executable(path)
 
1160
        if tree.kind(file_id) != "file":
 
1161
            return False
 
1162
        return tree.is_executable(file_id)
1113
1163
 
1114
1164
    @staticmethod
1115
 
    def kind(tree, path):
 
1165
    def kind(tree, file_id):
1116
1166
        """Determine the kind of a file-id (used as a key method)."""
1117
 
        try:
1118
 
            return tree.kind(path)
1119
 
        except errors.NoSuchFile:
 
1167
        if not tree.has_id(file_id):
1120
1168
            return None
 
1169
        return tree.kind(file_id)
1121
1170
 
1122
1171
    @staticmethod
1123
1172
    def _three_way(base, other, this):
1160
1209
        base_val, lca_vals = bases
1161
1210
        # Remove 'base_val' from the lca_vals, because it is not interesting
1162
1211
        filtered_lca_vals = [lca_val for lca_val in lca_vals
1163
 
                             if lca_val != base_val]
 
1212
                                      if lca_val != base_val]
1164
1213
        if len(filtered_lca_vals) == 0:
1165
1214
            return Merge3Merger._three_way(base_val, other, this)
1166
1215
 
1185
1234
        # At this point, the lcas disagree, and the tip disagree
1186
1235
        return 'conflict'
1187
1236
 
1188
 
    def _merge_names(self, file_id, paths, parents, names, resolver):
 
1237
    @staticmethod
 
1238
    @deprecated_method(deprecated_in((2, 2, 0)))
 
1239
    def scalar_three_way(this_tree, base_tree, other_tree, file_id, key):
 
1240
        """Do a three-way test on a scalar.
 
1241
        Return "this", "other" or "conflict", depending whether a value wins.
 
1242
        """
 
1243
        key_base = key(base_tree, file_id)
 
1244
        key_other = key(other_tree, file_id)
 
1245
        #if base == other, either they all agree, or only THIS has changed.
 
1246
        if key_base == key_other:
 
1247
            return "this"
 
1248
        key_this = key(this_tree, file_id)
 
1249
        # "Ambiguous clean merge"
 
1250
        if key_this == key_other:
 
1251
            return "this"
 
1252
        elif key_this == key_base:
 
1253
            return "other"
 
1254
        else:
 
1255
            return "conflict"
 
1256
 
 
1257
    def merge_names(self, file_id):
 
1258
        def get_entry(tree):
 
1259
            if tree.has_id(file_id):
 
1260
                return tree.inventory[file_id]
 
1261
            else:
 
1262
                return None
 
1263
        this_entry = get_entry(self.this_tree)
 
1264
        other_entry = get_entry(self.other_tree)
 
1265
        base_entry = get_entry(self.base_tree)
 
1266
        entries = (base_entry, other_entry, this_entry)
 
1267
        names = []
 
1268
        parents = []
 
1269
        for entry in entries:
 
1270
            if entry is None:
 
1271
                names.append(None)
 
1272
                parents.append(None)
 
1273
            else:
 
1274
                names.append(entry.name)
 
1275
                parents.append(entry.parent_id)
 
1276
        return self._merge_names(file_id, parents, names,
 
1277
                                 resolver=self._three_way)
 
1278
 
 
1279
    def _merge_names(self, file_id, parents, names, resolver):
1189
1280
        """Perform a merge on file_id names and parents"""
1190
1281
        base_name, other_name, this_name = names
1191
1282
        base_parent, other_parent, this_parent = parents
1192
 
        unused_base_path, other_path, this_path = paths
1193
1283
 
1194
1284
        name_winner = resolver(*names)
1195
1285
 
1209
1299
            self._raw_conflicts.append(('path conflict', trans_id, file_id,
1210
1300
                                        this_parent, this_name,
1211
1301
                                        other_parent, other_name))
1212
 
        if other_path is None:
 
1302
        if other_name is None:
1213
1303
            # it doesn't matter whether the result was 'other' or
1214
 
            # 'conflict'-- if it has no file id, we leave it alone.
 
1304
            # 'conflict'-- if there's no 'other', we leave it alone.
1215
1305
            return
1216
1306
        parent_id = parents[self.winner_idx[parent_id_winner]]
1217
 
        name = names[self.winner_idx[name_winner]]
1218
 
        if parent_id is not None or name is not None:
 
1307
        if parent_id is not None:
1219
1308
            # if we get here, name_winner and parent_winner are set to safe
1220
1309
            # values.
1221
 
            if parent_id is None and name is not None:
1222
 
                # if parent_id is None and name is non-None, current file is
1223
 
                # the tree root.
1224
 
                if names[self.winner_idx[parent_id_winner]] != '':
1225
 
                    raise AssertionError(
1226
 
                        'File looks like a root, but named %s' %
1227
 
                        names[self.winner_idx[parent_id_winner]])
1228
 
                parent_trans_id = transform.ROOT_PARENT
1229
 
            else:
1230
 
                parent_trans_id = self.tt.trans_id_file_id(parent_id)
1231
 
            self.tt.adjust_path(name, parent_trans_id,
 
1310
            self.tt.adjust_path(names[self.winner_idx[name_winner]],
 
1311
                                self.tt.trans_id_file_id(parent_id),
1232
1312
                                self.tt.trans_id_file_id(file_id))
1233
1313
 
1234
 
    def _do_merge_contents(self, paths, file_id):
 
1314
    def _do_merge_contents(self, file_id):
1235
1315
        """Performs a merge on file_id contents."""
1236
 
        def contents_pair(tree, path):
1237
 
            if path is None:
1238
 
                return (None, None)
1239
 
            try:
1240
 
                kind = tree.kind(path)
1241
 
            except errors.NoSuchFile:
1242
 
                return (None, None)
 
1316
        def contents_pair(tree):
 
1317
            if file_id not in tree:
 
1318
                return (None, None)
 
1319
            kind = tree.kind(file_id)
1243
1320
            if kind == "file":
1244
 
                contents = tree.get_file_sha1(path)
 
1321
                contents = tree.get_file_sha1(file_id)
1245
1322
            elif kind == "symlink":
1246
 
                contents = tree.get_symlink_target(path)
 
1323
                contents = tree.get_symlink_target(file_id)
1247
1324
            else:
1248
1325
                contents = None
1249
1326
            return kind, contents
1250
1327
 
1251
 
        base_path, other_path, this_path = paths
1252
1328
        # See SPOT run.  run, SPOT, run.
1253
1329
        # So we're not QUITE repeating ourselves; we do tricky things with
1254
1330
        # file kind...
1255
 
        other_pair = contents_pair(self.other_tree, other_path)
1256
 
        this_pair = contents_pair(self.this_tree, this_path)
 
1331
        base_pair = contents_pair(self.base_tree)
 
1332
        other_pair = contents_pair(self.other_tree)
1257
1333
        if self._lca_trees:
1258
 
            (base_path, lca_paths) = base_path
1259
 
            base_pair = contents_pair(self.base_tree, base_path)
1260
 
            lca_pairs = [contents_pair(tree, path)
1261
 
                         for tree, path in zip(self._lca_trees, lca_paths)]
 
1334
            this_pair = contents_pair(self.this_tree)
 
1335
            lca_pairs = [contents_pair(tree) for tree in self._lca_trees]
1262
1336
            winner = self._lca_multi_way((base_pair, lca_pairs), other_pair,
1263
1337
                                         this_pair, allow_overriding_lca=False)
1264
1338
        else:
1265
 
            base_pair = contents_pair(self.base_tree, base_path)
1266
1339
            if base_pair == other_pair:
1267
1340
                winner = 'this'
1268
1341
            else:
1269
1342
                # We delayed evaluating this_pair as long as we can to avoid
1270
1343
                # unnecessary sha1 calculation
1271
 
                this_pair = contents_pair(self.this_tree, this_path)
 
1344
                this_pair = contents_pair(self.this_tree)
1272
1345
                winner = self._three_way(base_pair, other_pair, this_pair)
1273
1346
        if winner == 'this':
1274
1347
            # No interesting changes introduced by OTHER
1276
1349
        # We have a hypothetical conflict, but if we have files, then we
1277
1350
        # can try to merge the content
1278
1351
        trans_id = self.tt.trans_id_file_id(file_id)
1279
 
        params = MergeFileHookParams(
1280
 
            self, file_id, (base_path, other_path,
1281
 
                            this_path), trans_id, this_pair[0],
 
1352
        params = MergeHookParams(self, file_id, trans_id, this_pair[0],
1282
1353
            other_pair[0], winner)
1283
1354
        hooks = self.active_hooks
1284
1355
        hook_status = 'not_applicable'
1287
1358
            if hook_status != 'not_applicable':
1288
1359
                # Don't try any more hooks, this one applies.
1289
1360
                break
1290
 
        # If the merge ends up replacing the content of the file, we get rid of
1291
 
        # it at the end of this method (this variable is used to track the
1292
 
        # exceptions to this rule).
1293
 
        keep_this = False
1294
1361
        result = "modified"
1295
1362
        if hook_status == 'not_applicable':
1296
 
            # No merge hook was able to resolve the situation. Two cases exist:
1297
 
            # a content conflict or a duplicate one.
 
1363
            # This is a contents conflict, because none of the available
 
1364
            # functions could merge it.
1298
1365
            result = None
1299
1366
            name = self.tt.final_name(trans_id)
1300
1367
            parent_id = self.tt.final_parent(trans_id)
1301
 
            inhibit_content_conflict = False
1302
 
            if params.this_kind is None:  # file_id is not in THIS
1303
 
                # Is the name used for a different file_id ?
1304
 
                if self.this_tree.is_versioned(other_path):
1305
 
                    # Two entries for the same path
1306
 
                    keep_this = True
1307
 
                    # versioning the merged file will trigger a duplicate
1308
 
                    # conflict
1309
 
                    self.tt.version_file(file_id, trans_id)
1310
 
                    transform.create_from_tree(
1311
 
                        self.tt, trans_id, self.other_tree,
1312
 
                        other_path, file_id=file_id,
1313
 
                        filter_tree_path=self._get_filter_tree_path(file_id))
1314
 
                    inhibit_content_conflict = True
1315
 
            elif params.other_kind is None:  # file_id is not in OTHER
1316
 
                # Is the name used for a different file_id ?
1317
 
                if self.other_tree.is_versioned(this_path):
1318
 
                    # Two entries for the same path again, but here, the other
1319
 
                    # entry will also be merged.  We simply inhibit the
1320
 
                    # 'content' conflict creation because we know OTHER will
1321
 
                    # create (or has already created depending on ordering) an
1322
 
                    # entry at the same path. This will trigger a 'duplicate'
1323
 
                    # conflict later.
1324
 
                    keep_this = True
1325
 
                    inhibit_content_conflict = True
1326
 
            if not inhibit_content_conflict:
1327
 
                if params.this_kind is not None:
1328
 
                    self.tt.unversion_file(trans_id)
1329
 
                # This is a contents conflict, because none of the available
1330
 
                # functions could merge it.
1331
 
                file_group = self._dump_conflicts(
1332
 
                    name, (base_path, other_path, this_path), parent_id,
1333
 
                    file_id, set_version=True)
1334
 
                self._raw_conflicts.append(('contents conflict', file_group))
 
1368
            if self.this_tree.has_id(file_id):
 
1369
                self.tt.unversion_file(trans_id)
 
1370
            file_group = self._dump_conflicts(name, parent_id, file_id,
 
1371
                                              set_version=True)
 
1372
            self._raw_conflicts.append(('contents conflict', file_group))
1335
1373
        elif hook_status == 'success':
1336
1374
            self.tt.create_file(lines, trans_id)
1337
1375
        elif hook_status == 'conflicted':
1341
1379
            self._raw_conflicts.append(('text conflict', trans_id))
1342
1380
            name = self.tt.final_name(trans_id)
1343
1381
            parent_id = self.tt.final_parent(trans_id)
1344
 
            self._dump_conflicts(
1345
 
                name, (base_path, other_path, this_path), parent_id, file_id)
 
1382
            self._dump_conflicts(name, parent_id, file_id)
1346
1383
        elif hook_status == 'delete':
1347
1384
            self.tt.unversion_file(trans_id)
1348
1385
            result = "deleted"
1352
1389
            pass
1353
1390
        else:
1354
1391
            raise AssertionError('unknown hook_status: %r' % (hook_status,))
1355
 
        if not this_path and result == "modified":
 
1392
        if not self.this_tree.has_id(file_id) and result == "modified":
1356
1393
            self.tt.version_file(file_id, trans_id)
1357
 
        if not keep_this:
1358
 
            # The merge has been performed and produced a new content, so the
1359
 
            # old contents should not be retained.
 
1394
        # The merge has been performed, so the old contents should not be
 
1395
        # retained.
 
1396
        try:
1360
1397
            self.tt.delete_contents(trans_id)
 
1398
        except errors.NoSuchFile:
 
1399
            pass
1361
1400
        return result
1362
1401
 
1363
1402
    def _default_other_winner_merge(self, merge_hook_params):
1364
1403
        """Replace this contents with other."""
1365
1404
        file_id = merge_hook_params.file_id
1366
1405
        trans_id = merge_hook_params.trans_id
1367
 
        if merge_hook_params.other_path is not None:
 
1406
        file_in_this = self.this_tree.has_id(file_id)
 
1407
        if self.other_tree.has_id(file_id):
1368
1408
            # OTHER changed the file
1369
 
            transform.create_from_tree(
1370
 
                self.tt, trans_id, self.other_tree,
1371
 
                merge_hook_params.other_path, file_id=file_id,
1372
 
                filter_tree_path=self._get_filter_tree_path(file_id))
 
1409
            wt = self.this_tree
 
1410
            if wt.supports_content_filtering():
 
1411
                # We get the path from the working tree if it exists.
 
1412
                # That fails though when OTHER is adding a file, so
 
1413
                # we fall back to the other tree to find the path if
 
1414
                # it doesn't exist locally.
 
1415
                try:
 
1416
                    filter_tree_path = wt.id2path(file_id)
 
1417
                except errors.NoSuchId:
 
1418
                    filter_tree_path = self.other_tree.id2path(file_id)
 
1419
            else:
 
1420
                # Skip the id2path lookup for older formats
 
1421
                filter_tree_path = None
 
1422
            transform.create_from_tree(self.tt, trans_id,
 
1423
                             self.other_tree, file_id,
 
1424
                             filter_tree_path=filter_tree_path)
1373
1425
            return 'done', None
1374
 
        elif merge_hook_params.this_path is not None:
 
1426
        elif file_in_this:
1375
1427
            # OTHER deleted the file
1376
1428
            return 'delete', None
1377
1429
        else:
1382
1434
    def merge_contents(self, merge_hook_params):
1383
1435
        """Fallback merge logic after user installed hooks."""
1384
1436
        # This function is used in merge hooks as the fallback instance.
1385
 
        # Perhaps making this function and the functions it calls be a
 
1437
        # Perhaps making this function and the functions it calls be a 
1386
1438
        # a separate class would be better.
1387
1439
        if merge_hook_params.winner == 'other':
1388
1440
            # OTHER is a straight winner, so replace this contents with other
1392
1444
            # BASE is a file, or both converted to files, so at least we
1393
1445
            # have agreement that output should be a file.
1394
1446
            try:
1395
 
                self.text_merge(merge_hook_params.trans_id,
1396
 
                                merge_hook_params.paths, merge_hook_params.file_id)
 
1447
                self.text_merge(merge_hook_params.file_id,
 
1448
                    merge_hook_params.trans_id)
1397
1449
            except errors.BinaryFile:
1398
1450
                return 'not_applicable', None
1399
1451
            return 'done', None
1400
1452
        else:
1401
1453
            return 'not_applicable', None
1402
1454
 
1403
 
    def get_lines(self, tree, path):
 
1455
    def get_lines(self, tree, file_id):
1404
1456
        """Return the lines in a file, or an empty list."""
1405
 
        if path is None:
1406
 
            return []
1407
 
        try:
1408
 
            kind = tree.kind(path)
1409
 
        except errors.NoSuchFile:
1410
 
            return []
 
1457
        if tree.has_id(file_id):
 
1458
            return tree.get_file(file_id).readlines()
1411
1459
        else:
1412
 
            if kind != 'file':
1413
 
                return []
1414
 
            return tree.get_file_lines(path)
 
1460
            return []
1415
1461
 
1416
 
    def text_merge(self, trans_id, paths, file_id):
 
1462
    def text_merge(self, file_id, trans_id):
1417
1463
        """Perform a three-way text merge on a file_id"""
1418
1464
        # it's possible that we got here with base as a different type.
1419
1465
        # if so, we just want two-way text conflicts.
1420
 
        base_path, other_path, this_path = paths
1421
 
        base_lines = self.get_lines(self.base_tree, base_path)
1422
 
        other_lines = self.get_lines(self.other_tree, other_path)
1423
 
        this_lines = self.get_lines(self.this_tree, this_path)
 
1466
        if self.base_tree.has_id(file_id) and \
 
1467
            self.base_tree.kind(file_id) == "file":
 
1468
            base_lines = self.get_lines(self.base_tree, file_id)
 
1469
        else:
 
1470
            base_lines = []
 
1471
        other_lines = self.get_lines(self.other_tree, file_id)
 
1472
        this_lines = self.get_lines(self.this_tree, file_id)
1424
1473
        m3 = merge3.Merge3(base_lines, this_lines, other_lines,
1425
1474
                           is_cherrypick=self.cherrypick)
1426
 
        start_marker = b"!START OF MERGE CONFLICT!" + b"I HOPE THIS IS UNIQUE"
 
1475
        start_marker = "!START OF MERGE CONFLICT!" + "I HOPE THIS IS UNIQUE"
1427
1476
        if self.show_base is True:
1428
 
            base_marker = b'|' * 7
 
1477
            base_marker = '|' * 7
1429
1478
        else:
1430
1479
            base_marker = None
1431
1480
 
1432
1481
        def iter_merge3(retval):
1433
1482
            retval["text_conflicts"] = False
1434
 
            for line in m3.merge_lines(name_a=b"TREE",
1435
 
                                       name_b=b"MERGE-SOURCE",
1436
 
                                       name_base=b"BASE-REVISION",
 
1483
            for line in m3.merge_lines(name_a = "TREE",
 
1484
                                       name_b = "MERGE-SOURCE",
 
1485
                                       name_base = "BASE-REVISION",
1437
1486
                                       start_marker=start_marker,
1438
1487
                                       base_marker=base_marker,
1439
1488
                                       reprocess=self.reprocess):
1440
1489
                if line.startswith(start_marker):
1441
1490
                    retval["text_conflicts"] = True
1442
 
                    yield line.replace(start_marker, b'<' * 7)
 
1491
                    yield line.replace(start_marker, '<' * 7)
1443
1492
                else:
1444
1493
                    yield line
1445
1494
        retval = {}
1449
1498
            self._raw_conflicts.append(('text conflict', trans_id))
1450
1499
            name = self.tt.final_name(trans_id)
1451
1500
            parent_id = self.tt.final_parent(trans_id)
1452
 
            file_group = self._dump_conflicts(name, paths, parent_id, file_id,
 
1501
            file_group = self._dump_conflicts(name, parent_id, file_id,
1453
1502
                                              this_lines, base_lines,
1454
1503
                                              other_lines)
1455
1504
            file_group.append(trans_id)
1456
1505
 
1457
 
    def _get_filter_tree_path(self, file_id):
1458
 
        if self.this_tree.supports_content_filtering():
1459
 
            # We get the path from the working tree if it exists.
1460
 
            # That fails though when OTHER is adding a file, so
1461
 
            # we fall back to the other tree to find the path if
1462
 
            # it doesn't exist locally.
1463
 
            try:
1464
 
                return self.this_tree.id2path(file_id)
1465
 
            except errors.NoSuchId:
1466
 
                return self.other_tree.id2path(file_id)
1467
 
        # Skip the id2path lookup for older formats
1468
 
        return None
1469
 
 
1470
 
    def _dump_conflicts(self, name, paths, parent_id, file_id, this_lines=None,
 
1506
    def _dump_conflicts(self, name, parent_id, file_id, this_lines=None,
1471
1507
                        base_lines=None, other_lines=None, set_version=False,
1472
1508
                        no_base=False):
1473
1509
        """Emit conflict files.
1475
1511
        determined automatically.  If set_version is true, the .OTHER, .THIS
1476
1512
        or .BASE (in that order) will be created as versioned files.
1477
1513
        """
1478
 
        base_path, other_path, this_path = paths
1479
 
        data = [('OTHER', self.other_tree, other_path, other_lines),
1480
 
                ('THIS', self.this_tree, this_path, this_lines)]
 
1514
        data = [('OTHER', self.other_tree, other_lines),
 
1515
                ('THIS', self.this_tree, this_lines)]
1481
1516
        if not no_base:
1482
 
            data.append(('BASE', self.base_tree, base_path, base_lines))
 
1517
            data.append(('BASE', self.base_tree, base_lines))
1483
1518
 
1484
1519
        # We need to use the actual path in the working tree of the file here,
1485
1520
        # ignoring the conflict suffixes
1496
1531
 
1497
1532
        versioned = False
1498
1533
        file_group = []
1499
 
        for suffix, tree, path, lines in data:
1500
 
            if path is not None:
1501
 
                trans_id = self._conflict_file(
1502
 
                    name, parent_id, path, tree, file_id, suffix, lines,
1503
 
                    filter_tree_path)
 
1534
        for suffix, tree, lines in data:
 
1535
            if tree.has_id(file_id):
 
1536
                trans_id = self._conflict_file(name, parent_id, tree, file_id,
 
1537
                                               suffix, lines, filter_tree_path)
1504
1538
                file_group.append(trans_id)
1505
1539
                if set_version and not versioned:
1506
1540
                    self.tt.version_file(file_id, trans_id)
1507
1541
                    versioned = True
1508
1542
        return file_group
1509
1543
 
1510
 
    def _conflict_file(self, name, parent_id, path, tree, file_id, suffix,
 
1544
    def _conflict_file(self, name, parent_id, tree, file_id, suffix,
1511
1545
                       lines=None, filter_tree_path=None):
1512
1546
        """Emit a single conflict file."""
1513
1547
        name = name + '.' + suffix
1514
1548
        trans_id = self.tt.create_path(name, parent_id)
1515
 
        transform.create_from_tree(
1516
 
            self.tt, trans_id, tree, path,
1517
 
            file_id=file_id, chunks=lines,
1518
 
            filter_tree_path=filter_tree_path)
 
1549
        transform.create_from_tree(self.tt, trans_id, tree, file_id, lines,
 
1550
            filter_tree_path)
1519
1551
        return trans_id
1520
1552
 
1521
 
    def merge_executable(self, paths, file_id, file_status):
 
1553
    def merge_executable(self, file_id, file_status):
1522
1554
        """Perform a merge on the execute bit."""
1523
 
        executable = [self.executable(t, p, file_id)
1524
 
                      for t, p in zip([self.base_tree, self.other_tree, self.this_tree], paths)]
1525
 
        self._merge_executable(paths, file_id, executable, file_status,
 
1555
        executable = [self.executable(t, file_id) for t in (self.base_tree,
 
1556
                      self.other_tree, self.this_tree)]
 
1557
        self._merge_executable(file_id, executable, file_status,
1526
1558
                               resolver=self._three_way)
1527
1559
 
1528
 
    def _merge_executable(self, paths, file_id, executable, file_status,
 
1560
    def _merge_executable(self, file_id, executable, file_status,
1529
1561
                          resolver):
1530
1562
        """Perform a merge on the execute bit."""
1531
1563
        base_executable, other_executable, this_executable = executable
1532
 
        base_path, other_path, this_path = paths
1533
1564
        if file_status == "deleted":
1534
1565
            return
1535
1566
        winner = resolver(*executable)
1536
1567
        if winner == "conflict":
1537
 
            # There must be a None in here, if we have a conflict, but we
1538
 
            # need executability since file status was not deleted.
1539
 
            if other_path is None:
 
1568
        # There must be a None in here, if we have a conflict, but we
 
1569
        # need executability since file status was not deleted.
 
1570
            if self.executable(self.other_tree, file_id) is None:
1540
1571
                winner = "this"
1541
1572
            else:
1542
1573
                winner = "other"
1543
1574
        if winner == 'this' and file_status != "modified":
1544
1575
            return
1545
1576
        trans_id = self.tt.trans_id_file_id(file_id)
1546
 
        if self.tt.final_kind(trans_id) != "file":
 
1577
        try:
 
1578
            if self.tt.final_kind(trans_id) != "file":
 
1579
                return
 
1580
        except errors.NoSuchFile:
1547
1581
            return
1548
1582
        if winner == "this":
1549
1583
            executability = this_executable
1550
1584
        else:
1551
 
            if other_path is not None:
 
1585
            if self.other_tree.has_id(file_id):
1552
1586
                executability = other_executable
1553
 
            elif this_path is not None:
 
1587
            elif self.this_tree.has_id(file_id):
1554
1588
                executability = this_executable
1555
 
            elif base_path is not None:
 
1589
            elif self.base_tree_has_id(file_id):
1556
1590
                executability = base_executable
1557
1591
        if executability is not None:
1558
1592
            trans_id = self.tt.trans_id_file_id(file_id)
1560
1594
 
1561
1595
    def cook_conflicts(self, fs_conflicts):
1562
1596
        """Convert all conflicts into a form that doesn't depend on trans_id"""
1563
 
        content_conflict_file_ids = set()
1564
 
        cooked_conflicts = transform.cook_conflicts(fs_conflicts, self.tt)
 
1597
        self.cooked_conflicts.extend(transform.cook_conflicts(
 
1598
                fs_conflicts, self.tt))
1565
1599
        fp = transform.FinalPaths(self.tt)
1566
1600
        for conflict in self._raw_conflicts:
1567
1601
            conflict_type = conflict[0]
1568
1602
            if conflict_type == 'path conflict':
1569
1603
                (trans_id, file_id,
1570
 
                 this_parent, this_name,
1571
 
                 other_parent, other_name) = conflict[1:]
 
1604
                this_parent, this_name,
 
1605
                other_parent, other_name) = conflict[1:]
1572
1606
                if this_parent is None or this_name is None:
1573
1607
                    this_path = '<deleted>'
1574
1608
                else:
1575
 
                    parent_path = fp.get_path(
 
1609
                    parent_path =  fp.get_path(
1576
1610
                        self.tt.trans_id_file_id(this_parent))
1577
1611
                    this_path = osutils.pathjoin(parent_path, this_name)
1578
1612
                if other_parent is None or other_name is None:
1579
1613
                    other_path = '<deleted>'
1580
1614
                else:
1581
 
                    if other_parent == self.other_tree.get_root_id():
1582
 
                        # The tree transform doesn't know about the other root,
1583
 
                        # so we special case here to avoid a NoFinalPath
1584
 
                        # exception
1585
 
                        parent_path = ''
1586
 
                    else:
1587
 
                        parent_path = fp.get_path(
1588
 
                            self.tt.trans_id_file_id(other_parent))
 
1615
                    parent_path =  fp.get_path(
 
1616
                        self.tt.trans_id_file_id(other_parent))
1589
1617
                    other_path = osutils.pathjoin(parent_path, other_name)
1590
1618
                c = _mod_conflicts.Conflict.factory(
1591
1619
                    'path conflict', path=this_path,
1595
1623
                for trans_id in conflict[1]:
1596
1624
                    file_id = self.tt.final_file_id(trans_id)
1597
1625
                    if file_id is not None:
1598
 
                        # Ok we found the relevant file-id
1599
1626
                        break
1600
1627
                path = fp.get_path(trans_id)
1601
1628
                for suffix in ('.BASE', '.THIS', '.OTHER'):
1602
1629
                    if path.endswith(suffix):
1603
 
                        # Here is the raw path
1604
1630
                        path = path[:-len(suffix)]
1605
1631
                        break
1606
1632
                c = _mod_conflicts.Conflict.factory(conflict_type,
1607
1633
                                                    path=path, file_id=file_id)
1608
 
                content_conflict_file_ids.add(file_id)
1609
1634
            elif conflict_type == 'text conflict':
1610
1635
                trans_id = conflict[1]
1611
1636
                path = fp.get_path(trans_id)
1614
1639
                                                    path=path, file_id=file_id)
1615
1640
            else:
1616
1641
                raise AssertionError('bad conflict type: %r' % (conflict,))
1617
 
            cooked_conflicts.append(c)
1618
 
 
1619
 
        self.cooked_conflicts = []
1620
 
        # We want to get rid of path conflicts when a corresponding contents
1621
 
        # conflict exists. This can occur when one branch deletes a file while
1622
 
        # the other renames *and* modifies it. In this case, the content
1623
 
        # conflict is enough.
1624
 
        for c in cooked_conflicts:
1625
 
            if (c.typestring == 'path conflict'
1626
 
                    and c.file_id in content_conflict_file_ids):
1627
 
                continue
1628
1642
            self.cooked_conflicts.append(c)
1629
1643
        self.cooked_conflicts.sort(key=_mod_conflicts.Conflict.sort_key)
1630
1644
 
1635
1649
    supports_show_base = False
1636
1650
    supports_reverse_cherrypick = False
1637
1651
    history_based = True
1638
 
    requires_file_merge_plan = True
1639
1652
 
1640
1653
    def _generate_merge_plan(self, file_id, base):
1641
1654
        return self.this_tree.plan_file_merge(file_id, self.other_tree,
1655
1668
            plan = list(plan)
1656
1669
            trans_id = self.tt.trans_id_file_id(file_id)
1657
1670
            name = self.tt.final_name(trans_id) + '.plan'
1658
 
            contents = (b'%11s|%s' % l for l in plan)
 
1671
            contents = ('%11s|%s' % l for l in plan)
1659
1672
            self.tt.new_file(name, self.tt.final_parent(trans_id), contents)
1660
 
        textmerge = versionedfile.PlanWeaveMerge(plan, b'<<<<<<< TREE\n',
1661
 
                                                 b'>>>>>>> MERGE-SOURCE\n')
 
1673
        textmerge = versionedfile.PlanWeaveMerge(plan, '<<<<<<< TREE\n',
 
1674
                                                 '>>>>>>> MERGE-SOURCE\n')
1662
1675
        lines, conflicts = textmerge.merge_lines(self.reprocess)
1663
1676
        if conflicts:
1664
1677
            base_lines = textmerge.base_from_plan()
1666
1679
            base_lines = None
1667
1680
        return lines, base_lines
1668
1681
 
1669
 
    def text_merge(self, trans_id, paths, file_id):
 
1682
    def text_merge(self, file_id, trans_id):
1670
1683
        """Perform a (weave) text merge for a given file and file-id.
1671
1684
        If conflicts are encountered, .THIS and .OTHER files will be emitted,
1672
1685
        and a conflict will be noted.
1673
1686
        """
1674
 
        base_path, other_path, this_path = paths
1675
1687
        lines, base_lines = self._merged_lines(file_id)
1676
1688
        lines = list(lines)
1677
1689
        # Note we're checking whether the OUTPUT is binary in this case,
1683
1695
            self._raw_conflicts.append(('text conflict', trans_id))
1684
1696
            name = self.tt.final_name(trans_id)
1685
1697
            parent_id = self.tt.final_parent(trans_id)
1686
 
            file_group = self._dump_conflicts(name, paths, parent_id, file_id,
 
1698
            file_group = self._dump_conflicts(name, parent_id, file_id,
1687
1699
                                              no_base=False,
1688
1700
                                              base_lines=base_lines)
1689
1701
            file_group.append(trans_id)
1691
1703
 
1692
1704
class LCAMerger(WeaveMerger):
1693
1705
 
1694
 
    requires_file_merge_plan = True
1695
 
 
1696
1706
    def _generate_merge_plan(self, file_id, base):
1697
1707
        return self.this_tree.plan_file_lca_merge(file_id, self.other_tree,
1698
1708
                                                  base=base)
1699
1709
 
1700
 
 
1701
1710
class Diff3Merger(Merge3Merger):
1702
1711
    """Three-way merger using external diff3 for text merging"""
1703
1712
 
1704
 
    requires_file_merge_plan = False
1705
 
 
1706
 
    def dump_file(self, temp_dir, name, tree, path):
 
1713
    def dump_file(self, temp_dir, name, tree, file_id):
1707
1714
        out_path = osutils.pathjoin(temp_dir, name)
1708
 
        with open(out_path, "wb") as out_file:
1709
 
            in_file = tree.get_file(path)
 
1715
        out_file = open(out_path, "wb")
 
1716
        try:
 
1717
            in_file = tree.get_file(file_id)
1710
1718
            for line in in_file:
1711
1719
                out_file.write(line)
 
1720
        finally:
 
1721
            out_file.close()
1712
1722
        return out_path
1713
1723
 
1714
 
    def text_merge(self, trans_id, paths, file_id):
 
1724
    def text_merge(self, file_id, trans_id):
1715
1725
        """Perform a diff3 merge using a specified file-id and trans-id.
1716
1726
        If conflicts are encountered, .BASE, .THIS. and .OTHER conflict files
1717
1727
        will be dumped, and a will be conflict noted.
1718
1728
        """
1719
 
        import breezy.patch
1720
 
        base_path, other_path, this_path = paths
 
1729
        import bzrlib.patch
1721
1730
        temp_dir = osutils.mkdtemp(prefix="bzr-")
1722
1731
        try:
1723
1732
            new_file = osutils.pathjoin(temp_dir, "new")
1724
 
            this = self.dump_file(
1725
 
                temp_dir, "this", self.this_tree, this_path)
1726
 
            base = self.dump_file(
1727
 
                temp_dir, "base", self.base_tree, base_path)
1728
 
            other = self.dump_file(
1729
 
                temp_dir, "other", self.other_tree, other_path)
1730
 
            status = breezy.patch.diff3(new_file, this, base, other)
 
1733
            this = self.dump_file(temp_dir, "this", self.this_tree, file_id)
 
1734
            base = self.dump_file(temp_dir, "base", self.base_tree, file_id)
 
1735
            other = self.dump_file(temp_dir, "other", self.other_tree, file_id)
 
1736
            status = bzrlib.patch.diff3(new_file, this, base, other)
1731
1737
            if status not in (0, 1):
1732
1738
                raise errors.BzrError("Unhandled diff3 exit code")
1733
 
            with open(new_file, 'rb') as f:
 
1739
            f = open(new_file, 'rb')
 
1740
            try:
1734
1741
                self.tt.create_file(f, trans_id)
 
1742
            finally:
 
1743
                f.close()
1735
1744
            if status == 1:
1736
1745
                name = self.tt.final_name(trans_id)
1737
1746
                parent_id = self.tt.final_parent(trans_id)
1738
 
                self._dump_conflicts(name, paths, parent_id, file_id)
 
1747
                self._dump_conflicts(name, parent_id, file_id)
1739
1748
                self._raw_conflicts.append(('text conflict', trans_id))
1740
1749
        finally:
1741
1750
            osutils.rmtree(temp_dir)
1742
1751
 
1743
1752
 
1744
 
class PathNotInTree(errors.BzrError):
1745
 
 
1746
 
    _fmt = """Merge-into failed because %(tree)s does not contain %(path)s."""
1747
 
 
1748
 
    def __init__(self, path, tree):
1749
 
        errors.BzrError.__init__(self, path=path, tree=tree)
1750
 
 
1751
 
 
1752
 
class MergeIntoMerger(Merger):
1753
 
    """Merger that understands other_tree will be merged into a subdir.
1754
 
 
1755
 
    This also changes the Merger api so that it uses real Branch, revision_id,
1756
 
    and RevisonTree objects, rather than using revision specs.
1757
 
    """
1758
 
 
1759
 
    def __init__(self, this_tree, other_branch, other_tree, target_subdir,
1760
 
                 source_subpath, other_rev_id=None):
1761
 
        """Create a new MergeIntoMerger object.
1762
 
 
1763
 
        source_subpath in other_tree will be effectively copied to
1764
 
        target_subdir in this_tree.
1765
 
 
1766
 
        :param this_tree: The tree that we will be merging into.
1767
 
        :param other_branch: The Branch we will be merging from.
1768
 
        :param other_tree: The RevisionTree object we want to merge.
1769
 
        :param target_subdir: The relative path where we want to merge
1770
 
            other_tree into this_tree
1771
 
        :param source_subpath: The relative path specifying the subtree of
1772
 
            other_tree to merge into this_tree.
1773
 
        """
1774
 
        # It is assumed that we are merging a tree that is not in our current
1775
 
        # ancestry, which means we are using the "EmptyTree" as our basis.
1776
 
        null_ancestor_tree = this_tree.branch.repository.revision_tree(
1777
 
            _mod_revision.NULL_REVISION)
1778
 
        super(MergeIntoMerger, self).__init__(
1779
 
            this_branch=this_tree.branch,
1780
 
            this_tree=this_tree,
1781
 
            other_tree=other_tree,
1782
 
            base_tree=null_ancestor_tree,
1783
 
            )
1784
 
        self._target_subdir = target_subdir
1785
 
        self._source_subpath = source_subpath
1786
 
        self.other_branch = other_branch
1787
 
        if other_rev_id is None:
1788
 
            other_rev_id = other_tree.get_revision_id()
1789
 
        self.other_rev_id = self.other_basis = other_rev_id
1790
 
        self.base_is_ancestor = True
1791
 
        self.backup_files = True
1792
 
        self.merge_type = Merge3Merger
1793
 
        self.show_base = False
1794
 
        self.reprocess = False
1795
 
        self.interesting_files = None
1796
 
        self.merge_type = _MergeTypeParameterizer(MergeIntoMergeType,
1797
 
                                                  target_subdir=self._target_subdir,
1798
 
                                                  source_subpath=self._source_subpath)
1799
 
        if self._source_subpath != '':
1800
 
            # If this isn't a partial merge make sure the revisions will be
1801
 
            # present.
1802
 
            self._maybe_fetch(self.other_branch, self.this_branch,
1803
 
                              self.other_basis)
1804
 
 
1805
 
    def set_pending(self):
1806
 
        if self._source_subpath != '':
1807
 
            return
1808
 
        Merger.set_pending(self)
1809
 
 
1810
 
 
1811
 
class _MergeTypeParameterizer(object):
1812
 
    """Wrap a merge-type class to provide extra parameters.
1813
 
 
1814
 
    This is hack used by MergeIntoMerger to pass some extra parameters to its
1815
 
    merge_type.  Merger.do_merge() sets up its own set of parameters to pass to
1816
 
    the 'merge_type' member.  It is difficult override do_merge without
1817
 
    re-writing the whole thing, so instead we create a wrapper which will pass
1818
 
    the extra parameters.
1819
 
    """
1820
 
 
1821
 
    def __init__(self, merge_type, **kwargs):
1822
 
        self._extra_kwargs = kwargs
1823
 
        self._merge_type = merge_type
1824
 
 
1825
 
    def __call__(self, *args, **kwargs):
1826
 
        kwargs.update(self._extra_kwargs)
1827
 
        return self._merge_type(*args, **kwargs)
1828
 
 
1829
 
    def __getattr__(self, name):
1830
 
        return getattr(self._merge_type, name)
1831
 
 
1832
 
 
1833
 
class MergeIntoMergeType(Merge3Merger):
1834
 
    """Merger that incorporates a tree (or part of a tree) into another."""
1835
 
 
1836
 
    def __init__(self, *args, **kwargs):
1837
 
        """Initialize the merger object.
1838
 
 
1839
 
        :param args: See Merge3Merger.__init__'s args.
1840
 
        :param kwargs: See Merge3Merger.__init__'s keyword args, except for
1841
 
            source_subpath and target_subdir.
1842
 
        :keyword source_subpath: The relative path specifying the subtree of
1843
 
            other_tree to merge into this_tree.
1844
 
        :keyword target_subdir: The relative path where we want to merge
1845
 
            other_tree into this_tree
1846
 
        """
1847
 
        # All of the interesting work happens during Merge3Merger.__init__(),
1848
 
        # so we have have to hack in to get our extra parameters set.
1849
 
        self._source_subpath = kwargs.pop('source_subpath')
1850
 
        self._target_subdir = kwargs.pop('target_subdir')
1851
 
        super(MergeIntoMergeType, self).__init__(*args, **kwargs)
1852
 
 
1853
 
    def _compute_transform(self):
1854
 
        with ui.ui_factory.nested_progress_bar() as child_pb:
1855
 
            entries = self._entries_to_incorporate()
1856
 
            entries = list(entries)
1857
 
            for num, (entry, parent_id, relpath) in enumerate(entries):
1858
 
                child_pb.update(gettext('Preparing file merge'),
1859
 
                                num, len(entries))
1860
 
                parent_trans_id = self.tt.trans_id_file_id(parent_id)
1861
 
                path = osutils.pathjoin(self._source_subpath, relpath)
1862
 
                trans_id = transform.new_by_entry(path, self.tt, entry,
1863
 
                                                  parent_trans_id, self.other_tree)
1864
 
        self._finish_computing_transform()
1865
 
 
1866
 
    def _entries_to_incorporate(self):
1867
 
        """Yields pairs of (inventory_entry, new_parent)."""
1868
 
        other_inv = self.other_tree.root_inventory
1869
 
        subdir_id = other_inv.path2id(self._source_subpath)
1870
 
        if subdir_id is None:
1871
 
            # XXX: The error would be clearer if it gave the URL of the source
1872
 
            # branch, but we don't have a reference to that here.
1873
 
            raise PathNotInTree(self._source_subpath, "Source tree")
1874
 
        subdir = other_inv.get_entry(subdir_id)
1875
 
        parent_in_target = osutils.dirname(self._target_subdir)
1876
 
        target_id = self.this_tree.path2id(parent_in_target)
1877
 
        if target_id is None:
1878
 
            raise PathNotInTree(self._target_subdir, "Target tree")
1879
 
        name_in_target = osutils.basename(self._target_subdir)
1880
 
        merge_into_root = subdir.copy()
1881
 
        merge_into_root.name = name_in_target
1882
 
        if self.this_tree.has_id(merge_into_root.file_id):
1883
 
            # Give the root a new file-id.
1884
 
            # This can happen fairly easily if the directory we are
1885
 
            # incorporating is the root, and both trees have 'TREE_ROOT' as
1886
 
            # their root_id.  Users will expect this to Just Work, so we
1887
 
            # change the file-id here.
1888
 
            # Non-root file-ids could potentially conflict too.  That's really
1889
 
            # an edge case, so we don't do anything special for those.  We let
1890
 
            # them cause conflicts.
1891
 
            merge_into_root.file_id = generate_ids.gen_file_id(name_in_target)
1892
 
        yield (merge_into_root, target_id, '')
1893
 
        if subdir.kind != 'directory':
1894
 
            # No children, so we are done.
1895
 
            return
1896
 
        for path, entry in other_inv.iter_entries_by_dir(subdir_id):
1897
 
            parent_id = entry.parent_id
1898
 
            if parent_id == subdir.file_id:
1899
 
                # The root's parent ID has changed, so make sure children of
1900
 
                # the root refer to the new ID.
1901
 
                parent_id = merge_into_root.file_id
1902
 
            yield (entry, parent_id, path)
1903
 
 
1904
 
 
1905
1753
def merge_inner(this_branch, other_tree, base_tree, ignore_zero=False,
1906
1754
                backup_files=False,
1907
1755
                merge_type=Merge3Merger,
 
1756
                interesting_ids=None,
1908
1757
                show_base=False,
1909
1758
                reprocess=False,
1910
1759
                other_rev_id=None,
1911
1760
                interesting_files=None,
1912
1761
                this_tree=None,
 
1762
                pb=None,
1913
1763
                change_reporter=None):
1914
1764
    """Primary interface for merging.
1915
1765
 
1916
 
    Typical use is probably::
1917
 
 
1918
 
        merge_inner(branch, branch.get_revision_tree(other_revision),
1919
 
                    branch.get_revision_tree(base_revision))
1920
 
    """
 
1766
        typical use is probably
 
1767
        'merge_inner(branch, branch.get_revision_tree(other_revision),
 
1768
                     branch.get_revision_tree(base_revision))'
 
1769
        """
1921
1770
    if this_tree is None:
1922
 
        raise errors.BzrError("breezy.merge.merge_inner requires a this_tree "
1923
 
                              "parameter")
 
1771
        raise errors.BzrError("bzrlib.merge.merge_inner requires a this_tree "
 
1772
                              "parameter as of bzrlib version 0.8.")
1924
1773
    merger = Merger(this_branch, other_tree, base_tree, this_tree=this_tree,
1925
 
                    change_reporter=change_reporter)
 
1774
                    pb=pb, change_reporter=change_reporter)
1926
1775
    merger.backup_files = backup_files
1927
1776
    merger.merge_type = merge_type
 
1777
    merger.interesting_ids = interesting_ids
1928
1778
    merger.ignore_zero = ignore_zero
1929
 
    merger.interesting_files = interesting_files
 
1779
    if interesting_files:
 
1780
        if interesting_ids:
 
1781
            raise ValueError('Only supply interesting_ids'
 
1782
                             ' or interesting_files')
 
1783
        merger.interesting_files = interesting_files
1930
1784
    merger.show_base = show_base
1931
1785
    merger.reprocess = reprocess
1932
1786
    merger.other_rev_id = other_rev_id
1938
1792
    merger.set_base_revision(get_revision_id(), this_branch)
1939
1793
    return merger.do_merge()
1940
1794
 
1941
 
 
1942
 
merge_type_registry = registry.Registry()
1943
 
merge_type_registry.register('diff3', Diff3Merger,
1944
 
                             "Merge using external diff3.")
1945
 
merge_type_registry.register('lca', LCAMerger,
1946
 
                             "LCA-newness merge.")
1947
 
merge_type_registry.register('merge3', Merge3Merger,
1948
 
                             "Native diff3-style merge.")
1949
 
merge_type_registry.register('weave', WeaveMerger,
1950
 
                             "Weave-based merge.")
1951
 
 
1952
 
 
1953
1795
def get_merge_type_registry():
1954
 
    """Merge type registry was previously in breezy.option
 
1796
    """Merge type registry is in bzrlib.option to avoid circular imports.
1955
1797
 
1956
 
    This method provides a backwards compatible way to retrieve it.
 
1798
    This method provides a sanctioned way to retrieve it.
1957
1799
    """
1958
 
    return merge_type_registry
 
1800
    from bzrlib import option
 
1801
    return option._merge_type_registry
1959
1802
 
1960
1803
 
1961
1804
def _plan_annotate_merge(annotated_a, annotated_b, ancestors_a, ancestors_b):
2067
1910
                else:
2068
1911
                    yield 'killed-a', self.lines_b[b_index]
2069
1912
            # handle common lines
2070
 
            for a_index in range(i, i + n):
 
1913
            for a_index in range(i, i+n):
2071
1914
                yield 'unchanged', self.lines_a[a_index]
2072
 
            last_i = i + n
2073
 
            last_j = j + n
 
1915
            last_i = i+n
 
1916
            last_j = j+n
2074
1917
 
2075
1918
    def _get_matching_blocks(self, left_revision, right_revision):
2076
1919
        """Return a description of which sections of two revisions match.
2132
1975
        for i, j, n in matcher.get_matching_blocks():
2133
1976
            for jj in range(last_j, j):
2134
1977
                yield new_plan[jj]
2135
 
            for jj in range(j, j + n):
 
1978
            for jj in range(j, j+n):
2136
1979
                plan_line = new_plan[jj]
2137
1980
                if plan_line[0] == 'new-b':
2138
1981
                    pass
2182
2025
        while True:
2183
2026
            next_lcas = self.graph.find_lca(*cur_ancestors)
2184
2027
            # Map a plain NULL_REVISION to a simple no-ancestors
2185
 
            if next_lcas == {_mod_revision.NULL_REVISION}:
 
2028
            if next_lcas == set([_mod_revision.NULL_REVISION]):
2186
2029
                next_lcas = ()
2187
2030
            # Order the lca's based on when they were merged into the tip
2188
2031
            # While the actual merge portion of weave merge uses a set() of
2276
2119
        filtered_parent_map = {}
2277
2120
        child_map = {}
2278
2121
        tails = []
2279
 
        for key, parent_keys in viewitems(parent_map):
 
2122
        for key, parent_keys in parent_map.iteritems():
2280
2123
            culled_parent_keys = [p for p in parent_keys if p in parent_map]
2281
2124
            if not culled_parent_keys:
2282
2125
                tails.append(key)
2332
2175
        return all_texts
2333
2176
 
2334
2177
    def _build_weave(self):
2335
 
        from .bzr import weave
 
2178
        from bzrlib import weave
2336
2179
        self._weave = weave.Weave(weave_name='in_memory_weave',
2337
2180
                                  allow_reserved=True)
2338
2181
        parent_map = self._find_recursive_lcas()
2369
2212
        are combined, they are written out in the format described in
2370
2213
        VersionedFile.plan_merge
2371
2214
        """
2372
 
        if self._head_key is not None:  # There was a single head
 
2215
        if self._head_key is not None: # There was a single head
2373
2216
            if self._head_key == self.a_key:
2374
2217
                plan = 'new-a'
2375
2218
            else:
2386
2229
class _PlanLCAMerge(_PlanMergeBase):
2387
2230
    """
2388
2231
    This merge algorithm differs from _PlanMerge in that:
2389
 
 
2390
2232
    1. comparisons are done against LCAs only
2391
2233
    2. cases where a contested line is new versus one LCA but old versus
2392
2234
       another are marked as conflicts, by emitting the line as conflicted-a
2433
2275
 
2434
2276
        If a line is killed and new, this indicates that the two merge
2435
2277
        revisions contain differing conflict resolutions.
2436
 
 
2437
2278
        :param revision_id: The id of the revision in which the lines are
2438
2279
            unique
2439
2280
        :param unique_line_numbers: The line numbers of unique lines.
2440
 
        :return: a tuple of (new_this, killed_other)
 
2281
        :return a tuple of (new_this, killed_other):
2441
2282
        """
2442
2283
        new = set()
2443
2284
        killed = set()