~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/conflicts.py

  • Committer: Robert Collins
  • Date: 2006-07-20 13:00:31 UTC
  • mto: (1852.9.1 Tree.compare().)
  • mto: This revision was merged to the branch mainline in revision 1890.
  • Revision ID: robertc@robertcollins.net-20060720130031-d26103a427ea10f3
StartĀ treeĀ implementationĀ tests.

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
210
        :return: a pair of ConflictLists: (not_selected, selected)
297
211
        """
314
228
                if cpath in path_set:
315
229
                    selected = True
316
230
                    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
231
            for key in ('file_id', 'conflict_file_id'):
323
232
                cfile_id = getattr(conflict, key, None)
324
233
                if cfile_id is None:
341
250
                    print "%s is not conflicted" % path
342
251
        return new_conflicts, selected_conflicts
343
252
 
344
 
 
 
253
 
345
254
class Conflict(object):
346
255
    """Base class for all types of conflict"""
347
256
 
348
 
    # FIXME: cleanup should take care of that ? -- vila 091229
349
257
    has_files = False
350
258
 
351
259
    def __init__(self, path, file_id=None):
352
260
        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)
 
261
        self.file_id = file_id
356
262
 
357
263
    def as_stanza(self):
358
 
        s = rio.Stanza(type=self.typestring, path=self.path)
 
264
        s = Stanza(type=self.typestring, path=self.path)
359
265
        if self.file_id is not None:
360
 
            # Stanza requires Unicode apis
361
 
            s.add('file_id', self.file_id.decode('utf8'))
 
266
            s.add('file_id', self.file_id)
362
267
        return s
363
268
 
364
269
    def _cmp_list(self):
400
305
        else:
401
306
            return None, conflict.typestring
402
307
 
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
308
 
430
309
class PathConflict(Conflict):
431
310
    """A conflict was encountered merging file paths"""
435
314
    format = 'Path conflict: %(path)s / %(conflict_path)s'
436
315
 
437
316
    rformat = '%(class)s(%(path)r, %(conflict_path)r, %(file_id)r)'
438
 
 
439
317
    def __init__(self, path, conflict_path=None, file_id=None):
440
318
        Conflict.__init__(self, path, file_id)
441
319
        self.conflict_path = conflict_path
446
324
            s.add('conflict_path', self.conflict_path)
447
325
        return s
448
326
 
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
327
 
461
328
class ContentsConflict(PathConflict):
462
329
    """The files are of different types, or not present"""
467
334
 
468
335
    format = 'Contents conflict in %(path)s'
469
336
 
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.
 
337
 
495
338
class TextConflict(PathConflict):
496
339
    """The merge algorithm could not resolve all differences encountered."""
497
340
 
501
344
 
502
345
    format = 'Text conflict in %(path)s'
503
346
 
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
347
 
513
348
class HandledConflict(Conflict):
514
349
    """A path problem that has been provisionally resolved.
516
351
    """
517
352
 
518
353
    rformat = "%(class)s(%(action)r, %(path)r, %(file_id)r)"
519
 
 
 
354
    
520
355
    def __init__(self, action, path, file_id=None):
521
356
        Conflict.__init__(self, path, file_id)
522
357
        self.action = action
529
364
        s.add('action', self.action)
530
365
        return s
531
366
 
532
 
    def cleanup(self, tree):
533
 
        """Nothing to cleanup."""
534
 
        pass
535
 
 
536
367
 
537
368
class HandledPathConflict(HandledConflict):
538
369
    """A provisionally-resolved path problem involving two paths.
545
376
    def __init__(self, action, path, conflict_path, file_id=None,
546
377
                 conflict_file_id=None):
547
378
        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
 
 
 
379
        self.conflict_path = conflict_path 
 
380
        self.conflict_file_id = conflict_file_id
 
381
        
554
382
    def _cmp_list(self):
555
 
        return HandledConflict._cmp_list(self) + [self.conflict_path,
 
383
        return HandledConflict._cmp_list(self) + [self.conflict_path, 
556
384
                                                  self.conflict_file_id]
557
385
 
558
386
    def as_stanza(self):
559
387
        s = HandledConflict.as_stanza(self)
560
388
        s.add('conflict_path', self.conflict_path)
561
389
        if self.conflict_file_id is not None:
562
 
            s.add('conflict_file_id', self.conflict_file_id.decode('utf8'))
563
 
 
 
390
            s.add('conflict_file_id', self.conflict_file_id)
 
391
            
564
392
        return s
565
393
 
566
394
 
579
407
 
580
408
    format = 'Conflict adding file %(conflict_path)s.  %(action)s %(path)s.'
581
409
 
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
410
 
590
411
class ParentLoop(HandledPathConflict):
591
412
    """An attempt to create an infinitely-looping directory structure.
592
413
    This is rare, but can be produced like so:
593
414
 
594
415
    tree A:
595
 
      mv foo bar
 
416
      mv foo/bar
596
417
    tree B:
597
 
      mv bar foo
 
418
      mv bar/foo
598
419
    merge A and B
599
420
    """
600
421
 
602
423
 
603
424
    format = 'Conflict moving %(conflict_path)s into %(path)s.  %(action)s.'
604
425
 
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
426
 
627
427
class UnversionedParent(HandledConflict):
628
 
    """An attempt to version a file whose parent directory is not versioned.
 
428
    """An attempt to version an file whose parent directory is not versioned.
629
429
    Typically, the result of a merge where one tree unversioned the directory
630
430
    and the other added a versioned file to it.
631
431
    """
632
432
 
633
433
    typestring = 'unversioned parent'
634
434
 
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
 
435
    format = 'Conflict adding versioned files to %(path)s.  %(action)s.'
646
436
 
647
437
 
648
438
class MissingParent(HandledConflict):
649
439
    """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)
 
440
    Typically, the result of a merge where one tree deleted the directory and
 
441
    the other added a file to it.
653
442
    """
654
443
 
655
444
    typestring = 'missing parent'
656
445
 
657
446
    format = 'Conflict adding files to %(path)s.  %(action)s.'
658
447
 
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
448
 
719
449
 
720
450
ctype = {}
726
456
    for conflict_type in conflict_types:
727
457
        ctype[conflict_type.typestring] = conflict_type
728
458
 
 
459
 
729
460
register_types(ContentsConflict, TextConflict, PathConflict, DuplicateID,
730
 
               DuplicateEntry, ParentLoop, UnversionedParent, MissingParent,
731
 
               DeletingParent, NonDirectoryParent)
 
461
               DuplicateEntry, ParentLoop, UnversionedParent, MissingParent,)