/brz/remove-bazaar

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