~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/conflicts.py

  • Committer: Aaron Bentley
  • Date: 2006-05-25 17:56:25 UTC
  • mto: This revision was merged to the branch mainline in revision 1738.
  • Revision ID: abentley@panoramicfeedback.com-20060525175625-c239c2e6526ffe6e
Cleanups to prepare for review

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2005, 2007, 2009, 2010 Canonical Ltd
2
 
#
 
1
# Copyright (C) 2005 by Aaron Bentley
 
2
 
3
3
# This program is free software; you can redistribute it and/or modify
4
4
# it under the terms of the GNU General Public License as published by
5
5
# the Free Software Foundation; either version 2 of the License, or
6
6
# (at your option) any later version.
7
 
#
 
7
 
8
8
# This program is distributed in the hope that it will be useful,
9
9
# but WITHOUT ANY WARRANTY; without even the implied warranty of
10
10
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
11
11
# GNU General Public License for more details.
12
 
#
 
12
 
13
13
# You should have received a copy of the GNU General Public License
14
14
# along with this program; if not, write to the Free Software
15
 
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
16
 
 
17
 
# TODO: 'bzr resolve' should accept a directory name and work from that
 
15
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
 
16
 
 
17
# TODO: Move this into builtins
 
18
 
 
19
# TODO: 'bzr resolve' should accept a directory name and work from that 
18
20
# point down
19
21
 
20
22
import os
21
 
import re
22
 
 
23
 
from bzrlib.lazy_import import lazy_import
24
 
lazy_import(globals(), """
25
23
import errno
26
24
 
27
 
from bzrlib import (
28
 
    builtins,
29
 
    commands,
30
 
    errors,
31
 
    osutils,
32
 
    rio,
33
 
    trace,
34
 
    transform,
35
 
    workingtree,
36
 
    )
37
 
""")
38
 
from bzrlib import (
39
 
    option,
40
 
    registry,
41
 
    )
 
25
import bzrlib
 
26
from bzrlib.commands import register_command
 
27
from bzrlib.errors import BzrCommandError, NotConflicted, UnsupportedOperation
 
28
from bzrlib.option import Option
 
29
from bzrlib.osutils import rename, delete_any
 
30
from bzrlib.rio import Stanza
42
31
 
43
32
 
44
33
CONFLICT_SUFFIXES = ('.THIS', '.BASE', '.OTHER')
45
34
 
46
35
 
47
 
class cmd_conflicts(commands.Command):
 
36
class cmd_conflicts(bzrlib.commands.Command):
48
37
    """List files with conflicts.
49
38
 
50
39
    Merge will do its best to combine the changes in two branches, but there
52
41
    it will mark a conflict.  A conflict means that you need to fix something,
53
42
    before you should commit.
54
43
 
55
 
    Conflicts normally are listed as short, human-readable messages.  If --text
56
 
    is supplied, the pathnames of files with text conflicts are listed,
57
 
    instead.  (This is useful for editing all files with text conflicts.)
58
 
 
59
44
    Use bzr resolve when you have fixed a problem.
 
45
 
 
46
    (conflicts are determined by the presence of .BASE .TREE, and .OTHER 
 
47
    files.)
 
48
 
 
49
    See also bzr resolve.
60
50
    """
61
 
    takes_options = [
62
 
            option.Option('text',
63
 
                          help='List paths of files with text conflicts.'),
64
 
        ]
65
 
    _see_also = ['resolve', 'conflict-types']
66
 
 
67
 
    def run(self, text=False):
68
 
        wt = workingtree.WorkingTree.open_containing(u'.')[0]
 
51
    def run(self):
 
52
        from bzrlib.workingtree import WorkingTree
 
53
        wt = WorkingTree.open_containing(u'.')[0]
69
54
        for conflict in wt.conflicts():
70
 
            if text:
71
 
                if conflict.typestring != 'text conflict':
72
 
                    continue
73
 
                self.outf.write(conflict.path + '\n')
74
 
            else:
75
 
                self.outf.write(str(conflict) + '\n')
76
 
 
77
 
 
78
 
resolve_action_registry = registry.Registry()
79
 
 
80
 
 
81
 
resolve_action_registry.register(
82
 
    'done', 'done', 'Marks the conflict as resolved' )
83
 
resolve_action_registry.register(
84
 
    'take-this', 'take_this',
85
 
    'Resolve the conflict preserving the version in the working tree' )
86
 
resolve_action_registry.register(
87
 
    'take-other', 'take_other',
88
 
    'Resolve the conflict taking the merged version into account' )
89
 
resolve_action_registry.default_key = 'done'
90
 
 
91
 
class ResolveActionOption(option.RegistryOption):
92
 
 
93
 
    def __init__(self):
94
 
        super(ResolveActionOption, self).__init__(
95
 
            'action', 'How to resolve the conflict.',
96
 
            value_switches=True,
97
 
            registry=resolve_action_registry)
98
 
 
99
 
 
100
 
class cmd_resolve(commands.Command):
 
55
            print conflict
 
56
 
 
57
 
 
58
class cmd_resolve(bzrlib.commands.Command):
101
59
    """Mark a conflict as resolved.
102
60
 
103
61
    Merge will do its best to combine the changes in two branches, but there
105
63
    it will mark a conflict.  A conflict means that you need to fix something,
106
64
    before you should commit.
107
65
 
108
 
    Once you have fixed a problem, use "bzr resolve" to automatically mark
109
 
    text conflicts as fixed, "bzr resolve FILE" to mark a specific conflict as
110
 
    resolved, or "bzr resolve --all" to mark all conflicts as resolved.
 
66
    Once you have fixed a problem, use "bzr resolve FILE.." to mark
 
67
    individual files as fixed, or "bzr resolve --all" to mark all conflicts as
 
68
    resolved.
 
69
 
 
70
    See also bzr conflicts.
111
71
    """
112
72
    aliases = ['resolved']
113
73
    takes_args = ['file*']
114
 
    takes_options = [
115
 
            option.Option('all', help='Resolve all conflicts in this tree.'),
116
 
            ResolveActionOption(),
117
 
            ]
118
 
    _see_also = ['conflicts']
119
 
    def run(self, file_list=None, all=False, action=None):
 
74
    takes_options = [Option('all', help='Resolve all conflicts in this tree')]
 
75
    def run(self, file_list=None, all=False):
 
76
        from bzrlib.workingtree import WorkingTree
120
77
        if all:
121
78
            if file_list:
122
 
                raise errors.BzrCommandError("If --all is specified,"
123
 
                                             " no FILE may be provided")
124
 
            tree = workingtree.WorkingTree.open_containing('.')[0]
125
 
            if action is None:
126
 
                action = 'done'
127
 
        else:
128
 
            tree, file_list = builtins.tree_files(file_list)
129
 
            if file_list is None:
130
 
                if action is None:
131
 
                    # FIXME: There is a special case here related to the option
132
 
                    # handling that could be clearer and easier to discover by
133
 
                    # providing an --auto action (bug #344013 and #383396) and
134
 
                    # make it mandatory instead of implicit and active only
135
 
                    # when no file_list is provided -- vila 091229
136
 
                    action = 'auto'
137
 
            else:
138
 
                if action is None:
139
 
                    action = 'done'
140
 
        if action == 'auto':
141
 
            if file_list is None:
142
 
                un_resolved, resolved = tree.auto_resolve()
143
 
                if len(un_resolved) > 0:
144
 
                    trace.note('%d conflict(s) auto-resolved.', len(resolved))
145
 
                    trace.note('Remaining conflicts:')
146
 
                    for conflict in un_resolved:
147
 
                        trace.note(conflict)
148
 
                    return 1
149
 
                else:
150
 
                    trace.note('All conflicts resolved.')
151
 
                    return 0
152
 
            else:
153
 
                # FIXME: This can never occur but the block above needs some
154
 
                # refactoring to transfer tree.auto_resolve() to
155
 
                # conflict.auto(tree) --vila 091242
156
 
                pass
157
 
        else:
158
 
            resolve(tree, file_list, action=action)
159
 
 
160
 
 
161
 
def resolve(tree, paths=None, ignore_misses=False, recursive=False,
162
 
            action='done'):
163
 
    """Resolve some or all of the conflicts in a working tree.
164
 
 
165
 
    :param paths: If None, resolve all conflicts.  Otherwise, select only
166
 
        specified conflicts.
167
 
    :param recursive: If True, then elements of paths which are directories
168
 
        have all their children resolved, etc.  When invoked as part of
169
 
        recursive commands like revert, this should be True.  For commands
170
 
        or applications wishing finer-grained control, like the resolve
171
 
        command, this should be False.
172
 
    :param ignore_misses: If False, warnings will be printed if the supplied
173
 
        paths do not have conflicts.
174
 
    :param action: How the conflict should be resolved,
175
 
    """
176
 
    tree.lock_tree_write()
 
79
                raise BzrCommandError("If --all is specified, no FILE may be provided")
 
80
            tree = WorkingTree.open_containing('.')[0]
 
81
            resolve(tree)
 
82
        else:
 
83
            if file_list is None:
 
84
                raise BzrCommandError("command 'resolve' needs one or more FILE, or --all")
 
85
            tree = WorkingTree.open_containing(file_list[0])[0]
 
86
            to_resolve = [tree.relpath(p) for p in file_list]
 
87
            resolve(tree, to_resolve)
 
88
 
 
89
 
 
90
def resolve(tree, paths=None, ignore_misses=False):
 
91
    tree.lock_write()
177
92
    try:
178
93
        tree_conflicts = tree.conflicts()
179
94
        if paths is None:
180
95
            new_conflicts = ConflictList()
181
 
            to_process = tree_conflicts
 
96
            selected_conflicts = tree_conflicts
182
97
        else:
183
 
            new_conflicts, to_process = tree_conflicts.select_conflicts(
184
 
                tree, paths, ignore_misses, recursive)
185
 
        for conflict in to_process:
186
 
            try:
187
 
                conflict._do(action, tree)
188
 
                conflict.cleanup(tree)
189
 
            except NotImplementedError:
190
 
                new_conflicts.append(conflict)
 
98
            new_conflicts, selected_conflicts = \
 
99
                tree_conflicts.select_conflicts(tree, paths, ignore_misses)
191
100
        try:
192
101
            tree.set_conflicts(new_conflicts)
193
 
        except errors.UnsupportedOperation:
 
102
        except UnsupportedOperation:
194
103
            pass
 
104
        selected_conflicts.remove_files(tree)
195
105
    finally:
196
106
        tree.unlock()
197
107
 
198
108
 
199
109
def restore(filename):
200
 
    """Restore a conflicted file to the state it was in before merging.
201
 
 
202
 
    Only text restoration is supported at present.
 
110
    """\
 
111
    Restore a conflicted file to the state it was in before merging.
 
112
    Only text restoration supported at present.
203
113
    """
204
114
    conflicted = False
205
115
    try:
206
 
        osutils.rename(filename + ".THIS", filename)
 
116
        rename(filename + ".THIS", filename)
207
117
        conflicted = True
208
118
    except OSError, e:
209
119
        if e.errno != errno.ENOENT:
221
131
        if e.errno != errno.ENOENT:
222
132
            raise
223
133
    if not conflicted:
224
 
        raise errors.NotConflicted(filename)
 
134
        raise NotConflicted(filename)
225
135
 
226
136
 
227
137
class ConflictList(object):
275
185
        """Generator of stanzas"""
276
186
        for conflict in self:
277
187
            yield conflict.as_stanza()
278
 
 
 
188
            
279
189
    def to_strings(self):
280
190
        """Generate strings for the provided conflicts"""
281
191
        for conflict in self:
286
196
        for conflict in self:
287
197
            if not conflict.has_files:
288
198
                continue
289
 
            conflict.cleanup(tree)
 
199
            for suffix in CONFLICT_SUFFIXES:
 
200
                try:
 
201
                    delete_any(tree.abspath(conflict.path+suffix))
 
202
                except OSError, e:
 
203
                    if e.errno != errno.ENOENT:
 
204
                        raise
290
205
 
291
 
    def select_conflicts(self, tree, paths, ignore_misses=False,
292
 
                         recurse=False):
 
206
    def select_conflicts(self, tree, paths, ignore_misses=False):
293
207
        """Select the conflicts associated with paths in a tree.
294
 
 
 
208
        
295
209
        File-ids are also used for this.
296
 
        :return: a pair of ConflictLists: (not_selected, selected)
297
210
        """
298
211
        path_set = set(paths)
299
212
        ids = {}
314
227
                if cpath in path_set:
315
228
                    selected = True
316
229
                    selected_paths.add(cpath)
317
 
                if recurse:
318
 
                    if osutils.is_inside_any(path_set, cpath):
319
 
                        selected = True
320
 
                        selected_paths.add(cpath)
321
 
 
322
230
            for key in ('file_id', 'conflict_file_id'):
323
231
                cfile_id = getattr(conflict, key, None)
324
232
                if cfile_id is None:
341
249
                    print "%s is not conflicted" % path
342
250
        return new_conflicts, selected_conflicts
343
251
 
344
 
 
 
252
 
345
253
class Conflict(object):
346
254
    """Base class for all types of conflict"""
347
255
 
348
 
    # FIXME: cleanup should take care of that ? -- vila 091229
349
256
    has_files = False
350
257
 
351
258
    def __init__(self, path, file_id=None):
352
259
        self.path = path
353
 
        # warn turned off, because the factory blindly transfers the Stanza
354
 
        # values to __init__ and Stanza is purely a Unicode api.
355
 
        self.file_id = osutils.safe_file_id(file_id, warn=False)
 
260
        self.file_id = file_id
356
261
 
357
262
    def as_stanza(self):
358
 
        s = rio.Stanza(type=self.typestring, path=self.path)
 
263
        s = Stanza(type=self.typestring, path=self.path)
359
264
        if self.file_id is not None:
360
 
            # Stanza requires Unicode apis
361
 
            s.add('file_id', self.file_id.decode('utf8'))
 
265
            s.add('file_id', self.file_id)
362
266
        return s
363
267
 
364
268
    def _cmp_list(self):
369
273
            return -1
370
274
        return cmp(self._cmp_list(), other._cmp_list())
371
275
 
372
 
    def __hash__(self):
373
 
        return hash((type(self), self.path, self.file_id))
374
 
 
375
276
    def __eq__(self, other):
376
277
        return self.__cmp__(other) == 0
377
278
 
400
301
        else:
401
302
            return None, conflict.typestring
402
303
 
403
 
    def _do(self, action, tree):
404
 
        """Apply the specified action to the conflict.
405
 
 
406
 
        :param action: The method name to call.
407
 
 
408
 
        :param tree: The tree passed as a parameter to the method.
409
 
        """
410
 
        meth = getattr(self, 'action_%s' % action, None)
411
 
        if meth is None:
412
 
            raise NotImplementedError(self.__class__.__name__ + '.' + action)
413
 
        meth(tree)
414
 
 
415
 
    def cleanup(self, tree):
416
 
        raise NotImplementedError(self.cleanup)
417
 
 
418
 
    def action_done(self, tree):
419
 
        """Mark the conflict as solved once it has been handled."""
420
 
        # This method does nothing but simplifies the design of upper levels.
421
 
        pass
422
 
 
423
 
    def action_take_this(self, tree):
424
 
        raise NotImplementedError(self.action_take_this)
425
 
 
426
 
    def action_take_other(self, tree):
427
 
        raise NotImplementedError(self.action_take_other)
428
 
 
429
304
 
430
305
class PathConflict(Conflict):
431
306
    """A conflict was encountered merging file paths"""
435
310
    format = 'Path conflict: %(path)s / %(conflict_path)s'
436
311
 
437
312
    rformat = '%(class)s(%(path)r, %(conflict_path)r, %(file_id)r)'
438
 
 
439
313
    def __init__(self, path, conflict_path=None, file_id=None):
440
314
        Conflict.__init__(self, path, file_id)
441
315
        self.conflict_path = conflict_path
446
320
            s.add('conflict_path', self.conflict_path)
447
321
        return s
448
322
 
449
 
    def cleanup(self, tree):
450
 
        # No additional files have been generated here
451
 
        pass
452
 
 
453
 
    def action_take_this(self, tree):
454
 
        tree.rename_one(self.conflict_path, self.path)
455
 
 
456
 
    def action_take_other(self, tree):
457
 
        # just acccept bzr proposal
458
 
        pass
459
 
 
460
323
 
461
324
class ContentsConflict(PathConflict):
462
325
    """The files are of different types, or not present"""
467
330
 
468
331
    format = 'Contents conflict in %(path)s'
469
332
 
470
 
    def cleanup(self, tree):
471
 
        for suffix in ('.BASE', '.OTHER'):
472
 
            try:
473
 
                osutils.delete_any(tree.abspath(self.path + suffix))
474
 
            except OSError, e:
475
 
                if e.errno != errno.ENOENT:
476
 
                    raise
477
 
 
478
 
    # FIXME: I smell something weird here and it seems we should be able to be
479
 
    # more coherent with some other conflict ? bzr *did* a choice there but
480
 
    # neither action_take_this nor action_take_other reflect that...
481
 
    # -- vila 20091224
482
 
    def action_take_this(self, tree):
483
 
        tree.remove([self.path + '.OTHER'], force=True, keep_files=False)
484
 
 
485
 
    def action_take_other(self, tree):
486
 
        tree.remove([self.path], force=True, keep_files=False)
487
 
 
488
 
 
489
 
 
490
 
# FIXME: TextConflict is about a single file-id, there never is a conflict_path
491
 
# attribute so we shouldn't inherit from PathConflict but simply from Conflict
492
 
 
493
 
# TODO: There should be a base revid attribute to better inform the user about
494
 
# how the conflicts were generated.
 
333
 
495
334
class TextConflict(PathConflict):
496
335
    """The merge algorithm could not resolve all differences encountered."""
497
336
 
501
340
 
502
341
    format = 'Text conflict in %(path)s'
503
342
 
504
 
    def cleanup(self, tree):
505
 
        for suffix in CONFLICT_SUFFIXES:
506
 
            try:
507
 
                osutils.delete_any(tree.abspath(self.path+suffix))
508
 
            except OSError, e:
509
 
                if e.errno != errno.ENOENT:
510
 
                    raise
511
 
 
512
343
 
513
344
class HandledConflict(Conflict):
514
345
    """A path problem that has been provisionally resolved.
516
347
    """
517
348
 
518
349
    rformat = "%(class)s(%(action)r, %(path)r, %(file_id)r)"
519
 
 
 
350
    
520
351
    def __init__(self, action, path, file_id=None):
521
352
        Conflict.__init__(self, path, file_id)
522
353
        self.action = action
529
360
        s.add('action', self.action)
530
361
        return s
531
362
 
532
 
    def cleanup(self, tree):
533
 
        """Nothing to cleanup."""
534
 
        pass
535
 
 
536
363
 
537
364
class HandledPathConflict(HandledConflict):
538
365
    """A provisionally-resolved path problem involving two paths.
545
372
    def __init__(self, action, path, conflict_path, file_id=None,
546
373
                 conflict_file_id=None):
547
374
        HandledConflict.__init__(self, action, path, file_id)
548
 
        self.conflict_path = conflict_path
549
 
        # warn turned off, because the factory blindly transfers the Stanza
550
 
        # values to __init__.
551
 
        self.conflict_file_id = osutils.safe_file_id(conflict_file_id,
552
 
                                                     warn=False)
553
 
 
 
375
        self.conflict_path = conflict_path 
 
376
        self.conflict_file_id = conflict_file_id
 
377
        
554
378
    def _cmp_list(self):
555
 
        return HandledConflict._cmp_list(self) + [self.conflict_path,
 
379
        return HandledConflict._cmp_list(self) + [self.conflict_path, 
556
380
                                                  self.conflict_file_id]
557
381
 
558
382
    def as_stanza(self):
559
383
        s = HandledConflict.as_stanza(self)
560
384
        s.add('conflict_path', self.conflict_path)
561
385
        if self.conflict_file_id is not None:
562
 
            s.add('conflict_file_id', self.conflict_file_id.decode('utf8'))
563
 
 
 
386
            s.add('conflict_file_id', self.conflict_file_id)
 
387
            
564
388
        return s
565
389
 
566
390
 
579
403
 
580
404
    format = 'Conflict adding file %(conflict_path)s.  %(action)s %(path)s.'
581
405
 
582
 
    def action_take_this(self, tree):
583
 
        tree.remove([self.conflict_path], force=True, keep_files=False)
584
 
        tree.rename_one(self.path, self.conflict_path)
585
 
 
586
 
    def action_take_other(self, tree):
587
 
        tree.remove([self.path], force=True, keep_files=False)
588
 
 
589
406
 
590
407
class ParentLoop(HandledPathConflict):
591
408
    """An attempt to create an infinitely-looping directory structure.
592
409
    This is rare, but can be produced like so:
593
410
 
594
411
    tree A:
595
 
      mv foo bar
 
412
      mv foo/bar
596
413
    tree B:
597
 
      mv bar foo
 
414
      mv bar/foo
598
415
    merge A and B
599
416
    """
600
417
 
602
419
 
603
420
    format = 'Conflict moving %(conflict_path)s into %(path)s.  %(action)s.'
604
421
 
605
 
    def action_take_this(self, tree):
606
 
        # just acccept bzr proposal
607
 
        pass
608
 
 
609
 
    def action_take_other(self, tree):
610
 
        # FIXME: We shouldn't have to manipulate so many paths here (and there
611
 
        # is probably a bug or two...)
612
 
        base_path = osutils.basename(self.path)
613
 
        conflict_base_path = osutils.basename(self.conflict_path)
614
 
        tt = transform.TreeTransform(tree)
615
 
        try:
616
 
            p_tid = tt.trans_id_file_id(self.file_id)
617
 
            parent_tid = tt.get_tree_parent(p_tid)
618
 
            cp_tid = tt.trans_id_file_id(self.conflict_file_id)
619
 
            cparent_tid = tt.get_tree_parent(cp_tid)
620
 
            tt.adjust_path(base_path, cparent_tid, cp_tid)
621
 
            tt.adjust_path(conflict_base_path, parent_tid, p_tid)
622
 
            tt.apply()
623
 
        finally:
624
 
            tt.finalize()
625
 
 
626
422
 
627
423
class UnversionedParent(HandledConflict):
628
 
    """An attempt to version a file whose parent directory is not versioned.
 
424
    """An attempt to version an file whose parent directory is not versioned.
629
425
    Typically, the result of a merge where one tree unversioned the directory
630
426
    and the other added a versioned file to it.
631
427
    """
632
428
 
633
429
    typestring = 'unversioned parent'
634
430
 
635
 
    format = 'Conflict because %(path)s is not versioned, but has versioned'\
636
 
             ' children.  %(action)s.'
637
 
 
638
 
    # FIXME: We silently do nothing to make tests pass, but most probably the
639
 
    # conflict shouldn't exist (the long story is that the conflict is
640
 
    # generated with another one that can be resolved properly) -- vila 091224
641
 
    def action_take_this(self, tree):
642
 
        pass
643
 
 
644
 
    def action_take_other(self, tree):
645
 
        pass
 
431
    format = 'Conflict adding versioned files to %(path)s.  %(action)s.'
646
432
 
647
433
 
648
434
class MissingParent(HandledConflict):
649
435
    """An attempt to add files to a directory that is not present.
650
 
    Typically, the result of a merge where THIS deleted the directory and
651
 
    the OTHER added a file to it.
652
 
    See also: DeletingParent (same situation, THIS and OTHER reversed)
 
436
    Typically, the result of a merge where one tree deleted the directory and
 
437
    the other added a file to it.
653
438
    """
654
439
 
655
440
    typestring = 'missing parent'
656
441
 
657
442
    format = 'Conflict adding files to %(path)s.  %(action)s.'
658
443
 
659
 
    def action_take_this(self, tree):
660
 
        tree.remove([self.path], force=True, keep_files=False)
661
 
 
662
 
    def action_take_other(self, tree):
663
 
        # just acccept bzr proposal
664
 
        pass
665
 
 
666
 
 
667
 
class DeletingParent(HandledConflict):
668
 
    """An attempt to add files to a directory that is not present.
669
 
    Typically, the result of a merge where one OTHER deleted the directory and
670
 
    the THIS added a file to it.
671
 
    """
672
 
 
673
 
    typestring = 'deleting parent'
674
 
 
675
 
    format = "Conflict: can't delete %(path)s because it is not empty.  "\
676
 
             "%(action)s."
677
 
 
678
 
    # FIXME: It's a bit strange that the default action is not coherent with
679
 
    # MissingParent from the *user* pov.
680
 
 
681
 
    def action_take_this(self, tree):
682
 
        # just acccept bzr proposal
683
 
        pass
684
 
 
685
 
    def action_take_other(self, tree):
686
 
        tree.remove([self.path], force=True, keep_files=False)
687
 
 
688
 
 
689
 
class NonDirectoryParent(HandledConflict):
690
 
    """An attempt to add files to a directory that is not a directory or
691
 
    an attempt to change the kind of a directory with files.
692
 
    """
693
 
 
694
 
    typestring = 'non-directory parent'
695
 
 
696
 
    format = "Conflict: %(path)s is not a directory, but has files in it."\
697
 
             "  %(action)s."
698
 
 
699
 
    # FIXME: .OTHER should be used instead of .new when the conflict is created
700
 
 
701
 
    def action_take_this(self, tree):
702
 
        # FIXME: we should preserve that path when the conflict is generated !
703
 
        if self.path.endswith('.new'):
704
 
            conflict_path = self.path[:-(len('.new'))]
705
 
            tree.remove([self.path], force=True, keep_files=False)
706
 
            tree.add(conflict_path)
707
 
        else:
708
 
            raise NotImplementedError(self.action_take_this)
709
 
 
710
 
    def action_take_other(self, tree):
711
 
        # FIXME: we should preserve that path when the conflict is generated !
712
 
        if self.path.endswith('.new'):
713
 
            conflict_path = self.path[:-(len('.new'))]
714
 
            tree.remove([conflict_path], force=True, keep_files=False)
715
 
            tree.rename_one(self.path, conflict_path)
716
 
        else:
717
 
            raise NotImplementedError(self.action_take_other)
718
444
 
719
445
 
720
446
ctype = {}
726
452
    for conflict_type in conflict_types:
727
453
        ctype[conflict_type.typestring] = conflict_type
728
454
 
 
455
 
729
456
register_types(ContentsConflict, TextConflict, PathConflict, DuplicateID,
730
 
               DuplicateEntry, ParentLoop, UnversionedParent, MissingParent,
731
 
               DeletingParent, NonDirectoryParent)
 
457
               DuplicateEntry, ParentLoop, UnversionedParent, MissingParent,)