bzr branch
http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar
|
5557.1.7
by John Arbash Meinel
Merge in the bzr.dev 5582 |
1 |
# Copyright (C) 2005, 2006, 2007, 2009, 2010, 2011 Canonical Ltd
|
|
1887.1.1
by Adeodato Simó
Do not separate paragraphs in the copyright statement with blank lines, |
2 |
#
|
|
1185.14.3
by Aaron Bentley
Copied conflict lister in |
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.
|
|
|
1887.1.1
by Adeodato Simó
Do not separate paragraphs in the copyright statement with blank lines, |
7 |
#
|
|
1185.14.3
by Aaron Bentley
Copied conflict lister in |
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.
|
|
|
1887.1.1
by Adeodato Simó
Do not separate paragraphs in the copyright statement with blank lines, |
12 |
#
|
|
1185.14.3
by Aaron Bentley
Copied conflict lister in |
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
|
|
1185.14.3
by Aaron Bentley
Copied conflict lister in |
16 |
|
|
6622.1.28
by Jelmer Vernooij
More renames; commands in output, environment variables. |
17 |
# TODO: 'brz resolve' should accept a directory name and work from that
|
|
1185.16.11
by Martin Pool
todo |
18 |
# point down
|
19 |
||
|
6379.6.1
by Jelmer Vernooij
Import absolute_import in a few places. |
20 |
from __future__ import absolute_import |
21 |
||
|
1185.16.33
by Martin Pool
- move 'conflict' and 'resolved' from shipped plugin to regular builtins |
22 |
import os |
|
6543.2.1
by Martin Packman
Refactor auto_resolve tree method into auto action on conflicts |
23 |
import re |
|
1996.3.33
by John Arbash Meinel
make bzrlib/conflicts.py lazy |
24 |
|
|
6624
by Jelmer Vernooij
Merge Python3 porting work ('py3 pokes') |
25 |
from .lazy_import import lazy_import |
|
1996.3.33
by John Arbash Meinel
make bzrlib/conflicts.py lazy |
26 |
lazy_import(globals(), """ |
|
1185.16.33
by Martin Pool
- move 'conflict' and 'resolved' from shipped plugin to regular builtins |
27 |
import errno
|
28 |
||
|
6622.1.34
by Jelmer Vernooij
Rename brzlib => breezy. |
29 |
from breezy import (
|
|
4597.2.29
by Vincent Ladeuil
Fixed as per Andrew's review comments. |
30 |
cleanup,
|
|
1996.3.33
by John Arbash Meinel
make bzrlib/conflicts.py lazy |
31 |
errors,
|
32 |
osutils,
|
|
33 |
rio,
|
|
|
2120.7.3
by Aaron Bentley
Update resolve command to automatically mark conflicts as resolved |
34 |
trace,
|
|
4597.3.64
by Vincent Ladeuil
Fixed as per Aaron's remark. |
35 |
transform,
|
|
4597.3.24
by Vincent Ladeuil
Fix imports in bzrlib/conflicts.py. |
36 |
workingtree,
|
|
1996.3.33
by John Arbash Meinel
make bzrlib/conflicts.py lazy |
37 |
)
|
|
6622.1.34
by Jelmer Vernooij
Rename brzlib => breezy. |
38 |
from breezy.i18n import gettext, ngettext
|
|
1996.3.33
by John Arbash Meinel
make bzrlib/conflicts.py lazy |
39 |
""") |
|
6624
by Jelmer Vernooij
Merge Python3 porting work ('py3 pokes') |
40 |
from . import ( |
|
6630.1.1
by Jelmer Vernooij
Remove deprecated functionality. |
41 |
cache_utf8, |
|
5904.1.2
by Martin Pool
Various pyflakes import fixes. |
42 |
commands, |
|
4597.3.51
by Vincent Ladeuil
Implement conflicts.ResolveActionOption. |
43 |
option, |
44 |
registry, |
|
45 |
)
|
|
|
6791.2.3
by Jelmer Vernooij
Fix more imports. |
46 |
from .sixish import text_type |
|
1534.10.6
by Aaron Bentley
Conflict serialization working for WorkingTree3 |
47 |
|
48 |
||
49 |
CONFLICT_SUFFIXES = ('.THIS', '.BASE', '.OTHER') |
|
50 |
||
|
1185.14.3
by Aaron Bentley
Copied conflict lister in |
51 |
|
|
1996.3.33
by John Arbash Meinel
make bzrlib/conflicts.py lazy |
52 |
class cmd_conflicts(commands.Command): |
|
5131.2.1
by Martin
Permit bzrlib to run under python -OO by explictly assigning to __doc__ for user-visible docstrings |
53 |
__doc__ = """List files with conflicts. |
|
1551.2.18
by Aaron Bentley
Updated docs to clarify conflict handling |
54 |
|
55 |
Merge will do its best to combine the changes in two branches, but there
|
|
56 |
are some kinds of problems only a human can fix. When it encounters those,
|
|
57 |
it will mark a conflict. A conflict means that you need to fix something,
|
|
|
6437.2.2
by Vincent Ladeuil
Release 2.5b5 |
58 |
before you can commit.
|
|
1551.2.18
by Aaron Bentley
Updated docs to clarify conflict handling |
59 |
|
|
1551.9.8
by Aaron Bentley
Add --text parameter to conflicts |
60 |
Conflicts normally are listed as short, human-readable messages. If --text
|
61 |
is supplied, the pathnames of files with text conflicts are listed,
|
|
62 |
instead. (This is useful for editing all files with text conflicts.)
|
|
63 |
||
|
6622.1.28
by Jelmer Vernooij
More renames; commands in output, environment variables. |
64 |
Use brz resolve when you have fixed a problem.
|
|
1185.14.3
by Aaron Bentley
Copied conflict lister in |
65 |
"""
|
|
2598.1.1
by Martin Pool
Add test for and documentation of option style, fix up existing options to comply |
66 |
takes_options = [ |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
67 |
'directory', |
68 |
option.Option('text', |
|
69 |
help='List paths of files with text conflicts.'), |
|
|
2598.1.1
by Martin Pool
Add test for and documentation of option style, fix up existing options to comply |
70 |
]
|
|
4927.2.1
by Ian Clatworthy
conflicts topic renamed to conflict-types and other minor help clean-ups |
71 |
_see_also = ['resolve', 'conflict-types'] |
|
1551.9.8
by Aaron Bentley
Add --text parameter to conflicts |
72 |
|
|
5171.3.13
by Martin von Gagern
Add --directory option to 7 more commands. |
73 |
def run(self, text=False, directory=u'.'): |
74 |
wt = workingtree.WorkingTree.open_containing(directory)[0] |
|
|
1534.10.24
by Aaron Bentley
Eliminated conflicts_to_strings, made remove_files a ConflictList member |
75 |
for conflict in wt.conflicts(): |
|
1551.9.8
by Aaron Bentley
Add --text parameter to conflicts |
76 |
if text: |
77 |
if conflict.typestring != 'text conflict': |
|
78 |
continue
|
|
79 |
self.outf.write(conflict.path + '\n') |
|
80 |
else: |
|
|
6791.2.3
by Jelmer Vernooij
Fix more imports. |
81 |
self.outf.write(text_type(conflict) + '\n') |
|
1185.14.3
by Aaron Bentley
Copied conflict lister in |
82 |
|
|
1652.1.1
by Martin Pool
Fix 'bzr resolve' run from subdirectory |
83 |
|
|
4597.3.51
by Vincent Ladeuil
Implement conflicts.ResolveActionOption. |
84 |
resolve_action_registry = registry.Registry() |
85 |
||
86 |
||
87 |
resolve_action_registry.register( |
|
|
6543.2.1
by Martin Packman
Refactor auto_resolve tree method into auto action on conflicts |
88 |
'auto', 'auto', 'Detect whether conflict has been resolved by user.') |
89 |
resolve_action_registry.register( |
|
|
6259.2.8
by Martin Packman
Add full stops to various registry help strings |
90 |
'done', 'done', 'Marks the conflict as resolved.') |
|
4597.3.51
by Vincent Ladeuil
Implement conflicts.ResolveActionOption. |
91 |
resolve_action_registry.register( |
|
4597.3.67
by Vincent Ladeuil
Settle with --take-this and --take-other as action names. |
92 |
'take-this', 'take_this', |
|
6259.2.8
by Martin Packman
Add full stops to various registry help strings |
93 |
'Resolve the conflict preserving the version in the working tree.') |
|
4597.3.51
by Vincent Ladeuil
Implement conflicts.ResolveActionOption. |
94 |
resolve_action_registry.register( |
|
4597.3.67
by Vincent Ladeuil
Settle with --take-this and --take-other as action names. |
95 |
'take-other', 'take_other', |
|
6259.2.8
by Martin Packman
Add full stops to various registry help strings |
96 |
'Resolve the conflict taking the merged version into account.') |
|
4597.3.51
by Vincent Ladeuil
Implement conflicts.ResolveActionOption. |
97 |
resolve_action_registry.default_key = 'done' |
98 |
||
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
99 |
|
|
4597.3.51
by Vincent Ladeuil
Implement conflicts.ResolveActionOption. |
100 |
class ResolveActionOption(option.RegistryOption): |
101 |
||
102 |
def __init__(self): |
|
103 |
super(ResolveActionOption, self).__init__( |
|
104 |
'action', 'How to resolve the conflict.', |
|
105 |
value_switches=True, |
|
106 |
registry=resolve_action_registry) |
|
107 |
||
108 |
||
|
1996.3.33
by John Arbash Meinel
make bzrlib/conflicts.py lazy |
109 |
class cmd_resolve(commands.Command): |
|
5131.2.1
by Martin
Permit bzrlib to run under python -OO by explictly assigning to __doc__ for user-visible docstrings |
110 |
__doc__ = """Mark a conflict as resolved. |
|
1551.2.18
by Aaron Bentley
Updated docs to clarify conflict handling |
111 |
|
112 |
Merge will do its best to combine the changes in two branches, but there
|
|
113 |
are some kinds of problems only a human can fix. When it encounters those,
|
|
114 |
it will mark a conflict. A conflict means that you need to fix something,
|
|
|
6437.2.2
by Vincent Ladeuil
Release 2.5b5 |
115 |
before you can commit.
|
|
1551.2.18
by Aaron Bentley
Updated docs to clarify conflict handling |
116 |
|
|
6622.1.28
by Jelmer Vernooij
More renames; commands in output, environment variables. |
117 |
Once you have fixed a problem, use "brz resolve" to automatically mark
|
118 |
text conflicts as fixed, "brz resolve FILE" to mark a specific conflict as
|
|
119 |
resolved, or "brz resolve --all" to mark all conflicts as resolved.
|
|
|
1185.14.3
by Aaron Bentley
Copied conflict lister in |
120 |
"""
|
|
1185.33.24
by Martin Pool
Add alias 'resolved' |
121 |
aliases = ['resolved'] |
|
1185.14.3
by Aaron Bentley
Copied conflict lister in |
122 |
takes_args = ['file*'] |
|
2598.1.2
by Martin Pool
Also check that option help ends in a period, and fix those that don't |
123 |
takes_options = [ |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
124 |
'directory', |
125 |
option.Option('all', help='Resolve all conflicts in this tree.'), |
|
126 |
ResolveActionOption(), |
|
127 |
]
|
|
|
4798.7.1
by Neil Martinsen-Burrell
fix some formatting and see also usage |
128 |
_see_also = ['conflicts'] |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
129 |
|
|
5321.1.75
by Gordon Tyler
Reverted more command-related stuff and my own idiocy. |
130 |
def run(self, file_list=None, all=False, action=None, directory=None): |
|
1652.1.3
by Martin Pool
Improved bzr resolve command line handling |
131 |
if all: |
132 |
if file_list: |
|
|
6138.3.7
by Jonathan Riddell
add gettext() to BzrCommandError uses |
133 |
raise errors.BzrCommandError(gettext("If --all is specified," |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
134 |
" no FILE may be provided")) |
|
5521.1.1
by Vincent Ladeuil
Handle --directory when paths are also provided to shelve and restore. |
135 |
if directory is None: |
136 |
directory = u'.' |
|
|
5171.3.13
by Martin von Gagern
Add --directory option to 7 more commands. |
137 |
tree = workingtree.WorkingTree.open_containing(directory)[0] |
|
4597.3.52
by Vincent Ladeuil
Replace --interactive by --action. |
138 |
if action is None: |
139 |
action = 'done' |
|
|
1185.14.3
by Aaron Bentley
Copied conflict lister in |
140 |
else: |
|
5346.4.4
by Martin Pool
Use open_containing_paths |
141 |
tree, file_list = workingtree.WorkingTree.open_containing_paths( |
|
5521.1.1
by Vincent Ladeuil
Handle --directory when paths are also provided to shelve and restore. |
142 |
file_list, directory) |
|
6543.2.1
by Martin Packman
Refactor auto_resolve tree method into auto action on conflicts |
143 |
if action is None: |
144 |
if file_list is None: |
|
|
4597.3.52
by Vincent Ladeuil
Replace --interactive by --action. |
145 |
action = 'auto' |
|
6543.2.1
by Martin Packman
Refactor auto_resolve tree method into auto action on conflicts |
146 |
else: |
|
4597.3.52
by Vincent Ladeuil
Replace --interactive by --action. |
147 |
action = 'done' |
|
6543.2.1
by Martin Packman
Refactor auto_resolve tree method into auto action on conflicts |
148 |
before, after = resolve(tree, file_list, action=action) |
149 |
# GZ 2012-07-27: Should unify UI below now that auto is less magical.
|
|
150 |
if action == 'auto' and file_list is None: |
|
151 |
if after > 0: |
|
|
7290.25.1
by Jelmer Vernooij
merge lp:~gz/bzr/resolve_auto_refactor |
152 |
trace.note( |
153 |
ngettext('%d conflict auto-resolved.', |
|
154 |
'%d conflicts auto-resolved.', before - after), |
|
|
6543.2.1
by Martin Packman
Refactor auto_resolve tree method into auto action on conflicts |
155 |
before - after) |
156 |
trace.note(gettext('Remaining conflicts:')) |
|
157 |
for conflict in tree.conflicts(): |
|
|
7290.25.1
by Jelmer Vernooij
merge lp:~gz/bzr/resolve_auto_refactor |
158 |
trace.note(text_type(conflict)) |
|
6543.2.1
by Martin Packman
Refactor auto_resolve tree method into auto action on conflicts |
159 |
return 1 |
|
2120.7.3
by Aaron Bentley
Update resolve command to automatically mark conflicts as resolved |
160 |
else: |
|
6543.2.1
by Martin Packman
Refactor auto_resolve tree method into auto action on conflicts |
161 |
trace.note(gettext('All conflicts resolved.')) |
162 |
return 0 |
|
|
4597.3.52
by Vincent Ladeuil
Replace --interactive by --action. |
163 |
else: |
|
6138.3.4
by Jonathan Riddell
add gettext() to uses of trace.note() |
164 |
trace.note(ngettext('{0} conflict resolved, {1} remaining', |
165 |
'{0} conflicts resolved, {1} remaining', |
|
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
166 |
before - after).format(before - after, after)) |
|
4597.3.52
by Vincent Ladeuil
Replace --interactive by --action. |
167 |
|
168 |
||
169 |
def resolve(tree, paths=None, ignore_misses=False, recursive=False, |
|
170 |
action='done'): |
|
|
3017.2.2
by Aaron Bentley
Add docstring to resolve |
171 |
"""Resolve some or all of the conflicts in a working tree. |
172 |
||
173 |
:param paths: If None, resolve all conflicts. Otherwise, select only
|
|
174 |
specified conflicts.
|
|
175 |
:param recursive: If True, then elements of paths which are directories
|
|
176 |
have all their children resolved, etc. When invoked as part of
|
|
177 |
recursive commands like revert, this should be True. For commands
|
|
178 |
or applications wishing finer-grained control, like the resolve
|
|
179 |
command, this should be False.
|
|
|
4597.3.52
by Vincent Ladeuil
Replace --interactive by --action. |
180 |
:param ignore_misses: If False, warnings will be printed if the supplied
|
181 |
paths do not have conflicts.
|
|
182 |
:param action: How the conflict should be resolved,
|
|
|
3017.2.2
by Aaron Bentley
Add docstring to resolve |
183 |
"""
|
|
4597.9.19
by Vincent Ladeuil
resolve now reports conflicts resolved/remaining. |
184 |
nb_conflicts_after = None |
|
6754.8.9
by Jelmer Vernooij
Fix more tests. |
185 |
with tree.lock_tree_write(): |
|
1534.10.23
by Aaron Bentley
Removed conflicts_to_stanzas and stanzas_to_conflicts |
186 |
tree_conflicts = tree.conflicts() |
|
4597.9.19
by Vincent Ladeuil
resolve now reports conflicts resolved/remaining. |
187 |
nb_conflicts_before = len(tree_conflicts) |
|
1534.10.10
by Aaron Bentley
Resolve uses the new stuff. |
188 |
if paths is None: |
|
1534.10.25
by Aaron Bentley
Move select_conflicts into ConflictList |
189 |
new_conflicts = ConflictList() |
|
4597.3.52
by Vincent Ladeuil
Replace --interactive by --action. |
190 |
to_process = tree_conflicts |
|
1534.10.10
by Aaron Bentley
Resolve uses the new stuff. |
191 |
else: |
|
4597.3.52
by Vincent Ladeuil
Replace --interactive by --action. |
192 |
new_conflicts, to_process = tree_conflicts.select_conflicts( |
193 |
tree, paths, ignore_misses, recursive) |
|
194 |
for conflict in to_process: |
|
195 |
try: |
|
196 |
conflict._do(action, tree) |
|
197 |
conflict.cleanup(tree) |
|
198 |
except NotImplementedError: |
|
199 |
new_conflicts.append(conflict) |
|
|
1534.10.10
by Aaron Bentley
Resolve uses the new stuff. |
200 |
try: |
|
4597.9.19
by Vincent Ladeuil
resolve now reports conflicts resolved/remaining. |
201 |
nb_conflicts_after = len(new_conflicts) |
|
1534.10.25
by Aaron Bentley
Move select_conflicts into ConflictList |
202 |
tree.set_conflicts(new_conflicts) |
|
1996.3.33
by John Arbash Meinel
make bzrlib/conflicts.py lazy |
203 |
except errors.UnsupportedOperation: |
|
1534.10.10
by Aaron Bentley
Resolve uses the new stuff. |
204 |
pass
|
|
4597.9.19
by Vincent Ladeuil
resolve now reports conflicts resolved/remaining. |
205 |
if nb_conflicts_after is None: |
206 |
nb_conflicts_after = nb_conflicts_before |
|
207 |
return nb_conflicts_before, nb_conflicts_after |
|
|
4597.3.26
by Vincent Ladeuil
Tests passing for a minimal --interactive implementation. |
208 |
|
209 |
||
|
1185.35.1
by Aaron Bentley
Implemented conflicts.restore |
210 |
def restore(filename): |
|
4597.2.6
by Vincent Ladeuil
Cleanup doc string. |
211 |
"""Restore a conflicted file to the state it was in before merging. |
212 |
||
213 |
Only text restoration is supported at present.
|
|
|
1185.35.1
by Aaron Bentley
Implemented conflicts.restore |
214 |
"""
|
215 |
conflicted = False |
|
216 |
try: |
|
|
1996.3.33
by John Arbash Meinel
make bzrlib/conflicts.py lazy |
217 |
osutils.rename(filename + ".THIS", filename) |
|
1185.35.1
by Aaron Bentley
Implemented conflicts.restore |
218 |
conflicted = True |
|
6619.3.2
by Jelmer Vernooij
Apply 2to3 except fix. |
219 |
except OSError as e: |
|
1185.35.1
by Aaron Bentley
Implemented conflicts.restore |
220 |
if e.errno != errno.ENOENT: |
221 |
raise
|
|
222 |
try: |
|
223 |
os.unlink(filename + ".BASE") |
|
224 |
conflicted = True |
|
|
6619.3.2
by Jelmer Vernooij
Apply 2to3 except fix. |
225 |
except OSError as e: |
|
1185.35.1
by Aaron Bentley
Implemented conflicts.restore |
226 |
if e.errno != errno.ENOENT: |
227 |
raise
|
|
228 |
try: |
|
229 |
os.unlink(filename + ".OTHER") |
|
230 |
conflicted = True |
|
|
6619.3.2
by Jelmer Vernooij
Apply 2to3 except fix. |
231 |
except OSError as e: |
|
1185.35.1
by Aaron Bentley
Implemented conflicts.restore |
232 |
if e.errno != errno.ENOENT: |
233 |
raise
|
|
234 |
if not conflicted: |
|
|
1996.3.33
by John Arbash Meinel
make bzrlib/conflicts.py lazy |
235 |
raise errors.NotConflicted(filename) |
|
1534.10.4
by Aaron Bentley
Implemented conflict serialization |
236 |
|
237 |
||
|
1534.10.22
by Aaron Bentley
Got ConflictList implemented |
238 |
class ConflictList(object): |
|
1652.1.1
by Martin Pool
Fix 'bzr resolve' run from subdirectory |
239 |
"""List of conflicts. |
240 |
||
241 |
Typically obtained from WorkingTree.conflicts()
|
|
242 |
||
|
1534.10.22
by Aaron Bentley
Got ConflictList implemented |
243 |
Can be instantiated from stanzas or from Conflict subclasses.
|
244 |
"""
|
|
245 |
||
246 |
def __init__(self, conflicts=None): |
|
247 |
object.__init__(self) |
|
248 |
if conflicts is None: |
|
249 |
self.__list = [] |
|
250 |
else: |
|
251 |
self.__list = conflicts |
|
252 |
||
|
1652.1.1
by Martin Pool
Fix 'bzr resolve' run from subdirectory |
253 |
def is_empty(self): |
254 |
return len(self.__list) == 0 |
|
255 |
||
|
1534.10.22
by Aaron Bentley
Got ConflictList implemented |
256 |
def __len__(self): |
257 |
return len(self.__list) |
|
258 |
||
259 |
def __iter__(self): |
|
260 |
return iter(self.__list) |
|
261 |
||
262 |
def __getitem__(self, key): |
|
263 |
return self.__list[key] |
|
264 |
||
265 |
def append(self, conflict): |
|
266 |
return self.__list.append(conflict) |
|
267 |
||
268 |
def __eq__(self, other_list): |
|
269 |
return list(self) == list(other_list) |
|
270 |
||
271 |
def __ne__(self, other_list): |
|
272 |
return not (self == other_list) |
|
273 |
||
274 |
def __repr__(self): |
|
275 |
return "ConflictList(%r)" % self.__list |
|
276 |
||
277 |
@staticmethod
|
|
278 |
def from_stanzas(stanzas): |
|
279 |
"""Produce a new ConflictList from an iterable of stanzas""" |
|
280 |
conflicts = ConflictList() |
|
281 |
for stanza in stanzas: |
|
282 |
conflicts.append(Conflict.factory(**stanza.as_dict())) |
|
283 |
return conflicts |
|
284 |
||
285 |
def to_stanzas(self): |
|
286 |
"""Generator of stanzas""" |
|
287 |
for conflict in self: |
|
288 |
yield conflict.as_stanza() |
|
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
289 |
|
|
1534.10.24
by Aaron Bentley
Eliminated conflicts_to_strings, made remove_files a ConflictList member |
290 |
def to_strings(self): |
291 |
"""Generate strings for the provided conflicts""" |
|
292 |
for conflict in self: |
|
|
6791.2.3
by Jelmer Vernooij
Fix more imports. |
293 |
yield text_type(conflict) |
|
1534.10.24
by Aaron Bentley
Eliminated conflicts_to_strings, made remove_files a ConflictList member |
294 |
|
295 |
def remove_files(self, tree): |
|
|
1534.10.25
by Aaron Bentley
Move select_conflicts into ConflictList |
296 |
"""Remove the THIS, BASE and OTHER files for listed conflicts""" |
|
1534.10.24
by Aaron Bentley
Eliminated conflicts_to_strings, made remove_files a ConflictList member |
297 |
for conflict in self: |
298 |
if not conflict.has_files: |
|
299 |
continue
|
|
|
4597.3.52
by Vincent Ladeuil
Replace --interactive by --action. |
300 |
conflict.cleanup(tree) |
|
1534.10.21
by Aaron Bentley
Moved and renamed conflict functions |
301 |
|
|
1551.15.58
by Aaron Bentley
Status honours selected paths for conflicts (#127606) |
302 |
def select_conflicts(self, tree, paths, ignore_misses=False, |
303 |
recurse=False): |
|
|
1534.10.25
by Aaron Bentley
Move select_conflicts into ConflictList |
304 |
"""Select the conflicts associated with paths in a tree. |
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
305 |
|
|
1534.10.25
by Aaron Bentley
Move select_conflicts into ConflictList |
306 |
File-ids are also used for this.
|
|
1551.7.10
by Aaron Bentley
Remerge doesn't clear unrelated conflicts |
307 |
:return: a pair of ConflictLists: (not_selected, selected)
|
|
1534.10.25
by Aaron Bentley
Move select_conflicts into ConflictList |
308 |
"""
|
309 |
path_set = set(paths) |
|
310 |
ids = {} |
|
311 |
selected_paths = set() |
|
312 |
new_conflicts = ConflictList() |
|
313 |
selected_conflicts = ConflictList() |
|
314 |
for path in paths: |
|
315 |
file_id = tree.path2id(path) |
|
316 |
if file_id is not None: |
|
317 |
ids[file_id] = path |
|
318 |
||
319 |
for conflict in self: |
|
320 |
selected = False |
|
321 |
for key in ('path', 'conflict_path'): |
|
322 |
cpath = getattr(conflict, key, None) |
|
323 |
if cpath is None: |
|
324 |
continue
|
|
325 |
if cpath in path_set: |
|
326 |
selected = True |
|
327 |
selected_paths.add(cpath) |
|
|
1551.15.58
by Aaron Bentley
Status honours selected paths for conflicts (#127606) |
328 |
if recurse: |
329 |
if osutils.is_inside_any(path_set, cpath): |
|
330 |
selected = True |
|
331 |
selected_paths.add(cpath) |
|
332 |
||
|
1534.10.25
by Aaron Bentley
Move select_conflicts into ConflictList |
333 |
for key in ('file_id', 'conflict_file_id'): |
334 |
cfile_id = getattr(conflict, key, None) |
|
335 |
if cfile_id is None: |
|
336 |
continue
|
|
337 |
try: |
|
338 |
cpath = ids[cfile_id] |
|
339 |
except KeyError: |
|
340 |
continue
|
|
341 |
selected = True |
|
342 |
selected_paths.add(cpath) |
|
343 |
if selected: |
|
344 |
selected_conflicts.append(conflict) |
|
345 |
else: |
|
346 |
new_conflicts.append(conflict) |
|
347 |
if ignore_misses is not True: |
|
348 |
for path in [p for p in paths if p not in selected_paths]: |
|
349 |
if not os.path.exists(tree.abspath(path)): |
|
|
6619.3.3
by Jelmer Vernooij
Apply 2to3 print fix. |
350 |
print("%s does not exist" % path) |
|
1534.10.25
by Aaron Bentley
Move select_conflicts into ConflictList |
351 |
else: |
|
6619.3.3
by Jelmer Vernooij
Apply 2to3 print fix. |
352 |
print("%s is not conflicted" % path) |
|
1534.10.25
by Aaron Bentley
Move select_conflicts into ConflictList |
353 |
return new_conflicts, selected_conflicts |
354 |
||
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
355 |
|
|
1534.10.18
by Aaron Bentley
Defined all new Conflict types |
356 |
class Conflict(object): |
357 |
"""Base class for all types of conflict""" |
|
|
1534.10.24
by Aaron Bentley
Eliminated conflicts_to_strings, made remove_files a ConflictList member |
358 |
|
|
4597.3.56
by Vincent Ladeuil
Add a NEWS entry and prepare submission. |
359 |
# FIXME: cleanup should take care of that ? -- vila 091229
|
|
1534.10.24
by Aaron Bentley
Eliminated conflicts_to_strings, made remove_files a ConflictList member |
360 |
has_files = False |
361 |
||
|
1534.10.19
by Aaron Bentley
Stanza conversion, cooking |
362 |
def __init__(self, path, file_id=None): |
|
1534.10.18
by Aaron Bentley
Defined all new Conflict types |
363 |
self.path = path |
|
6630.1.1
by Jelmer Vernooij
Remove deprecated functionality. |
364 |
# the factory blindly transfers the Stanza values to __init__ and
|
365 |
# Stanza is purely a Unicode api.
|
|
|
6791.2.3
by Jelmer Vernooij
Fix more imports. |
366 |
if isinstance(file_id, text_type): |
|
6630.1.1
by Jelmer Vernooij
Remove deprecated functionality. |
367 |
file_id = cache_utf8.encode(file_id) |
368 |
self.file_id = osutils.safe_file_id(file_id) |
|
|
1534.10.19
by Aaron Bentley
Stanza conversion, cooking |
369 |
|
370 |
def as_stanza(self): |
|
|
1996.3.33
by John Arbash Meinel
make bzrlib/conflicts.py lazy |
371 |
s = rio.Stanza(type=self.typestring, path=self.path) |
|
1534.10.19
by Aaron Bentley
Stanza conversion, cooking |
372 |
if self.file_id is not None: |
|
2309.4.13
by John Arbash Meinel
Conflicts go through Stanza so the need to be aware of utf8 versus unicode file ids. |
373 |
# Stanza requires Unicode apis
|
374 |
s.add('file_id', self.file_id.decode('utf8')) |
|
|
1534.10.19
by Aaron Bentley
Stanza conversion, cooking |
375 |
return s |
376 |
||
|
1534.10.22
by Aaron Bentley
Got ConflictList implemented |
377 |
def _cmp_list(self): |
378 |
return [type(self), self.path, self.file_id] |
|
379 |
||
|
1534.10.19
by Aaron Bentley
Stanza conversion, cooking |
380 |
def __cmp__(self, other): |
|
1534.10.22
by Aaron Bentley
Got ConflictList implemented |
381 |
if getattr(other, "_cmp_list", None) is None: |
382 |
return -1 |
|
|
6973.5.10
by Jelmer Vernooij
Random bunch of python3 bee-improvements. |
383 |
x = self._cmp_list() |
384 |
y = other._cmp_list() |
|
385 |
return (x > y) - (x < y) |
|
|
1534.10.19
by Aaron Bentley
Stanza conversion, cooking |
386 |
|
|
1551.7.11
by Aaron Bentley
Add WorkingTree.add_conflicts |
387 |
def __hash__(self): |
388 |
return hash((type(self), self.path, self.file_id)) |
|
389 |
||
|
1534.10.19
by Aaron Bentley
Stanza conversion, cooking |
390 |
def __eq__(self, other): |
391 |
return self.__cmp__(other) == 0 |
|
392 |
||
393 |
def __ne__(self, other): |
|
394 |
return not self.__eq__(other) |
|
|
1534.10.18
by Aaron Bentley
Defined all new Conflict types |
395 |
|
|
5898.1.5
by Martin
Provide __unicode__ not __str__ so callers fall back to repr unless they handle non-ascii strings |
396 |
def __unicode__(self): |
|
6973.6.4
by Jelmer Vernooij
Avoid text_type() |
397 |
return self.describe() |
398 |
||
|
7045.2.1
by Jelmer Vernooij
Fix some conflicts tests. |
399 |
def __str__(self): |
400 |
return self.describe() |
|
401 |
||
|
6973.6.4
by Jelmer Vernooij
Avoid text_type() |
402 |
def describe(self): |
|
1534.10.20
by Aaron Bentley
Got all tests passing |
403 |
return self.format % self.__dict__ |
404 |
||
|
1534.10.22
by Aaron Bentley
Got ConflictList implemented |
405 |
def __repr__(self): |
406 |
rdict = dict(self.__dict__) |
|
407 |
rdict['class'] = self.__class__.__name__ |
|
408 |
return self.rformat % rdict |
|
409 |
||
|
1534.10.18
by Aaron Bentley
Defined all new Conflict types |
410 |
@staticmethod
|
|
1534.10.19
by Aaron Bentley
Stanza conversion, cooking |
411 |
def factory(type, **kwargs): |
|
1534.10.18
by Aaron Bentley
Defined all new Conflict types |
412 |
global ctype |
|
1534.10.19
by Aaron Bentley
Stanza conversion, cooking |
413 |
return ctype[type](**kwargs) |
|
1534.10.18
by Aaron Bentley
Defined all new Conflict types |
414 |
|
|
1666.1.4
by Robert Collins
* 'Metadir' is now the default disk format. This improves behaviour in |
415 |
@staticmethod
|
416 |
def sort_key(conflict): |
|
417 |
if conflict.path is not None: |
|
418 |
return conflict.path, conflict.typestring |
|
419 |
elif getattr(conflict, "conflict_path", None) is not None: |
|
420 |
return conflict.conflict_path, conflict.typestring |
|
421 |
else: |
|
422 |
return None, conflict.typestring |
|
423 |
||
|
4597.3.52
by Vincent Ladeuil
Replace --interactive by --action. |
424 |
def _do(self, action, tree): |
425 |
"""Apply the specified action to the conflict. |
|
426 |
||
427 |
:param action: The method name to call.
|
|
428 |
||
429 |
:param tree: The tree passed as a parameter to the method.
|
|
430 |
"""
|
|
|
4597.3.73
by Vincent Ladeuil
All Conflict action method names start with 'action_' to avoid potential namespace collisions |
431 |
meth = getattr(self, 'action_%s' % action, None) |
|
4597.3.52
by Vincent Ladeuil
Replace --interactive by --action. |
432 |
if meth is None: |
433 |
raise NotImplementedError(self.__class__.__name__ + '.' + action) |
|
434 |
meth(tree) |
|
435 |
||
|
5013.2.4
by Vincent Ladeuil
``bzr add`` won't blindly add conflict related files. |
436 |
def associated_filenames(self): |
437 |
"""The names of the files generated to help resolve the conflict.""" |
|
438 |
raise NotImplementedError(self.associated_filenames) |
|
439 |
||
|
4597.3.52
by Vincent Ladeuil
Replace --interactive by --action. |
440 |
def cleanup(self, tree): |
|
5013.2.4
by Vincent Ladeuil
``bzr add`` won't blindly add conflict related files. |
441 |
for fname in self.associated_filenames(): |
442 |
try: |
|
443 |
osutils.delete_any(tree.abspath(fname)) |
|
|
6619.3.2
by Jelmer Vernooij
Apply 2to3 except fix. |
444 |
except OSError as e: |
|
5013.2.4
by Vincent Ladeuil
``bzr add`` won't blindly add conflict related files. |
445 |
if e.errno != errno.ENOENT: |
446 |
raise
|
|
|
4597.3.52
by Vincent Ladeuil
Replace --interactive by --action. |
447 |
|
|
6543.2.1
by Martin Packman
Refactor auto_resolve tree method into auto action on conflicts |
448 |
def action_auto(self, tree): |
449 |
raise NotImplementedError(self.action_auto) |
|
450 |
||
|
4597.3.73
by Vincent Ladeuil
All Conflict action method names start with 'action_' to avoid potential namespace collisions |
451 |
def action_done(self, tree): |
|
4597.3.54
by Vincent Ladeuil
There is no point to define Conflict.done() multiple times. |
452 |
"""Mark the conflict as solved once it has been handled.""" |
453 |
# This method does nothing but simplifies the design of upper levels.
|
|
454 |
pass
|
|
|
4597.3.52
by Vincent Ladeuil
Replace --interactive by --action. |
455 |
|
|
4597.3.73
by Vincent Ladeuil
All Conflict action method names start with 'action_' to avoid potential namespace collisions |
456 |
def action_take_this(self, tree): |
457 |
raise NotImplementedError(self.action_take_this) |
|
|
4597.3.52
by Vincent Ladeuil
Replace --interactive by --action. |
458 |
|
|
4597.3.73
by Vincent Ladeuil
All Conflict action method names start with 'action_' to avoid potential namespace collisions |
459 |
def action_take_other(self, tree): |
460 |
raise NotImplementedError(self.action_take_other) |
|
|
4597.3.52
by Vincent Ladeuil
Replace --interactive by --action. |
461 |
|
|
4597.7.8
by Vincent Ladeuil
Some refactoring to help address bug #531967. |
462 |
def _resolve_with_cleanups(self, tree, *args, **kwargs): |
463 |
tt = transform.TreeTransform(tree) |
|
464 |
op = cleanup.OperationWithCleanups(self._resolve) |
|
465 |
op.add_cleanup(tt.finalize) |
|
466 |
op.run_simple(tt, *args, **kwargs) |
|
467 |
||
|
1534.10.18
by Aaron Bentley
Defined all new Conflict types |
468 |
|
469 |
class PathConflict(Conflict): |
|
|
1534.10.22
by Aaron Bentley
Got ConflictList implemented |
470 |
"""A conflict was encountered merging file paths""" |
471 |
||
|
1534.10.18
by Aaron Bentley
Defined all new Conflict types |
472 |
typestring = 'path conflict' |
|
1534.10.22
by Aaron Bentley
Got ConflictList implemented |
473 |
|
|
1534.10.20
by Aaron Bentley
Got all tests passing |
474 |
format = 'Path conflict: %(path)s / %(conflict_path)s' |
|
1534.10.22
by Aaron Bentley
Got ConflictList implemented |
475 |
|
476 |
rformat = '%(class)s(%(path)r, %(conflict_path)r, %(file_id)r)' |
|
|
4597.3.52
by Vincent Ladeuil
Replace --interactive by --action. |
477 |
|
|
1534.10.20
by Aaron Bentley
Got all tests passing |
478 |
def __init__(self, path, conflict_path=None, file_id=None): |
|
1534.10.19
by Aaron Bentley
Stanza conversion, cooking |
479 |
Conflict.__init__(self, path, file_id) |
|
1534.10.18
by Aaron Bentley
Defined all new Conflict types |
480 |
self.conflict_path = conflict_path |
481 |
||
|
1534.10.19
by Aaron Bentley
Stanza conversion, cooking |
482 |
def as_stanza(self): |
483 |
s = Conflict.as_stanza(self) |
|
|
1534.10.20
by Aaron Bentley
Got all tests passing |
484 |
if self.conflict_path is not None: |
485 |
s.add('conflict_path', self.conflict_path) |
|
|
1534.10.19
by Aaron Bentley
Stanza conversion, cooking |
486 |
return s |
487 |
||
|
5013.2.4
by Vincent Ladeuil
``bzr add`` won't blindly add conflict related files. |
488 |
def associated_filenames(self): |
|
4597.7.14
by Vincent Ladeuil
Tweak the fix to not use helpers but let resolve restore the old version. |
489 |
# No additional files have been generated here
|
490 |
return [] |
|
|
4597.3.52
by Vincent Ladeuil
Replace --interactive by --action. |
491 |
|
|
4597.7.13
by Vincent Ladeuil
Remove some duplication. |
492 |
def _resolve(self, tt, file_id, path, winner): |
|
4597.7.8
by Vincent Ladeuil
Some refactoring to help address bug #531967. |
493 |
"""Resolve the conflict. |
494 |
||
495 |
:param tt: The TreeTransform where the conflict is resolved.
|
|
496 |
:param file_id: The retained file id.
|
|
497 |
:param path: The retained path.
|
|
|
4597.7.13
by Vincent Ladeuil
Remove some duplication. |
498 |
:param winner: 'this' or 'other' indicates which side is the winner.
|
|
4597.7.8
by Vincent Ladeuil
Some refactoring to help address bug #531967. |
499 |
"""
|
|
4597.7.14
by Vincent Ladeuil
Tweak the fix to not use helpers but let resolve restore the old version. |
500 |
path_to_create = None |
|
4597.7.13
by Vincent Ladeuil
Remove some duplication. |
501 |
if winner == 'this': |
502 |
if self.path == '<deleted>': |
|
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
503 |
return # Nothing to do |
|
4597.7.13
by Vincent Ladeuil
Remove some duplication. |
504 |
if self.conflict_path == '<deleted>': |
|
4597.7.14
by Vincent Ladeuil
Tweak the fix to not use helpers but let resolve restore the old version. |
505 |
path_to_create = self.path |
506 |
revid = tt._tree.get_parent_ids()[0] |
|
|
4597.7.13
by Vincent Ladeuil
Remove some duplication. |
507 |
elif winner == 'other': |
508 |
if self.conflict_path == '<deleted>': |
|
509 |
return # Nothing to do |
|
510 |
if self.path == '<deleted>': |
|
|
4597.7.14
by Vincent Ladeuil
Tweak the fix to not use helpers but let resolve restore the old version. |
511 |
path_to_create = self.conflict_path |
512 |
# FIXME: If there are more than two parents we may need to
|
|
513 |
# iterate. Taking the last parent is the safer bet in the mean
|
|
514 |
# time. -- vila 20100309
|
|
515 |
revid = tt._tree.get_parent_ids()[-1] |
|
|
4597.7.18
by Vincent Ladeuil
Start addressing Andrew's concerns. |
516 |
else: |
517 |
# Programmer error
|
|
518 |
raise AssertionError('bad winner: %r' % (winner,)) |
|
|
4597.7.14
by Vincent Ladeuil
Tweak the fix to not use helpers but let resolve restore the old version. |
519 |
if path_to_create is not None: |
520 |
tid = tt.trans_id_tree_path(path_to_create) |
|
|
6809.4.7
by Jelmer Vernooij
Swap arguments for get_symlink_target and kind/stored_kind. |
521 |
tree = self._revision_tree(tt._tree, revid) |
|
4597.7.14
by Vincent Ladeuil
Tweak the fix to not use helpers but let resolve restore the old version. |
522 |
transform.create_from_tree( |
|
6809.4.7
by Jelmer Vernooij
Swap arguments for get_symlink_target and kind/stored_kind. |
523 |
tt, tid, tree, tree.id2path(file_id), file_id=file_id) |
|
4597.7.14
by Vincent Ladeuil
Tweak the fix to not use helpers but let resolve restore the old version. |
524 |
tt.version_file(file_id, tid) |
|
5988.2.1
by Vincent Ladeuil
Do not generate path conflicts if a corresponding content conflict exists |
525 |
else: |
526 |
tid = tt.trans_id_file_id(file_id) |
|
|
4597.7.9
by Vincent Ladeuil
Resolve the PathConflict when the file-id has been lost due to bug #531967. |
527 |
# Adjust the path for the retained file id
|
|
4597.7.8
by Vincent Ladeuil
Some refactoring to help address bug #531967. |
528 |
parent_tid = tt.get_tree_parent(tid) |
|
5050.63.6
by Vincent Ladeuil
Same fix for path conflicts. |
529 |
tt.adjust_path(osutils.basename(path), parent_tid, tid) |
|
4597.7.8
by Vincent Ladeuil
Some refactoring to help address bug #531967. |
530 |
tt.apply() |
531 |
||
|
4597.7.15
by Vincent Ladeuil
Workaround bug #533547. |
532 |
def _revision_tree(self, tree, revid): |
533 |
return tree.branch.repository.revision_tree(revid) |
|
534 |
||
|
4597.7.16
by Vincent Ladeuil
Some cleanup. |
535 |
def _infer_file_id(self, tree): |
|
4597.7.8
by Vincent Ladeuil
Some refactoring to help address bug #531967. |
536 |
# Prior to bug #531967, file_id wasn't always set, there may still be
|
537 |
# conflict files in the wild so we need to cope with them
|
|
|
4597.7.9
by Vincent Ladeuil
Resolve the PathConflict when the file-id has been lost due to bug #531967. |
538 |
# Establish which path we should use to find back the file-id
|
539 |
possible_paths = [] |
|
540 |
for p in (self.path, self.conflict_path): |
|
541 |
if p == '<deleted>': |
|
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
542 |
# special hard-coded path
|
|
4597.7.9
by Vincent Ladeuil
Resolve the PathConflict when the file-id has been lost due to bug #531967. |
543 |
continue
|
544 |
if p is not None: |
|
545 |
possible_paths.append(p) |
|
546 |
# Search the file-id in the parents with any path available
|
|
547 |
file_id = None |
|
548 |
for revid in tree.get_parent_ids(): |
|
|
4597.7.15
by Vincent Ladeuil
Workaround bug #533547. |
549 |
revtree = self._revision_tree(tree, revid) |
|
4597.7.9
by Vincent Ladeuil
Resolve the PathConflict when the file-id has been lost due to bug #531967. |
550 |
for p in possible_paths: |
551 |
file_id = revtree.path2id(p) |
|
552 |
if file_id is not None: |
|
553 |
return revtree, file_id |
|
554 |
return None, None |
|
|
4597.7.8
by Vincent Ladeuil
Some refactoring to help address bug #531967. |
555 |
|
|
4597.3.73
by Vincent Ladeuil
All Conflict action method names start with 'action_' to avoid potential namespace collisions |
556 |
def action_take_this(self, tree): |
|
4597.7.9
by Vincent Ladeuil
Resolve the PathConflict when the file-id has been lost due to bug #531967. |
557 |
if self.file_id is not None: |
|
4597.7.11
by Vincent Ladeuil
Fix #531967 by creating helpers for PathConflicts when a deletion |
558 |
self._resolve_with_cleanups(tree, self.file_id, self.path, |
|
4597.7.13
by Vincent Ladeuil
Remove some duplication. |
559 |
winner='this') |
|
4597.7.9
by Vincent Ladeuil
Resolve the PathConflict when the file-id has been lost due to bug #531967. |
560 |
else: |
561 |
# Prior to bug #531967 we need to find back the file_id and restore
|
|
562 |
# the content from there
|
|
|
4597.7.16
by Vincent Ladeuil
Some cleanup. |
563 |
revtree, file_id = self._infer_file_id(tree) |
|
4597.7.9
by Vincent Ladeuil
Resolve the PathConflict when the file-id has been lost due to bug #531967. |
564 |
tree.revert([revtree.id2path(file_id)], |
565 |
old_tree=revtree, backups=False) |
|
|
4597.3.33
by Vincent Ladeuil
Implement --interactive for PathConflict. |
566 |
|
|
4597.3.73
by Vincent Ladeuil
All Conflict action method names start with 'action_' to avoid potential namespace collisions |
567 |
def action_take_other(self, tree): |
|
4597.7.9
by Vincent Ladeuil
Resolve the PathConflict when the file-id has been lost due to bug #531967. |
568 |
if self.file_id is not None: |
|
4597.7.11
by Vincent Ladeuil
Fix #531967 by creating helpers for PathConflicts when a deletion |
569 |
self._resolve_with_cleanups(tree, self.file_id, |
570 |
self.conflict_path, |
|
|
4597.7.13
by Vincent Ladeuil
Remove some duplication. |
571 |
winner='other') |
|
4597.7.9
by Vincent Ladeuil
Resolve the PathConflict when the file-id has been lost due to bug #531967. |
572 |
else: |
573 |
# Prior to bug #531967 we need to find back the file_id and restore
|
|
574 |
# the content from there
|
|
|
4597.7.16
by Vincent Ladeuil
Some cleanup. |
575 |
revtree, file_id = self._infer_file_id(tree) |
|
4597.7.9
by Vincent Ladeuil
Resolve the PathConflict when the file-id has been lost due to bug #531967. |
576 |
tree.revert([revtree.id2path(file_id)], |
577 |
old_tree=revtree, backups=False) |
|
|
4597.3.33
by Vincent Ladeuil
Implement --interactive for PathConflict. |
578 |
|
|
1534.10.20
by Aaron Bentley
Got all tests passing |
579 |
|
580 |
class ContentsConflict(PathConflict): |
|
|
4597.8.1
by Vincent Ladeuil
Fix doc string. |
581 |
"""The files are of different types (or both binary), or not present""" |
|
1534.10.22
by Aaron Bentley
Got ConflictList implemented |
582 |
|
|
1534.10.24
by Aaron Bentley
Eliminated conflicts_to_strings, made remove_files a ConflictList member |
583 |
has_files = True |
584 |
||
|
1534.10.20
by Aaron Bentley
Got all tests passing |
585 |
typestring = 'contents conflict' |
|
1534.10.22
by Aaron Bentley
Got ConflictList implemented |
586 |
|
|
1534.10.20
by Aaron Bentley
Got all tests passing |
587 |
format = 'Contents conflict in %(path)s' |
588 |
||
|
5013.2.4
by Vincent Ladeuil
``bzr add`` won't blindly add conflict related files. |
589 |
def associated_filenames(self): |
590 |
return [self.path + suffix for suffix in ('.BASE', '.OTHER')] |
|
|
4597.3.52
by Vincent Ladeuil
Replace --interactive by --action. |
591 |
|
|
4597.7.8
by Vincent Ladeuil
Some refactoring to help address bug #531967. |
592 |
def _resolve(self, tt, suffix_to_remove): |
|
4597.2.26
by Vincent Ladeuil
Fix bug #529968 by renaming the kept file on content conflicts. |
593 |
"""Resolve the conflict. |
594 |
||
|
4597.2.29
by Vincent Ladeuil
Fixed as per Andrew's review comments. |
595 |
:param tt: The TreeTransform where the conflict is resolved.
|
|
4597.2.27
by Vincent Ladeuil
Address some Andrew's review points. |
596 |
:param suffix_to_remove: Either 'THIS' or 'OTHER'
|
|
4597.2.26
by Vincent Ladeuil
Fix bug #529968 by renaming the kept file on content conflicts. |
597 |
|
|
5972.1.1
by Vincent Ladeuil
Make ContentConflict resolution more robust |
598 |
The resolution is symmetric: when taking THIS, OTHER is deleted and
|
|
4597.2.26
by Vincent Ladeuil
Fix bug #529968 by renaming the kept file on content conflicts. |
599 |
item.THIS is renamed into item and vice-versa.
|
600 |
"""
|
|
|
4597.2.29
by Vincent Ladeuil
Fixed as per Andrew's review comments. |
601 |
try: |
602 |
# Delete 'item.THIS' or 'item.OTHER' depending on
|
|
603 |
# suffix_to_remove
|
|
604 |
tt.delete_contents( |
|
605 |
tt.trans_id_tree_path(self.path + '.' + suffix_to_remove)) |
|
606 |
except errors.NoSuchFile: |
|
607 |
# There are valid cases where 'item.suffix_to_remove' either
|
|
608 |
# never existed or was already deleted (including the case
|
|
609 |
# where the user deleted it)
|
|
610 |
pass
|
|
|
5972.1.1
by Vincent Ladeuil
Make ContentConflict resolution more robust |
611 |
try: |
|
5972.1.2
by Vincent Ladeuil
Get closer to the exception even if this means ~duplicating tt.trans_id_tree_path to make the code clearer (suggested by jam). |
612 |
this_path = tt._tree.id2path(self.file_id) |
|
5972.1.1
by Vincent Ladeuil
Make ContentConflict resolution more robust |
613 |
except errors.NoSuchId: |
614 |
# The file is not present anymore. This may happen if the user
|
|
615 |
# deleted the file either manually or when resolving a conflict on
|
|
616 |
# the parent. We may raise some exception to indicate that the
|
|
617 |
# conflict doesn't exist anymore and as such doesn't need to be
|
|
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
618 |
# resolved ? -- vila 20110615
|
|
5972.1.1
by Vincent Ladeuil
Make ContentConflict resolution more robust |
619 |
this_tid = None |
|
5972.1.2
by Vincent Ladeuil
Get closer to the exception even if this means ~duplicating tt.trans_id_tree_path to make the code clearer (suggested by jam). |
620 |
else: |
621 |
this_tid = tt.trans_id_tree_path(this_path) |
|
|
5972.1.1
by Vincent Ladeuil
Make ContentConflict resolution more robust |
622 |
if this_tid is not None: |
623 |
# Rename 'item.suffix_to_remove' (note that if
|
|
624 |
# 'item.suffix_to_remove' has been deleted, this is a no-op)
|
|
625 |
parent_tid = tt.get_tree_parent(this_tid) |
|
626 |
tt.adjust_path(osutils.basename(self.path), parent_tid, this_tid) |
|
627 |
tt.apply() |
|
|
4597.2.29
by Vincent Ladeuil
Fixed as per Andrew's review comments. |
628 |
|
|
4597.3.73
by Vincent Ladeuil
All Conflict action method names start with 'action_' to avoid potential namespace collisions |
629 |
def action_take_this(self, tree): |
|
4597.7.8
by Vincent Ladeuil
Some refactoring to help address bug #531967. |
630 |
self._resolve_with_cleanups(tree, 'OTHER') |
|
4597.3.28
by Vincent Ladeuil
Implement --interactive for ContentsConflict. |
631 |
|
|
4597.3.73
by Vincent Ladeuil
All Conflict action method names start with 'action_' to avoid potential namespace collisions |
632 |
def action_take_other(self, tree): |
|
4597.7.8
by Vincent Ladeuil
Some refactoring to help address bug #531967. |
633 |
self._resolve_with_cleanups(tree, 'THIS') |
|
4597.3.30
by Vincent Ladeuil
Light changes learned while starting to understand multiple conflicts on |
634 |
|
|
4597.3.52
by Vincent Ladeuil
Replace --interactive by --action. |
635 |
|
|
4597.3.48
by Vincent Ladeuil
Start cleaning BRANCH.TODO by pushing changes to either the python modules or the documentation. |
636 |
# TODO: There should be a base revid attribute to better inform the user about
|
637 |
# how the conflicts were generated.
|
|
|
5050.51.1
by Vincent Ladeuil
Don't crash when --take-other (or --take-this) is called for a text conflict |
638 |
class TextConflict(Conflict): |
|
1534.10.22
by Aaron Bentley
Got ConflictList implemented |
639 |
"""The merge algorithm could not resolve all differences encountered.""" |
640 |
||
|
1534.10.24
by Aaron Bentley
Eliminated conflicts_to_strings, made remove_files a ConflictList member |
641 |
has_files = True |
642 |
||
|
1534.10.20
by Aaron Bentley
Got all tests passing |
643 |
typestring = 'text conflict' |
|
1534.10.22
by Aaron Bentley
Got ConflictList implemented |
644 |
|
|
1534.10.20
by Aaron Bentley
Got all tests passing |
645 |
format = 'Text conflict in %(path)s' |
646 |
||
|
5050.51.1
by Vincent Ladeuil
Don't crash when --take-other (or --take-this) is called for a text conflict |
647 |
rformat = '%(class)s(%(path)r, %(file_id)r)' |
648 |
||
|
7290.25.4
by Jelmer Vernooij
Fix byteness for conflict markers. |
649 |
_conflict_re = re.compile(b'^(<{7}|={7}|>{7})') |
|
6543.2.1
by Martin Packman
Refactor auto_resolve tree method into auto action on conflicts |
650 |
|
|
5013.2.4
by Vincent Ladeuil
``bzr add`` won't blindly add conflict related files. |
651 |
def associated_filenames(self): |
652 |
return [self.path + suffix for suffix in CONFLICT_SUFFIXES] |
|
|
4597.3.33
by Vincent Ladeuil
Implement --interactive for PathConflict. |
653 |
|
|
4597.14.2
by Vincent Ladeuil
Implements --take-this and --take-other when resolving text conflicts |
654 |
def _resolve(self, tt, winner_suffix): |
655 |
"""Resolve the conflict by copying one of .THIS or .OTHER into file. |
|
656 |
||
657 |
:param tt: The TreeTransform where the conflict is resolved.
|
|
658 |
:param winner_suffix: Either 'THIS' or 'OTHER'
|
|
659 |
||
660 |
The resolution is symmetric, when taking THIS, item.THIS is renamed
|
|
661 |
into item and vice-versa. This takes one of the files as a whole
|
|
662 |
ignoring every difference that could have been merged cleanly.
|
|
663 |
"""
|
|
664 |
# To avoid useless copies, we switch item and item.winner_suffix, only
|
|
665 |
# item will exist after the conflict has been resolved anyway.
|
|
666 |
item_tid = tt.trans_id_file_id(self.file_id) |
|
667 |
item_parent_tid = tt.get_tree_parent(item_tid) |
|
668 |
winner_path = self.path + '.' + winner_suffix |
|
669 |
winner_tid = tt.trans_id_tree_path(winner_path) |
|
670 |
winner_parent_tid = tt.get_tree_parent(winner_tid) |
|
671 |
# Switch the paths to preserve the content
|
|
|
5609.17.2
by Vincent Ladeuil
Fix bug #715058 and cleanup ParentLoop.action_take_other which did the right thing before everyone else ;-/ |
672 |
tt.adjust_path(osutils.basename(self.path), |
673 |
winner_parent_tid, winner_tid) |
|
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
674 |
tt.adjust_path(osutils.basename(winner_path), |
675 |
item_parent_tid, item_tid) |
|
|
4597.14.2
by Vincent Ladeuil
Implements --take-this and --take-other when resolving text conflicts |
676 |
# Associate the file_id to the right content
|
677 |
tt.unversion_file(item_tid) |
|
678 |
tt.version_file(self.file_id, winner_tid) |
|
679 |
tt.apply() |
|
680 |
||
|
6543.2.1
by Martin Packman
Refactor auto_resolve tree method into auto action on conflicts |
681 |
def action_auto(self, tree): |
682 |
# GZ 2012-07-27: Using NotImplementedError to signal that a conflict
|
|
683 |
# can't be auto resolved does not seem ideal.
|
|
|
7290.25.6
by Jelmer Vernooij
Some more tests. |
684 |
try: |
685 |
kind = tree.kind(self.path) |
|
686 |
except errors.NoSuchFile: |
|
687 |
return
|
|
688 |
if kind != 'file': |
|
|
6543.2.1
by Martin Packman
Refactor auto_resolve tree method into auto action on conflicts |
689 |
raise NotImplementedError("Conflict is not a file") |
690 |
conflict_markers_in_line = self._conflict_re.search |
|
|
6543.2.4
by Martin Packman
Note issue over looking for conflict markers in a removed file |
691 |
# GZ 2012-07-27: What if not tree.has_id(self.file_id) due to removal?
|
|
7290.25.1
by Jelmer Vernooij
merge lp:~gz/bzr/resolve_auto_refactor |
692 |
with tree.get_file(self.path) as f: |
|
6543.2.1
by Martin Packman
Refactor auto_resolve tree method into auto action on conflicts |
693 |
for line in f: |
694 |
if conflict_markers_in_line(line): |
|
695 |
raise NotImplementedError("Conflict markers present") |
|
696 |
||
|
4597.14.2
by Vincent Ladeuil
Implements --take-this and --take-other when resolving text conflicts |
697 |
def action_take_this(self, tree): |
698 |
self._resolve_with_cleanups(tree, 'THIS') |
|
699 |
||
700 |
def action_take_other(self, tree): |
|
701 |
self._resolve_with_cleanups(tree, 'OTHER') |
|
702 |
||
|
1534.10.20
by Aaron Bentley
Got all tests passing |
703 |
|
|
1534.10.18
by Aaron Bentley
Defined all new Conflict types |
704 |
class HandledConflict(Conflict): |
|
1534.10.22
by Aaron Bentley
Got ConflictList implemented |
705 |
"""A path problem that has been provisionally resolved. |
706 |
This is intended to be a base class.
|
|
707 |
"""
|
|
708 |
||
709 |
rformat = "%(class)s(%(action)r, %(path)r, %(file_id)r)" |
|
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
710 |
|
|
1534.10.19
by Aaron Bentley
Stanza conversion, cooking |
711 |
def __init__(self, action, path, file_id=None): |
712 |
Conflict.__init__(self, path, file_id) |
|
|
1534.10.18
by Aaron Bentley
Defined all new Conflict types |
713 |
self.action = action |
714 |
||
|
1534.10.22
by Aaron Bentley
Got ConflictList implemented |
715 |
def _cmp_list(self): |
716 |
return Conflict._cmp_list(self) + [self.action] |
|
717 |
||
|
1534.10.19
by Aaron Bentley
Stanza conversion, cooking |
718 |
def as_stanza(self): |
719 |
s = Conflict.as_stanza(self) |
|
720 |
s.add('action', self.action) |
|
721 |
return s |
|
722 |
||
|
5013.2.4
by Vincent Ladeuil
``bzr add`` won't blindly add conflict related files. |
723 |
def associated_filenames(self): |
724 |
# Nothing has been generated here
|
|
725 |
return [] |
|
|
4597.3.52
by Vincent Ladeuil
Replace --interactive by --action. |
726 |
|
|
1534.10.20
by Aaron Bentley
Got all tests passing |
727 |
|
|
1534.10.18
by Aaron Bentley
Defined all new Conflict types |
728 |
class HandledPathConflict(HandledConflict): |
|
1534.10.22
by Aaron Bentley
Got ConflictList implemented |
729 |
"""A provisionally-resolved path problem involving two paths. |
730 |
This is intended to be a base class.
|
|
731 |
"""
|
|
732 |
||
733 |
rformat = "%(class)s(%(action)r, %(path)r, %(conflict_path)r,"\ |
|
734 |
" %(file_id)r, %(conflict_file_id)r)" |
|
735 |
||
|
1534.10.19
by Aaron Bentley
Stanza conversion, cooking |
736 |
def __init__(self, action, path, conflict_path, file_id=None, |
|
1534.10.18
by Aaron Bentley
Defined all new Conflict types |
737 |
conflict_file_id=None): |
|
1534.10.19
by Aaron Bentley
Stanza conversion, cooking |
738 |
HandledConflict.__init__(self, action, path, file_id) |
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
739 |
self.conflict_path = conflict_path |
|
6630.1.1
by Jelmer Vernooij
Remove deprecated functionality. |
740 |
# the factory blindly transfers the Stanza values to __init__,
|
741 |
# so they can be unicode.
|
|
|
6791.2.3
by Jelmer Vernooij
Fix more imports. |
742 |
if isinstance(conflict_file_id, text_type): |
|
6630.1.1
by Jelmer Vernooij
Remove deprecated functionality. |
743 |
conflict_file_id = cache_utf8.encode(conflict_file_id) |
744 |
self.conflict_file_id = osutils.safe_file_id(conflict_file_id) |
|
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
745 |
|
|
1534.10.22
by Aaron Bentley
Got ConflictList implemented |
746 |
def _cmp_list(self): |
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
747 |
return HandledConflict._cmp_list(self) + [self.conflict_path, |
|
1534.10.22
by Aaron Bentley
Got ConflictList implemented |
748 |
self.conflict_file_id] |
749 |
||
|
1534.10.19
by Aaron Bentley
Stanza conversion, cooking |
750 |
def as_stanza(self): |
751 |
s = HandledConflict.as_stanza(self) |
|
752 |
s.add('conflict_path', self.conflict_path) |
|
753 |
if self.conflict_file_id is not None: |
|
|
2309.4.13
by John Arbash Meinel
Conflicts go through Stanza so the need to be aware of utf8 versus unicode file ids. |
754 |
s.add('conflict_file_id', self.conflict_file_id.decode('utf8')) |
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
755 |
|
|
1534.10.19
by Aaron Bentley
Stanza conversion, cooking |
756 |
return s |
|
1534.10.20
by Aaron Bentley
Got all tests passing |
757 |
|
758 |
||
|
1534.10.18
by Aaron Bentley
Defined all new Conflict types |
759 |
class DuplicateID(HandledPathConflict): |
|
1534.10.22
by Aaron Bentley
Got ConflictList implemented |
760 |
"""Two files want the same file_id.""" |
761 |
||
|
1534.10.18
by Aaron Bentley
Defined all new Conflict types |
762 |
typestring = 'duplicate id' |
|
1534.10.22
by Aaron Bentley
Got ConflictList implemented |
763 |
|
|
1534.10.20
by Aaron Bentley
Got all tests passing |
764 |
format = 'Conflict adding id to %(conflict_path)s. %(action)s %(path)s.' |
765 |
||
|
1534.10.18
by Aaron Bentley
Defined all new Conflict types |
766 |
|
767 |
class DuplicateEntry(HandledPathConflict): |
|
|
1534.10.22
by Aaron Bentley
Got ConflictList implemented |
768 |
"""Two directory entries want to have the same name.""" |
769 |
||
|
1534.10.18
by Aaron Bentley
Defined all new Conflict types |
770 |
typestring = 'duplicate' |
|
1534.10.22
by Aaron Bentley
Got ConflictList implemented |
771 |
|
|
1534.10.20
by Aaron Bentley
Got all tests passing |
772 |
format = 'Conflict adding file %(conflict_path)s. %(action)s %(path)s.' |
773 |
||
|
4597.3.73
by Vincent Ladeuil
All Conflict action method names start with 'action_' to avoid potential namespace collisions |
774 |
def action_take_this(self, tree): |
|
4597.3.26
by Vincent Ladeuil
Tests passing for a minimal --interactive implementation. |
775 |
tree.remove([self.conflict_path], force=True, keep_files=False) |
776 |
tree.rename_one(self.path, self.conflict_path) |
|
777 |
||
|
4597.3.73
by Vincent Ladeuil
All Conflict action method names start with 'action_' to avoid potential namespace collisions |
778 |
def action_take_other(self, tree): |
|
4597.3.26
by Vincent Ladeuil
Tests passing for a minimal --interactive implementation. |
779 |
tree.remove([self.path], force=True, keep_files=False) |
780 |
||
|
1534.10.18
by Aaron Bentley
Defined all new Conflict types |
781 |
|
782 |
class ParentLoop(HandledPathConflict): |
|
|
1534.10.22
by Aaron Bentley
Got ConflictList implemented |
783 |
"""An attempt to create an infinitely-looping directory structure. |
784 |
This is rare, but can be produced like so:
|
|
785 |
||
786 |
tree A:
|
|
|
4597.3.50
by Vincent Ladeuil
Integrate most of BRANCH.TOTO into bzrlib/help_topics/en/conflicts.txt, fix a typo in a test. |
787 |
mv foo bar
|
|
1534.10.22
by Aaron Bentley
Got ConflictList implemented |
788 |
tree B:
|
|
4597.3.50
by Vincent Ladeuil
Integrate most of BRANCH.TOTO into bzrlib/help_topics/en/conflicts.txt, fix a typo in a test. |
789 |
mv bar foo
|
|
1534.10.22
by Aaron Bentley
Got ConflictList implemented |
790 |
merge A and B
|
791 |
"""
|
|
792 |
||
|
1534.10.18
by Aaron Bentley
Defined all new Conflict types |
793 |
typestring = 'parent loop' |
|
1534.10.22
by Aaron Bentley
Got ConflictList implemented |
794 |
|
|
4597.8.8
by Vincent Ladeuil
Exhibit bug #537956. |
795 |
format = 'Conflict moving %(path)s into %(conflict_path)s. %(action)s.' |
|
1534.10.20
by Aaron Bentley
Got all tests passing |
796 |
|
|
4597.3.73
by Vincent Ladeuil
All Conflict action method names start with 'action_' to avoid potential namespace collisions |
797 |
def action_take_this(self, tree): |
|
6622.1.28
by Jelmer Vernooij
More renames; commands in output, environment variables. |
798 |
# just acccept brz proposal
|
|
4597.3.34
by Vincent Ladeuil
Implement --interactive for ParentLoop. |
799 |
pass
|
800 |
||
|
4597.3.73
by Vincent Ladeuil
All Conflict action method names start with 'action_' to avoid potential namespace collisions |
801 |
def action_take_other(self, tree): |
|
4597.3.64
by Vincent Ladeuil
Fixed as per Aaron's remark. |
802 |
tt = transform.TreeTransform(tree) |
803 |
try: |
|
804 |
p_tid = tt.trans_id_file_id(self.file_id) |
|
805 |
parent_tid = tt.get_tree_parent(p_tid) |
|
806 |
cp_tid = tt.trans_id_file_id(self.conflict_file_id) |
|
807 |
cparent_tid = tt.get_tree_parent(cp_tid) |
|
|
5609.17.2
by Vincent Ladeuil
Fix bug #715058 and cleanup ParentLoop.action_take_other which did the right thing before everyone else ;-/ |
808 |
tt.adjust_path(osutils.basename(self.path), cparent_tid, cp_tid) |
809 |
tt.adjust_path(osutils.basename(self.conflict_path), |
|
810 |
parent_tid, p_tid) |
|
|
4597.3.64
by Vincent Ladeuil
Fixed as per Aaron's remark. |
811 |
tt.apply() |
812 |
finally: |
|
813 |
tt.finalize() |
|
|
4597.3.34
by Vincent Ladeuil
Implement --interactive for ParentLoop. |
814 |
|
|
1534.10.18
by Aaron Bentley
Defined all new Conflict types |
815 |
|
816 |
class UnversionedParent(HandledConflict): |
|
|
4597.2.1
by Vincent Ladeuil
Fix some typos. |
817 |
"""An attempt to version a file whose parent directory is not versioned. |
|
1534.10.22
by Aaron Bentley
Got ConflictList implemented |
818 |
Typically, the result of a merge where one tree unversioned the directory
|
819 |
and the other added a versioned file to it.
|
|
820 |
"""
|
|
821 |
||
|
1534.10.18
by Aaron Bentley
Defined all new Conflict types |
822 |
typestring = 'unversioned parent' |
|
1534.10.22
by Aaron Bentley
Got ConflictList implemented |
823 |
|
|
1551.8.22
by Aaron Bentley
Improve message when OTHER deletes an in-use tree |
824 |
format = 'Conflict because %(path)s is not versioned, but has versioned'\ |
825 |
' children. %(action)s.' |
|
|
1534.10.20
by Aaron Bentley
Got all tests passing |
826 |
|
|
4597.3.52
by Vincent Ladeuil
Replace --interactive by --action. |
827 |
# FIXME: We silently do nothing to make tests pass, but most probably the
|
828 |
# conflict shouldn't exist (the long story is that the conflict is
|
|
829 |
# generated with another one that can be resolved properly) -- vila 091224
|
|
|
4597.3.73
by Vincent Ladeuil
All Conflict action method names start with 'action_' to avoid potential namespace collisions |
830 |
def action_take_this(self, tree): |
|
4597.3.52
by Vincent Ladeuil
Replace --interactive by --action. |
831 |
pass
|
832 |
||
|
4597.3.73
by Vincent Ladeuil
All Conflict action method names start with 'action_' to avoid potential namespace collisions |
833 |
def action_take_other(self, tree): |
|
4597.3.52
by Vincent Ladeuil
Replace --interactive by --action. |
834 |
pass
|
835 |
||
|
1534.10.18
by Aaron Bentley
Defined all new Conflict types |
836 |
|
837 |
class MissingParent(HandledConflict): |
|
|
1534.10.22
by Aaron Bentley
Got ConflictList implemented |
838 |
"""An attempt to add files to a directory that is not present. |
|
1551.8.22
by Aaron Bentley
Improve message when OTHER deletes an in-use tree |
839 |
Typically, the result of a merge where THIS deleted the directory and
|
840 |
the OTHER added a file to it.
|
|
|
4597.3.50
by Vincent Ladeuil
Integrate most of BRANCH.TOTO into bzrlib/help_topics/en/conflicts.txt, fix a typo in a test. |
841 |
See also: DeletingParent (same situation, THIS and OTHER reversed)
|
|
1534.10.22
by Aaron Bentley
Got ConflictList implemented |
842 |
"""
|
843 |
||
|
1534.10.18
by Aaron Bentley
Defined all new Conflict types |
844 |
typestring = 'missing parent' |
|
1534.10.22
by Aaron Bentley
Got ConflictList implemented |
845 |
|
|
1534.10.20
by Aaron Bentley
Got all tests passing |
846 |
format = 'Conflict adding files to %(path)s. %(action)s.' |
847 |
||
|
4597.3.73
by Vincent Ladeuil
All Conflict action method names start with 'action_' to avoid potential namespace collisions |
848 |
def action_take_this(self, tree): |
|
4597.3.31
by Vincent Ladeuil
Implement --interactive for MissingParent. |
849 |
tree.remove([self.path], force=True, keep_files=False) |
850 |
||
|
4597.3.73
by Vincent Ladeuil
All Conflict action method names start with 'action_' to avoid potential namespace collisions |
851 |
def action_take_other(self, tree): |
|
6622.1.28
by Jelmer Vernooij
More renames; commands in output, environment variables. |
852 |
# just acccept brz proposal
|
|
4597.3.31
by Vincent Ladeuil
Implement --interactive for MissingParent. |
853 |
pass
|
854 |
||
|
1534.10.20
by Aaron Bentley
Got all tests passing |
855 |
|
|
1551.8.22
by Aaron Bentley
Improve message when OTHER deletes an in-use tree |
856 |
class DeletingParent(HandledConflict): |
857 |
"""An attempt to add files to a directory that is not present. |
|
858 |
Typically, the result of a merge where one OTHER deleted the directory and
|
|
859 |
the THIS added a file to it.
|
|
860 |
"""
|
|
861 |
||
862 |
typestring = 'deleting parent' |
|
863 |
||
|
1551.8.23
by Aaron Bentley
Tweaked conflict message to be more understandable |
864 |
format = "Conflict: can't delete %(path)s because it is not empty. "\ |
865 |
"%(action)s." |
|
|
1551.8.22
by Aaron Bentley
Improve message when OTHER deletes an in-use tree |
866 |
|
|
4597.3.32
by Vincent Ladeuil
Implement --interactive for DeletingParent noting the inconsistency. |
867 |
# FIXME: It's a bit strange that the default action is not coherent with
|
868 |
# MissingParent from the *user* pov.
|
|
869 |
||
|
4597.3.73
by Vincent Ladeuil
All Conflict action method names start with 'action_' to avoid potential namespace collisions |
870 |
def action_take_this(self, tree): |
|
6622.1.28
by Jelmer Vernooij
More renames; commands in output, environment variables. |
871 |
# just acccept brz proposal
|
|
4597.3.32
by Vincent Ladeuil
Implement --interactive for DeletingParent noting the inconsistency. |
872 |
pass
|
873 |
||
|
4597.3.73
by Vincent Ladeuil
All Conflict action method names start with 'action_' to avoid potential namespace collisions |
874 |
def action_take_other(self, tree): |
|
4597.3.32
by Vincent Ladeuil
Implement --interactive for DeletingParent noting the inconsistency. |
875 |
tree.remove([self.path], force=True, keep_files=False) |
876 |
||
|
1534.10.18
by Aaron Bentley
Defined all new Conflict types |
877 |
|
|
3144.4.2
by Aaron Bentley
Handle non-directory parent conflicts (abentley, #177390) |
878 |
class NonDirectoryParent(HandledConflict): |
|
4597.3.35
by Vincent Ladeuil
Implement --interactive for NonDirectoryParent, sort of. |
879 |
"""An attempt to add files to a directory that is not a directory or |
|
3144.4.2
by Aaron Bentley
Handle non-directory parent conflicts (abentley, #177390) |
880 |
an attempt to change the kind of a directory with files.
|
881 |
"""
|
|
882 |
||
883 |
typestring = 'non-directory parent' |
|
884 |
||
885 |
format = "Conflict: %(path)s is not a directory, but has files in it."\ |
|
886 |
" %(action)s." |
|
887 |
||
|
4597.3.68
by Vincent Ladeuil
Don't use rest syntax for text displayed as-is to users. |
888 |
# FIXME: .OTHER should be used instead of .new when the conflict is created
|
889 |
||
|
4597.3.73
by Vincent Ladeuil
All Conflict action method names start with 'action_' to avoid potential namespace collisions |
890 |
def action_take_this(self, tree): |
|
4597.3.55
by Vincent Ladeuil
More cleanup. |
891 |
# FIXME: we should preserve that path when the conflict is generated !
|
|
4597.3.35
by Vincent Ladeuil
Implement --interactive for NonDirectoryParent, sort of. |
892 |
if self.path.endswith('.new'): |
893 |
conflict_path = self.path[:-(len('.new'))] |
|
894 |
tree.remove([self.path], force=True, keep_files=False) |
|
895 |
tree.add(conflict_path) |
|
896 |
else: |
|
|
4597.3.73
by Vincent Ladeuil
All Conflict action method names start with 'action_' to avoid potential namespace collisions |
897 |
raise NotImplementedError(self.action_take_this) |
|
4597.3.35
by Vincent Ladeuil
Implement --interactive for NonDirectoryParent, sort of. |
898 |
|
|
4597.3.73
by Vincent Ladeuil
All Conflict action method names start with 'action_' to avoid potential namespace collisions |
899 |
def action_take_other(self, tree): |
|
4597.3.55
by Vincent Ladeuil
More cleanup. |
900 |
# FIXME: we should preserve that path when the conflict is generated !
|
|
4597.3.35
by Vincent Ladeuil
Implement --interactive for NonDirectoryParent, sort of. |
901 |
if self.path.endswith('.new'): |
902 |
conflict_path = self.path[:-(len('.new'))] |
|
903 |
tree.remove([conflict_path], force=True, keep_files=False) |
|
904 |
tree.rename_one(self.path, conflict_path) |
|
905 |
else: |
|
|
4597.3.73
by Vincent Ladeuil
All Conflict action method names start with 'action_' to avoid potential namespace collisions |
906 |
raise NotImplementedError(self.action_take_other) |
|
4597.3.35
by Vincent Ladeuil
Implement --interactive for NonDirectoryParent, sort of. |
907 |
|
908 |
||
|
1534.10.18
by Aaron Bentley
Defined all new Conflict types |
909 |
ctype = {} |
|
1534.10.20
by Aaron Bentley
Got all tests passing |
910 |
|
911 |
||
|
1534.10.18
by Aaron Bentley
Defined all new Conflict types |
912 |
def register_types(*conflict_types): |
913 |
"""Register a Conflict subclass for serialization purposes""" |
|
914 |
global ctype |
|
915 |
for conflict_type in conflict_types: |
|
916 |
ctype[conflict_type.typestring] = conflict_type |
|
917 |
||
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
918 |
|
|
1534.10.18
by Aaron Bentley
Defined all new Conflict types |
919 |
register_types(ContentsConflict, TextConflict, PathConflict, DuplicateID, |
|
1551.8.22
by Aaron Bentley
Improve message when OTHER deletes an in-use tree |
920 |
DuplicateEntry, ParentLoop, UnversionedParent, MissingParent, |
|
3144.4.2
by Aaron Bentley
Handle non-directory parent conflicts (abentley, #177390) |
921 |
DeletingParent, NonDirectoryParent) |