← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/conflicts.py

  • Committer: Robert Collins
  • Date: 2010-05-06 23:41:35 UTC
  • mto: This revision was merged to the branch mainline in revision 5223.
  • Revision ID: robertc@robertcollins.net-20100506234135-yivbzczw1sejxnxc
Lock methods on ``Tree``, ``Branch`` and ``Repository`` are now
expected to return an object which can be used to unlock them. This reduces
duplicate code when using cleanups. The previous 'tokens's returned by
``Branch.lock_write`` and ``Repository.lock_write`` are now attributes
on the result of the lock_write. ``repository.RepositoryWriteLockResult``
and ``branch.BranchWriteLockResult`` document this. (Robert Collins)

``log._get_info_for_log_files`` now takes an add_cleanup callable.
(Robert Collins)

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/conflicts.py
1
 
# Copyright (C) 2005, 2007 Canonical Ltd
 
1
# Copyright (C) 2005, 2006, 2007, 2009, 2010 Canonical Ltd
2
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
14
14
# along with this program; if not, write to the Free Software
15
15
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
16
16
 
17
 
# TODO: Move this into builtins
18
 
 
19
17
# TODO: 'bzr resolve' should accept a directory name and work from that
20
18
# point down
21
19
 
27
25
 
28
26
from bzrlib import (
29
27
    builtins,
 
28
    cleanup,
30
29
    commands,
31
30
    errors,
32
31
    osutils,
33
32
    rio,
34
33
    trace,
 
34
    transform,
 
35
    workingtree,
35
36
    )
36
37
""")
37
 
from bzrlib.option import Option
 
38
from bzrlib import (
 
39
    option,
 
40
    registry,
 
41
    )
38
42
 
39
43
 
40
44
CONFLICT_SUFFIXES = ('.THIS', '.BASE', '.OTHER')
41
45
 
42
46
 
43
47
class cmd_conflicts(commands.Command):
44
 
    """List files with conflicts.
 
48
    __doc__ = """List files with conflicts.
45
49
 
46
50
    Merge will do its best to combine the changes in two branches, but there
47
51
    are some kinds of problems only a human can fix.  When it encounters those,
53
57
    instead.  (This is useful for editing all files with text conflicts.)
54
58
 
55
59
    Use bzr resolve when you have fixed a problem.
56
 
 
57
 
    See also bzr resolve.
58
60
    """
59
61
    takes_options = [
60
 
            Option('text',
61
 
                   help='List paths of files with text conflicts.'),
 
62
            option.Option('text',
 
63
                          help='List paths of files with text conflicts.'),
62
64
        ]
 
65
    _see_also = ['resolve', 'conflict-types']
63
66
 
64
67
    def run(self, text=False):
65
 
        from bzrlib.workingtree import WorkingTree
66
 
        wt = WorkingTree.open_containing(u'.')[0]
 
68
        wt = workingtree.WorkingTree.open_containing(u'.')[0]
67
69
        for conflict in wt.conflicts():
68
70
            if text:
69
71
                if conflict.typestring != 'text conflict':
73
75
                self.outf.write(str(conflict) + '\n')
74
76
 
75
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
 
76
100
class cmd_resolve(commands.Command):
77
 
    """Mark a conflict as resolved.
 
101
    __doc__ = """Mark a conflict as resolved.
78
102
 
79
103
    Merge will do its best to combine the changes in two branches, but there
80
104
    are some kinds of problems only a human can fix.  When it encounters those,
82
106
    before you should commit.
83
107
 
84
108
    Once you have fixed a problem, use "bzr resolve" to automatically mark
85
 
    text conflicts as fixed, resolve FILE to mark a specific conflict as
 
109
    text conflicts as fixed, "bzr resolve FILE" to mark a specific conflict as
86
110
    resolved, or "bzr resolve --all" to mark all conflicts as resolved.
87
 
 
88
 
    See also bzr conflicts.
89
111
    """
90
112
    aliases = ['resolved']
91
113
    takes_args = ['file*']
92
114
    takes_options = [
93
 
            Option('all', help='Resolve all conflicts in this tree.'),
 
115
            option.Option('all', help='Resolve all conflicts in this tree.'),
 
116
            ResolveActionOption(),
94
117
            ]
95
 
    def run(self, file_list=None, all=False):
96
 
        from bzrlib.workingtree import WorkingTree
 
118
    _see_also = ['conflicts']
 
119
    def run(self, file_list=None, all=False, action=None):
97
120
        if all:
98
121
            if file_list:
99
122
                raise errors.BzrCommandError("If --all is specified,"
100
123
                                             " no FILE may be provided")
101
 
            tree = WorkingTree.open_containing('.')[0]
102
 
            resolve(tree)
 
124
            tree = workingtree.WorkingTree.open_containing('.')[0]
 
125
            if action is None:
 
126
                action = 'done'
103
127
        else:
104
128
            tree, file_list = builtins.tree_files(file_list)
105
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:
106
142
                un_resolved, resolved = tree.auto_resolve()
107
143
                if len(un_resolved) > 0:
108
144
                    trace.note('%d conflict(s) auto-resolved.', len(resolved))
114
150
                    trace.note('All conflicts resolved.')
115
151
                    return 0
116
152
            else:
117
 
                resolve(tree, file_list)
118
 
 
119
 
 
120
 
def resolve(tree, paths=None, ignore_misses=False, recursive=False):
 
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'):
121
163
    """Resolve some or all of the conflicts in a working tree.
122
164
 
123
165
    :param paths: If None, resolve all conflicts.  Otherwise, select only
127
169
        recursive commands like revert, this should be True.  For commands
128
170
        or applications wishing finer-grained control, like the resolve
129
171
        command, this should be False.
130
 
    :ignore_misses: If False, warnings will be printed if the supplied paths
131
 
        do not have conflicts.
 
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,
132
175
    """
133
176
    tree.lock_tree_write()
134
177
    try:
135
178
        tree_conflicts = tree.conflicts()
136
179
        if paths is None:
137
180
            new_conflicts = ConflictList()
138
 
            selected_conflicts = tree_conflicts
 
181
            to_process = tree_conflicts
139
182
        else:
140
 
            new_conflicts, selected_conflicts = \
141
 
                tree_conflicts.select_conflicts(tree, paths, ignore_misses,
142
 
                    recursive)
 
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)
143
191
        try:
144
192
            tree.set_conflicts(new_conflicts)
145
193
        except errors.UnsupportedOperation:
146
194
            pass
147
 
        selected_conflicts.remove_files(tree)
148
195
    finally:
149
196
        tree.unlock()
150
197
 
151
198
 
152
199
def restore(filename):
153
 
    """\
154
 
    Restore a conflicted file to the state it was in before merging.
155
 
    Only text restoration supported at present.
 
200
    """Restore a conflicted file to the state it was in before merging.
 
201
 
 
202
    Only text restoration is supported at present.
156
203
    """
157
204
    conflicted = False
158
205
    try:
239
286
        for conflict in self:
240
287
            if not conflict.has_files:
241
288
                continue
242
 
            for suffix in CONFLICT_SUFFIXES:
243
 
                try:
244
 
                    osutils.delete_any(tree.abspath(conflict.path+suffix))
245
 
                except OSError, e:
246
 
                    if e.errno != errno.ENOENT:
247
 
                        raise
 
289
            conflict.cleanup(tree)
248
290
 
249
291
    def select_conflicts(self, tree, paths, ignore_misses=False,
250
292
                         recurse=False):
303
345
class Conflict(object):
304
346
    """Base class for all types of conflict"""
305
347
 
 
348
    # FIXME: cleanup should take care of that ? -- vila 091229
306
349
    has_files = False
307
350
 
308
351
    def __init__(self, path, file_id=None):
357
400
        else:
358
401
            return None, conflict.typestring
359
402
 
 
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 associated_filenames(self):
 
416
        """The names of the files generated to help resolve the conflict."""
 
417
        raise NotImplementedError(self.associated_filenames)
 
418
 
 
419
    def cleanup(self, tree):
 
420
        for fname in self.associated_filenames():
 
421
            try:
 
422
                osutils.delete_any(tree.abspath(fname))
 
423
            except OSError, e:
 
424
                if e.errno != errno.ENOENT:
 
425
                    raise
 
426
 
 
427
    def action_done(self, tree):
 
428
        """Mark the conflict as solved once it has been handled."""
 
429
        # This method does nothing but simplifies the design of upper levels.
 
430
        pass
 
431
 
 
432
    def action_take_this(self, tree):
 
433
        raise NotImplementedError(self.action_take_this)
 
434
 
 
435
    def action_take_other(self, tree):
 
436
        raise NotImplementedError(self.action_take_other)
 
437
 
 
438
    def _resolve_with_cleanups(self, tree, *args, **kwargs):
 
439
        tt = transform.TreeTransform(tree)
 
440
        op = cleanup.OperationWithCleanups(self._resolve)
 
441
        op.add_cleanup(tt.finalize)
 
442
        op.run_simple(tt, *args, **kwargs)
 
443
 
360
444
 
361
445
class PathConflict(Conflict):
362
446
    """A conflict was encountered merging file paths"""
366
450
    format = 'Path conflict: %(path)s / %(conflict_path)s'
367
451
 
368
452
    rformat = '%(class)s(%(path)r, %(conflict_path)r, %(file_id)r)'
 
453
 
369
454
    def __init__(self, path, conflict_path=None, file_id=None):
370
455
        Conflict.__init__(self, path, file_id)
371
456
        self.conflict_path = conflict_path
376
461
            s.add('conflict_path', self.conflict_path)
377
462
        return s
378
463
 
 
464
    def associated_filenames(self):
 
465
        # No additional files have been generated here
 
466
        return []
 
467
 
 
468
    def _resolve(self, tt, file_id, path, winner):
 
469
        """Resolve the conflict.
 
470
 
 
471
        :param tt: The TreeTransform where the conflict is resolved.
 
472
        :param file_id: The retained file id.
 
473
        :param path: The retained path.
 
474
        :param winner: 'this' or 'other' indicates which side is the winner.
 
475
        """
 
476
        path_to_create = None
 
477
        if winner == 'this':
 
478
            if self.path == '<deleted>':
 
479
                return # Nothing to do
 
480
            if self.conflict_path == '<deleted>':
 
481
                path_to_create = self.path
 
482
                revid = tt._tree.get_parent_ids()[0]
 
483
        elif winner == 'other':
 
484
            if self.conflict_path == '<deleted>':
 
485
                return  # Nothing to do
 
486
            if self.path == '<deleted>':
 
487
                path_to_create = self.conflict_path
 
488
                # FIXME: If there are more than two parents we may need to
 
489
                # iterate. Taking the last parent is the safer bet in the mean
 
490
                # time. -- vila 20100309
 
491
                revid = tt._tree.get_parent_ids()[-1]
 
492
        else:
 
493
            # Programmer error
 
494
            raise AssertionError('bad winner: %r' % (winner,))
 
495
        if path_to_create is not None:
 
496
            tid = tt.trans_id_tree_path(path_to_create)
 
497
            transform.create_from_tree(
 
498
                tt, tt.trans_id_tree_path(path_to_create),
 
499
                self._revision_tree(tt._tree, revid), file_id)
 
500
            tt.version_file(file_id, tid)
 
501
 
 
502
        # Adjust the path for the retained file id
 
503
        tid = tt.trans_id_file_id(file_id)
 
504
        parent_tid = tt.get_tree_parent(tid)
 
505
        tt.adjust_path(path, parent_tid, tid)
 
506
        tt.apply()
 
507
 
 
508
    def _revision_tree(self, tree, revid):
 
509
        return tree.branch.repository.revision_tree(revid)
 
510
 
 
511
    def _infer_file_id(self, tree):
 
512
        # Prior to bug #531967, file_id wasn't always set, there may still be
 
513
        # conflict files in the wild so we need to cope with them
 
514
        # Establish which path we should use to find back the file-id
 
515
        possible_paths = []
 
516
        for p in (self.path, self.conflict_path):
 
517
            if p == '<deleted>':
 
518
                # special hard-coded path 
 
519
                continue
 
520
            if p is not None:
 
521
                possible_paths.append(p)
 
522
        # Search the file-id in the parents with any path available
 
523
        file_id = None
 
524
        for revid in tree.get_parent_ids():
 
525
            revtree = self._revision_tree(tree, revid)
 
526
            for p in possible_paths:
 
527
                file_id = revtree.path2id(p)
 
528
                if file_id is not None:
 
529
                    return revtree, file_id
 
530
        return None, None
 
531
 
 
532
    def action_take_this(self, tree):
 
533
        if self.file_id is not None:
 
534
            self._resolve_with_cleanups(tree, self.file_id, self.path,
 
535
                                        winner='this')
 
536
        else:
 
537
            # Prior to bug #531967 we need to find back the file_id and restore
 
538
            # the content from there
 
539
            revtree, file_id = self._infer_file_id(tree)
 
540
            tree.revert([revtree.id2path(file_id)],
 
541
                        old_tree=revtree, backups=False)
 
542
 
 
543
    def action_take_other(self, tree):
 
544
        if self.file_id is not None:
 
545
            self._resolve_with_cleanups(tree, self.file_id,
 
546
                                        self.conflict_path,
 
547
                                        winner='other')
 
548
        else:
 
549
            # Prior to bug #531967 we need to find back the file_id and restore
 
550
            # the content from there
 
551
            revtree, file_id = self._infer_file_id(tree)
 
552
            tree.revert([revtree.id2path(file_id)],
 
553
                        old_tree=revtree, backups=False)
 
554
 
379
555
 
380
556
class ContentsConflict(PathConflict):
381
 
    """The files are of different types, or not present"""
 
557
    """The files are of different types (or both binary), or not present"""
382
558
 
383
559
    has_files = True
384
560
 
386
562
 
387
563
    format = 'Contents conflict in %(path)s'
388
564
 
389
 
 
 
565
    def associated_filenames(self):
 
566
        return [self.path + suffix for suffix in ('.BASE', '.OTHER')]
 
567
 
 
568
    def _resolve(self, tt, suffix_to_remove):
 
569
        """Resolve the conflict.
 
570
 
 
571
        :param tt: The TreeTransform where the conflict is resolved.
 
572
        :param suffix_to_remove: Either 'THIS' or 'OTHER'
 
573
 
 
574
        The resolution is symmetric, when taking THIS, OTHER is deleted and
 
575
        item.THIS is renamed into item and vice-versa.
 
576
        """
 
577
        try:
 
578
            # Delete 'item.THIS' or 'item.OTHER' depending on
 
579
            # suffix_to_remove
 
580
            tt.delete_contents(
 
581
                tt.trans_id_tree_path(self.path + '.' + suffix_to_remove))
 
582
        except errors.NoSuchFile:
 
583
            # There are valid cases where 'item.suffix_to_remove' either
 
584
            # never existed or was already deleted (including the case
 
585
            # where the user deleted it)
 
586
            pass
 
587
        # Rename 'item.suffix_to_remove' (note that if
 
588
        # 'item.suffix_to_remove' has been deleted, this is a no-op)
 
589
        this_tid = tt.trans_id_file_id(self.file_id)
 
590
        parent_tid = tt.get_tree_parent(this_tid)
 
591
        tt.adjust_path(self.path, parent_tid, this_tid)
 
592
        tt.apply()
 
593
 
 
594
    def action_take_this(self, tree):
 
595
        self._resolve_with_cleanups(tree, 'OTHER')
 
596
 
 
597
    def action_take_other(self, tree):
 
598
        self._resolve_with_cleanups(tree, 'THIS')
 
599
 
 
600
 
 
601
# FIXME: TextConflict is about a single file-id, there never is a conflict_path
 
602
# attribute so we shouldn't inherit from PathConflict but simply from Conflict
 
603
 
 
604
# TODO: There should be a base revid attribute to better inform the user about
 
605
# how the conflicts were generated.
390
606
class TextConflict(PathConflict):
391
607
    """The merge algorithm could not resolve all differences encountered."""
392
608
 
396
612
 
397
613
    format = 'Text conflict in %(path)s'
398
614
 
 
615
    def associated_filenames(self):
 
616
        return [self.path + suffix for suffix in CONFLICT_SUFFIXES]
 
617
 
399
618
 
400
619
class HandledConflict(Conflict):
401
620
    """A path problem that has been provisionally resolved.
416
635
        s.add('action', self.action)
417
636
        return s
418
637
 
 
638
    def associated_filenames(self):
 
639
        # Nothing has been generated here
 
640
        return []
 
641
 
419
642
 
420
643
class HandledPathConflict(HandledConflict):
421
644
    """A provisionally-resolved path problem involving two paths.
462
685
 
463
686
    format = 'Conflict adding file %(conflict_path)s.  %(action)s %(path)s.'
464
687
 
 
688
    def action_take_this(self, tree):
 
689
        tree.remove([self.conflict_path], force=True, keep_files=False)
 
690
        tree.rename_one(self.path, self.conflict_path)
 
691
 
 
692
    def action_take_other(self, tree):
 
693
        tree.remove([self.path], force=True, keep_files=False)
 
694
 
465
695
 
466
696
class ParentLoop(HandledPathConflict):
467
697
    """An attempt to create an infinitely-looping directory structure.
468
698
    This is rare, but can be produced like so:
469
699
 
470
700
    tree A:
471
 
      mv foo/bar
 
701
      mv foo bar
472
702
    tree B:
473
 
      mv bar/foo
 
703
      mv bar foo
474
704
    merge A and B
475
705
    """
476
706
 
477
707
    typestring = 'parent loop'
478
708
 
479
 
    format = 'Conflict moving %(conflict_path)s into %(path)s.  %(action)s.'
 
709
    format = 'Conflict moving %(path)s into %(conflict_path)s. %(action)s.'
 
710
 
 
711
    def action_take_this(self, tree):
 
712
        # just acccept bzr proposal
 
713
        pass
 
714
 
 
715
    def action_take_other(self, tree):
 
716
        # FIXME: We shouldn't have to manipulate so many paths here (and there
 
717
        # is probably a bug or two...)
 
718
        base_path = osutils.basename(self.path)
 
719
        conflict_base_path = osutils.basename(self.conflict_path)
 
720
        tt = transform.TreeTransform(tree)
 
721
        try:
 
722
            p_tid = tt.trans_id_file_id(self.file_id)
 
723
            parent_tid = tt.get_tree_parent(p_tid)
 
724
            cp_tid = tt.trans_id_file_id(self.conflict_file_id)
 
725
            cparent_tid = tt.get_tree_parent(cp_tid)
 
726
            tt.adjust_path(base_path, cparent_tid, cp_tid)
 
727
            tt.adjust_path(conflict_base_path, parent_tid, p_tid)
 
728
            tt.apply()
 
729
        finally:
 
730
            tt.finalize()
480
731
 
481
732
 
482
733
class UnversionedParent(HandledConflict):
483
 
    """An attempt to version an file whose parent directory is not versioned.
 
734
    """An attempt to version a file whose parent directory is not versioned.
484
735
    Typically, the result of a merge where one tree unversioned the directory
485
736
    and the other added a versioned file to it.
486
737
    """
490
741
    format = 'Conflict because %(path)s is not versioned, but has versioned'\
491
742
             ' children.  %(action)s.'
492
743
 
 
744
    # FIXME: We silently do nothing to make tests pass, but most probably the
 
745
    # conflict shouldn't exist (the long story is that the conflict is
 
746
    # generated with another one that can be resolved properly) -- vila 091224
 
747
    def action_take_this(self, tree):
 
748
        pass
 
749
 
 
750
    def action_take_other(self, tree):
 
751
        pass
 
752
 
493
753
 
494
754
class MissingParent(HandledConflict):
495
755
    """An attempt to add files to a directory that is not present.
496
756
    Typically, the result of a merge where THIS deleted the directory and
497
757
    the OTHER added a file to it.
498
 
    See also: DeletingParent (same situation, reversed THIS and OTHER)
 
758
    See also: DeletingParent (same situation, THIS and OTHER reversed)
499
759
    """
500
760
 
501
761
    typestring = 'missing parent'
502
762
 
503
763
    format = 'Conflict adding files to %(path)s.  %(action)s.'
504
764
 
 
765
    def action_take_this(self, tree):
 
766
        tree.remove([self.path], force=True, keep_files=False)
 
767
 
 
768
    def action_take_other(self, tree):
 
769
        # just acccept bzr proposal
 
770
        pass
 
771
 
505
772
 
506
773
class DeletingParent(HandledConflict):
507
774
    """An attempt to add files to a directory that is not present.
514
781
    format = "Conflict: can't delete %(path)s because it is not empty.  "\
515
782
             "%(action)s."
516
783
 
 
784
    # FIXME: It's a bit strange that the default action is not coherent with
 
785
    # MissingParent from the *user* pov.
 
786
 
 
787
    def action_take_this(self, tree):
 
788
        # just acccept bzr proposal
 
789
        pass
 
790
 
 
791
    def action_take_other(self, tree):
 
792
        tree.remove([self.path], force=True, keep_files=False)
 
793
 
517
794
 
518
795
class NonDirectoryParent(HandledConflict):
519
 
    """An attempt to add files to a directory that is not a director or
 
796
    """An attempt to add files to a directory that is not a directory or
520
797
    an attempt to change the kind of a directory with files.
521
798
    """
522
799
 
525
802
    format = "Conflict: %(path)s is not a directory, but has files in it."\
526
803
             "  %(action)s."
527
804
 
 
805
    # FIXME: .OTHER should be used instead of .new when the conflict is created
 
806
 
 
807
    def action_take_this(self, tree):
 
808
        # FIXME: we should preserve that path when the conflict is generated !
 
809
        if self.path.endswith('.new'):
 
810
            conflict_path = self.path[:-(len('.new'))]
 
811
            tree.remove([self.path], force=True, keep_files=False)
 
812
            tree.add(conflict_path)
 
813
        else:
 
814
            raise NotImplementedError(self.action_take_this)
 
815
 
 
816
    def action_take_other(self, tree):
 
817
        # FIXME: we should preserve that path when the conflict is generated !
 
818
        if self.path.endswith('.new'):
 
819
            conflict_path = self.path[:-(len('.new'))]
 
820
            tree.remove([conflict_path], force=True, keep_files=False)
 
821
            tree.rename_one(self.path, conflict_path)
 
822
        else:
 
823
            raise NotImplementedError(self.action_take_other)
 
824
 
 
825
 
528
826
ctype = {}
529
827
 
530
828
 
534
832
    for conflict_type in conflict_types:
535
833
        ctype[conflict_type.typestring] = conflict_type
536
834
 
537
 
 
538
835
register_types(ContentsConflict, TextConflict, PathConflict, DuplicateID,
539
836
               DuplicateEntry, ParentLoop, UnversionedParent, MissingParent,
540
837
               DeletingParent, NonDirectoryParent)