~bzr-pqm/bzr/bzr.dev

4988.1.1 by John Arbash Meinel
Merge the per-file-hook updates.
1
# Copyright (C) 2005-2010 Canonical Ltd
1773.4.1 by Martin Pool
Add pyflakes makefile target; fix many warnings
2
#
1110 by Martin Pool
- merge aaron's merge improvements:
3
# This program is free software; you can redistribute it and/or modify
4
# it under the terms of the GNU General Public License as published by
5
# the Free Software Foundation; either version 2 of the License, or
6
# (at your option) any later version.
1773.4.1 by Martin Pool
Add pyflakes makefile target; fix many warnings
7
#
1110 by Martin Pool
- merge aaron's merge improvements:
8
# This program is distributed in the hope that it will be useful,
9
# but WITHOUT ANY WARRANTY; without even the implied warranty of
10
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
11
# GNU General Public License for more details.
1773.4.1 by Martin Pool
Add pyflakes makefile target; fix many warnings
12
#
1110 by Martin Pool
- merge aaron's merge improvements:
13
# You should have received a copy of the GNU General Public License
14
# along with this program; if not, write to the Free Software
4183.7.1 by Sabin Iacob
update FSF mailing address
15
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
1110 by Martin Pool
- merge aaron's merge improvements:
16
4961.2.11 by Martin Pool
Pull out pbs and ProgressPhases stored in object state; just use them in single functions
17
import warnings
1545.2.3 by Aaron Bentley
Updated following j-a-meinel's comments
18
5279.1.1 by Andrew Bennetts
lazy_import most things in merge.py; add a few representative modules to the import tariff tests; tweak a couple of other modules so that patiencediff is not necessarily imported; remove a bunch of unused imports from test_knit.py.
19
from bzrlib.lazy_import import lazy_import
20
lazy_import(globals(), """
1996.3.18 by John Arbash Meinel
Now that mkdtemp and rmtree are lazy, they should not be directly improted.
21
from bzrlib import (
4721.3.1 by Vincent Ladeuil
Cleanup imports.
22
    branch as _mod_branch,
23
    conflicts as _mod_conflicts,
1551.19.17 by Aaron Bentley
Add debugging flag for merges
24
    debug,
2590.2.11 by Aaron Bentley
Aggressively cache trees, use dirstate. re-mplement _add_parent.
25
    errors,
3514.2.14 by John Arbash Meinel
Bring in the code to collapse linear portions of the graph.
26
    graph as _mod_graph,
4721.3.1 by Vincent Ladeuil
Cleanup imports.
27
    merge3,
1996.3.18 by John Arbash Meinel
Now that mkdtemp and rmtree are lazy, they should not be directly improted.
28
    osutils,
1551.15.46 by Aaron Bentley
Move plan merge to tree
29
    patiencediff,
2598.5.1 by Aaron Bentley
Start eliminating the use of None to indicate null revision
30
    revision as _mod_revision,
4721.3.1 by Vincent Ladeuil
Cleanup imports.
31
    textfile,
32
    trace,
33
    transform,
3514.4.5 by John Arbash Meinel
Initial work on _entries_lca.
34
    tree as _mod_tree,
3514.2.5 by John Arbash Meinel
Switch to the get_parent_map design I had settled on.
35
    tsort,
4721.3.1 by Vincent Ladeuil
Cleanup imports.
36
    ui,
5279.1.1 by Andrew Bennetts
lazy_import most things in merge.py; add a few representative modules to the import tariff tests; tweak a couple of other modules so that patiencediff is not necessarily imported; remove a bunch of unused imports from test_knit.py.
37
    versionedfile,
1996.3.18 by John Arbash Meinel
Now that mkdtemp and rmtree are lazy, they should not be directly improted.
38
    )
4797.16.1 by Andrew Bennetts
Replace many try/finally blocks in merge.py with more robust OperationWithCleanups.
39
from bzrlib.cleanup import OperationWithCleanups
5279.1.1 by Andrew Bennetts
lazy_import most things in merge.py; add a few representative modules to the import tariff tests; tweak a couple of other modules so that patiencediff is not necessarily imported; remove a bunch of unused imports from test_knit.py.
40
""")
41
from bzrlib import (
42
    decorators,
43
    hooks,
44
    )
4721.3.2 by Vincent Ladeuil
Simplify mutable_tree.has_changes() and update call sites.
45
from bzrlib.symbol_versioning import (
46
    deprecated_in,
47
    deprecated_method,
48
    )
1545.2.6 by Aaron Bentley
Removed _merge, renamed MergeConflictHandler to _MergeConflictHandler
49
# TODO: Report back as changes are merged in
50
2325.3.1 by John Arbash Meinel
New helper function for merge, which allows us to re-use the existing workingtree, rather than opening it again.
51
1185.35.4 by Aaron Bentley
Implemented remerge
52
def transform_tree(from_tree, to_tree, interesting_ids=None):
4595.13.1 by Alexander Belchenko
[cherrypick revno.4635 from bzr.dev] (robertc) Fix many locking errors on windows due to a small bug in merge.transform_tree. (Robert Collins)
53
    from_tree.lock_tree_write()
4797.16.1 by Andrew Bennetts
Replace many try/finally blocks in merge.py with more robust OperationWithCleanups.
54
    operation = OperationWithCleanups(merge_inner)
55
    operation.add_cleanup(from_tree.unlock)
56
    operation.run_simple(from_tree.branch, to_tree, from_tree,
57
        ignore_zero=True, interesting_ids=interesting_ids, this_tree=from_tree)
1185.12.76 by Aaron Bentley
Refactored merge and merge_inner to use Merger
58
1457.1.12 by Robert Collins
Update comment to reflect author.
59
4869.3.1 by Andrew Bennetts
Basic per-file merge hook.
60
class MergeHooks(hooks.Hooks):
61
62
    def __init__(self):
63
        hooks.Hooks.__init__(self)
64
        self.create_hook(hooks.HookPoint('merge_file_content',
4797.5.1 by Robert Collins
Support state on per-file merging to permit more efficient use of configuration data.
65
            "Called with a bzrlib.merge.Merger object to create a per file "
66
            "merge object when starting a merge. "
67
            "Should return either None or a subclass of "
68
            "``bzrlib.merge.AbstractPerFileMerger``. "
69
            "Such objects will then be called per file "
70
            "that needs to be merged (including when one "
71
            "side has deleted the file and the other has changed it). "
72
            "See the AbstractPerFileMerger API docs for details on how it is "
73
            "used by merge.",
4869.3.9 by Andrew Bennetts
Document hook point better
74
            (2, 1), None))
4869.3.1 by Andrew Bennetts
Basic per-file merge hook.
75
76
4797.5.1 by Robert Collins
Support state on per-file merging to permit more efficient use of configuration data.
77
class AbstractPerFileMerger(object):
78
    """PerFileMerger objects are used by plugins extending merge for bzrlib.
79
80
    See ``bzrlib.plugins.news_merge.news_merge`` for an example concrete class.
81
    
82
    :ivar merger: The Merge3Merger performing the merge.
83
    """
84
85
    def __init__(self, merger):
86
        """Create a PerFileMerger for use with merger."""
87
        self.merger = merger
88
89
    def merge_contents(self, merge_params):
90
        """Attempt to merge the contents of a single file.
91
        
92
        :param merge_params: A bzrlib.merge.MergeHookParams
93
        :return : A tuple of (status, chunks), where status is one of
94
            'not_applicable', 'success', 'conflicted', or 'delete'.  If status
95
            is 'success' or 'conflicted', then chunks should be an iterable of
96
            strings for the new file contents.
97
        """
98
        return ('not applicable', None)
99
100
5168.2.2 by Andrew Bennetts
Create a PerFileMerger helper class that is midway between AbstractPerFileMerger and ConfigurableFileMerger. It allows the example merge plugin in the docs (the always conflict *.xml files) to be much, much simpler.
101
class PerFileMerger(AbstractPerFileMerger):
102
    """Merge individual files when self.file_matches returns True.
103
104
    This class is intended to be subclassed.  The file_matches and
105
    merge_matching methods should be overridden with concrete implementations.
106
    """
107
108
    def file_matches(self, params):
109
        """Return True if merge_matching should be called on this file.
110
111
        Only called with merges of plain files with no clear winner.
112
113
        Subclasses must override this.
114
        """
115
        raise NotImplementedError(self.file_matches)
116
117
    def get_filename(self, params, tree):
118
        """Lookup the filename (i.e. basename, not path), given a Tree (e.g.
119
        self.merger.this_tree) and a MergeHookParams.
120
        """
5168.2.3 by Andrew Bennetts
Change implementation of get_filename as suggested by John.
121
        return osutils.basename(tree.id2path(params.file_id))
5168.2.2 by Andrew Bennetts
Create a PerFileMerger helper class that is midway between AbstractPerFileMerger and ConfigurableFileMerger. It allows the example merge plugin in the docs (the always conflict *.xml files) to be much, much simpler.
122
123
    def get_filepath(self, params, tree):
124
        """Calculate the path to the file in a tree.
125
126
        :param params: A MergeHookParams describing the file to merge
127
        :param tree: a Tree, e.g. self.merger.this_tree.
128
        """
129
        return tree.id2path(params.file_id)
130
131
    def merge_contents(self, params):
132
        """Merge the contents of a single file."""
133
        # Check whether this custom merge logic should be used.
134
        if (
135
            # OTHER is a straight winner, rely on default merge.
136
            params.winner == 'other' or
137
            # THIS and OTHER aren't both files.
138
            not params.is_file_merge() or
139
            # The filename doesn't match *.xml
140
            not self.file_matches(params)):
141
            return 'not_applicable', None
5168.2.3 by Andrew Bennetts
Change implementation of get_filename as suggested by John.
142
        return self.merge_matching(params)
5168.2.2 by Andrew Bennetts
Create a PerFileMerger helper class that is midway between AbstractPerFileMerger and ConfigurableFileMerger. It allows the example merge plugin in the docs (the always conflict *.xml files) to be much, much simpler.
143
144
    def merge_matching(self, params):
145
        """Merge the contents of a single file that has matched the criteria
146
        in PerFileMerger.merge_contents (is a conflict, is a file,
147
        self.file_matches is True).
148
149
        Subclasses must override this.
150
        """
151
        raise NotImplementedError(self.merge_matching)
152
153
154
class ConfigurableFileMerger(PerFileMerger):
4797.5.2 by Robert Collins
Refactor NewsMerger into a reusable base class merge.ConfigurableFileMerger.
155
    """Merge individual files when configured via a .conf file.
156
157
    This is a base class for concrete custom file merging logic. Concrete
158
    classes should implement ``merge_text``.
159
4797.9.1 by Vincent Ladeuil
No test was exercising the faulty code path.
160
    See ``bzrlib.plugins.news_merge.news_merge`` for an example concrete class.
161
    
4797.5.2 by Robert Collins
Refactor NewsMerger into a reusable base class merge.ConfigurableFileMerger.
162
    :ivar affected_files: The configured file paths to merge.
4797.9.1 by Vincent Ladeuil
No test was exercising the faulty code path.
163
4797.5.3 by Robert Collins
Tweak ConfigurableFileMerger to use class variables rather than requiring __init__ wrapping as future proofing for helper functions.
164
    :cvar name_prefix: The prefix to use when looking up configuration
4797.9.1 by Vincent Ladeuil
No test was exercising the faulty code path.
165
        details. <name_prefix>_merge_files describes the files targeted by the
166
        hook for example.
167
        
4797.5.3 by Robert Collins
Tweak ConfigurableFileMerger to use class variables rather than requiring __init__ wrapping as future proofing for helper functions.
168
    :cvar default_files: The default file paths to merge when no configuration
4797.5.2 by Robert Collins
Refactor NewsMerger into a reusable base class merge.ConfigurableFileMerger.
169
        is present.
170
    """
171
4797.5.3 by Robert Collins
Tweak ConfigurableFileMerger to use class variables rather than requiring __init__ wrapping as future proofing for helper functions.
172
    name_prefix = None
173
    default_files = None
174
175
    def __init__(self, merger):
4797.5.2 by Robert Collins
Refactor NewsMerger into a reusable base class merge.ConfigurableFileMerger.
176
        super(ConfigurableFileMerger, self).__init__(merger)
177
        self.affected_files = None
4797.5.3 by Robert Collins
Tweak ConfigurableFileMerger to use class variables rather than requiring __init__ wrapping as future proofing for helper functions.
178
        self.default_files = self.__class__.default_files or []
179
        self.name_prefix = self.__class__.name_prefix
180
        if self.name_prefix is None:
181
            raise ValueError("name_prefix must be set.")
4797.5.2 by Robert Collins
Refactor NewsMerger into a reusable base class merge.ConfigurableFileMerger.
182
5168.2.2 by Andrew Bennetts
Create a PerFileMerger helper class that is midway between AbstractPerFileMerger and ConfigurableFileMerger. It allows the example merge plugin in the docs (the always conflict *.xml files) to be much, much simpler.
183
    def file_matches(self, params):
4797.9.1 by Vincent Ladeuil
No test was exercising the faulty code path.
184
        """Check whether the file should call the merge hook.
185
186
        <name_prefix>_merge_files configuration variable is a list of files
187
        that should use the hook.
188
        """
4797.5.2 by Robert Collins
Refactor NewsMerger into a reusable base class merge.ConfigurableFileMerger.
189
        affected_files = self.affected_files
190
        if affected_files is None:
4797.21.1 by Aaron Bentley
Fix merge when this_tree is not a WorkingTree.
191
            config = self.merger.this_branch.get_config()
4797.5.2 by Robert Collins
Refactor NewsMerger into a reusable base class merge.ConfigurableFileMerger.
192
            # Until bzr provides a better policy for caching the config, we
193
            # just add the part we're interested in to the params to avoid
194
            # reading the config files repeatedly (bazaar.conf, location.conf,
195
            # branch.conf).
196
            config_key = self.name_prefix + '_merge_files'
197
            affected_files = config.get_user_option_as_list(config_key)
198
            if affected_files is None:
199
                # If nothing was specified in the config, use the default.
200
                affected_files = self.default_files
201
            self.affected_files = affected_files
202
        if affected_files:
5168.2.2 by Andrew Bennetts
Create a PerFileMerger helper class that is midway between AbstractPerFileMerger and ConfigurableFileMerger. It allows the example merge plugin in the docs (the always conflict *.xml files) to be much, much simpler.
203
            filepath = self.get_filepath(params, self.merger.this_tree)
204
            if filepath in affected_files:
4797.5.2 by Robert Collins
Refactor NewsMerger into a reusable base class merge.ConfigurableFileMerger.
205
                return True
206
        return False
207
5168.2.2 by Andrew Bennetts
Create a PerFileMerger helper class that is midway between AbstractPerFileMerger and ConfigurableFileMerger. It allows the example merge plugin in the docs (the always conflict *.xml files) to be much, much simpler.
208
    def merge_matching(self, params):
4797.9.1 by Vincent Ladeuil
No test was exercising the faulty code path.
209
        return self.merge_text(params)
4797.5.2 by Robert Collins
Refactor NewsMerger into a reusable base class merge.ConfigurableFileMerger.
210
211
    def merge_text(self, params):
212
        """Merge the byte contents of a single file.
213
214
        This is called after checking that the merge should be performed in
215
        merge_contents, and it should behave as per
216
        ``bzrlib.merge.AbstractPerFileMerger.merge_contents``.
217
        """
218
        raise NotImplementedError(self.merge_text)
219
220
4869.3.1 by Andrew Bennetts
Basic per-file merge hook.
221
class MergeHookParams(object):
4869.3.11 by Andrew Bennetts
More docs.
222
    """Object holding parameters passed to merge_file_content hooks.
223
4797.5.1 by Robert Collins
Support state on per-file merging to permit more efficient use of configuration data.
224
    There are some fields hooks can access:
4869.3.11 by Andrew Bennetts
More docs.
225
226
    :ivar file_id: the file ID of the file being merged
4869.3.24 by Andrew Bennetts
Mention 'winner' attribute in MergeHookParams' docstring.
227
    :ivar trans_id: the transform ID for the merge of this file
4869.3.21 by Andrew Bennetts
Pass kinds (not pairs) to MergeHookParams.
228
    :ivar this_kind: kind of file_id in 'this' tree
229
    :ivar other_kind: kind of file_id in 'other' tree
4869.3.24 by Andrew Bennetts
Mention 'winner' attribute in MergeHookParams' docstring.
230
    :ivar winner: one of 'this', 'other', 'conflict'
4869.3.11 by Andrew Bennetts
More docs.
231
    """
4869.3.1 by Andrew Bennetts
Basic per-file merge hook.
232
4869.3.21 by Andrew Bennetts
Pass kinds (not pairs) to MergeHookParams.
233
    def __init__(self, merger, file_id, trans_id, this_kind, other_kind,
4869.3.16 by Andrew Bennetts
Move handling of winner==OTHER case into generic hook path too.
234
            winner):
4797.5.1 by Robert Collins
Support state on per-file merging to permit more efficient use of configuration data.
235
        self._merger = merger
4869.3.1 by Andrew Bennetts
Basic per-file merge hook.
236
        self.file_id = file_id
237
        self.trans_id = trans_id
4869.3.21 by Andrew Bennetts
Pass kinds (not pairs) to MergeHookParams.
238
        self.this_kind = this_kind
239
        self.other_kind = other_kind
4869.3.16 by Andrew Bennetts
Move handling of winner==OTHER case into generic hook path too.
240
        self.winner = winner
4869.3.34 by Vincent Ladeuil
Finish the patch based on reviews.
241
4869.3.15 by Andrew Bennetts
Refactor so that the default text merge is the final hook tried. Other hooks can now invoke it directly via merger.default_text_merge if they wish.
242
    def is_file_merge(self):
4869.3.21 by Andrew Bennetts
Pass kinds (not pairs) to MergeHookParams.
243
        """True if this_kind and other_kind are both 'file'."""
244
        return self.this_kind == 'file' and self.other_kind == 'file'
4869.3.34 by Vincent Ladeuil
Finish the patch based on reviews.
245
4869.3.13 by Andrew Bennetts
Add simple cachedproperty decorator, and add {this,other,base}_lines cachedproperties to MergeHookParams.
246
    @decorators.cachedproperty
247
    def base_lines(self):
248
        """The lines of the 'base' version of the file."""
4797.5.1 by Robert Collins
Support state on per-file merging to permit more efficient use of configuration data.
249
        return self._merger.get_lines(self._merger.base_tree, self.file_id)
4869.3.13 by Andrew Bennetts
Add simple cachedproperty decorator, and add {this,other,base}_lines cachedproperties to MergeHookParams.
250
251
    @decorators.cachedproperty
252
    def this_lines(self):
253
        """The lines of the 'this' version of the file."""
4797.5.1 by Robert Collins
Support state on per-file merging to permit more efficient use of configuration data.
254
        return self._merger.get_lines(self._merger.this_tree, self.file_id)
4869.3.13 by Andrew Bennetts
Add simple cachedproperty decorator, and add {this,other,base}_lines cachedproperties to MergeHookParams.
255
256
    @decorators.cachedproperty
257
    def other_lines(self):
258
        """The lines of the 'other' version of the file."""
4797.5.1 by Robert Collins
Support state on per-file merging to permit more efficient use of configuration data.
259
        return self._merger.get_lines(self._merger.other_tree, self.file_id)
4869.3.13 by Andrew Bennetts
Add simple cachedproperty decorator, and add {this,other,base}_lines cachedproperties to MergeHookParams.
260
4869.3.1 by Andrew Bennetts
Basic per-file merge hook.
261
1185.12.76 by Aaron Bentley
Refactored merge and merge_inner to use Merger
262
class Merger(object):
4869.3.1 by Andrew Bennetts
Basic per-file merge hook.
263
264
    hooks = MergeHooks()
265
2255.2.31 by Robert Collins
Work in progress to make merge_inner work with dirstate trees.
266
    def __init__(self, this_branch, other_tree=None, base_tree=None,
1551.21.8 by Aaron Bentley
Remove now-redundant DummyProgress from Merger.__init__
267
                 this_tree=None, pb=None, change_reporter=None,
3146.5.3 by Aaron Bentley
Avoid retrieving revision graph twice
268
                 recurse='down', revision_graph=None):
1185.12.76 by Aaron Bentley
Refactored merge and merge_inner to use Merger
269
        object.__init__(self)
270
        self.this_branch = this_branch
2598.5.4 by Aaron Bentley
Restore original Branch.last_revision behavior, fix bits that care
271
        self.this_basis = _mod_revision.ensure_null(
272
            this_branch.last_revision())
1185.12.76 by Aaron Bentley
Refactored merge and merge_inner to use Merger
273
        self.this_rev_id = None
1534.4.26 by Robert Collins
Move working tree initialisation out from Branch.initialize, deprecated Branch.initialize to Branch.create.
274
        self.this_tree = this_tree
1185.12.83 by Aaron Bentley
Preliminary weave merge support
275
        self.this_revision_tree = None
1185.35.5 by Aaron Bentley
Made weave merge succeed if interesting files match history
276
        self.this_basis_tree = None
1185.12.76 by Aaron Bentley
Refactored merge and merge_inner to use Merger
277
        self.other_tree = other_tree
2100.3.29 by Aaron Bentley
Get merge working initially
278
        self.other_branch = None
1185.12.76 by Aaron Bentley
Refactored merge and merge_inner to use Merger
279
        self.base_tree = base_tree
280
        self.ignore_zero = False
281
        self.backup_files = False
282
        self.interesting_ids = None
2590.2.5 by Aaron Bentley
Allow selected files to be specified instead of selected ids
283
        self.interesting_files = None
1185.12.76 by Aaron Bentley
Refactored merge and merge_inner to use Merger
284
        self.show_base = False
1185.24.3 by Aaron Bentley
Integrated reprocessing into the rest of the merge stuff
285
        self.reprocess = False
4961.2.11 by Martin Pool
Pull out pbs and ProgressPhases stored in object state; just use them in single functions
286
        if pb is not None:
287
            warnings.warn("pb parameter to Merger() is deprecated and ignored")
1551.2.32 by Aaron Bentley
Handle progress phases more nicely in merge
288
        self.pp = None
2100.3.29 by Aaron Bentley
Get merge working initially
289
        self.recurse = recurse
1551.11.9 by Aaron Bentley
Apply change reporting to merge
290
        self.change_reporter = change_reporter
2590.2.11 by Aaron Bentley
Aggressively cache trees, use dirstate. re-mplement _add_parent.
291
        self._cached_trees = {}
3146.5.3 by Aaron Bentley
Avoid retrieving revision graph twice
292
        self._revision_graph = revision_graph
3146.5.1 by Aaron Bentley
Make merge --uncommitted work with merge-type weave
293
        self._base_is_ancestor = None
294
        self._base_is_other_ancestor = None
3514.4.39 by John Arbash Meinel
Use self._lca_multi_way rather than Merge3Merger._lca_multi_way by Aaron's request.
295
        self._is_criss_cross = None
3514.4.2 by John Arbash Meinel
Extract the lca trees and the base tree when we find a criss-cross
296
        self._lca_trees = None
3146.5.1 by Aaron Bentley
Make merge --uncommitted work with merge-type weave
297
4595.13.2 by Alexander Belchenko
[cherrypick revno 4650 from bzr.dev] Fix shelve on windows. (Robert Collins, #305006)
298
    def cache_trees_with_revision_ids(self, trees):
299
        """Cache any tree in trees if it has a revision_id."""
300
        for maybe_tree in trees:
301
            if maybe_tree is None:
302
                continue
303
            try:
304
                rev_id = maybe_tree.get_revision_id()
305
            except AttributeError:
306
                continue
307
            self._cached_trees[rev_id] = maybe_tree
308
3146.5.1 by Aaron Bentley
Make merge --uncommitted work with merge-type weave
309
    @property
310
    def revision_graph(self):
311
        if self._revision_graph is None:
3146.5.3 by Aaron Bentley
Avoid retrieving revision graph twice
312
            self._revision_graph = self.this_branch.repository.get_graph()
3146.5.1 by Aaron Bentley
Make merge --uncommitted work with merge-type weave
313
        return self._revision_graph
314
315
    def _set_base_is_ancestor(self, value):
316
        self._base_is_ancestor = value
317
318
    def _get_base_is_ancestor(self):
319
        if self._base_is_ancestor is None:
320
            self._base_is_ancestor = self.revision_graph.is_ancestor(
321
                self.base_rev_id, self.this_basis)
322
        return self._base_is_ancestor
323
324
    base_is_ancestor = property(_get_base_is_ancestor, _set_base_is_ancestor)
325
326
    def _set_base_is_other_ancestor(self, value):
327
        self._base_is_other_ancestor = value
328
329
    def _get_base_is_other_ancestor(self):
330
        if self._base_is_other_ancestor is None:
3249.3.1 by John Arbash Meinel
Implement cherrypick support for Merge3
331
            if self.other_basis is None:
332
                return True
333
            self._base_is_other_ancestor = self.revision_graph.is_ancestor(
3146.5.3 by Aaron Bentley
Avoid retrieving revision graph twice
334
                self.base_rev_id, self.other_basis)
3146.5.1 by Aaron Bentley
Make merge --uncommitted work with merge-type weave
335
        return self._base_is_other_ancestor
336
337
    base_is_other_ancestor = property(_get_base_is_other_ancestor,
338
                                      _set_base_is_other_ancestor)
2590.2.11 by Aaron Bentley
Aggressively cache trees, use dirstate. re-mplement _add_parent.
339
1551.15.67 by Aaron Bentley
Stop using _merge_helper for merging
340
    @staticmethod
1551.21.7 by Aaron Bentley
Fix progress warning
341
    def from_uncommitted(tree, other_tree, pb=None, base_tree=None):
1551.15.74 by Aaron Bentley
Textual updates from review
342
        """Return a Merger for uncommitted changes in other_tree.
343
344
        :param tree: The tree to merge into
345
        :param other_tree: The tree to get uncommitted changes from
346
        :param pb: A progress indicator
3363.17.17 by Aaron Bentley
Start testing merging PreviewTree as OTHER
347
        :param base_tree: The basis to use for the merge.  If unspecified,
348
            other_tree.basis_tree() will be used.
1551.15.74 by Aaron Bentley
Textual updates from review
349
        """
3363.17.17 by Aaron Bentley
Start testing merging PreviewTree as OTHER
350
        if base_tree is None:
351
            base_tree = other_tree.basis_tree()
352
        merger = Merger(tree.branch, other_tree, base_tree, tree, pb)
1551.15.67 by Aaron Bentley
Stop using _merge_helper for merging
353
        merger.base_rev_id = merger.base_tree.get_revision_id()
354
        merger.other_rev_id = None
3146.5.1 by Aaron Bentley
Make merge --uncommitted work with merge-type weave
355
        merger.other_basis = merger.base_rev_id
1551.15.67 by Aaron Bentley
Stop using _merge_helper for merging
356
        return merger
357
358
    @classmethod
359
    def from_mergeable(klass, tree, mergeable, pb):
1551.15.74 by Aaron Bentley
Textual updates from review
360
        """Return a Merger for a bundle or merge directive.
361
362
        :param tree: The tree to merge changes into
363
        :param mergeable: A merge directive or bundle
364
        :param pb: A progress indicator
365
        """
1551.15.67 by Aaron Bentley
Stop using _merge_helper for merging
366
        mergeable.install_revisions(tree.branch.repository)
367
        base_revision_id, other_revision_id, verified =\
368
            mergeable.get_merge_request(tree.branch.repository)
3146.5.3 by Aaron Bentley
Avoid retrieving revision graph twice
369
        revision_graph = tree.branch.repository.get_graph()
1551.19.44 by Aaron Bentley
Fix handling of old merge directives with stricter get_parent_map
370
        if base_revision_id is not None:
371
            if (base_revision_id != _mod_revision.NULL_REVISION and
372
                revision_graph.is_ancestor(
373
                base_revision_id, tree.branch.last_revision())):
374
                base_revision_id = None
375
            else:
4721.3.1 by Vincent Ladeuil
Cleanup imports.
376
                trace.warning('Performing cherrypick')
1551.15.67 by Aaron Bentley
Stop using _merge_helper for merging
377
        merger = klass.from_revision_ids(pb, tree, other_revision_id,
3146.5.3 by Aaron Bentley
Avoid retrieving revision graph twice
378
                                         base_revision_id, revision_graph=
379
                                         revision_graph)
1551.15.67 by Aaron Bentley
Stop using _merge_helper for merging
380
        return merger, verified
381
382
    @staticmethod
3249.3.1 by John Arbash Meinel
Implement cherrypick support for Merge3
383
    def from_revision_ids(pb, tree, other, base=None, other_branch=None,
3363.17.1 by Aaron Bentley
Avoid inventory for merge and transform code
384
                          base_branch=None, revision_graph=None,
385
                          tree_branch=None):
1551.15.74 by Aaron Bentley
Textual updates from review
386
        """Return a Merger for revision-ids.
387
3363.17.15 by Aaron Bentley
Update docs
388
        :param pb: A progress indicator
1551.15.74 by Aaron Bentley
Textual updates from review
389
        :param tree: The tree to merge changes into
390
        :param other: The revision-id to use as OTHER
391
        :param base: The revision-id to use as BASE.  If not specified, will
392
            be auto-selected.
393
        :param other_branch: A branch containing the other revision-id.  If
3249.3.1 by John Arbash Meinel
Implement cherrypick support for Merge3
394
            not supplied, tree.branch is used.
1551.15.74 by Aaron Bentley
Textual updates from review
395
        :param base_branch: A branch containing the base revision-id.  If
3249.3.1 by John Arbash Meinel
Implement cherrypick support for Merge3
396
            not supplied, other_branch or tree.branch will be used.
397
        :param revision_graph: If you have a revision_graph precomputed, pass
398
            it in, otherwise it will be created for you.
3363.17.15 by Aaron Bentley
Update docs
399
        :param tree_branch: The branch associated with tree.  If not supplied,
400
            tree.branch will be used.
1551.15.74 by Aaron Bentley
Textual updates from review
401
        """
3363.17.1 by Aaron Bentley
Avoid inventory for merge and transform code
402
        if tree_branch is None:
403
            tree_branch = tree.branch
404
        merger = Merger(tree_branch, this_tree=tree, pb=pb,
3146.5.3 by Aaron Bentley
Avoid retrieving revision graph twice
405
                        revision_graph=revision_graph)
1551.15.67 by Aaron Bentley
Stop using _merge_helper for merging
406
        if other_branch is None:
3249.3.1 by John Arbash Meinel
Implement cherrypick support for Merge3
407
            other_branch = tree.branch
1551.15.67 by Aaron Bentley
Stop using _merge_helper for merging
408
        merger.set_other_revision(other, other_branch)
409
        if base is None:
410
            merger.find_base()
411
        else:
412
            if base_branch is None:
413
                base_branch = other_branch
414
            merger.set_base_revision(base, base_branch)
415
        return merger
416
2590.2.11 by Aaron Bentley
Aggressively cache trees, use dirstate. re-mplement _add_parent.
417
    def revision_tree(self, revision_id, branch=None):
418
        if revision_id not in self._cached_trees:
419
            if branch is None:
420
                branch = self.this_branch
421
            try:
422
                tree = self.this_tree.revision_tree(revision_id)
423
            except errors.NoSuchRevisionInTree:
424
                tree = branch.repository.revision_tree(revision_id)
425
            self._cached_trees[revision_id] = tree
426
        return self._cached_trees[revision_id]
427
2485.8.58 by Vincent Ladeuil
merge bzr.dev@1617
428
    def _get_tree(self, treespec, possible_transports=None):
2590.2.11 by Aaron Bentley
Aggressively cache trees, use dirstate. re-mplement _add_parent.
429
        from bzrlib import workingtree
430
        location, revno = treespec
431
        if revno is None:
432
            tree = workingtree.WorkingTree.open_containing(location)[0]
433
            return tree.branch, tree
4721.3.1 by Vincent Ladeuil
Cleanup imports.
434
        branch = _mod_branch.Branch.open_containing(
435
            location, possible_transports)[0]
2590.2.11 by Aaron Bentley
Aggressively cache trees, use dirstate. re-mplement _add_parent.
436
        if revno == -1:
437
            revision_id = branch.last_revision()
438
        else:
439
            revision_id = branch.get_rev_id(revno)
4721.3.1 by Vincent Ladeuil
Cleanup imports.
440
        revision_id = _mod_revision.ensure_null(revision_id)
2590.2.11 by Aaron Bentley
Aggressively cache trees, use dirstate. re-mplement _add_parent.
441
        return branch, self.revision_tree(revision_id, branch)
1185.12.83 by Aaron Bentley
Preliminary weave merge support
442
4721.3.2 by Vincent Ladeuil
Simplify mutable_tree.has_changes() and update call sites.
443
    @deprecated_method(deprecated_in((2, 1, 0)))
1185.12.83 by Aaron Bentley
Preliminary weave merge support
444
    def ensure_revision_trees(self):
445
        if self.this_revision_tree is None:
2590.2.11 by Aaron Bentley
Aggressively cache trees, use dirstate. re-mplement _add_parent.
446
            self.this_basis_tree = self.revision_tree(self.this_basis)
1185.35.5 by Aaron Bentley
Made weave merge succeed if interesting files match history
447
            if self.this_basis == self.this_rev_id:
448
                self.this_revision_tree = self.this_basis_tree
449
1185.12.83 by Aaron Bentley
Preliminary weave merge support
450
        if self.other_rev_id is None:
451
            other_basis_tree = self.revision_tree(self.other_basis)
4503.1.2 by Vincent Ladeuil
Use tree.has_changes() where appropriate (the test suite caught a
452
            if other_basis_tree.has_changes(self.other_tree):
4869.2.1 by Andrew Bennetts
Trivial improvements to bzrlib/merge.py suggested by pyflakes.
453
                raise errors.WorkingTreeNotRevision(self.this_tree)
1773.4.1 by Martin Pool
Add pyflakes makefile target; fix many warnings
454
            other_rev_id = self.other_basis
1185.12.83 by Aaron Bentley
Preliminary weave merge support
455
            self.other_tree = other_basis_tree
456
4721.3.2 by Vincent Ladeuil
Simplify mutable_tree.has_changes() and update call sites.
457
    @deprecated_method(deprecated_in((2, 1, 0)))
1185.12.83 by Aaron Bentley
Preliminary weave merge support
458
    def file_revisions(self, file_id):
459
        self.ensure_revision_trees()
460
        def get_id(tree, file_id):
461
            revision_id = tree.inventory[file_id].revision
462
            return revision_id
1185.35.5 by Aaron Bentley
Made weave merge succeed if interesting files match history
463
        if self.this_rev_id is None:
464
            if self.this_basis_tree.get_file_sha1(file_id) != \
465
                self.this_tree.get_file_sha1(file_id):
4721.3.1 by Vincent Ladeuil
Cleanup imports.
466
                raise errors.WorkingTreeNotRevision(self.this_tree)
1185.35.5 by Aaron Bentley
Made weave merge succeed if interesting files match history
467
468
        trees = (self.this_basis_tree, self.other_tree)
1185.12.83 by Aaron Bentley
Preliminary weave merge support
469
        return [get_id(tree, file_id) for tree in trees]
470
4721.3.2 by Vincent Ladeuil
Simplify mutable_tree.has_changes() and update call sites.
471
    @deprecated_method(deprecated_in((2, 1, 0)))
1185.82.44 by Aaron Bentley
Switch to merge_changeset in test suite
472
    def check_basis(self, check_clean, require_commits=True):
473
        if self.this_basis is None and require_commits is True:
4721.3.1 by Vincent Ladeuil
Cleanup imports.
474
            raise errors.BzrCommandError(
475
                "This branch has no commits."
476
                " (perhaps you would prefer 'bzr pull')")
1185.12.76 by Aaron Bentley
Refactored merge and merge_inner to use Merger
477
        if check_clean:
478
            self.compare_basis()
479
            if self.this_basis != self.this_rev_id:
2796.2.1 by Aaron Bentley
Begin work on reconfigure command
480
                raise errors.UncommittedChanges(self.this_tree)
1185.12.76 by Aaron Bentley
Refactored merge and merge_inner to use Merger
481
4721.3.2 by Vincent Ladeuil
Simplify mutable_tree.has_changes() and update call sites.
482
    @deprecated_method(deprecated_in((2, 1, 0)))
1185.12.76 by Aaron Bentley
Refactored merge and merge_inner to use Merger
483
    def compare_basis(self):
2590.2.20 by Aaron Bentley
Fix handling of ghost base trees
484
        try:
485
            basis_tree = self.revision_tree(self.this_tree.last_revision())
3350.6.4 by Robert Collins
First cut at pluralised VersionedFiles. Some rather massive API incompatabilities, primarily because of the difficulty of coherence among competing stores.
486
        except errors.NoSuchRevision:
2590.2.20 by Aaron Bentley
Fix handling of ghost base trees
487
            basis_tree = self.this_tree.basis_tree()
4503.1.2 by Vincent Ladeuil
Use tree.has_changes() where appropriate (the test suite caught a
488
        if not self.this_tree.has_changes(basis_tree):
1185.12.76 by Aaron Bentley
Refactored merge and merge_inner to use Merger
489
            self.this_rev_id = self.this_basis
490
491
    def set_interesting_files(self, file_list):
2590.2.7 by Aaron Bentley
Misc cleanup
492
        self.interesting_files = file_list
1457.1.8 by Robert Collins
Replace the WorkingTree.revert method algorithm with a call to merge_inner.
493
1185.12.76 by Aaron Bentley
Refactored merge and merge_inner to use Merger
494
    def set_pending(self):
4721.3.2 by Vincent Ladeuil
Simplify mutable_tree.has_changes() and update call sites.
495
        if (not self.base_is_ancestor or not self.base_is_other_ancestor
496
            or self.other_rev_id is None):
1185.12.77 by Aaron Bentley
Prevented all ancestors from being marked as pending merges
497
            return
2590.2.11 by Aaron Bentley
Aggressively cache trees, use dirstate. re-mplement _add_parent.
498
        self._add_parent()
499
500
    def _add_parent(self):
501
        new_parents = self.this_tree.get_parent_ids() + [self.other_rev_id]
2590.2.20 by Aaron Bentley
Fix handling of ghost base trees
502
        new_parent_trees = []
4797.16.1 by Andrew Bennetts
Replace many try/finally blocks in merge.py with more robust OperationWithCleanups.
503
        operation = OperationWithCleanups(self.this_tree.set_parent_trees)
2590.2.20 by Aaron Bentley
Fix handling of ghost base trees
504
        for revision_id in new_parents:
505
            try:
506
                tree = self.revision_tree(revision_id)
3350.6.4 by Robert Collins
First cut at pluralised VersionedFiles. Some rather massive API incompatabilities, primarily because of the difficulty of coherence among competing stores.
507
            except errors.NoSuchRevision:
2590.2.20 by Aaron Bentley
Fix handling of ghost base trees
508
                tree = None
509
            else:
510
                tree.lock_read()
4797.16.1 by Andrew Bennetts
Replace many try/finally blocks in merge.py with more robust OperationWithCleanups.
511
                operation.add_cleanup(tree.unlock)
2590.2.20 by Aaron Bentley
Fix handling of ghost base trees
512
            new_parent_trees.append((revision_id, tree))
4797.16.1 by Andrew Bennetts
Replace many try/finally blocks in merge.py with more robust OperationWithCleanups.
513
        operation.run_simple(new_parent_trees, allow_leftmost_as_ghost=True)
1185.12.76 by Aaron Bentley
Refactored merge and merge_inner to use Merger
514
2485.8.37 by Vincent Ladeuil
Fix merge multiple connections. Test suite *not* passing (sftp
515
    def set_other(self, other_revision, possible_transports=None):
1979.2.1 by Robert Collins
(robertc) adds a convenience method "merge_from_branch" to WorkingTree.
516
        """Set the revision and tree to merge from.
517
518
        This sets the other_tree, other_rev_id, other_basis attributes.
519
520
        :param other_revision: The [path, revision] list to merge from.
521
        """
2485.8.58 by Vincent Ladeuil
merge bzr.dev@1617
522
        self.other_branch, self.other_tree = self._get_tree(other_revision,
523
                                                            possible_transports)
1185.12.76 by Aaron Bentley
Refactored merge and merge_inner to use Merger
524
        if other_revision[1] == -1:
2598.5.4 by Aaron Bentley
Restore original Branch.last_revision behavior, fix bits that care
525
            self.other_rev_id = _mod_revision.ensure_null(
526
                self.other_branch.last_revision())
2598.5.1 by Aaron Bentley
Start eliminating the use of None to indicate null revision
527
            if _mod_revision.is_null(self.other_rev_id):
4721.3.1 by Vincent Ladeuil
Cleanup imports.
528
                raise errors.NoCommits(self.other_branch)
1185.12.76 by Aaron Bentley
Refactored merge and merge_inner to use Merger
529
            self.other_basis = self.other_rev_id
530
        elif other_revision[1] is not None:
2100.3.31 by Aaron Bentley
Merged bzr.dev (17 tests failing)
531
            self.other_rev_id = self.other_branch.get_rev_id(other_revision[1])
1185.12.76 by Aaron Bentley
Refactored merge and merge_inner to use Merger
532
            self.other_basis = self.other_rev_id
533
        else:
534
            self.other_rev_id = None
2100.3.31 by Aaron Bentley
Merged bzr.dev (17 tests failing)
535
            self.other_basis = self.other_branch.last_revision()
1185.12.76 by Aaron Bentley
Refactored merge and merge_inner to use Merger
536
            if self.other_basis is None:
4721.3.1 by Vincent Ladeuil
Cleanup imports.
537
                raise errors.NoCommits(self.other_branch)
2590.2.11 by Aaron Bentley
Aggressively cache trees, use dirstate. re-mplement _add_parent.
538
        if self.other_rev_id is not None:
539
            self._cached_trees[self.other_rev_id] = self.other_tree
2590.2.19 by Aaron Bentley
Avoid fetch within a repository
540
        self._maybe_fetch(self.other_branch,self.this_branch, self.other_basis)
1185.12.76 by Aaron Bentley
Refactored merge and merge_inner to use Merger
541
2100.3.29 by Aaron Bentley
Get merge working initially
542
    def set_other_revision(self, revision_id, other_branch):
543
        """Set 'other' based on a branch and revision id
544
545
        :param revision_id: The revision to use for a tree
546
        :param other_branch: The branch containing this tree
547
        """
548
        self.other_rev_id = revision_id
549
        self.other_branch = other_branch
2590.2.19 by Aaron Bentley
Avoid fetch within a repository
550
        self._maybe_fetch(other_branch, self.this_branch, self.other_rev_id)
2100.3.29 by Aaron Bentley
Get merge working initially
551
        self.other_tree = self.revision_tree(revision_id)
552
        self.other_basis = revision_id
553
2520.4.110 by Aaron Bentley
Implement cherrypick support for merge directives
554
    def set_base_revision(self, revision_id, branch):
555
        """Set 'base' based on a branch and revision id
556
557
        :param revision_id: The revision to use for a tree
558
        :param branch: The branch containing this tree
559
        """
560
        self.base_rev_id = revision_id
561
        self.base_branch = branch
2520.4.132 by Aaron Bentley
Merge from bzr.dev
562
        self._maybe_fetch(branch, self.this_branch, revision_id)
2520.4.110 by Aaron Bentley
Implement cherrypick support for merge directives
563
        self.base_tree = self.revision_tree(revision_id)
2520.4.132 by Aaron Bentley
Merge from bzr.dev
564
2590.2.19 by Aaron Bentley
Avoid fetch within a repository
565
    def _maybe_fetch(self, source, target, revision_id):
2665.5.2 by Aaron Bentley
Switch commit and merge to Repository.has_same_location
566
        if not source.repository.has_same_location(target.repository):
2590.2.19 by Aaron Bentley
Avoid fetch within a repository
567
            target.fetch(source, revision_id)
2520.4.110 by Aaron Bentley
Implement cherrypick support for merge directives
568
1185.82.25 by Aaron Bentley
Added changeset-merging functionality
569
    def find_base(self):
4721.3.1 by Vincent Ladeuil
Cleanup imports.
570
        revisions = [_mod_revision.ensure_null(self.this_basis),
571
                     _mod_revision.ensure_null(self.other_basis)]
572
        if _mod_revision.NULL_REVISION in revisions:
573
            self.base_rev_id = _mod_revision.NULL_REVISION
3514.4.36 by John Arbash Meinel
Optimizing how we extracted multiple trees left a path that didn't create a base_tree.
574
            self.base_tree = self.revision_tree(self.base_rev_id)
3514.4.39 by John Arbash Meinel
Use self._lca_multi_way rather than Merge3Merger._lca_multi_way by Aaron's request.
575
            self._is_criss_cross = False
2590.2.22 by Aaron Bentley
Remove cruft
576
        else:
3514.4.2 by John Arbash Meinel
Extract the lca trees and the base tree when we find a criss-cross
577
            lcas = self.revision_graph.find_lca(revisions[0], revisions[1])
3514.4.39 by John Arbash Meinel
Use self._lca_multi_way rather than Merge3Merger._lca_multi_way by Aaron's request.
578
            self._is_criss_cross = False
3514.4.2 by John Arbash Meinel
Extract the lca trees and the base tree when we find a criss-cross
579
            if len(lcas) == 0:
4721.3.1 by Vincent Ladeuil
Cleanup imports.
580
                self.base_rev_id = _mod_revision.NULL_REVISION
3514.4.2 by John Arbash Meinel
Extract the lca trees and the base tree when we find a criss-cross
581
            elif len(lcas) == 1:
582
                self.base_rev_id = list(lcas)[0]
583
            else: # len(lcas) > 1
3514.4.23 by John Arbash Meinel
Handle when there are more than 2 LCAs while searching for the unique lca.
584
                if len(lcas) > 2:
585
                    # find_unique_lca can only handle 2 nodes, so we have to
586
                    # start back at the beginning. It is a shame to traverse
587
                    # the graph again, but better than re-implementing
588
                    # find_unique_lca.
589
                    self.base_rev_id = self.revision_graph.find_unique_lca(
590
                                            revisions[0], revisions[1])
591
                else:
592
                    self.base_rev_id = self.revision_graph.find_unique_lca(
593
                                            *lcas)
3514.4.2 by John Arbash Meinel
Extract the lca trees and the base tree when we find a criss-cross
594
                self._is_criss_cross = True
4721.3.1 by Vincent Ladeuil
Cleanup imports.
595
            if self.base_rev_id == _mod_revision.NULL_REVISION:
596
                raise errors.UnrelatedBranches()
3514.4.2 by John Arbash Meinel
Extract the lca trees and the base tree when we find a criss-cross
597
            if self._is_criss_cross:
4721.3.1 by Vincent Ladeuil
Cleanup imports.
598
                trace.warning('Warning: criss-cross merge encountered.  See bzr'
599
                              ' help criss-cross.')
600
                trace.mutter('Criss-cross lcas: %r' % lcas)
3514.4.2 by John Arbash Meinel
Extract the lca trees and the base tree when we find a criss-cross
601
                interesting_revision_ids = [self.base_rev_id]
602
                interesting_revision_ids.extend(lcas)
603
                interesting_trees = dict((t.get_revision_id(), t)
604
                    for t in self.this_branch.repository.revision_trees(
605
                        interesting_revision_ids))
606
                self._cached_trees.update(interesting_trees)
607
                self.base_tree = interesting_trees.pop(self.base_rev_id)
3514.4.13 by John Arbash Meinel
Switch the lca_trees to be in 'find_merge_order'.
608
                sorted_lca_keys = self.revision_graph.find_merge_order(
609
                    revisions[0], lcas)
610
                self._lca_trees = [interesting_trees[key]
611
                                   for key in sorted_lca_keys]
3514.4.2 by John Arbash Meinel
Extract the lca trees and the base tree when we find a criss-cross
612
            else:
613
                self.base_tree = self.revision_tree(self.base_rev_id)
2590.2.13 by Aaron Bentley
Make find_base implement the base_finding code
614
        self.base_is_ancestor = True
2590.2.18 by Aaron Bentley
Merge is_ancestor fix
615
        self.base_is_other_ancestor = True
4721.3.1 by Vincent Ladeuil
Cleanup imports.
616
        trace.mutter('Base revid: %r' % self.base_rev_id)
1185.82.25 by Aaron Bentley
Added changeset-merging functionality
617
1185.12.76 by Aaron Bentley
Refactored merge and merge_inner to use Merger
618
    def set_base(self, base_revision):
1979.2.1 by Robert Collins
(robertc) adds a convenience method "merge_from_branch" to WorkingTree.
619
        """Set the base revision to use for the merge.
620
621
        :param base_revision: A 2-list containing a path and revision number.
622
        """
4721.3.1 by Vincent Ladeuil
Cleanup imports.
623
        trace.mutter("doing merge() with no base_revision specified")
1185.12.76 by Aaron Bentley
Refactored merge and merge_inner to use Merger
624
        if base_revision == [None, None]:
2590.2.13 by Aaron Bentley
Make find_base implement the base_finding code
625
            self.find_base()
1185.12.76 by Aaron Bentley
Refactored merge and merge_inner to use Merger
626
        else:
2590.2.11 by Aaron Bentley
Aggressively cache trees, use dirstate. re-mplement _add_parent.
627
            base_branch, self.base_tree = self._get_tree(base_revision)
1185.12.76 by Aaron Bentley
Refactored merge and merge_inner to use Merger
628
            if base_revision[1] == -1:
629
                self.base_rev_id = base_branch.last_revision()
630
            elif base_revision[1] is None:
2598.5.3 by Aaron Bentley
Push NULL_REVISION deeper
631
                self.base_rev_id = _mod_revision.NULL_REVISION
493 by Martin Pool
- Merge aaron's merge command
632
            else:
2598.5.3 by Aaron Bentley
Push NULL_REVISION deeper
633
                self.base_rev_id = _mod_revision.ensure_null(
634
                    base_branch.get_rev_id(base_revision[1]))
2590.2.19 by Aaron Bentley
Avoid fetch within a repository
635
            self._maybe_fetch(base_branch, self.this_branch, self.base_rev_id)
1185.12.76 by Aaron Bentley
Refactored merge and merge_inner to use Merger
636
3008.1.8 by Michael Hudson
extract merger creation into a method
637
    def make_merger(self):
2255.2.31 by Robert Collins
Work in progress to make merge_inner work with dirstate trees.
638
        kwargs = {'working_tree':self.this_tree, 'this_tree': self.this_tree,
639
                  'other_tree': self.other_tree,
1551.2.32 by Aaron Bentley
Handle progress phases more nicely in merge
640
                  'interesting_ids': self.interesting_ids,
2590.2.5 by Aaron Bentley
Allow selected files to be specified instead of selected ids
641
                  'interesting_files': self.interesting_files,
4961.2.20 by Martin Pool
Resolve conflicts with trunk
642
                  'this_branch': self.this_branch,
3008.1.11 by Michael Hudson
restore the default behaviour of Merge3Merger.__init__().
643
                  'do_merge': False}
1534.7.84 by Aaron Bentley
Added reprocess support, support for varying merge types
644
        if self.merge_type.requires_base:
645
            kwargs['base_tree'] = self.base_tree
1534.7.137 by Aaron Bentley
Avoided generating a new tree for every weave merge
646
        if self.merge_type.supports_reprocess:
647
            kwargs['reprocess'] = self.reprocess
648
        elif self.reprocess:
4721.3.1 by Vincent Ladeuil
Cleanup imports.
649
            raise errors.BzrError(
650
                "Conflict reduction is not supported for merge"
651
                " type %s." % self.merge_type)
1534.7.137 by Aaron Bentley
Avoided generating a new tree for every weave merge
652
        if self.merge_type.supports_show_base:
653
            kwargs['show_base'] = self.show_base
654
        elif self.show_base:
4721.3.1 by Vincent Ladeuil
Cleanup imports.
655
            raise errors.BzrError("Showing base is not supported for this"
656
                                  " merge type. %s" % self.merge_type)
3062.2.7 by Aaron Bentley
Prevent reverse cherry-picking with weave
657
        if (not getattr(self.merge_type, 'supports_reverse_cherrypick', True)
658
            and not self.base_is_other_ancestor):
659
            raise errors.CannotReverseCherrypick()
3249.3.1 by John Arbash Meinel
Implement cherrypick support for Merge3
660
        if self.merge_type.supports_cherrypick:
3062.2.9 by Aaron Bentley
Don't use the base if not cherrypicking
661
            kwargs['cherrypick'] = (not self.base_is_ancestor or
662
                                    not self.base_is_other_ancestor)
3514.4.4 by John Arbash Meinel
Test that the lca_trees are passed down to the Merger object when appropriate.
663
        if self._is_criss_cross and getattr(self.merge_type,
664
                                            'supports_lca_trees', False):
665
            kwargs['lca_trees'] = self._lca_trees
4961.2.11 by Martin Pool
Pull out pbs and ProgressPhases stored in object state; just use them in single functions
666
        return self.merge_type(pb=None,
3008.1.8 by Michael Hudson
extract merger creation into a method
667
                               change_reporter=self.change_reporter,
668
                               **kwargs)
669
4797.16.1 by Andrew Bennetts
Replace many try/finally blocks in merge.py with more robust OperationWithCleanups.
670
    def _do_merge_to(self):
671
        merge = self.make_merger()
4273.1.9 by Aaron Bentley
Cleanup
672
        if self.other_branch is not None:
673
            self.other_branch.update_references(self.this_branch)
3619.6.4 by Mark Hammond
do_merge gets one 'finally' per lock so locks always released on failure.
674
        merge.do_merge()
675
        if self.recurse == 'down':
676
            for relpath, file_id in self.this_tree.iter_references():
677
                sub_tree = self.this_tree.get_nested_tree(file_id, relpath)
678
                other_revision = self.other_tree.get_reference_revision(
679
                    file_id, relpath)
680
                if  other_revision == sub_tree.last_revision():
681
                    continue
682
                sub_merge = Merger(sub_tree.branch, this_tree=sub_tree)
683
                sub_merge.merge_type = self.merge_type
684
                other_branch = self.other_branch.reference_parent(file_id, relpath)
685
                sub_merge.set_other_revision(other_revision, other_branch)
686
                base_revision = self.base_tree.get_reference_revision(file_id)
687
                sub_merge.base_tree = \
688
                    sub_tree.branch.repository.revision_tree(base_revision)
689
                sub_merge.base_rev_id = base_revision
690
                sub_merge.do_merge()
4797.16.1 by Andrew Bennetts
Replace many try/finally blocks in merge.py with more robust OperationWithCleanups.
691
        return merge
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
692
3008.1.8 by Michael Hudson
extract merger creation into a method
693
    def do_merge(self):
4797.16.1 by Andrew Bennetts
Replace many try/finally blocks in merge.py with more robust OperationWithCleanups.
694
        operation = OperationWithCleanups(self._do_merge_to)
2255.2.50 by John Arbash Meinel
merge only needs a lock_tree_write() on the working tree, not a full lock_write()
695
        self.this_tree.lock_tree_write()
4797.16.1 by Andrew Bennetts
Replace many try/finally blocks in merge.py with more robust OperationWithCleanups.
696
        operation.add_cleanup(self.this_tree.unlock)
697
        if self.base_tree is not None:
698
            self.base_tree.lock_read()
699
            operation.add_cleanup(self.base_tree.unlock)
700
        if self.other_tree is not None:
701
            self.other_tree.lock_read()
702
            operation.add_cleanup(self.other_tree.unlock)
703
        merge = operation.run_simple()
1534.7.151 by Aaron Bentley
Fixed all changes applied successfully
704
        if len(merge.cooked_conflicts) == 0:
4721.3.1 by Vincent Ladeuil
Cleanup imports.
705
            if not self.ignore_zero and not trace.is_quiet():
706
                trace.note("All changes applied successfully.")
1534.7.141 by Aaron Bentley
Added conflict reporting
707
        else:
4721.3.1 by Vincent Ladeuil
Cleanup imports.
708
            trace.note("%d conflicts encountered."
709
                       % len(merge.cooked_conflicts))
1534.7.141 by Aaron Bentley
Added conflict reporting
710
1534.7.134 by Aaron Bentley
Hid raw conflicts
711
        return len(merge.cooked_conflicts)
1185.12.76 by Aaron Bentley
Refactored merge and merge_inner to use Merger
712
1545.2.4 by Aaron Bentley
PEP8 fixes
713
3514.4.20 by John Arbash Meinel
Use the _lca_multi_way to work out if there is actually a kind/parent/name/content change.
714
class _InventoryNoneEntry(object):
715
    """This represents an inventory entry which *isn't there*.
716
717
    It simplifies the merging logic if we always have an InventoryEntry, even
718
    if it isn't actually present
719
    """
720
    executable = None
721
    kind = None
722
    name = None
723
    parent_id = None
724
    revision = None
725
    symlink_target = None
726
    text_sha1 = None
727
728
_none_entry = _InventoryNoneEntry()
729
730
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
731
class Merge3Merger(object):
1534.7.167 by Aaron Bentley
PEP8 and comment cleanups
732
    """Three-way merger that uses the merge3 text merger"""
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
733
    requires_base = True
734
    supports_reprocess = True
735
    supports_show_base = True
736
    history_based = False
3249.3.1 by John Arbash Meinel
Implement cherrypick support for Merge3
737
    supports_cherrypick = True
3062.2.7 by Aaron Bentley
Prevent reverse cherry-picking with weave
738
    supports_reverse_cherrypick = True
2590.2.7 by Aaron Bentley
Misc cleanup
739
    winner_idx = {"this": 2, "other": 1, "conflict": 1}
3514.4.4 by John Arbash Meinel
Test that the lca_trees are passed down to the Merger object when appropriate.
740
    supports_lca_trees = True
1534.7.167 by Aaron Bentley
PEP8 and comment cleanups
741
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
742
    def __init__(self, working_tree, this_tree, base_tree, other_tree,
1558.2.2 by Aaron Bentley
Make remerge honour interesting-ids
743
                 interesting_ids=None, reprocess=False, show_base=False,
4961.2.9 by Martin Pool
Rip out most remaining uses of DummyProgressBar
744
                 pb=None, pp=None, change_reporter=None,
3249.3.1 by John Arbash Meinel
Implement cherrypick support for Merge3
745
                 interesting_files=None, do_merge=True,
4869.3.27 by Andrew Bennetts
Move news_merge plugin from contrib to bzrlib/plugins, change it to be enabled via a 'news_merge_files' config option, move more code out of the __init__ to minimise overhead, and add lots of docstrings, add NEWS entry.
746
                 cherrypick=False, lca_trees=None, this_branch=None):
2590.2.10 by Aaron Bentley
Updates from review
747
        """Initialize the merger object and perform the merge.
748
749
        :param working_tree: The working tree to apply the merge to
750
        :param this_tree: The local tree in the merge operation
751
        :param base_tree: The common tree in the merge operation
4031.3.1 by Frank Aspell
Fixing various typos
752
        :param other_tree: The other tree to merge changes from
5162.3.1 by Aaron Bentley
Fix switch/merge when a ConfigurableFileMerger is used.
753
        :param this_branch: The branch associated with this_tree.  Defaults to
754
            this_tree.branch if not supplied.
2590.2.10 by Aaron Bentley
Updates from review
755
        :param interesting_ids: The file_ids of files that should be
756
            participate in the merge.  May not be combined with
757
            interesting_files.
758
        :param: reprocess If True, perform conflict-reduction processing.
759
        :param show_base: If True, show the base revision in text conflicts.
760
            (incompatible with reprocess)
4961.2.9 by Martin Pool
Rip out most remaining uses of DummyProgressBar
761
        :param pb: ignored
2590.2.10 by Aaron Bentley
Updates from review
762
        :param pp: A ProgressPhase object
763
        :param change_reporter: An object that should report changes made
764
        :param interesting_files: The tree-relative paths of files that should
765
            participate in the merge.  If these paths refer to directories,
766
            the contents of those directories will also be included.  May not
767
            be combined with interesting_ids.  If neither interesting_files nor
768
            interesting_ids is specified, all files may participate in the
769
            merge.
3514.4.4 by John Arbash Meinel
Test that the lca_trees are passed down to the Merger object when appropriate.
770
        :param lca_trees: Can be set to a dictionary of {revision_id:rev_tree}
771
            if the ancestry was found to include a criss-cross merge.
772
            Otherwise should be None.
2590.2.10 by Aaron Bentley
Updates from review
773
        """
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
774
        object.__init__(self)
3376.2.4 by Martin Pool
Remove every assert statement from bzrlib!
775
        if interesting_files is not None and interesting_ids is not None:
776
            raise ValueError(
777
                'specify either interesting_ids or interesting_files')
5162.3.1 by Aaron Bentley
Fix switch/merge when a ConfigurableFileMerger is used.
778
        if this_branch is None:
779
            this_branch = this_tree.branch
2590.2.4 by Aaron Bentley
Move entry generation to a helper
780
        self.interesting_ids = interesting_ids
2590.2.5 by Aaron Bentley
Allow selected files to be specified instead of selected ids
781
        self.interesting_files = interesting_files
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
782
        self.this_tree = working_tree
783
        self.base_tree = base_tree
784
        self.other_tree = other_tree
4869.3.27 by Andrew Bennetts
Move news_merge plugin from contrib to bzrlib/plugins, change it to be enabled via a 'news_merge_files' config option, move more code out of the __init__ to minimise overhead, and add lots of docstrings, add NEWS entry.
785
        self.this_branch = this_branch
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
786
        self._raw_conflicts = []
787
        self.cooked_conflicts = []
788
        self.reprocess = reprocess
789
        self.show_base = show_base
3514.4.4 by John Arbash Meinel
Test that the lca_trees are passed down to the Merger object when appropriate.
790
        self._lca_trees = lca_trees
3514.4.37 by John Arbash Meinel
Tests pass when using _entries_lca as the default algorithm.
791
        # Uncommenting this will change the default algorithm to always use
792
        # _entries_lca. This can be useful for running the test suite and
793
        # making sure we haven't missed any corner cases.
3514.4.34 by John Arbash Meinel
Handle symlinks properly when objects are not RevisionTrees
794
        # if lca_trees is None:
795
        #     self._lca_trees = [self.base_tree]
1551.11.9 by Aaron Bentley
Apply change reporting to merge
796
        self.change_reporter = change_reporter
3249.3.1 by John Arbash Meinel
Implement cherrypick support for Merge3
797
        self.cherrypick = cherrypick
3008.1.11 by Michael Hudson
restore the default behaviour of Merge3Merger.__init__().
798
        if do_merge:
799
            self.do_merge()
4961.2.12 by Martin Pool
Further pb/pp removals
800
        if pp is not None:
801
            warnings.warn("pp argument to Merge3Merger is deprecated")
802
        if pb is not None:
803
            warnings.warn("pb argument to Merge3Merger is deprecated")
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
804
3008.1.6 by Michael Hudson
chop up Merge3Merger.__init__ into pieces
805
    def do_merge(self):
4797.16.1 by Andrew Bennetts
Replace many try/finally blocks in merge.py with more robust OperationWithCleanups.
806
        operation = OperationWithCleanups(self._do_merge)
3008.1.6 by Michael Hudson
chop up Merge3Merger.__init__ into pieces
807
        self.this_tree.lock_tree_write()
4797.16.1 by Andrew Bennetts
Replace many try/finally blocks in merge.py with more robust OperationWithCleanups.
808
        operation.add_cleanup(self.this_tree.unlock)
3008.1.6 by Michael Hudson
chop up Merge3Merger.__init__ into pieces
809
        self.base_tree.lock_read()
4797.16.1 by Andrew Bennetts
Replace many try/finally blocks in merge.py with more robust OperationWithCleanups.
810
        operation.add_cleanup(self.base_tree.unlock)
3008.1.6 by Michael Hudson
chop up Merge3Merger.__init__ into pieces
811
        self.other_tree.lock_read()
4797.16.1 by Andrew Bennetts
Replace many try/finally blocks in merge.py with more robust OperationWithCleanups.
812
        operation.add_cleanup(self.other_tree.unlock)
813
        operation.run()
814
815
    def _do_merge(self, operation):
5031.1.2 by Andrew Bennetts
Merge lp:bzr/2.1
816
        self.tt = transform.TreeTransform(self.this_tree, None)
4797.16.1 by Andrew Bennetts
Replace many try/finally blocks in merge.py with more robust OperationWithCleanups.
817
        operation.add_cleanup(self.tt.finalize)
818
        self._compute_transform()
819
        results = self.tt.apply(no_conflicts=True)
820
        self.write_modified(results)
5031.1.3 by Andrew Bennetts
Fix syntax error from bad conflict resolution.
821
        try:
4797.16.1 by Andrew Bennetts
Replace many try/finally blocks in merge.py with more robust OperationWithCleanups.
822
            self.this_tree.add_conflicts(self.cooked_conflicts)
823
        except errors.UnsupportedOperation:
824
            pass
1534.7.192 by Aaron Bentley
Record hashes produced by merges
825
3008.1.9 by Michael Hudson
wanton hacking that lets me write an efficient version of get_diff_as_merged
826
    def make_preview_transform(self):
4797.16.1 by Andrew Bennetts
Replace many try/finally blocks in merge.py with more robust OperationWithCleanups.
827
        operation = OperationWithCleanups(self._make_preview_transform)
3008.1.9 by Michael Hudson
wanton hacking that lets me write an efficient version of get_diff_as_merged
828
        self.base_tree.lock_read()
4797.16.1 by Andrew Bennetts
Replace many try/finally blocks in merge.py with more robust OperationWithCleanups.
829
        operation.add_cleanup(self.base_tree.unlock)
3008.1.9 by Michael Hudson
wanton hacking that lets me write an efficient version of get_diff_as_merged
830
        self.other_tree.lock_read()
4797.16.1 by Andrew Bennetts
Replace many try/finally blocks in merge.py with more robust OperationWithCleanups.
831
        operation.add_cleanup(self.other_tree.unlock)
832
        return operation.run_simple()
833
834
    def _make_preview_transform(self):
4721.3.1 by Vincent Ladeuil
Cleanup imports.
835
        self.tt = transform.TransformPreview(self.this_tree)
4797.16.1 by Andrew Bennetts
Replace many try/finally blocks in merge.py with more robust OperationWithCleanups.
836
        self._compute_transform()
3008.1.9 by Michael Hudson
wanton hacking that lets me write an efficient version of get_diff_as_merged
837
        return self.tt
838
3008.1.21 by Aaron Bentley
Make compute_transform private, test make_preview_transform
839
    def _compute_transform(self):
3514.4.21 by John Arbash Meinel
Hook up the lca-way logic into the path handlers.
840
        if self._lca_trees is None:
841
            entries = self._entries3()
842
            resolver = self._three_way
843
        else:
844
            entries = self._entries_lca()
845
            resolver = self._lca_multi_way
3008.1.6 by Michael Hudson
chop up Merge3Merger.__init__ into pieces
846
        child_pb = ui.ui_factory.nested_progress_bar()
847
        try:
4797.5.1 by Robert Collins
Support state on per-file merging to permit more efficient use of configuration data.
848
            factories = Merger.hooks['merge_file_content']
849
            hooks = [factory(self) for factory in factories] + [self]
850
            self.active_hooks = [hook for hook in hooks if hook is not None]
3008.1.6 by Michael Hudson
chop up Merge3Merger.__init__ into pieces
851
            for num, (file_id, changed, parents3, names3,
852
                      executable3) in enumerate(entries):
853
                child_pb.update('Preparing file merge', num, len(entries))
3514.4.21 by John Arbash Meinel
Hook up the lca-way logic into the path handlers.
854
                self._merge_names(file_id, parents3, names3, resolver=resolver)
3008.1.6 by Michael Hudson
chop up Merge3Merger.__init__ into pieces
855
                if changed:
4797.5.1 by Robert Collins
Support state on per-file merging to permit more efficient use of configuration data.
856
                    file_status = self._do_merge_contents(file_id)
3008.1.6 by Michael Hudson
chop up Merge3Merger.__init__ into pieces
857
                else:
858
                    file_status = 'unmodified'
859
                self._merge_executable(file_id,
3514.4.21 by John Arbash Meinel
Hook up the lca-way logic into the path handlers.
860
                    executable3, file_status, resolver=resolver)
3008.1.6 by Michael Hudson
chop up Merge3Merger.__init__ into pieces
861
        finally:
862
            child_pb.finished()
863
        self.fix_root()
864
        child_pb = ui.ui_factory.nested_progress_bar()
865
        try:
4721.3.1 by Vincent Ladeuil
Cleanup imports.
866
            fs_conflicts = transform.resolve_conflicts(self.tt, child_pb,
867
                lambda t, c: transform.conflict_pass(t, c, self.other_tree))
3008.1.6 by Michael Hudson
chop up Merge3Merger.__init__ into pieces
868
        finally:
869
            child_pb.finished()
870
        if self.change_reporter is not None:
871
            from bzrlib import delta
872
            delta.report_changes(
3254.1.1 by Aaron Bentley
Make Tree.iter_changes a public method
873
                self.tt.iter_changes(), self.change_reporter)
3008.1.6 by Michael Hudson
chop up Merge3Merger.__init__ into pieces
874
        self.cook_conflicts(fs_conflicts)
875
        for conflict in self.cooked_conflicts:
4721.3.1 by Vincent Ladeuil
Cleanup imports.
876
            trace.warning(conflict)
3008.1.6 by Michael Hudson
chop up Merge3Merger.__init__ into pieces
877
2590.2.4 by Aaron Bentley
Move entry generation to a helper
878
    def _entries3(self):
2590.2.7 by Aaron Bentley
Misc cleanup
879
        """Gather data about files modified between three trees.
880
881
        Return a list of tuples of file_id, changed, parents3, names3,
882
        executable3.  changed is a boolean indicating whether the file contents
883
        or kind were changed.  parents3 is a tuple of parent ids for base,
884
        other and this.  names3 is a tuple of names for base, other and this.
885
        executable3 is a tuple of execute-bit values for base, other and this.
886
        """
2590.2.4 by Aaron Bentley
Move entry generation to a helper
887
        result = []
3254.1.1 by Aaron Bentley
Make Tree.iter_changes a public method
888
        iterator = self.other_tree.iter_changes(self.base_tree,
3514.2.16 by John Arbash Meinel
Review feedback from Ian.
889
                include_unchanged=True, specific_files=self.interesting_files,
2590.2.5 by Aaron Bentley
Allow selected files to be specified instead of selected ids
890
                extra_trees=[self.this_tree])
3363.17.1 by Aaron Bentley
Avoid inventory for merge and transform code
891
        this_entries = dict((e.file_id, e) for p, e in
892
                            self.this_tree.iter_entries_by_dir(
893
                            self.interesting_ids))
2590.2.4 by Aaron Bentley
Move entry generation to a helper
894
        for (file_id, paths, changed, versioned, parents, names, kind,
2590.2.5 by Aaron Bentley
Allow selected files to be specified instead of selected ids
895
             executable) in iterator:
2590.2.4 by Aaron Bentley
Move entry generation to a helper
896
            if (self.interesting_ids is not None and
897
                file_id not in self.interesting_ids):
898
                continue
3363.17.1 by Aaron Bentley
Avoid inventory for merge and transform code
899
            entry = this_entries.get(file_id)
900
            if entry is not None:
2590.2.4 by Aaron Bentley
Move entry generation to a helper
901
                this_name = entry.name
902
                this_parent = entry.parent_id
903
                this_executable = entry.executable
904
            else:
905
                this_name = None
906
                this_parent = None
907
                this_executable = None
908
            parents3 = parents + (this_parent,)
909
            names3 = names + (this_name,)
910
            executable3 = executable + (this_executable,)
911
            result.append((file_id, changed, parents3, names3, executable3))
912
        return result
913
3514.4.5 by John Arbash Meinel
Initial work on _entries_lca.
914
    def _entries_lca(self):
915
        """Gather data about files modified between multiple trees.
916
917
        This compares OTHER versus all LCA trees, and for interesting entries,
918
        it then compares with THIS and BASE.
919
920
        For the multi-valued entries, the format will be (BASE, [lca1, lca2])
921
        :return: [(file_id, changed, parents, names, executable)]
922
            file_id     Simple file_id of the entry
923
            changed     Boolean, True if the kind or contents changed
924
                        else False
925
            parents     ((base, [parent_id, in, lcas]), parent_id_other,
926
                         parent_id_this)
927
            names       ((base, [name, in, lcas]), name_in_other, name_in_this)
928
            executable  ((base, [exec, in, lcas]), exec_in_other, exec_in_this)
929
        """
3514.4.24 by John Arbash Meinel
Implement support for 'interesting_files' and 'interesting_ids' for _entries_lca
930
        if self.interesting_files is not None:
931
            lookup_trees = [self.this_tree, self.base_tree]
932
            lookup_trees.extend(self._lca_trees)
933
            # I think we should include the lca trees as well
934
            interesting_ids = self.other_tree.paths2ids(self.interesting_files,
935
                                                        lookup_trees)
936
        else:
937
            interesting_ids = self.interesting_ids
3514.4.5 by John Arbash Meinel
Initial work on _entries_lca.
938
        result = []
3514.4.13 by John Arbash Meinel
Switch the lca_trees to be in 'find_merge_order'.
939
        walker = _mod_tree.MultiWalker(self.other_tree, self._lca_trees)
3514.4.5 by John Arbash Meinel
Initial work on _entries_lca.
940
3514.4.10 by John Arbash Meinel
Test the case where BASE is missing a file that is present in THIS, OTHER and all LCAs.
941
        base_inventory = self.base_tree.inventory
942
        this_inventory = self.this_tree.inventory
3514.4.5 by John Arbash Meinel
Initial work on _entries_lca.
943
        for path, file_id, other_ie, lca_values in walker.iter_all():
944
            # Is this modified at all from any of the other trees?
3514.4.15 by John Arbash Meinel
Handle when OTHER doesn't have the entry.
945
            if other_ie is None:
3514.4.20 by John Arbash Meinel
Use the _lca_multi_way to work out if there is actually a kind/parent/name/content change.
946
                other_ie = _none_entry
3514.4.24 by John Arbash Meinel
Implement support for 'interesting_files' and 'interesting_ids' for _entries_lca
947
            if interesting_ids is not None and file_id not in interesting_ids:
948
                continue
3514.4.15 by John Arbash Meinel
Handle when OTHER doesn't have the entry.
949
3514.4.30 by John Arbash Meinel
Several updates.
950
            # If other_revision is found in any of the lcas, that means this
951
            # node is uninteresting. This is because when merging, if there are
952
            # multiple heads(), we have to create a new node. So if we didn't,
953
            # we know that the ancestry is linear, and that OTHER did not
954
            # modify anything
955
            # See doc/developers/lca_merge_resolution.txt for details
3514.4.20 by John Arbash Meinel
Use the _lca_multi_way to work out if there is actually a kind/parent/name/content change.
956
            other_revision = other_ie.revision
3514.4.32 by John Arbash Meinel
Add a test for proper behavior when *everything* is a WT.
957
            if other_revision is not None:
958
                # We can't use this shortcut when other_revision is None,
959
                # because it may be None because things are WorkingTrees, and
960
                # not because it is *actually* None.
961
                is_unmodified = False
962
                for lca_path, ie in lca_values:
963
                    if ie is not None and ie.revision == other_revision:
964
                        is_unmodified = True
965
                        break
966
                if is_unmodified:
967
                    continue
3514.4.20 by John Arbash Meinel
Use the _lca_multi_way to work out if there is actually a kind/parent/name/content change.
968
969
            lca_entries = []
970
            for lca_path, lca_ie in lca_values:
971
                if lca_ie is None:
972
                    lca_entries.append(_none_entry)
973
                else:
974
                    lca_entries.append(lca_ie)
3514.4.19 by John Arbash Meinel
Add the _lca_multi_way function, and explicit tests.
975
3514.4.10 by John Arbash Meinel
Test the case where BASE is missing a file that is present in THIS, OTHER and all LCAs.
976
            if file_id in base_inventory:
3514.4.20 by John Arbash Meinel
Use the _lca_multi_way to work out if there is actually a kind/parent/name/content change.
977
                base_ie = base_inventory[file_id]
3514.4.10 by John Arbash Meinel
Test the case where BASE is missing a file that is present in THIS, OTHER and all LCAs.
978
            else:
3514.4.20 by John Arbash Meinel
Use the _lca_multi_way to work out if there is actually a kind/parent/name/content change.
979
                base_ie = _none_entry
980
3514.4.11 by John Arbash Meinel
Handle when an entry is missing in THIS
981
            if file_id in this_inventory:
982
                this_ie = this_inventory[file_id]
983
            else:
3514.4.20 by John Arbash Meinel
Use the _lca_multi_way to work out if there is actually a kind/parent/name/content change.
984
                this_ie = _none_entry
3514.4.11 by John Arbash Meinel
Handle when an entry is missing in THIS
985
3514.4.21 by John Arbash Meinel
Hook up the lca-way logic into the path handlers.
986
            lca_kinds = []
987
            lca_parent_ids = []
988
            lca_names = []
989
            lca_executable = []
990
            for lca_ie in lca_entries:
991
                lca_kinds.append(lca_ie.kind)
992
                lca_parent_ids.append(lca_ie.parent_id)
993
                lca_names.append(lca_ie.name)
994
                lca_executable.append(lca_ie.executable)
995
3514.4.39 by John Arbash Meinel
Use self._lca_multi_way rather than Merge3Merger._lca_multi_way by Aaron's request.
996
            kind_winner = self._lca_multi_way(
3514.4.21 by John Arbash Meinel
Hook up the lca-way logic into the path handlers.
997
                (base_ie.kind, lca_kinds),
3514.4.20 by John Arbash Meinel
Use the _lca_multi_way to work out if there is actually a kind/parent/name/content change.
998
                other_ie.kind, this_ie.kind)
3514.4.39 by John Arbash Meinel
Use self._lca_multi_way rather than Merge3Merger._lca_multi_way by Aaron's request.
999
            parent_id_winner = self._lca_multi_way(
3514.4.21 by John Arbash Meinel
Hook up the lca-way logic into the path handlers.
1000
                (base_ie.parent_id, lca_parent_ids),
3514.4.20 by John Arbash Meinel
Use the _lca_multi_way to work out if there is actually a kind/parent/name/content change.
1001
                other_ie.parent_id, this_ie.parent_id)
3514.4.39 by John Arbash Meinel
Use self._lca_multi_way rather than Merge3Merger._lca_multi_way by Aaron's request.
1002
            name_winner = self._lca_multi_way(
3514.4.21 by John Arbash Meinel
Hook up the lca-way logic into the path handlers.
1003
                (base_ie.name, lca_names),
3514.4.20 by John Arbash Meinel
Use the _lca_multi_way to work out if there is actually a kind/parent/name/content change.
1004
                other_ie.name, this_ie.name)
3514.4.21 by John Arbash Meinel
Hook up the lca-way logic into the path handlers.
1005
3514.4.20 by John Arbash Meinel
Use the _lca_multi_way to work out if there is actually a kind/parent/name/content change.
1006
            content_changed = True
1007
            if kind_winner == 'this':
1008
                # No kind change in OTHER, see if there are *any* changes
3948.1.1 by John Arbash Meinel
Fix an edge case with deleted files and criss-cross merges.
1009
                if other_ie.kind == 'directory':
3514.4.20 by John Arbash Meinel
Use the _lca_multi_way to work out if there is actually a kind/parent/name/content change.
1010
                    if parent_id_winner == 'this' and name_winner == 'this':
1011
                        # No change for this directory in OTHER, skip
1012
                        continue
1013
                    content_changed = False
3948.1.1 by John Arbash Meinel
Fix an edge case with deleted files and criss-cross merges.
1014
                elif other_ie.kind is None or other_ie.kind == 'file':
3514.4.30 by John Arbash Meinel
Several updates.
1015
                    def get_sha1(ie, tree):
3514.4.35 by John Arbash Meinel
Add a test that we only call get_symlink_target if the object should be a symlink.
1016
                        if ie.kind != 'file':
3514.4.30 by John Arbash Meinel
Several updates.
1017
                            return None
1018
                        return tree.get_file_sha1(file_id)
1019
                    base_sha1 = get_sha1(base_ie, self.base_tree)
1020
                    lca_sha1s = [get_sha1(ie, tree) for ie, tree
1021
                                 in zip(lca_entries, self._lca_trees)]
1022
                    this_sha1 = get_sha1(this_ie, self.this_tree)
1023
                    other_sha1 = get_sha1(other_ie, self.other_tree)
3514.4.39 by John Arbash Meinel
Use self._lca_multi_way rather than Merge3Merger._lca_multi_way by Aaron's request.
1024
                    sha1_winner = self._lca_multi_way(
3514.4.30 by John Arbash Meinel
Several updates.
1025
                        (base_sha1, lca_sha1s), other_sha1, this_sha1,
1026
                        allow_overriding_lca=False)
3514.4.39 by John Arbash Meinel
Use self._lca_multi_way rather than Merge3Merger._lca_multi_way by Aaron's request.
1027
                    exec_winner = self._lca_multi_way(
3514.4.22 by John Arbash Meinel
Handle executable bit changes as well.
1028
                        (base_ie.executable, lca_executable),
1029
                        other_ie.executable, this_ie.executable)
3514.4.20 by John Arbash Meinel
Use the _lca_multi_way to work out if there is actually a kind/parent/name/content change.
1030
                    if (parent_id_winner == 'this' and name_winner == 'this'
3514.4.22 by John Arbash Meinel
Handle executable bit changes as well.
1031
                        and sha1_winner == 'this' and exec_winner == 'this'):
1032
                        # No kind, parent, name, exec, or content change for
1033
                        # OTHER, so this node is not considered interesting
3514.4.20 by John Arbash Meinel
Use the _lca_multi_way to work out if there is actually a kind/parent/name/content change.
1034
                        continue
1035
                    if sha1_winner == 'this':
1036
                        content_changed = False
3514.4.27 by John Arbash Meinel
A couple of symlink tests, we need to do more.
1037
                elif other_ie.kind == 'symlink':
3514.4.34 by John Arbash Meinel
Handle symlinks properly when objects are not RevisionTrees
1038
                    def get_target(ie, tree):
3514.4.35 by John Arbash Meinel
Add a test that we only call get_symlink_target if the object should be a symlink.
1039
                        if ie.kind != 'symlink':
3514.4.34 by John Arbash Meinel
Handle symlinks properly when objects are not RevisionTrees
1040
                            return None
1041
                        return tree.get_symlink_target(file_id)
1042
                    base_target = get_target(base_ie, self.base_tree)
1043
                    lca_targets = [get_target(ie, tree) for ie, tree
1044
                                   in zip(lca_entries, self._lca_trees)]
1045
                    this_target = get_target(this_ie, self.this_tree)
1046
                    other_target = get_target(other_ie, self.other_tree)
3514.4.39 by John Arbash Meinel
Use self._lca_multi_way rather than Merge3Merger._lca_multi_way by Aaron's request.
1047
                    target_winner = self._lca_multi_way(
3514.4.34 by John Arbash Meinel
Handle symlinks properly when objects are not RevisionTrees
1048
                        (base_target, lca_targets),
1049
                        other_target, this_target)
3514.4.28 by John Arbash Meinel
More symlink tests.
1050
                    if (parent_id_winner == 'this' and name_winner == 'this'
1051
                        and target_winner == 'this'):
1052
                        # No kind, parent, name, or symlink target change
1053
                        # not interesting
1054
                        continue
1055
                    if target_winner == 'this':
1056
                        content_changed = False
3514.4.33 by John Arbash Meinel
Implement support for 'tree-reference'.
1057
                elif other_ie.kind == 'tree-reference':
1058
                    # The 'changed' information seems to be handled at a higher
1059
                    # level. At least, _entries3 returns False for content
1060
                    # changed, even when at a new revision_id.
1061
                    content_changed = False
1062
                    if (parent_id_winner == 'this' and name_winner == 'this'):
1063
                        # Nothing interesting
1064
                        continue
3514.4.21 by John Arbash Meinel
Hook up the lca-way logic into the path handlers.
1065
                else:
1066
                    raise AssertionError('unhandled kind: %s' % other_ie.kind)
3514.4.13 by John Arbash Meinel
Switch the lca_trees to be in 'find_merge_order'.
1067
1068
            # If we have gotten this far, that means something has changed
3514.4.20 by John Arbash Meinel
Use the _lca_multi_way to work out if there is actually a kind/parent/name/content change.
1069
            result.append((file_id, content_changed,
1070
                           ((base_ie.parent_id, lca_parent_ids),
1071
                            other_ie.parent_id, this_ie.parent_id),
1072
                           ((base_ie.name, lca_names),
1073
                            other_ie.name, this_ie.name),
1074
                           ((base_ie.executable, lca_executable),
1075
                            other_ie.executable, this_ie.executable)
3514.4.5 by John Arbash Meinel
Initial work on _entries_lca.
1076
                          ))
1077
        return result
1078
1079
1731.1.33 by Aaron Bentley
Revert no-special-root changes
1080
    def fix_root(self):
1081
        try:
1082
            self.tt.final_kind(self.tt.root)
4721.3.1 by Vincent Ladeuil
Cleanup imports.
1083
        except errors.NoSuchFile:
1731.1.33 by Aaron Bentley
Revert no-special-root changes
1084
            self.tt.cancel_deletion(self.tt.root)
1085
        if self.tt.final_file_id(self.tt.root) is None:
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
1086
            self.tt.version_file(self.tt.tree_file_id(self.tt.root),
1731.1.33 by Aaron Bentley
Revert no-special-root changes
1087
                                 self.tt.root)
2946.3.3 by John Arbash Meinel
Prefer tree.get_root_id() as more explicit than tree.path2id('')
1088
        other_root_file_id = self.other_tree.get_root_id()
3363.17.17 by Aaron Bentley
Start testing merging PreviewTree as OTHER
1089
        if other_root_file_id is None:
1090
            return
1731.1.33 by Aaron Bentley
Revert no-special-root changes
1091
        other_root = self.tt.trans_id_file_id(other_root_file_id)
1092
        if other_root == self.tt.root:
1093
            return
3649.3.1 by Jelmer Vernooij
Merging from a previously joined branch will no longer cause a traceback.
1094
        if self.other_tree.inventory.root.file_id in self.this_tree.inventory:
4634.149.3 by Vincent Ladeuil
Fix bug #3735898 by filtering out garbage paths.
1095
            # the other tree's root is a non-root in the current tree (as when
4634.149.6 by Vincent Ladeuil
Fix typo.
1096
            # a previously unrelated branch is merged into another)
3649.3.1 by Jelmer Vernooij
Merging from a previously joined branch will no longer cause a traceback.
1097
            return
4634.149.5 by Vincent Ladeuil
Fixed as per John's review comments.
1098
        try:
1099
            self.tt.final_kind(other_root)
1100
            other_root_is_present = True
4797.42.2 by Vincent Ladeuil
Fix pqm failures
1101
        except errors.NoSuchFile:
4634.149.5 by Vincent Ladeuil
Fixed as per John's review comments.
1102
            # other_root doesn't have a physical representation. We still need
1103
            # to move any references to the actual root of the tree.
1104
            other_root_is_present = False
1105
        # 'other_tree.inventory.root' is not present in this tree. We are
1106
        # calling adjust_path for children which *want* to be present with a
1107
        # correct place to go.
1108
        for thing, child in self.other_tree.inventory.root.children.iteritems():
1109
            trans_id = self.tt.trans_id_file_id(child.file_id)
1110
            if not other_root_is_present:
1111
                # FIXME: Make final_kind returns None instead of raising
1112
                # NoSuchFile to avoid the ugly construct below -- vila 20100402
1113
                try:
1114
                    self.tt.final_kind(trans_id)
1115
                    # The item exist in the final tree and has a defined place
1116
                    # to go already.
1117
                    continue
1118
                except errors.NoSuchFile, e:
1119
                    pass
1120
            # Move the item into the root
1121
            self.tt.adjust_path(self.tt.final_name(trans_id),
1122
                                self.tt.root, trans_id)
1123
        if other_root_is_present:
4634.149.3 by Vincent Ladeuil
Fix bug #3735898 by filtering out garbage paths.
1124
            self.tt.cancel_creation(other_root)
1125
            self.tt.cancel_versioning(other_root)
1731.1.33 by Aaron Bentley
Revert no-special-root changes
1126
1534.7.192 by Aaron Bentley
Record hashes produced by merges
1127
    def write_modified(self, results):
1128
        modified_hashes = {}
1129
        for path in results.modified_paths:
1130
            file_id = self.this_tree.path2id(self.this_tree.relpath(path))
1131
            if file_id is None:
1132
                continue
1133
            hash = self.this_tree.get_file_sha1(file_id)
1134
            if hash is None:
1135
                continue
1136
            modified_hashes[file_id] = hash
1137
        self.this_tree.set_merge_modified(modified_hashes)
1138
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1139
    @staticmethod
1140
    def parent(entry, file_id):
1534.7.157 by Aaron Bentley
Added more docs
1141
        """Determine the parent for a file_id (used as a key method)"""
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1142
        if entry is None:
1143
            return None
1144
        return entry.parent_id
1145
1146
    @staticmethod
1147
    def name(entry, file_id):
1534.7.157 by Aaron Bentley
Added more docs
1148
        """Determine the name for a file_id (used as a key method)"""
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1149
        if entry is None:
1150
            return None
1151
        return entry.name
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
1152
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1153
    @staticmethod
1154
    def contents_sha1(tree, file_id):
1534.7.157 by Aaron Bentley
Added more docs
1155
        """Determine the sha1 of the file contents (used as a key method)."""
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1156
        if file_id not in tree:
1157
            return None
1158
        return tree.get_file_sha1(file_id)
1159
1160
    @staticmethod
1161
    def executable(tree, file_id):
1534.7.157 by Aaron Bentley
Added more docs
1162
        """Determine the executability of a file-id (used as a key method)."""
4869.3.22 by Andrew Bennetts
Use 'tree.has_id(file_id)' rather than using 'file_id in tree' or using tree.inventory.
1163
        if not tree.has_id(file_id):
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1164
            return None
1165
        if tree.kind(file_id) != "file":
1166
            return False
1167
        return tree.is_executable(file_id)
1168
1169
    @staticmethod
1170
    def kind(tree, file_id):
1534.7.157 by Aaron Bentley
Added more docs
1171
        """Determine the kind of a file-id (used as a key method)."""
4869.3.22 by Andrew Bennetts
Use 'tree.has_id(file_id)' rather than using 'file_id in tree' or using tree.inventory.
1172
        if not tree.has_id(file_id):
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1173
            return None
1174
        return tree.kind(file_id)
1175
1176
    @staticmethod
2590.2.8 by Aaron Bentley
Restore conflict handling changes
1177
    def _three_way(base, other, this):
1178
        if base == other:
5128.1.1 by Vincent Ladeuil
Uncontroversial cleanups, mostly comments
1179
            # if 'base == other', either they all agree, or only 'this' has
1180
            # changed.
2590.2.8 by Aaron Bentley
Restore conflict handling changes
1181
            return 'this'
2590.2.10 by Aaron Bentley
Updates from review
1182
        elif this not in (base, other):
5128.1.1 by Vincent Ladeuil
Uncontroversial cleanups, mostly comments
1183
            # 'this' is neither 'base' nor 'other', so both sides changed
2590.2.8 by Aaron Bentley
Restore conflict handling changes
1184
            return 'conflict'
1185
        elif this == other:
5128.1.1 by Vincent Ladeuil
Uncontroversial cleanups, mostly comments
1186
            # "Ambiguous clean merge" -- both sides have made the same change.
2590.2.8 by Aaron Bentley
Restore conflict handling changes
1187
            return "this"
1188
        else:
5128.1.1 by Vincent Ladeuil
Uncontroversial cleanups, mostly comments
1189
            # this == base: only other has changed.
2590.2.8 by Aaron Bentley
Restore conflict handling changes
1190
            return "other"
1191
1192
    @staticmethod
3514.4.30 by John Arbash Meinel
Several updates.
1193
    def _lca_multi_way(bases, other, this, allow_overriding_lca=True):
3514.4.19 by John Arbash Meinel
Add the _lca_multi_way function, and explicit tests.
1194
        """Consider LCAs when determining whether a change has occurred.
1195
1196
        If LCAS are all identical, this is the same as a _three_way comparison.
1197
1198
        :param bases: value in (BASE, [LCAS])
1199
        :param other: value in OTHER
1200
        :param this: value in THIS
3514.4.30 by John Arbash Meinel
Several updates.
1201
        :param allow_overriding_lca: If there is more than one unique lca
1202
            value, allow OTHER to override THIS if it has a new value, and
1203
            THIS only has an lca value, or vice versa. This is appropriate for
1204
            truly scalar values, not as much for non-scalars.
3514.4.19 by John Arbash Meinel
Add the _lca_multi_way function, and explicit tests.
1205
        :return: 'this', 'other', or 'conflict' depending on whether an entry
1206
            changed or not.
1207
        """
3948.1.7 by Vincent Ladeuil
Slight refactoring and test fixing.
1208
        # See doc/developers/lca_tree_merging.txt for details about this
3514.4.30 by John Arbash Meinel
Several updates.
1209
        # algorithm.
3514.4.19 by John Arbash Meinel
Add the _lca_multi_way function, and explicit tests.
1210
        if other == this:
1211
            # Either Ambiguously clean, or nothing was actually changed. We
1212
            # don't really care
1213
            return 'this'
1214
        base_val, lca_vals = bases
1215
        # Remove 'base_val' from the lca_vals, because it is not interesting
1216
        filtered_lca_vals = [lca_val for lca_val in lca_vals
1217
                                      if lca_val != base_val]
1218
        if len(filtered_lca_vals) == 0:
1219
            return Merge3Merger._three_way(base_val, other, this)
3514.4.30 by John Arbash Meinel
Several updates.
1220
1221
        unique_lca_vals = set(filtered_lca_vals)
1222
        if len(unique_lca_vals) == 1:
1223
            return Merge3Merger._three_way(unique_lca_vals.pop(), other, this)
1224
1225
        if allow_overriding_lca:
1226
            if other in unique_lca_vals:
1227
                if this in unique_lca_vals:
1228
                    # Each side picked a different lca, conflict
1229
                    return 'conflict'
1230
                else:
1231
                    # This has a value which supersedes both lca values, and
1232
                    # other only has an lca value
1233
                    return 'this'
1234
            elif this in unique_lca_vals:
1235
                # OTHER has a value which supersedes both lca values, and this
3514.4.19 by John Arbash Meinel
Add the _lca_multi_way function, and explicit tests.
1236
                # only has an lca value
3514.4.30 by John Arbash Meinel
Several updates.
1237
                return 'other'
3514.4.19 by John Arbash Meinel
Add the _lca_multi_way function, and explicit tests.
1238
5128.1.1 by Vincent Ladeuil
Uncontroversial cleanups, mostly comments
1239
        # At this point, the lcas disagree, and the tip disagree
3514.4.19 by John Arbash Meinel
Add the _lca_multi_way function, and explicit tests.
1240
        return 'conflict'
1241
1242
    @staticmethod
5077.2.1 by Vincent Ladeuil
Deprecate Merge3Merger.scalar_three_way.
1243
    @deprecated_method(deprecated_in((2, 2, 0)))
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1244
    def scalar_three_way(this_tree, base_tree, other_tree, file_id, key):
1245
        """Do a three-way test on a scalar.
1246
        Return "this", "other" or "conflict", depending whether a value wins.
1247
        """
1248
        key_base = key(base_tree, file_id)
1249
        key_other = key(other_tree, file_id)
1250
        #if base == other, either they all agree, or only THIS has changed.
1251
        if key_base == key_other:
1252
            return "this"
1253
        key_this = key(this_tree, file_id)
3376.2.4 by Martin Pool
Remove every assert statement from bzrlib!
1254
        # "Ambiguous clean merge"
1255
        if key_this == key_other:
1256
            return "this"
1257
        elif key_this == key_base:
1258
            return "other"
1259
        else:
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1260
            return "conflict"
1261
1262
    def merge_names(self, file_id):
1263
        def get_entry(tree):
4869.3.22 by Andrew Bennetts
Use 'tree.has_id(file_id)' rather than using 'file_id in tree' or using tree.inventory.
1264
            if tree.has_id(file_id):
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1265
                return tree.inventory[file_id]
1266
            else:
1267
                return None
1268
        this_entry = get_entry(self.this_tree)
1269
        other_entry = get_entry(self.other_tree)
1270
        base_entry = get_entry(self.base_tree)
2590.2.1 by Aaron Bentley
Start work on merging names based on iter_changes
1271
        entries = (base_entry, other_entry, this_entry)
1272
        names = []
1273
        parents = []
1274
        for entry in entries:
1275
            if entry is None:
1276
                names.append(None)
1277
                parents.append(None)
1278
            else:
1279
                names.append(entry.name)
1280
                parents.append(entry.parent_id)
3514.4.40 by John Arbash Meinel
Make 'resolver' a required parameter.
1281
        return self._merge_names(file_id, parents, names,
1282
                                 resolver=self._three_way)
2590.2.1 by Aaron Bentley
Start work on merging names based on iter_changes
1283
3514.4.40 by John Arbash Meinel
Make 'resolver' a required parameter.
1284
    def _merge_names(self, file_id, parents, names, resolver):
2590.2.7 by Aaron Bentley
Misc cleanup
1285
        """Perform a merge on file_id names and parents"""
2590.2.1 by Aaron Bentley
Start work on merging names based on iter_changes
1286
        base_name, other_name, this_name = names
1287
        base_parent, other_parent, this_parent = parents
2590.2.2 by Aaron Bentley
Do most name merging from iter_changes output
1288
3514.4.21 by John Arbash Meinel
Hook up the lca-way logic into the path handlers.
1289
        name_winner = resolver(*names)
2590.2.1 by Aaron Bentley
Start work on merging names based on iter_changes
1290
3514.4.21 by John Arbash Meinel
Hook up the lca-way logic into the path handlers.
1291
        parent_id_winner = resolver(*parents)
2590.2.2 by Aaron Bentley
Do most name merging from iter_changes output
1292
        if this_name is None:
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1293
            if name_winner == "this":
1294
                name_winner = "other"
1295
            if parent_id_winner == "this":
1296
                parent_id_winner = "other"
1297
        if name_winner == "this" and parent_id_winner == "this":
1298
            return
4597.7.11 by Vincent Ladeuil
Fix #531967 by creating helpers for PathConflicts when a deletion
1299
        if name_winner == 'conflict' or parent_id_winner == 'conflict':
4597.7.18 by Vincent Ladeuil
Start addressing Andrew's concerns.
1300
            # Creating helpers (.OTHER or .THIS) here cause problems down the
1301
            # road if a ContentConflict needs to be created so we should not do
1302
            # that
4597.7.11 by Vincent Ladeuil
Fix #531967 by creating helpers for PathConflicts when a deletion
1303
            trans_id = self.tt.trans_id_file_id(file_id)
1304
            self._raw_conflicts.append(('path conflict', trans_id, file_id,
1305
                                        this_parent, this_name,
1306
                                        other_parent, other_name))
2590.2.2 by Aaron Bentley
Do most name merging from iter_changes output
1307
        if other_name is None:
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
1308
            # it doesn't matter whether the result was 'other' or
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1309
            # 'conflict'-- if there's no 'other', we leave it alone.
1310
            return
2590.2.7 by Aaron Bentley
Misc cleanup
1311
        parent_id = parents[self.winner_idx[parent_id_winner]]
1731.1.33 by Aaron Bentley
Revert no-special-root changes
1312
        if parent_id is not None:
4597.7.11 by Vincent Ladeuil
Fix #531967 by creating helpers for PathConflicts when a deletion
1313
            # if we get here, name_winner and parent_winner are set to safe
1314
            # values.
2590.2.7 by Aaron Bentley
Misc cleanup
1315
            self.tt.adjust_path(names[self.winner_idx[name_winner]],
4597.7.11 by Vincent Ladeuil
Fix #531967 by creating helpers for PathConflicts when a deletion
1316
                                self.tt.trans_id_file_id(parent_id),
1317
                                self.tt.trans_id_file_id(file_id))
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1318
4797.5.1 by Robert Collins
Support state on per-file merging to permit more efficient use of configuration data.
1319
    def _do_merge_contents(self, file_id):
3874.2.10 by Vincent Ladeuil
Fix typo.
1320
        """Performs a merge on file_id contents."""
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1321
        def contents_pair(tree):
1322
            if file_id not in tree:
1323
                return (None, None)
1324
            kind = tree.kind(file_id)
1325
            if kind == "file":
1326
                contents = tree.get_file_sha1(file_id)
1327
            elif kind == "symlink":
1328
                contents = tree.get_symlink_target(file_id)
1329
            else:
1330
                contents = None
1331
            return kind, contents
1558.15.3 by Aaron Bentley
Handle binary files for diff3 merges
1332
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1333
        # See SPOT run.  run, SPOT, run.
1334
        # So we're not QUITE repeating ourselves; we do tricky things with
1335
        # file kind...
1336
        base_pair = contents_pair(self.base_tree)
1337
        other_pair = contents_pair(self.other_tree)
3955.2.1 by John Arbash Meinel
Change the workings of merge_content to be lca aware.
1338
        if self._lca_trees:
3948.1.7 by Vincent Ladeuil
Slight refactoring and test fixing.
1339
            this_pair = contents_pair(self.this_tree)
3955.2.1 by John Arbash Meinel
Change the workings of merge_content to be lca aware.
1340
            lca_pairs = [contents_pair(tree) for tree in self._lca_trees]
1341
            winner = self._lca_multi_way((base_pair, lca_pairs), other_pair,
1342
                                         this_pair, allow_overriding_lca=False)
1343
        else:
3948.1.7 by Vincent Ladeuil
Slight refactoring and test fixing.
1344
            if base_pair == other_pair:
1345
                winner = 'this'
1346
            else:
1347
                # We delayed evaluating this_pair as long as we can to avoid
1348
                # unnecessary sha1 calculation
1349
                this_pair = contents_pair(self.this_tree)
1350
                winner = self._three_way(base_pair, other_pair, this_pair)
3955.2.1 by John Arbash Meinel
Change the workings of merge_content to be lca aware.
1351
        if winner == 'this':
1352
            # No interesting changes introduced by OTHER
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1353
            return "unmodified"
4869.3.15 by Andrew Bennetts
Refactor so that the default text merge is the final hook tried. Other hooks can now invoke it directly via merger.default_text_merge if they wish.
1354
        # We have a hypothetical conflict, but if we have files, then we
1355
        # can try to merge the content
4869.3.16 by Andrew Bennetts
Move handling of winner==OTHER case into generic hook path too.
1356
        trans_id = self.tt.trans_id_file_id(file_id)
4869.3.21 by Andrew Bennetts
Pass kinds (not pairs) to MergeHookParams.
1357
        params = MergeHookParams(self, file_id, trans_id, this_pair[0],
1358
            other_pair[0], winner)
4797.5.1 by Robert Collins
Support state on per-file merging to permit more efficient use of configuration data.
1359
        hooks = self.active_hooks
4869.3.15 by Andrew Bennetts
Refactor so that the default text merge is the final hook tried. Other hooks can now invoke it directly via merger.default_text_merge if they wish.
1360
        hook_status = 'not_applicable'
1361
        for hook in hooks:
4797.5.1 by Robert Collins
Support state on per-file merging to permit more efficient use of configuration data.
1362
            hook_status, lines = hook.merge_contents(params)
4869.3.15 by Andrew Bennetts
Refactor so that the default text merge is the final hook tried. Other hooks can now invoke it directly via merger.default_text_merge if they wish.
1363
            if hook_status != 'not_applicable':
1364
                # Don't try any more hooks, this one applies.
4869.3.5 by Andrew Bennetts
Hook is invoked for delete vs. change conflicts, and may choose to result in deleting the file.
1365
                break
4869.3.18 by Andrew Bennetts
Simplify slightly more.
1366
        result = "modified"
4869.3.15 by Andrew Bennetts
Refactor so that the default text merge is the final hook tried. Other hooks can now invoke it directly via merger.default_text_merge if they wish.
1367
        if hook_status == 'not_applicable':
4869.3.17 by Andrew Bennetts
Remove contents_conflict nested function, replace last callsite with simpler form.
1368
            # This is a contents conflict, because none of the available
1369
            # functions could merge it.
4869.3.18 by Andrew Bennetts
Simplify slightly more.
1370
            result = None
4869.3.17 by Andrew Bennetts
Remove contents_conflict nested function, replace last callsite with simpler form.
1371
            name = self.tt.final_name(trans_id)
1372
            parent_id = self.tt.final_parent(trans_id)
4869.3.22 by Andrew Bennetts
Use 'tree.has_id(file_id)' rather than using 'file_id in tree' or using tree.inventory.
1373
            if self.this_tree.has_id(file_id):
4869.3.17 by Andrew Bennetts
Remove contents_conflict nested function, replace last callsite with simpler form.
1374
                self.tt.unversion_file(trans_id)
1375
            file_group = self._dump_conflicts(name, parent_id, file_id,
1376
                                              set_version=True)
1377
            self._raw_conflicts.append(('contents conflict', file_group))
4869.3.15 by Andrew Bennetts
Refactor so that the default text merge is the final hook tried. Other hooks can now invoke it directly via merger.default_text_merge if they wish.
1378
        elif hook_status == 'success':
1379
            self.tt.create_file(lines, trans_id)
1380
        elif hook_status == 'conflicted':
1381
            # XXX: perhaps the hook should be able to provide
1382
            # the BASE/THIS/OTHER files?
1383
            self.tt.create_file(lines, trans_id)
1384
            self._raw_conflicts.append(('text conflict', trans_id))
1385
            name = self.tt.final_name(trans_id)
1386
            parent_id = self.tt.final_parent(trans_id)
4869.3.23 by Andrew Bennetts
Remove some confusing code that has no effect.
1387
            self._dump_conflicts(name, parent_id, file_id)
4869.3.15 by Andrew Bennetts
Refactor so that the default text merge is the final hook tried. Other hooks can now invoke it directly via merger.default_text_merge if they wish.
1388
        elif hook_status == 'delete':
1389
            self.tt.unversion_file(trans_id)
4869.3.18 by Andrew Bennetts
Simplify slightly more.
1390
            result = "deleted"
4869.3.15 by Andrew Bennetts
Refactor so that the default text merge is the final hook tried. Other hooks can now invoke it directly via merger.default_text_merge if they wish.
1391
        elif hook_status == 'done':
1392
            # The hook function did whatever it needs to do directly, no
1393
            # further action needed here.
1394
            pass
1395
        else:
4869.3.18 by Andrew Bennetts
Simplify slightly more.
1396
            raise AssertionError('unknown hook_status: %r' % (hook_status,))
4869.3.22 by Andrew Bennetts
Use 'tree.has_id(file_id)' rather than using 'file_id in tree' or using tree.inventory.
1397
        if not self.this_tree.has_id(file_id) and result == "modified":
4869.3.15 by Andrew Bennetts
Refactor so that the default text merge is the final hook tried. Other hooks can now invoke it directly via merger.default_text_merge if they wish.
1398
            self.tt.version_file(file_id, trans_id)
4869.3.31 by Andrew Bennetts
Add comment, improve doc string, remove redundant line.
1399
        # The merge has been performed, so the old contents should not be
1400
        # retained.
4869.3.15 by Andrew Bennetts
Refactor so that the default text merge is the final hook tried. Other hooks can now invoke it directly via merger.default_text_merge if they wish.
1401
        try:
1402
            self.tt.delete_contents(trans_id)
1403
        except errors.NoSuchFile:
1404
            pass
4869.3.18 by Andrew Bennetts
Simplify slightly more.
1405
        return result
4869.3.15 by Andrew Bennetts
Refactor so that the default text merge is the final hook tried. Other hooks can now invoke it directly via merger.default_text_merge if they wish.
1406
4869.3.16 by Andrew Bennetts
Move handling of winner==OTHER case into generic hook path too.
1407
    def _default_other_winner_merge(self, merge_hook_params):
1408
        """Replace this contents with other."""
1409
        file_id = merge_hook_params.file_id
1410
        trans_id = merge_hook_params.trans_id
4869.3.22 by Andrew Bennetts
Use 'tree.has_id(file_id)' rather than using 'file_id in tree' or using tree.inventory.
1411
        file_in_this = self.this_tree.has_id(file_id)
1412
        if self.other_tree.has_id(file_id):
4869.3.16 by Andrew Bennetts
Move handling of winner==OTHER case into generic hook path too.
1413
            # OTHER changed the file
1414
            wt = self.this_tree
1415
            if wt.supports_content_filtering():
1416
                # We get the path from the working tree if it exists.
1417
                # That fails though when OTHER is adding a file, so
1418
                # we fall back to the other tree to find the path if
1419
                # it doesn't exist locally.
1420
                try:
1421
                    filter_tree_path = wt.id2path(file_id)
1422
                except errors.NoSuchId:
1423
                    filter_tree_path = self.other_tree.id2path(file_id)
1424
            else:
1425
                # Skip the id2path lookup for older formats
1426
                filter_tree_path = None
1427
            transform.create_from_tree(self.tt, trans_id,
1428
                             self.other_tree, file_id,
1429
                             filter_tree_path=filter_tree_path)
1430
            return 'done', None
1431
        elif file_in_this:
1432
            # OTHER deleted the file
1433
            return 'delete', None
1434
        else:
1435
            raise AssertionError(
1436
                'winner is OTHER, but file_id %r not in THIS or OTHER tree'
1437
                % (file_id,))
1438
4797.5.1 by Robert Collins
Support state on per-file merging to permit more efficient use of configuration data.
1439
    def merge_contents(self, merge_hook_params):
1440
        """Fallback merge logic after user installed hooks."""
1441
        # This function is used in merge hooks as the fallback instance.
1442
        # Perhaps making this function and the functions it calls be a 
1443
        # a separate class would be better.
4869.3.16 by Andrew Bennetts
Move handling of winner==OTHER case into generic hook path too.
1444
        if merge_hook_params.winner == 'other':
1445
            # OTHER is a straight winner, so replace this contents with other
1446
            return self._default_other_winner_merge(merge_hook_params)
1447
        elif merge_hook_params.is_file_merge():
4869.3.15 by Andrew Bennetts
Refactor so that the default text merge is the final hook tried. Other hooks can now invoke it directly via merger.default_text_merge if they wish.
1448
            # THIS and OTHER are both files, so text merge.  Either
1449
            # BASE is a file, or both converted to files, so at least we
1450
            # have agreement that output should be a file.
1451
            try:
1452
                self.text_merge(merge_hook_params.file_id,
1453
                    merge_hook_params.trans_id)
1454
            except errors.BinaryFile:
1455
                return 'not_applicable', None
4869.3.16 by Andrew Bennetts
Move handling of winner==OTHER case into generic hook path too.
1456
            return 'done', None
4869.3.15 by Andrew Bennetts
Refactor so that the default text merge is the final hook tried. Other hooks can now invoke it directly via merger.default_text_merge if they wish.
1457
        else:
1458
            return 'not_applicable', None
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1459
1460
    def get_lines(self, tree, file_id):
1534.7.157 by Aaron Bentley
Added more docs
1461
        """Return the lines in a file, or an empty list."""
4869.3.22 by Andrew Bennetts
Use 'tree.has_id(file_id)' rather than using 'file_id in tree' or using tree.inventory.
1462
        if tree.has_id(file_id):
4708.2.1 by Martin
Ensure all files opened by bazaar proper are explicitly closed
1463
            return tree.get_file_lines(file_id)
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1464
        else:
1465
            return []
1466
1467
    def text_merge(self, file_id, trans_id):
1468
        """Perform a three-way text merge on a file_id"""
1469
        # it's possible that we got here with base as a different type.
1470
        # if so, we just want two-way text conflicts.
4869.3.22 by Andrew Bennetts
Use 'tree.has_id(file_id)' rather than using 'file_id in tree' or using tree.inventory.
1471
        if self.base_tree.has_id(file_id) and \
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1472
            self.base_tree.kind(file_id) == "file":
1473
            base_lines = self.get_lines(self.base_tree, file_id)
1474
        else:
1475
            base_lines = []
1476
        other_lines = self.get_lines(self.other_tree, file_id)
1477
        this_lines = self.get_lines(self.this_tree, file_id)
4721.3.1 by Vincent Ladeuil
Cleanup imports.
1478
        m3 = merge3.Merge3(base_lines, this_lines, other_lines,
1479
                           is_cherrypick=self.cherrypick)
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1480
        start_marker = "!START OF MERGE CONFLICT!" + "I HOPE THIS IS UNIQUE"
1481
        if self.show_base is True:
1482
            base_marker = '|' * 7
1483
        else:
1484
            base_marker = None
1485
1486
        def iter_merge3(retval):
1487
            retval["text_conflicts"] = False
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
1488
            for line in m3.merge_lines(name_a = "TREE",
1489
                                       name_b = "MERGE-SOURCE",
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1490
                                       name_base = "BASE-REVISION",
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
1491
                                       start_marker=start_marker,
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1492
                                       base_marker=base_marker,
1493
                                       reprocess=self.reprocess):
1494
                if line.startswith(start_marker):
1495
                    retval["text_conflicts"] = True
1496
                    yield line.replace(start_marker, '<' * 7)
1497
                else:
1498
                    yield line
1499
        retval = {}
1500
        merge3_iterator = iter_merge3(retval)
1501
        self.tt.create_file(merge3_iterator, trans_id)
1502
        if retval["text_conflicts"] is True:
1503
            self._raw_conflicts.append(('text conflict', trans_id))
1504
            name = self.tt.final_name(trans_id)
1505
            parent_id = self.tt.final_parent(trans_id)
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
1506
            file_group = self._dump_conflicts(name, parent_id, file_id,
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1507
                                              this_lines, base_lines,
1508
                                              other_lines)
1509
            file_group.append(trans_id)
1510
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
1511
    def _dump_conflicts(self, name, parent_id, file_id, this_lines=None,
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1512
                        base_lines=None, other_lines=None, set_version=False,
1513
                        no_base=False):
1534.7.157 by Aaron Bentley
Added more docs
1514
        """Emit conflict files.
1515
        If this_lines, base_lines, or other_lines are omitted, they will be
1516
        determined automatically.  If set_version is true, the .OTHER, .THIS
1517
        or .BASE (in that order) will be created as versioned files.
1518
        """
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
1519
        data = [('OTHER', self.other_tree, other_lines),
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1520
                ('THIS', self.this_tree, this_lines)]
1521
        if not no_base:
1522
            data.append(('BASE', self.base_tree, base_lines))
4443.2.1 by Ian Clatworthy
apply content filters when merging new files
1523
1524
        # We need to use the actual path in the working tree of the file here,
1525
        # ignoring the conflict suffixes
1526
        wt = self.this_tree
1527
        if wt.supports_content_filtering():
1528
            try:
1529
                filter_tree_path = wt.id2path(file_id)
1530
            except errors.NoSuchId:
1531
                # file has been deleted
1532
                filter_tree_path = None
1533
        else:
1534
            # Skip the id2path lookup for older formats
1535
            filter_tree_path = None
1536
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1537
        versioned = False
1538
        file_group = []
1539
        for suffix, tree, lines in data:
4869.3.22 by Andrew Bennetts
Use 'tree.has_id(file_id)' rather than using 'file_id in tree' or using tree.inventory.
1540
            if tree.has_id(file_id):
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1541
                trans_id = self._conflict_file(name, parent_id, tree, file_id,
4443.2.1 by Ian Clatworthy
apply content filters when merging new files
1542
                                               suffix, lines, filter_tree_path)
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1543
                file_group.append(trans_id)
1544
                if set_version and not versioned:
1545
                    self.tt.version_file(file_id, trans_id)
1546
                    versioned = True
1547
        return file_group
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
1548
3363.17.21 by Aaron Bentley
Conflicts are handled when merging from preview trees
1549
    def _conflict_file(self, name, parent_id, tree, file_id, suffix,
4443.2.1 by Ian Clatworthy
apply content filters when merging new files
1550
                       lines=None, filter_tree_path=None):
1534.7.157 by Aaron Bentley
Added more docs
1551
        """Emit a single conflict file."""
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1552
        name = name + '.' + suffix
1553
        trans_id = self.tt.create_path(name, parent_id)
4854.1.1 by Andrew Bennetts
Merge lp:bzr/2.0 into lp:bzr.
1554
        transform.create_from_tree(self.tt, trans_id, tree, file_id, lines,
4443.2.1 by Ian Clatworthy
apply content filters when merging new files
1555
            filter_tree_path)
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1556
        return trans_id
1557
1558
    def merge_executable(self, file_id, file_status):
1534.7.157 by Aaron Bentley
Added more docs
1559
        """Perform a merge on the execute bit."""
2590.2.3 by Aaron Bentley
Merge the execute bit based on iter_changes
1560
        executable = [self.executable(t, file_id) for t in (self.base_tree,
1561
                      self.other_tree, self.this_tree)]
3514.4.40 by John Arbash Meinel
Make 'resolver' a required parameter.
1562
        self._merge_executable(file_id, executable, file_status,
1563
                               resolver=self._three_way)
2590.2.3 by Aaron Bentley
Merge the execute bit based on iter_changes
1564
3514.4.21 by John Arbash Meinel
Hook up the lca-way logic into the path handlers.
1565
    def _merge_executable(self, file_id, executable, file_status,
3514.4.40 by John Arbash Meinel
Make 'resolver' a required parameter.
1566
                          resolver):
2590.2.3 by Aaron Bentley
Merge the execute bit based on iter_changes
1567
        """Perform a merge on the execute bit."""
1568
        base_executable, other_executable, this_executable = executable
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1569
        if file_status == "deleted":
1570
            return
3514.4.21 by John Arbash Meinel
Hook up the lca-way logic into the path handlers.
1571
        winner = resolver(*executable)
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1572
        if winner == "conflict":
1573
        # There must be a None in here, if we have a conflict, but we
1574
        # need executability since file status was not deleted.
1534.10.35 by Aaron Bentley
Merge handles contents + executable + deletion conflict
1575
            if self.executable(self.other_tree, file_id) is None:
1534.7.142 by Aaron Bentley
Fixed executability conflicts
1576
                winner = "this"
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1577
            else:
1534.7.142 by Aaron Bentley
Fixed executability conflicts
1578
                winner = "other"
1551.19.30 by Aaron Bentley
Accelerate merge by skipping file existence check when merging execute bit
1579
        if winner == 'this' and file_status != "modified":
1580
            return
1581
        trans_id = self.tt.trans_id_file_id(file_id)
1582
        try:
1583
            if self.tt.final_kind(trans_id) != "file":
1584
                return
4721.3.1 by Vincent Ladeuil
Cleanup imports.
1585
        except errors.NoSuchFile:
1551.19.30 by Aaron Bentley
Accelerate merge by skipping file existence check when merging execute bit
1586
            return
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1587
        if winner == "this":
1551.19.30 by Aaron Bentley
Accelerate merge by skipping file existence check when merging execute bit
1588
            executability = this_executable
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1589
        else:
4869.3.22 by Andrew Bennetts
Use 'tree.has_id(file_id)' rather than using 'file_id in tree' or using tree.inventory.
1590
            if self.other_tree.has_id(file_id):
2590.2.3 by Aaron Bentley
Merge the execute bit based on iter_changes
1591
                executability = other_executable
4869.3.22 by Andrew Bennetts
Use 'tree.has_id(file_id)' rather than using 'file_id in tree' or using tree.inventory.
1592
            elif self.this_tree.has_id(file_id):
2590.2.3 by Aaron Bentley
Merge the execute bit based on iter_changes
1593
                executability = this_executable
4869.3.22 by Andrew Bennetts
Use 'tree.has_id(file_id)' rather than using 'file_id in tree' or using tree.inventory.
1594
            elif self.base_tree_has_id(file_id):
2590.2.3 by Aaron Bentley
Merge the execute bit based on iter_changes
1595
                executability = base_executable
1551.19.30 by Aaron Bentley
Accelerate merge by skipping file existence check when merging execute bit
1596
        if executability is not None:
1597
            trans_id = self.tt.trans_id_file_id(file_id)
1598
            self.tt.set_executability(executability, trans_id)
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1599
1534.7.172 by Aaron Bentley
Integrated fs conflicts with merge conflicts.
1600
    def cook_conflicts(self, fs_conflicts):
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1601
        """Convert all conflicts into a form that doesn't depend on trans_id"""
4721.3.1 by Vincent Ladeuil
Cleanup imports.
1602
        self.cooked_conflicts.extend(transform.cook_conflicts(
1603
                fs_conflicts, self.tt))
1604
        fp = transform.FinalPaths(self.tt)
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1605
        for conflict in self._raw_conflicts:
1606
            conflict_type = conflict[0]
4597.7.11 by Vincent Ladeuil
Fix #531967 by creating helpers for PathConflicts when a deletion
1607
            if conflict_type == 'path conflict':
1608
                (trans_id, file_id,
1609
                this_parent, this_name,
1610
                other_parent, other_name) = conflict[1:]
1611
                if this_parent is None or this_name is None:
1612
                    this_path = '<deleted>'
1613
                else:
1614
                    parent_path =  fp.get_path(
1615
                        self.tt.trans_id_file_id(this_parent))
1616
                    this_path = osutils.pathjoin(parent_path, this_name)
1617
                if other_parent is None or other_name is None:
1618
                    other_path = '<deleted>'
1619
                else:
1620
                    parent_path =  fp.get_path(
1621
                        self.tt.trans_id_file_id(other_parent))
1622
                    other_path = osutils.pathjoin(parent_path, other_name)
1623
                c = _mod_conflicts.Conflict.factory(
1624
                    'path conflict', path=this_path,
1625
                    conflict_path=other_path,
1626
                    file_id=file_id)
1627
            elif conflict_type == 'contents conflict':
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1628
                for trans_id in conflict[1]:
1629
                    file_id = self.tt.final_file_id(trans_id)
1630
                    if file_id is not None:
1631
                        break
1632
                path = fp.get_path(trans_id)
1633
                for suffix in ('.BASE', '.THIS', '.OTHER'):
1634
                    if path.endswith(suffix):
1635
                        path = path[:-len(suffix)]
1636
                        break
4721.3.1 by Vincent Ladeuil
Cleanup imports.
1637
                c = _mod_conflicts.Conflict.factory(conflict_type,
1638
                                                    path=path, file_id=file_id)
4597.7.11 by Vincent Ladeuil
Fix #531967 by creating helpers for PathConflicts when a deletion
1639
            elif conflict_type == 'text conflict':
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1640
                trans_id = conflict[1]
1641
                path = fp.get_path(trans_id)
1642
                file_id = self.tt.final_file_id(trans_id)
4721.3.1 by Vincent Ladeuil
Cleanup imports.
1643
                c = _mod_conflicts.Conflict.factory(conflict_type,
1644
                                                    path=path, file_id=file_id)
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1645
            else:
4597.7.18 by Vincent Ladeuil
Start addressing Andrew's concerns.
1646
                raise AssertionError('bad conflict type: %r' % (conflict,))
1534.10.19 by Aaron Bentley
Stanza conversion, cooking
1647
            self.cooked_conflicts.append(c)
4721.3.1 by Vincent Ladeuil
Cleanup imports.
1648
        self.cooked_conflicts.sort(key=_mod_conflicts.Conflict.sort_key)
1534.7.141 by Aaron Bentley
Added conflict reporting
1649
1650
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1651
class WeaveMerger(Merge3Merger):
1534.7.167 by Aaron Bentley
PEP8 and comment cleanups
1652
    """Three-way tree merger, text weave merger."""
1551.6.8 by Aaron Bentley
Implemented reprocess for weave
1653
    supports_reprocess = True
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1654
    supports_show_base = False
3062.2.7 by Aaron Bentley
Prevent reverse cherry-picking with weave
1655
    supports_reverse_cherrypick = False
3062.2.9 by Aaron Bentley
Don't use the base if not cherrypicking
1656
    history_based = True
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1657
4634.101.7 by John Arbash Meinel
Start working on the ability to drop a .BASE file for --weave and --lca merge.
1658
    def _generate_merge_plan(self, file_id, base):
1659
        return self.this_tree.plan_file_merge(file_id, self.other_tree,
3062.2.9 by Aaron Bentley
Don't use the base if not cherrypicking
1660
                                              base=base)
4634.101.7 by John Arbash Meinel
Start working on the ability to drop a .BASE file for --weave and --lca merge.
1661
1662
    def _merged_lines(self, file_id):
1663
        """Generate the merged lines.
1664
        There is no distinction between lines that are meant to contain <<<<<<<
1665
        and conflicts.
1666
        """
1667
        if self.cherrypick:
1668
            base = self.base_tree
1669
        else:
1670
            base = None
1671
        plan = self._generate_merge_plan(file_id, base)
1551.19.17 by Aaron Bentley
Add debugging flag for merges
1672
        if 'merge' in debug.debug_flags:
1673
            plan = list(plan)
1674
            trans_id = self.tt.trans_id_file_id(file_id)
1675
            name = self.tt.final_name(trans_id) + '.plan'
4634.101.6 by John Arbash Meinel
killed-both and killed-base is 11 chars long.
1676
            contents = ('%11s|%s' % l for l in plan)
1551.19.17 by Aaron Bentley
Add debugging flag for merges
1677
            self.tt.new_file(name, self.tt.final_parent(trans_id), contents)
4721.3.1 by Vincent Ladeuil
Cleanup imports.
1678
        textmerge = versionedfile.PlanWeaveMerge(plan, '<<<<<<< TREE\n',
1679
                                                 '>>>>>>> MERGE-SOURCE\n')
4634.101.7 by John Arbash Meinel
Start working on the ability to drop a .BASE file for --weave and --lca merge.
1680
        lines, conflicts = textmerge.merge_lines(self.reprocess)
1681
        if conflicts:
1682
            base_lines = textmerge.base_from_plan()
1683
        else:
1684
            base_lines = None
1685
        return lines, base_lines
4869.2.2 by Andrew Bennetts
Remove some duplication.
1686
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1687
    def text_merge(self, file_id, trans_id):
1534.7.157 by Aaron Bentley
Added more docs
1688
        """Perform a (weave) text merge for a given file and file-id.
1689
        If conflicts are encountered, .THIS and .OTHER files will be emitted,
1690
        and a conflict will be noted.
1691
        """
4634.101.7 by John Arbash Meinel
Start working on the ability to drop a .BASE file for --weave and --lca merge.
1692
        lines, base_lines = self._merged_lines(file_id)
1558.15.10 by Aaron Bentley
Merge bzr.dev
1693
        lines = list(lines)
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
1694
        # Note we're checking whether the OUTPUT is binary in this case,
1558.15.5 by Aaron Bentley
Fixed binary handling in weave merge
1695
        # because we don't want to get into weave merge guts.
4721.3.1 by Vincent Ladeuil
Cleanup imports.
1696
        textfile.check_text_lines(lines)
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1697
        self.tt.create_file(lines, trans_id)
4634.101.7 by John Arbash Meinel
Start working on the ability to drop a .BASE file for --weave and --lca merge.
1698
        if base_lines is not None:
1699
            # Conflict
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1700
            self._raw_conflicts.append(('text conflict', trans_id))
1701
            name = self.tt.final_name(trans_id)
1702
            parent_id = self.tt.final_parent(trans_id)
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
1703
            file_group = self._dump_conflicts(name, parent_id, file_id,
4634.101.7 by John Arbash Meinel
Start working on the ability to drop a .BASE file for --weave and --lca merge.
1704
                                              no_base=False,
1705
                                              base_lines=base_lines)
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1706
            file_group.append(trans_id)
1707
1708
3144.3.1 by Aaron Bentley
Implement LCA merge, with problematic conflict markers
1709
class LCAMerger(WeaveMerger):
1710
4634.101.7 by John Arbash Meinel
Start working on the ability to drop a .BASE file for --weave and --lca merge.
1711
    def _generate_merge_plan(self, file_id, base):
1712
        return self.this_tree.plan_file_lca_merge(file_id, self.other_tree,
3144.3.1 by Aaron Bentley
Implement LCA merge, with problematic conflict markers
1713
                                                  base=base)
1714
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1715
class Diff3Merger(Merge3Merger):
1534.7.167 by Aaron Bentley
PEP8 and comment cleanups
1716
    """Three-way merger using external diff3 for text merging"""
1711.7.20 by John Arbash Meinel
always close files, minor PEP8 cleanup
1717
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1718
    def dump_file(self, temp_dir, name, tree, file_id):
4721.3.1 by Vincent Ladeuil
Cleanup imports.
1719
        out_path = osutils.pathjoin(temp_dir, name)
1711.7.20 by John Arbash Meinel
always close files, minor PEP8 cleanup
1720
        out_file = open(out_path, "wb")
1721
        try:
1722
            in_file = tree.get_file(file_id)
1723
            for line in in_file:
1724
                out_file.write(line)
1725
        finally:
1726
            out_file.close()
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1727
        return out_path
1728
1729
    def text_merge(self, file_id, trans_id):
1534.7.157 by Aaron Bentley
Added more docs
1730
        """Perform a diff3 merge using a specified file-id and trans-id.
1731
        If conflicts are encountered, .BASE, .THIS. and .OTHER conflict files
1732
        will be dumped, and a will be conflict noted.
1733
        """
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1734
        import bzrlib.patch
1996.3.18 by John Arbash Meinel
Now that mkdtemp and rmtree are lazy, they should not be directly improted.
1735
        temp_dir = osutils.mkdtemp(prefix="bzr-")
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1736
        try:
4721.3.1 by Vincent Ladeuil
Cleanup imports.
1737
            new_file = osutils.pathjoin(temp_dir, "new")
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1738
            this = self.dump_file(temp_dir, "this", self.this_tree, file_id)
1739
            base = self.dump_file(temp_dir, "base", self.base_tree, file_id)
1740
            other = self.dump_file(temp_dir, "other", self.other_tree, file_id)
1741
            status = bzrlib.patch.diff3(new_file, this, base, other)
1742
            if status not in (0, 1):
4721.3.1 by Vincent Ladeuil
Cleanup imports.
1743
                raise errors.BzrError("Unhandled diff3 exit code")
1711.7.20 by John Arbash Meinel
always close files, minor PEP8 cleanup
1744
            f = open(new_file, 'rb')
1745
            try:
1746
                self.tt.create_file(f, trans_id)
1747
            finally:
1748
                f.close()
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1749
            if status == 1:
1750
                name = self.tt.final_name(trans_id)
1751
                parent_id = self.tt.final_parent(trans_id)
1752
                self._dump_conflicts(name, parent_id, file_id)
1551.8.39 by Aaron Bentley
Fix diff3 conflict-reporting bug
1753
                self._raw_conflicts.append(('text conflict', trans_id))
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1754
        finally:
1996.3.18 by John Arbash Meinel
Now that mkdtemp and rmtree are lazy, they should not be directly improted.
1755
            osutils.rmtree(temp_dir)
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1756
1757
1758
def merge_inner(this_branch, other_tree, base_tree, ignore_zero=False,
2255.2.31 by Robert Collins
Work in progress to make merge_inner work with dirstate trees.
1759
                backup_files=False,
1760
                merge_type=Merge3Merger,
1761
                interesting_ids=None,
1762
                show_base=False,
1763
                reprocess=False,
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1764
                other_rev_id=None,
1765
                interesting_files=None,
1534.9.9 by Aaron Bentley
Added progress bar to pull
1766
                this_tree=None,
4961.2.9 by Martin Pool
Rip out most remaining uses of DummyProgressBar
1767
                pb=None,
1551.11.10 by Aaron Bentley
Add change reporting to pull
1768
                change_reporter=None):
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
1769
    """Primary interface for merging.
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1770
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
1771
        typical use is probably
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1772
        'merge_inner(branch, branch.get_revision_tree(other_revision),
1773
                     branch.get_revision_tree(base_revision))'
1774
        """
1775
    if this_tree is None:
4721.3.1 by Vincent Ladeuil
Cleanup imports.
1776
        raise errors.BzrError("bzrlib.merge.merge_inner requires a this_tree "
1777
                              "parameter as of bzrlib version 0.8.")
2255.2.31 by Robert Collins
Work in progress to make merge_inner work with dirstate trees.
1778
    merger = Merger(this_branch, other_tree, base_tree, this_tree=this_tree,
1551.11.10 by Aaron Bentley
Add change reporting to pull
1779
                    pb=pb, change_reporter=change_reporter)
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1780
    merger.backup_files = backup_files
1781
    merger.merge_type = merge_type
1782
    merger.interesting_ids = interesting_ids
1551.2.23 by Aaron Bentley
Got merge_inner's ignore_zero parameter working
1783
    merger.ignore_zero = ignore_zero
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1784
    if interesting_files:
3376.2.4 by Martin Pool
Remove every assert statement from bzrlib!
1785
        if interesting_ids:
1786
            raise ValueError('Only supply interesting_ids'
1787
                             ' or interesting_files')
2590.2.5 by Aaron Bentley
Allow selected files to be specified instead of selected ids
1788
        merger.interesting_files = interesting_files
1979.2.1 by Robert Collins
(robertc) adds a convenience method "merge_from_branch" to WorkingTree.
1789
    merger.show_base = show_base
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1790
    merger.reprocess = reprocess
1791
    merger.other_rev_id = other_rev_id
1792
    merger.other_basis = other_rev_id
3249.3.1 by John Arbash Meinel
Implement cherrypick support for Merge3
1793
    get_revision_id = getattr(base_tree, 'get_revision_id', None)
1794
    if get_revision_id is None:
1795
        get_revision_id = base_tree.last_revision
4595.13.2 by Alexander Belchenko
[cherrypick revno 4650 from bzr.dev] Fix shelve on windows. (Robert Collins, #305006)
1796
    merger.cache_trees_with_revision_ids([other_tree, base_tree, this_tree])
3249.3.1 by John Arbash Meinel
Implement cherrypick support for Merge3
1797
    merger.set_base_revision(get_revision_id(), this_branch)
1534.7.140 by Aaron Bentley
Moved the merge stuff into merge.py
1798
    return merger.do_merge()
1799
2221.4.15 by Aaron Bentley
Use RegistryOption for merge type
1800
def get_merge_type_registry():
2221.4.17 by Aaron Bentley
PEP8-ness
1801
    """Merge type registry is in bzrlib.option to avoid circular imports.
2221.4.15 by Aaron Bentley
Use RegistryOption for merge type
1802
1803
    This method provides a sanctioned way to retrieve it.
1804
    """
1805
    from bzrlib import option
2221.4.16 by Aaron Bentley
Add tests for get_merge_type_registry
1806
    return option._merge_type_registry
1551.15.46 by Aaron Bentley
Move plan merge to tree
1807
1808
1809
def _plan_annotate_merge(annotated_a, annotated_b, ancestors_a, ancestors_b):
1810
    def status_a(revision, text):
1811
        if revision in ancestors_b:
1812
            return 'killed-b', text
1813
        else:
1814
            return 'new-a', text
1815
1816
    def status_b(revision, text):
1817
        if revision in ancestors_a:
1818
            return 'killed-a', text
1819
        else:
1820
            return 'new-b', text
1821
1822
    plain_a = [t for (a, t) in annotated_a]
1823
    plain_b = [t for (a, t) in annotated_b]
1824
    matcher = patiencediff.PatienceSequenceMatcher(None, plain_a, plain_b)
1825
    blocks = matcher.get_matching_blocks()
1826
    a_cur = 0
1827
    b_cur = 0
1828
    for ai, bi, l in blocks:
1829
        # process all mismatched sections
1830
        # (last mismatched section is handled because blocks always
1831
        # includes a 0-length last block)
1832
        for revision, text in annotated_a[a_cur:ai]:
1833
            yield status_a(revision, text)
1834
        for revision, text in annotated_b[b_cur:bi]:
1835
            yield status_b(revision, text)
1836
        # and now the matched section
1837
        a_cur = ai + l
1838
        b_cur = bi + l
3376.2.8 by Martin Pool
Some review cleanups for assertion removal
1839
        for text_a in plain_a[ai:a_cur]:
1551.15.46 by Aaron Bentley
Move plan merge to tree
1840
            yield "unchanged", text_a
3062.1.9 by Aaron Bentley
Move PlanMerge into merge and _PlanMergeVersionedFile into versionedfile
1841
1842
3144.3.1 by Aaron Bentley
Implement LCA merge, with problematic conflict markers
1843
class _PlanMergeBase(object):
3062.1.9 by Aaron Bentley
Move PlanMerge into merge and _PlanMergeVersionedFile into versionedfile
1844
3350.6.10 by Martin Pool
VersionedFiles review cleanups
1845
    def __init__(self, a_rev, b_rev, vf, key_prefix):
3062.1.9 by Aaron Bentley
Move PlanMerge into merge and _PlanMergeVersionedFile into versionedfile
1846
        """Contructor.
1847
1848
        :param a_rev: Revision-id of one revision to merge
1849
        :param b_rev: Revision-id of the other revision to merge
3350.6.4 by Robert Collins
First cut at pluralised VersionedFiles. Some rather massive API incompatabilities, primarily because of the difficulty of coherence among competing stores.
1850
        :param vf: A VersionedFiles containing both revisions
3350.6.10 by Martin Pool
VersionedFiles review cleanups
1851
        :param key_prefix: A prefix for accessing keys in vf, typically
1852
            (file_id,).
3062.1.9 by Aaron Bentley
Move PlanMerge into merge and _PlanMergeVersionedFile into versionedfile
1853
        """
1854
        self.a_rev = a_rev
1855
        self.b_rev = b_rev
1856
        self.vf = vf
3062.1.12 by Aaron Bentley
Implement simple text cache
1857
        self._last_lines = None
1858
        self._last_lines_revision_id = None
3144.3.7 by Aaron Bentley
Update from review
1859
        self._cached_matching_blocks = {}
3350.6.10 by Martin Pool
VersionedFiles review cleanups
1860
        self._key_prefix = key_prefix
3514.2.2 by John Arbash Meinel
Restore a real weave merge to 'bzr merge --weave'.
1861
        self._precache_tip_lines()
1862
1863
    def _precache_tip_lines(self):
1864
        lines = self.get_lines([self.a_rev, self.b_rev])
1865
        self.lines_a = lines[self.a_rev]
1866
        self.lines_b = lines[self.b_rev]
3350.6.4 by Robert Collins
First cut at pluralised VersionedFiles. Some rather massive API incompatabilities, primarily because of the difficulty of coherence among competing stores.
1867
1868
    def get_lines(self, revisions):
1869
        """Get lines for revisions from the backing VersionedFiles.
3890.2.9 by John Arbash Meinel
Start using osutils.chunks_as_lines rather than osutils.split_lines.
1870
3350.6.10 by Martin Pool
VersionedFiles review cleanups
1871
        :raises RevisionNotPresent: on absent texts.
3350.6.4 by Robert Collins
First cut at pluralised VersionedFiles. Some rather massive API incompatabilities, primarily because of the difficulty of coherence among competing stores.
1872
        """
3350.6.10 by Martin Pool
VersionedFiles review cleanups
1873
        keys = [(self._key_prefix + (rev,)) for rev in revisions]
3350.6.4 by Robert Collins
First cut at pluralised VersionedFiles. Some rather massive API incompatabilities, primarily because of the difficulty of coherence among competing stores.
1874
        result = {}
1875
        for record in self.vf.get_record_stream(keys, 'unordered', True):
1876
            if record.storage_kind == 'absent':
1877
                raise errors.RevisionNotPresent(record.key, self.vf)
3890.2.9 by John Arbash Meinel
Start using osutils.chunks_as_lines rather than osutils.split_lines.
1878
            result[record.key[-1]] = osutils.chunks_to_lines(
1879
                record.get_bytes_as('chunked'))
3350.6.4 by Robert Collins
First cut at pluralised VersionedFiles. Some rather massive API incompatabilities, primarily because of the difficulty of coherence among competing stores.
1880
        return result
3062.1.9 by Aaron Bentley
Move PlanMerge into merge and _PlanMergeVersionedFile into versionedfile
1881
1882
    def plan_merge(self):
1883
        """Generate a 'plan' for merging the two revisions.
1884
1885
        This involves comparing their texts and determining the cause of
1886
        differences.  If text A has a line and text B does not, then either the
1887
        line was added to text A, or it was deleted from B.  Once the causes
1888
        are combined, they are written out in the format described in
1889
        VersionedFile.plan_merge
1890
        """
1891
        blocks = self._get_matching_blocks(self.a_rev, self.b_rev)
3144.3.1 by Aaron Bentley
Implement LCA merge, with problematic conflict markers
1892
        unique_a, unique_b = self._unique_lines(blocks)
1893
        new_a, killed_b = self._determine_status(self.a_rev, unique_a)
1894
        new_b, killed_a = self._determine_status(self.b_rev, unique_b)
1895
        return self._iter_plan(blocks, new_a, killed_b, new_b, killed_a)
1896
1897
    def _iter_plan(self, blocks, new_a, killed_b, new_b, killed_a):
3062.1.9 by Aaron Bentley
Move PlanMerge into merge and _PlanMergeVersionedFile into versionedfile
1898
        last_i = 0
1899
        last_j = 0
1900
        for i, j, n in blocks:
1901
            for a_index in range(last_i, i):
1902
                if a_index in new_a:
3144.3.2 by Aaron Bentley
Get conflict handling working
1903
                    if a_index in killed_b:
1904
                        yield 'conflicted-a', self.lines_a[a_index]
1905
                    else:
1906
                        yield 'new-a', self.lines_a[a_index]
1907
                else:
3144.3.1 by Aaron Bentley
Implement LCA merge, with problematic conflict markers
1908
                    yield 'killed-b', self.lines_a[a_index]
3062.1.9 by Aaron Bentley
Move PlanMerge into merge and _PlanMergeVersionedFile into versionedfile
1909
            for b_index in range(last_j, j):
1910
                if b_index in new_b:
3144.3.2 by Aaron Bentley
Get conflict handling working
1911
                    if b_index in killed_a:
3144.3.10 by Aaron Bentley
Use correct index when emitting conflicted-b
1912
                        yield 'conflicted-b', self.lines_b[b_index]
3144.3.2 by Aaron Bentley
Get conflict handling working
1913
                    else:
1914
                        yield 'new-b', self.lines_b[b_index]
1915
                else:
3144.3.1 by Aaron Bentley
Implement LCA merge, with problematic conflict markers
1916
                    yield 'killed-a', self.lines_b[b_index]
3062.1.9 by Aaron Bentley
Move PlanMerge into merge and _PlanMergeVersionedFile into versionedfile
1917
            # handle common lines
1918
            for a_index in range(i, i+n):
3144.3.1 by Aaron Bentley
Implement LCA merge, with problematic conflict markers
1919
                yield 'unchanged', self.lines_a[a_index]
3062.1.9 by Aaron Bentley
Move PlanMerge into merge and _PlanMergeVersionedFile into versionedfile
1920
            last_i = i+n
1921
            last_j = j+n
1922
1923
    def _get_matching_blocks(self, left_revision, right_revision):
1924
        """Return a description of which sections of two revisions match.
1925
1926
        See SequenceMatcher.get_matching_blocks
1927
        """
3144.3.7 by Aaron Bentley
Update from review
1928
        cached = self._cached_matching_blocks.get((left_revision,
1929
                                                   right_revision))
1930
        if cached is not None:
1931
            return cached
3062.1.12 by Aaron Bentley
Implement simple text cache
1932
        if self._last_lines_revision_id == left_revision:
1933
            left_lines = self._last_lines
3350.6.4 by Robert Collins
First cut at pluralised VersionedFiles. Some rather massive API incompatabilities, primarily because of the difficulty of coherence among competing stores.
1934
            right_lines = self.get_lines([right_revision])[right_revision]
3062.1.12 by Aaron Bentley
Implement simple text cache
1935
        else:
3350.6.4 by Robert Collins
First cut at pluralised VersionedFiles. Some rather massive API incompatabilities, primarily because of the difficulty of coherence among competing stores.
1936
            lines = self.get_lines([left_revision, right_revision])
1937
            left_lines = lines[left_revision]
1938
            right_lines = lines[right_revision]
3062.1.12 by Aaron Bentley
Implement simple text cache
1939
        self._last_lines = right_lines
1940
        self._last_lines_revision_id = right_revision
3062.1.9 by Aaron Bentley
Move PlanMerge into merge and _PlanMergeVersionedFile into versionedfile
1941
        matcher = patiencediff.PatienceSequenceMatcher(None, left_lines,
1942
                                                       right_lines)
1943
        return matcher.get_matching_blocks()
1944
1945
    def _unique_lines(self, matching_blocks):
1946
        """Analyse matching_blocks to determine which lines are unique
1947
1948
        :return: a tuple of (unique_left, unique_right), where the values are
1949
            sets of line numbers of unique lines.
1950
        """
1951
        last_i = 0
1952
        last_j = 0
1953
        unique_left = []
1954
        unique_right = []
1955
        for i, j, n in matching_blocks:
1956
            unique_left.extend(range(last_i, i))
1957
            unique_right.extend(range(last_j, j))
1958
            last_i = i + n
1959
            last_j = j + n
1960
        return unique_left, unique_right
1961
3144.3.1 by Aaron Bentley
Implement LCA merge, with problematic conflict markers
1962
    @staticmethod
1963
    def _subtract_plans(old_plan, new_plan):
3144.3.7 by Aaron Bentley
Update from review
1964
        """Remove changes from new_plan that came from old_plan.
1965
1966
        It is assumed that the difference between the old_plan and new_plan
1967
        is their choice of 'b' text.
1968
1969
        All lines from new_plan that differ from old_plan are emitted
1970
        verbatim.  All lines from new_plan that match old_plan but are
1971
        not about the 'b' revision are emitted verbatim.
1972
1973
        Lines that match and are about the 'b' revision are the lines we
1974
        don't want, so we convert 'killed-b' -> 'unchanged', and 'new-b'
1975
        is skipped entirely.
1976
        """
3144.3.1 by Aaron Bentley
Implement LCA merge, with problematic conflict markers
1977
        matcher = patiencediff.PatienceSequenceMatcher(None, old_plan,
1978
                                                       new_plan)
1979
        last_j = 0
1980
        for i, j, n in matcher.get_matching_blocks():
1981
            for jj in range(last_j, j):
1982
                yield new_plan[jj]
1983
            for jj in range(j, j+n):
1984
                plan_line = new_plan[jj]
1985
                if plan_line[0] == 'new-b':
1986
                    pass
1987
                elif plan_line[0] == 'killed-b':
1988
                    yield 'unchanged', plan_line[1]
1989
                else:
1990
                    yield plan_line
1991
            last_j = j + n
1992
1993
1994
class _PlanMerge(_PlanMergeBase):
1995
    """Plan an annotate merge using on-the-fly annotation"""
1996
3350.6.10 by Martin Pool
VersionedFiles review cleanups
1997
    def __init__(self, a_rev, b_rev, vf, key_prefix):
3514.2.2 by John Arbash Meinel
Restore a real weave merge to 'bzr merge --weave'.
1998
        super(_PlanMerge, self).__init__(a_rev, b_rev, vf, key_prefix)
1999
        self.a_key = self._key_prefix + (self.a_rev,)
2000
        self.b_key = self._key_prefix + (self.b_rev,)
4721.3.1 by Vincent Ladeuil
Cleanup imports.
2001
        self.graph = _mod_graph.Graph(self.vf)
3514.2.11 by John Arbash Meinel
Shortcut the case when one revision is in the ancestry of the other.
2002
        heads = self.graph.heads((self.a_key, self.b_key))
2003
        if len(heads) == 1:
2004
            # one side dominates, so we can just return its values, yay for
2005
            # per-file graphs
2006
            # Ideally we would know that before we get this far
2007
            self._head_key = heads.pop()
2008
            if self._head_key == self.a_key:
2009
                other = b_rev
2010
            else:
2011
                other = a_rev
4721.3.1 by Vincent Ladeuil
Cleanup imports.
2012
            trace.mutter('found dominating revision for %s\n%s > %s', self.vf,
2013
                         self._head_key[-1], other)
3514.2.11 by John Arbash Meinel
Shortcut the case when one revision is in the ancestry of the other.
2014
            self._weave = None
2015
        else:
2016
            self._head_key = None
2017
            self._build_weave()
3514.2.2 by John Arbash Meinel
Restore a real weave merge to 'bzr merge --weave'.
2018
2019
    def _precache_tip_lines(self):
2020
        # Turn this into a no-op, because we will do this later
2021
        pass
2022
2023
    def _find_recursive_lcas(self):
2024
        """Find all the ancestors back to a unique lca"""
2025
        cur_ancestors = (self.a_key, self.b_key)
2026
        # graph.find_lca(uncommon, keys) now returns plain NULL_REVISION,
3514.2.5 by John Arbash Meinel
Switch to the get_parent_map design I had settled on.
2027
        # rather than a key tuple. We will just map that directly to no common
2028
        # ancestors.
2029
        parent_map = {}
3514.2.2 by John Arbash Meinel
Restore a real weave merge to 'bzr merge --weave'.
2030
        while True:
2031
            next_lcas = self.graph.find_lca(*cur_ancestors)
3514.2.5 by John Arbash Meinel
Switch to the get_parent_map design I had settled on.
2032
            # Map a plain NULL_REVISION to a simple no-ancestors
4721.3.1 by Vincent Ladeuil
Cleanup imports.
2033
            if next_lcas == set([_mod_revision.NULL_REVISION]):
3514.2.5 by John Arbash Meinel
Switch to the get_parent_map design I had settled on.
2034
                next_lcas = ()
3514.2.9 by John Arbash Meinel
Add some debugging, and work on getting the graph right so we get the weave insertion order correct.
2035
            # Order the lca's based on when they were merged into the tip
2036
            # While the actual merge portion of weave merge uses a set() of
2037
            # active revisions, the order of insertion *does* effect the
2038
            # implicit ordering of the texts.
3514.2.5 by John Arbash Meinel
Switch to the get_parent_map design I had settled on.
2039
            for rev_key in cur_ancestors:
3514.2.9 by John Arbash Meinel
Add some debugging, and work on getting the graph right so we get the weave insertion order correct.
2040
                ordered_parents = tuple(self.graph.find_merge_order(rev_key,
2041
                                                                    next_lcas))
2042
                parent_map[rev_key] = ordered_parents
3514.2.2 by John Arbash Meinel
Restore a real weave merge to 'bzr merge --weave'.
2043
            if len(next_lcas) == 0:
2044
                break
2045
            elif len(next_lcas) == 1:
3514.2.9 by John Arbash Meinel
Add some debugging, and work on getting the graph right so we get the weave insertion order correct.
2046
                parent_map[list(next_lcas)[0]] = ()
3514.2.2 by John Arbash Meinel
Restore a real weave merge to 'bzr merge --weave'.
2047
                break
3514.2.10 by John Arbash Meinel
Handle more edge cases.
2048
            elif len(next_lcas) > 2:
3514.2.7 by John Arbash Meinel
Fix the failing test by implementing the fallback logic.
2049
                # More than 2 lca's, fall back to grabbing all nodes between
2050
                # this and the unique lca.
4721.3.1 by Vincent Ladeuil
Cleanup imports.
2051
                trace.mutter('More than 2 LCAs, falling back to all nodes for:'
2052
                             ' %s, %s\n=> %s',
2053
                             self.a_key, self.b_key, cur_ancestors)
3514.2.7 by John Arbash Meinel
Fix the failing test by implementing the fallback logic.
2054
                cur_lcas = next_lcas
2055
                while len(cur_lcas) > 1:
2056
                    cur_lcas = self.graph.find_lca(*cur_lcas)
2057
                if len(cur_lcas) == 0:
2058
                    # No common base to find, use the full ancestry
2059
                    unique_lca = None
2060
                else:
3514.2.8 by John Arbash Meinel
The insertion ordering into the weave has an impact on conflicts.
2061
                    unique_lca = list(cur_lcas)[0]
4721.3.1 by Vincent Ladeuil
Cleanup imports.
2062
                    if unique_lca == _mod_revision.NULL_REVISION:
3514.2.14 by John Arbash Meinel
Bring in the code to collapse linear portions of the graph.
2063
                        # find_lca will return a plain 'NULL_REVISION' rather
2064
                        # than a key tuple when there is no common ancestor, we
2065
                        # prefer to just use None, because it doesn't confuse
2066
                        # _get_interesting_texts()
2067
                        unique_lca = None
3514.2.7 by John Arbash Meinel
Fix the failing test by implementing the fallback logic.
2068
                parent_map.update(self._find_unique_parents(next_lcas,
2069
                                                            unique_lca))
2070
                break
3514.2.2 by John Arbash Meinel
Restore a real weave merge to 'bzr merge --weave'.
2071
            cur_ancestors = next_lcas
3514.2.5 by John Arbash Meinel
Switch to the get_parent_map design I had settled on.
2072
        return parent_map
3514.2.2 by John Arbash Meinel
Restore a real weave merge to 'bzr merge --weave'.
2073
3514.2.7 by John Arbash Meinel
Fix the failing test by implementing the fallback logic.
2074
    def _find_unique_parents(self, tip_keys, base_key):
2075
        """Find ancestors of tip that aren't ancestors of base.
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
2076
3514.2.7 by John Arbash Meinel
Fix the failing test by implementing the fallback logic.
2077
        :param tip_keys: Nodes that are interesting
2078
        :param base_key: Cull all ancestors of this node
2079
        :return: The parent map for all revisions between tip_keys and
2080
            base_key. base_key will be included. References to nodes outside of
2081
            the ancestor set will also be removed.
2082
        """
2083
        # TODO: this would be simpler if find_unique_ancestors took a list
2084
        #       instead of a single tip, internally it supports it, but it
2085
        #       isn't a "backwards compatible" api change.
2086
        if base_key is None:
2087
            parent_map = dict(self.graph.iter_ancestry(tip_keys))
3514.2.14 by John Arbash Meinel
Bring in the code to collapse linear portions of the graph.
2088
            # We remove NULL_REVISION because it isn't a proper tuple key, and
2089
            # thus confuses things like _get_interesting_texts, and our logic
2090
            # to add the texts into the memory weave.
4721.3.1 by Vincent Ladeuil
Cleanup imports.
2091
            if _mod_revision.NULL_REVISION in parent_map:
2092
                parent_map.pop(_mod_revision.NULL_REVISION)
3514.2.7 by John Arbash Meinel
Fix the failing test by implementing the fallback logic.
2093
        else:
2094
            interesting = set()
2095
            for tip in tip_keys:
2096
                interesting.update(
2097
                    self.graph.find_unique_ancestors(tip, [base_key]))
2098
            parent_map = self.graph.get_parent_map(interesting)
2099
            parent_map[base_key] = ()
3514.2.12 by John Arbash Meinel
Start refactoring into helper functions
2100
        culled_parent_map, child_map, tails = self._remove_external_references(
2101
            parent_map)
3514.2.13 by John Arbash Meinel
Add the ability to prune extra tails from the parent_map.
2102
        # Remove all the tails but base_key
3514.2.14 by John Arbash Meinel
Bring in the code to collapse linear portions of the graph.
2103
        if base_key is not None:
2104
            tails.remove(base_key)
2105
            self._prune_tails(culled_parent_map, child_map, tails)
2106
        # Now remove all the uninteresting 'linear' regions
3514.2.15 by John Arbash Meinel
Enable collapsing linear regions.
2107
        simple_map = _mod_graph.collapse_linear_regions(culled_parent_map)
2108
        return simple_map
3514.2.12 by John Arbash Meinel
Start refactoring into helper functions
2109
2110
    @staticmethod
2111
    def _remove_external_references(parent_map):
2112
        """Remove references that go outside of the parent map.
2113
2114
        :param parent_map: Something returned from Graph.get_parent_map(keys)
2115
        :return: (filtered_parent_map, child_map, tails)
2116
            filtered_parent_map is parent_map without external references
2117
            child_map is the {parent_key: [child_keys]} mapping
2118
            tails is a list of nodes that do not have any parents in the map
2119
        """
2120
        # TODO: The basic effect of this function seems more generic than
2121
        #       _PlanMerge. But the specific details of building a child_map,
2122
        #       and computing tails seems very specific to _PlanMerge.
2123
        #       Still, should this be in Graph land?
2124
        filtered_parent_map = {}
2125
        child_map = {}
2126
        tails = []
3514.2.7 by John Arbash Meinel
Fix the failing test by implementing the fallback logic.
2127
        for key, parent_keys in parent_map.iteritems():
3514.2.12 by John Arbash Meinel
Start refactoring into helper functions
2128
            culled_parent_keys = [p for p in parent_keys if p in parent_map]
2129
            if not culled_parent_keys:
2130
                tails.append(key)
2131
            for parent_key in culled_parent_keys:
2132
                child_map.setdefault(parent_key, []).append(key)
2133
            # TODO: Do we want to do this, it adds overhead for every node,
2134
            #       just to say that the node has no children
2135
            child_map.setdefault(key, [])
2136
            filtered_parent_map[key] = culled_parent_keys
2137
        return filtered_parent_map, child_map, tails
2138
3514.2.13 by John Arbash Meinel
Add the ability to prune extra tails from the parent_map.
2139
    @staticmethod
2140
    def _prune_tails(parent_map, child_map, tails_to_remove):
2141
        """Remove tails from the parent map.
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
2142
3514.2.13 by John Arbash Meinel
Add the ability to prune extra tails from the parent_map.
2143
        This will remove the supplied revisions until no more children have 0
2144
        parents.
2145
2146
        :param parent_map: A dict of {child: [parents]}, this dictionary will
2147
            be modified in place.
2148
        :param tails_to_remove: A list of tips that should be removed,
2149
            this list will be consumed
2150
        :param child_map: The reverse dict of parent_map ({parent: [children]})
2151
            this dict will be modified
2152
        :return: None, parent_map will be modified in place.
2153
        """
2154
        while tails_to_remove:
2155
            next = tails_to_remove.pop()
2156
            parent_map.pop(next)
2157
            children = child_map.pop(next)
2158
            for child in children:
2159
                child_parents = parent_map[child]
2160
                child_parents.remove(next)
2161
                if len(child_parents) == 0:
2162
                    tails_to_remove.append(child)
3514.2.7 by John Arbash Meinel
Fix the failing test by implementing the fallback logic.
2163
3514.2.5 by John Arbash Meinel
Switch to the get_parent_map design I had settled on.
2164
    def _get_interesting_texts(self, parent_map):
3514.2.2 by John Arbash Meinel
Restore a real weave merge to 'bzr merge --weave'.
2165
        """Return a dict of texts we are interested in.
2166
2167
        Note that the input is in key tuples, but the output is in plain
2168
        revision ids.
2169
3514.2.5 by John Arbash Meinel
Switch to the get_parent_map design I had settled on.
2170
        :param parent_map: The output from _find_recursive_lcas
3514.2.2 by John Arbash Meinel
Restore a real weave merge to 'bzr merge --weave'.
2171
        :return: A dict of {'revision_id':lines} as returned by
2172
            _PlanMergeBase.get_lines()
2173
        """
3514.2.5 by John Arbash Meinel
Switch to the get_parent_map design I had settled on.
2174
        all_revision_keys = set(parent_map)
2175
        all_revision_keys.add(self.a_key)
2176
        all_revision_keys.add(self.b_key)
3514.2.2 by John Arbash Meinel
Restore a real weave merge to 'bzr merge --weave'.
2177
3514.2.5 by John Arbash Meinel
Switch to the get_parent_map design I had settled on.
2178
        # Everything else is in 'keys' but get_lines is in 'revision_ids'
2179
        all_texts = self.get_lines([k[-1] for k in all_revision_keys])
3514.2.2 by John Arbash Meinel
Restore a real weave merge to 'bzr merge --weave'.
2180
        return all_texts
2181
2182
    def _build_weave(self):
2183
        from bzrlib import weave
2184
        self._weave = weave.Weave(weave_name='in_memory_weave',
2185
                                  allow_reserved=True)
3514.2.5 by John Arbash Meinel
Switch to the get_parent_map design I had settled on.
2186
        parent_map = self._find_recursive_lcas()
2187
2188
        all_texts = self._get_interesting_texts(parent_map)
2189
2190
        # Note: Unfortunately, the order given by topo_sort will effect the
2191
        # ordering resolution in the output. Specifically, if you add A then B,
2192
        # then in the output text A lines will show up before B lines. And, of
2193
        # course, topo_sort doesn't guarantee any real ordering.
3514.2.8 by John Arbash Meinel
The insertion ordering into the weave has an impact on conflicts.
2194
        # So we use merge_sort, and add a fake node on the tip.
2195
        # This ensures that left-hand parents will always be inserted into the
2196
        # weave before right-hand parents.
2197
        tip_key = self._key_prefix + (_mod_revision.CURRENT_REVISION,)
2198
        parent_map[tip_key] = (self.a_key, self.b_key)
2199
2200
        for seq_num, key, depth, eom in reversed(tsort.merge_sort(parent_map,
2201
                                                                  tip_key)):
2202
            if key == tip_key:
2203
                continue
3514.2.9 by John Arbash Meinel
Add some debugging, and work on getting the graph right so we get the weave insertion order correct.
2204
        # for key in tsort.topo_sort(parent_map):
3514.2.5 by John Arbash Meinel
Switch to the get_parent_map design I had settled on.
2205
            parent_keys = parent_map[key]
2206
            revision_id = key[-1]
2207
            parent_ids = [k[-1] for k in parent_keys]
2208
            self._weave.add_lines(revision_id, parent_ids,
2209
                                  all_texts[revision_id])
3514.2.2 by John Arbash Meinel
Restore a real weave merge to 'bzr merge --weave'.
2210
2211
    def plan_merge(self):
2212
        """Generate a 'plan' for merging the two revisions.
2213
2214
        This involves comparing their texts and determining the cause of
2215
        differences.  If text A has a line and text B does not, then either the
2216
        line was added to text A, or it was deleted from B.  Once the causes
2217
        are combined, they are written out in the format described in
2218
        VersionedFile.plan_merge
2219
        """
2220
        if self._head_key is not None: # There was a single head
2221
            if self._head_key == self.a_key:
2222
                plan = 'new-a'
3062.1.9 by Aaron Bentley
Move PlanMerge into merge and _PlanMergeVersionedFile into versionedfile
2223
            else:
3514.2.2 by John Arbash Meinel
Restore a real weave merge to 'bzr merge --weave'.
2224
                if self._head_key != self.b_key:
2225
                    raise AssertionError('There was an invalid head: %s != %s'
2226
                                         % (self.b_key, self._head_key))
2227
                plan = 'new-b'
3514.2.16 by John Arbash Meinel
Review feedback from Ian.
2228
            head_rev = self._head_key[-1]
2229
            lines = self.get_lines([head_rev])[head_rev]
3514.2.2 by John Arbash Meinel
Restore a real weave merge to 'bzr merge --weave'.
2230
            return ((plan, line) for line in lines)
2231
        return self._weave.plan_merge(self.a_rev, self.b_rev)
3062.2.1 by Aaron Bentley
Add support for plan-merge with a base
2232
3144.3.1 by Aaron Bentley
Implement LCA merge, with problematic conflict markers
2233
2234
class _PlanLCAMerge(_PlanMergeBase):
2235
    """
3144.3.7 by Aaron Bentley
Update from review
2236
    This merge algorithm differs from _PlanMerge in that:
3144.3.1 by Aaron Bentley
Implement LCA merge, with problematic conflict markers
2237
    1. comparisons are done against LCAs only
2238
    2. cases where a contested line is new versus one LCA but old versus
3144.3.7 by Aaron Bentley
Update from review
2239
       another are marked as conflicts, by emitting the line as conflicted-a
2240
       or conflicted-b.
3144.3.1 by Aaron Bentley
Implement LCA merge, with problematic conflict markers
2241
2242
    This is faster, and hopefully produces more useful output.
2243
    """
2244
3350.6.10 by Martin Pool
VersionedFiles review cleanups
2245
    def __init__(self, a_rev, b_rev, vf, key_prefix, graph):
2246
        _PlanMergeBase.__init__(self, a_rev, b_rev, vf, key_prefix)
2247
        lcas = graph.find_lca(key_prefix + (a_rev,), key_prefix + (b_rev,))
3350.6.5 by Robert Collins
Update to bzr.dev.
2248
        self.lcas = set()
2249
        for lca in lcas:
4721.3.1 by Vincent Ladeuil
Cleanup imports.
2250
            if lca == _mod_revision.NULL_REVISION:
3350.6.5 by Robert Collins
Update to bzr.dev.
2251
                self.lcas.add(lca)
2252
            else:
2253
                self.lcas.add(lca[-1])
3144.3.7 by Aaron Bentley
Update from review
2254
        for lca in self.lcas:
3287.17.1 by John Arbash Meinel
Fix bug #235715 by using the empty list as the text for a base of NULL_REVISION.
2255
            if _mod_revision.is_null(lca):
2256
                lca_lines = []
2257
            else:
3350.6.5 by Robert Collins
Update to bzr.dev.
2258
                lca_lines = self.get_lines([lca])[lca]
3144.3.7 by Aaron Bentley
Update from review
2259
            matcher = patiencediff.PatienceSequenceMatcher(None, self.lines_a,
2260
                                                           lca_lines)
2261
            blocks = list(matcher.get_matching_blocks())
2262
            self._cached_matching_blocks[(a_rev, lca)] = blocks
2263
            matcher = patiencediff.PatienceSequenceMatcher(None, self.lines_b,
2264
                                                           lca_lines)
2265
            blocks = list(matcher.get_matching_blocks())
2266
            self._cached_matching_blocks[(b_rev, lca)] = blocks
3144.3.1 by Aaron Bentley
Implement LCA merge, with problematic conflict markers
2267
3144.3.7 by Aaron Bentley
Update from review
2268
    def _determine_status(self, revision_id, unique_line_numbers):
3144.3.1 by Aaron Bentley
Implement LCA merge, with problematic conflict markers
2269
        """Determines the status unique lines versus all lcas.
2270
2271
        Basically, determines why the line is unique to this revision.
2272
2273
        A line may be determined new, killed, or both.
2274
3144.3.7 by Aaron Bentley
Update from review
2275
        If a line is determined new, that means it was not present in at least
2276
        one LCA, and is not present in the other merge revision.
3144.3.1 by Aaron Bentley
Implement LCA merge, with problematic conflict markers
2277
2278
        If a line is determined killed, that means the line was present in
2279
        at least one LCA.
2280
2281
        If a line is killed and new, this indicates that the two merge
2282
        revisions contain differing conflict resolutions.
3144.3.7 by Aaron Bentley
Update from review
2283
        :param revision_id: The id of the revision in which the lines are
2284
            unique
2285
        :param unique_line_numbers: The line numbers of unique lines.
3144.3.1 by Aaron Bentley
Implement LCA merge, with problematic conflict markers
2286
        :return a tuple of (new_this, killed_other):
2287
        """
2288
        new = set()
2289
        killed = set()
3144.3.7 by Aaron Bentley
Update from review
2290
        unique_line_numbers = set(unique_line_numbers)
3144.3.1 by Aaron Bentley
Implement LCA merge, with problematic conflict markers
2291
        for lca in self.lcas:
2292
            blocks = self._get_matching_blocks(revision_id, lca)
2293
            unique_vs_lca, _ignored = self._unique_lines(blocks)
3144.3.7 by Aaron Bentley
Update from review
2294
            new.update(unique_line_numbers.intersection(unique_vs_lca))
2295
            killed.update(unique_line_numbers.difference(unique_vs_lca))
3144.3.1 by Aaron Bentley
Implement LCA merge, with problematic conflict markers
2296
        return new, killed