← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/conflicts.py

  • Committer: John Arbash Meinel
  • Date: 2010-01-05 04:30:07 UTC
  • mfrom: (4932 +trunk)
  • mto: This revision was merged to the branch mainline in revision 4934.
  • Revision ID: john@arbash-meinel.com-20100105043007-ehgbldqd3q0gtyws
Merge bzr.dev, resolve conflicts.

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/conflicts.py
1
 
# Copyright (C) 2005, 2006, 2007, 2009, 2010, 2011 Canonical Ltd
 
1
# Copyright (C) 2005, 2007 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
 
17
19
# TODO: 'bzr resolve' should accept a directory name and work from that
18
20
# point down
19
21
 
24
26
import errno
25
27
 
26
28
from bzrlib import (
27
 
    cleanup,
 
29
    builtins,
 
30
    commands,
28
31
    errors,
29
32
    osutils,
30
33
    rio,
31
34
    trace,
32
 
    transform,
33
 
    workingtree,
34
35
    )
35
 
from bzrlib.i18n import gettext, ngettext
36
36
""")
37
 
from bzrlib import (
38
 
    commands,
39
 
    option,
40
 
    registry,
41
 
    )
 
37
from bzrlib.option import Option
42
38
 
43
39
 
44
40
CONFLICT_SUFFIXES = ('.THIS', '.BASE', '.OTHER')
45
41
 
46
42
 
47
43
class cmd_conflicts(commands.Command):
48
 
    __doc__ = """List files with conflicts.
 
44
    """List files with conflicts.
49
45
 
50
46
    Merge will do its best to combine the changes in two branches, but there
51
47
    are some kinds of problems only a human can fix.  When it encounters those,
59
55
    Use bzr resolve when you have fixed a problem.
60
56
    """
61
57
    takes_options = [
62
 
            'directory',
63
 
            option.Option('text',
64
 
                          help='List paths of files with text conflicts.'),
 
58
            Option('text',
 
59
                   help='List paths of files with text conflicts.'),
65
60
        ]
66
 
    _see_also = ['resolve', 'conflict-types']
 
61
    _see_also = ['resolve']
67
62
 
68
 
    def run(self, text=False, directory=u'.'):
69
 
        wt = workingtree.WorkingTree.open_containing(directory)[0]
 
63
    def run(self, text=False):
 
64
        from bzrlib.workingtree import WorkingTree
 
65
        wt = WorkingTree.open_containing(u'.')[0]
70
66
        for conflict in wt.conflicts():
71
67
            if text:
72
68
                if conflict.typestring != 'text conflict':
73
69
                    continue
74
70
                self.outf.write(conflict.path + '\n')
75
71
            else:
76
 
                self.outf.write(unicode(conflict) + '\n')
77
 
 
78
 
 
79
 
resolve_action_registry = registry.Registry()
80
 
 
81
 
 
82
 
resolve_action_registry.register(
83
 
    'done', 'done', 'Marks the conflict as resolved' )
84
 
resolve_action_registry.register(
85
 
    'take-this', 'take_this',
86
 
    'Resolve the conflict preserving the version in the working tree' )
87
 
resolve_action_registry.register(
88
 
    'take-other', 'take_other',
89
 
    'Resolve the conflict taking the merged version into account' )
90
 
resolve_action_registry.default_key = 'done'
91
 
 
92
 
class ResolveActionOption(option.RegistryOption):
93
 
 
94
 
    def __init__(self):
95
 
        super(ResolveActionOption, self).__init__(
96
 
            'action', 'How to resolve the conflict.',
97
 
            value_switches=True,
98
 
            registry=resolve_action_registry)
 
72
                self.outf.write(str(conflict) + '\n')
99
73
 
100
74
 
101
75
class cmd_resolve(commands.Command):
102
 
    __doc__ = """Mark a conflict as resolved.
 
76
    """Mark a conflict as resolved.
103
77
 
104
78
    Merge will do its best to combine the changes in two branches, but there
105
79
    are some kinds of problems only a human can fix.  When it encounters those,
113
87
    aliases = ['resolved']
114
88
    takes_args = ['file*']
115
89
    takes_options = [
116
 
            'directory',
117
 
            option.Option('all', help='Resolve all conflicts in this tree.'),
118
 
            ResolveActionOption(),
 
90
            Option('all', help='Resolve all conflicts in this tree.'),
119
91
            ]
120
92
    _see_also = ['conflicts']
121
 
    def run(self, file_list=None, all=False, action=None, directory=None):
 
93
    def run(self, file_list=None, all=False):
 
94
        from bzrlib.workingtree import WorkingTree
122
95
        if all:
123
96
            if file_list:
124
 
                raise errors.BzrCommandError(gettext("If --all is specified,"
125
 
                                             " no FILE may be provided"))
126
 
            if directory is None:
127
 
                directory = u'.'
128
 
            tree = workingtree.WorkingTree.open_containing(directory)[0]
129
 
            if action is None:
130
 
                action = 'done'
 
97
                raise errors.BzrCommandError("If --all is specified,"
 
98
                                             " no FILE may be provided")
 
99
            tree = WorkingTree.open_containing('.')[0]
 
100
            resolve(tree)
131
101
        else:
132
 
            tree, file_list = workingtree.WorkingTree.open_containing_paths(
133
 
                file_list, directory)
134
 
            if file_list is None:
135
 
                if action is None:
136
 
                    # FIXME: There is a special case here related to the option
137
 
                    # handling that could be clearer and easier to discover by
138
 
                    # providing an --auto action (bug #344013 and #383396) and
139
 
                    # make it mandatory instead of implicit and active only
140
 
                    # when no file_list is provided -- vila 091229
141
 
                    action = 'auto'
142
 
            else:
143
 
                if action is None:
144
 
                    action = 'done'
145
 
        if action == 'auto':
 
102
            tree, file_list = builtins.tree_files(file_list)
146
103
            if file_list is None:
147
104
                un_resolved, resolved = tree.auto_resolve()
148
105
                if len(un_resolved) > 0:
149
 
                    trace.note(ngettext('%d conflict auto-resolved.',
150
 
                        '%d conflicts auto-resolved.', len(resolved)),
151
 
                        len(resolved))
152
 
                    trace.note(gettext('Remaining conflicts:'))
 
106
                    trace.note('%d conflict(s) auto-resolved.', len(resolved))
 
107
                    trace.note('Remaining conflicts:')
153
108
                    for conflict in un_resolved:
154
 
                        trace.note(unicode(conflict))
 
109
                        trace.note(conflict)
155
110
                    return 1
156
111
                else:
157
 
                    trace.note(gettext('All conflicts resolved.'))
 
112
                    trace.note('All conflicts resolved.')
158
113
                    return 0
159
114
            else:
160
 
                # FIXME: This can never occur but the block above needs some
161
 
                # refactoring to transfer tree.auto_resolve() to
162
 
                # conflict.auto(tree) --vila 091242
163
 
                pass
164
 
        else:
165
 
            before, after = resolve(tree, file_list, action=action)
166
 
            trace.note(ngettext('{0} conflict resolved, {1} remaining',
167
 
                                '{0} conflicts resolved, {1} remaining',
168
 
                                before-after).format(before - after, after))
169
 
 
170
 
 
171
 
def resolve(tree, paths=None, ignore_misses=False, recursive=False,
172
 
            action='done'):
 
115
                resolve(tree, file_list)
 
116
 
 
117
 
 
118
def resolve(tree, paths=None, ignore_misses=False, recursive=False):
173
119
    """Resolve some or all of the conflicts in a working tree.
174
120
 
175
121
    :param paths: If None, resolve all conflicts.  Otherwise, select only
179
125
        recursive commands like revert, this should be True.  For commands
180
126
        or applications wishing finer-grained control, like the resolve
181
127
        command, this should be False.
182
 
    :param ignore_misses: If False, warnings will be printed if the supplied
183
 
        paths do not have conflicts.
184
 
    :param action: How the conflict should be resolved,
 
128
    :ignore_misses: If False, warnings will be printed if the supplied paths
 
129
        do not have conflicts.
185
130
    """
186
131
    tree.lock_tree_write()
187
 
    nb_conflicts_after = None
188
132
    try:
189
133
        tree_conflicts = tree.conflicts()
190
 
        nb_conflicts_before = len(tree_conflicts)
191
134
        if paths is None:
192
135
            new_conflicts = ConflictList()
193
 
            to_process = tree_conflicts
 
136
            selected_conflicts = tree_conflicts
194
137
        else:
195
 
            new_conflicts, to_process = tree_conflicts.select_conflicts(
196
 
                tree, paths, ignore_misses, recursive)
197
 
        for conflict in to_process:
198
 
            try:
199
 
                conflict._do(action, tree)
200
 
                conflict.cleanup(tree)
201
 
            except NotImplementedError:
202
 
                new_conflicts.append(conflict)
 
138
            new_conflicts, selected_conflicts = \
 
139
                tree_conflicts.select_conflicts(tree, paths, ignore_misses,
 
140
                    recursive)
203
141
        try:
204
 
            nb_conflicts_after = len(new_conflicts)
205
142
            tree.set_conflicts(new_conflicts)
206
143
        except errors.UnsupportedOperation:
207
144
            pass
 
145
        selected_conflicts.remove_files(tree)
208
146
    finally:
209
147
        tree.unlock()
210
 
    if nb_conflicts_after is None:
211
 
        nb_conflicts_after = nb_conflicts_before
212
 
    return nb_conflicts_before, nb_conflicts_after
213
148
 
214
149
 
215
150
def restore(filename):
295
230
    def to_strings(self):
296
231
        """Generate strings for the provided conflicts"""
297
232
        for conflict in self:
298
 
            yield unicode(conflict)
 
233
            yield str(conflict)
299
234
 
300
235
    def remove_files(self, tree):
301
236
        """Remove the THIS, BASE and OTHER files for listed conflicts"""
302
237
        for conflict in self:
303
238
            if not conflict.has_files:
304
239
                continue
305
 
            conflict.cleanup(tree)
 
240
            for suffix in CONFLICT_SUFFIXES:
 
241
                try:
 
242
                    osutils.delete_any(tree.abspath(conflict.path+suffix))
 
243
                except OSError, e:
 
244
                    if e.errno != errno.ENOENT:
 
245
                        raise
306
246
 
307
247
    def select_conflicts(self, tree, paths, ignore_misses=False,
308
248
                         recurse=False):
361
301
class Conflict(object):
362
302
    """Base class for all types of conflict"""
363
303
 
364
 
    # FIXME: cleanup should take care of that ? -- vila 091229
365
304
    has_files = False
366
305
 
367
306
    def __init__(self, path, file_id=None):
394
333
    def __ne__(self, other):
395
334
        return not self.__eq__(other)
396
335
 
397
 
    def __unicode__(self):
 
336
    def __str__(self):
398
337
        return self.format % self.__dict__
399
338
 
400
339
    def __repr__(self):
416
355
        else:
417
356
            return None, conflict.typestring
418
357
 
419
 
    def _do(self, action, tree):
420
 
        """Apply the specified action to the conflict.
421
 
 
422
 
        :param action: The method name to call.
423
 
 
424
 
        :param tree: The tree passed as a parameter to the method.
425
 
        """
426
 
        meth = getattr(self, 'action_%s' % action, None)
427
 
        if meth is None:
428
 
            raise NotImplementedError(self.__class__.__name__ + '.' + action)
429
 
        meth(tree)
430
 
 
431
 
    def associated_filenames(self):
432
 
        """The names of the files generated to help resolve the conflict."""
433
 
        raise NotImplementedError(self.associated_filenames)
434
 
 
435
 
    def cleanup(self, tree):
436
 
        for fname in self.associated_filenames():
437
 
            try:
438
 
                osutils.delete_any(tree.abspath(fname))
439
 
            except OSError, e:
440
 
                if e.errno != errno.ENOENT:
441
 
                    raise
442
 
 
443
 
    def action_done(self, tree):
444
 
        """Mark the conflict as solved once it has been handled."""
445
 
        # This method does nothing but simplifies the design of upper levels.
446
 
        pass
447
 
 
448
 
    def action_take_this(self, tree):
449
 
        raise NotImplementedError(self.action_take_this)
450
 
 
451
 
    def action_take_other(self, tree):
452
 
        raise NotImplementedError(self.action_take_other)
453
 
 
454
 
    def _resolve_with_cleanups(self, tree, *args, **kwargs):
455
 
        tt = transform.TreeTransform(tree)
456
 
        op = cleanup.OperationWithCleanups(self._resolve)
457
 
        op.add_cleanup(tt.finalize)
458
 
        op.run_simple(tt, *args, **kwargs)
459
 
 
460
358
 
461
359
class PathConflict(Conflict):
462
360
    """A conflict was encountered merging file paths"""
466
364
    format = 'Path conflict: %(path)s / %(conflict_path)s'
467
365
 
468
366
    rformat = '%(class)s(%(path)r, %(conflict_path)r, %(file_id)r)'
469
 
 
470
367
    def __init__(self, path, conflict_path=None, file_id=None):
471
368
        Conflict.__init__(self, path, file_id)
472
369
        self.conflict_path = conflict_path
477
374
            s.add('conflict_path', self.conflict_path)
478
375
        return s
479
376
 
480
 
    def associated_filenames(self):
481
 
        # No additional files have been generated here
482
 
        return []
483
 
 
484
 
    def _resolve(self, tt, file_id, path, winner):
485
 
        """Resolve the conflict.
486
 
 
487
 
        :param tt: The TreeTransform where the conflict is resolved.
488
 
        :param file_id: The retained file id.
489
 
        :param path: The retained path.
490
 
        :param winner: 'this' or 'other' indicates which side is the winner.
491
 
        """
492
 
        path_to_create = None
493
 
        if winner == 'this':
494
 
            if self.path == '<deleted>':
495
 
                return # Nothing to do
496
 
            if self.conflict_path == '<deleted>':
497
 
                path_to_create = self.path
498
 
                revid = tt._tree.get_parent_ids()[0]
499
 
        elif winner == 'other':
500
 
            if self.conflict_path == '<deleted>':
501
 
                return  # Nothing to do
502
 
            if self.path == '<deleted>':
503
 
                path_to_create = self.conflict_path
504
 
                # FIXME: If there are more than two parents we may need to
505
 
                # iterate. Taking the last parent is the safer bet in the mean
506
 
                # time. -- vila 20100309
507
 
                revid = tt._tree.get_parent_ids()[-1]
508
 
        else:
509
 
            # Programmer error
510
 
            raise AssertionError('bad winner: %r' % (winner,))
511
 
        if path_to_create is not None:
512
 
            tid = tt.trans_id_tree_path(path_to_create)
513
 
            transform.create_from_tree(
514
 
                tt, tid, self._revision_tree(tt._tree, revid), file_id)
515
 
            tt.version_file(file_id, tid)
516
 
        else:
517
 
            tid = tt.trans_id_file_id(file_id)
518
 
        # Adjust the path for the retained file id
519
 
        parent_tid = tt.get_tree_parent(tid)
520
 
        tt.adjust_path(osutils.basename(path), parent_tid, tid)
521
 
        tt.apply()
522
 
 
523
 
    def _revision_tree(self, tree, revid):
524
 
        return tree.branch.repository.revision_tree(revid)
525
 
 
526
 
    def _infer_file_id(self, tree):
527
 
        # Prior to bug #531967, file_id wasn't always set, there may still be
528
 
        # conflict files in the wild so we need to cope with them
529
 
        # Establish which path we should use to find back the file-id
530
 
        possible_paths = []
531
 
        for p in (self.path, self.conflict_path):
532
 
            if p == '<deleted>':
533
 
                # special hard-coded path 
534
 
                continue
535
 
            if p is not None:
536
 
                possible_paths.append(p)
537
 
        # Search the file-id in the parents with any path available
538
 
        file_id = None
539
 
        for revid in tree.get_parent_ids():
540
 
            revtree = self._revision_tree(tree, revid)
541
 
            for p in possible_paths:
542
 
                file_id = revtree.path2id(p)
543
 
                if file_id is not None:
544
 
                    return revtree, file_id
545
 
        return None, None
546
 
 
547
 
    def action_take_this(self, tree):
548
 
        if self.file_id is not None:
549
 
            self._resolve_with_cleanups(tree, self.file_id, self.path,
550
 
                                        winner='this')
551
 
        else:
552
 
            # Prior to bug #531967 we need to find back the file_id and restore
553
 
            # the content from there
554
 
            revtree, file_id = self._infer_file_id(tree)
555
 
            tree.revert([revtree.id2path(file_id)],
556
 
                        old_tree=revtree, backups=False)
557
 
 
558
 
    def action_take_other(self, tree):
559
 
        if self.file_id is not None:
560
 
            self._resolve_with_cleanups(tree, self.file_id,
561
 
                                        self.conflict_path,
562
 
                                        winner='other')
563
 
        else:
564
 
            # Prior to bug #531967 we need to find back the file_id and restore
565
 
            # the content from there
566
 
            revtree, file_id = self._infer_file_id(tree)
567
 
            tree.revert([revtree.id2path(file_id)],
568
 
                        old_tree=revtree, backups=False)
569
 
 
570
377
 
571
378
class ContentsConflict(PathConflict):
572
 
    """The files are of different types (or both binary), or not present"""
 
379
    """The files are of different types, or not present"""
573
380
 
574
381
    has_files = True
575
382
 
577
384
 
578
385
    format = 'Contents conflict in %(path)s'
579
386
 
580
 
    def associated_filenames(self):
581
 
        return [self.path + suffix for suffix in ('.BASE', '.OTHER')]
582
 
 
583
 
    def _resolve(self, tt, suffix_to_remove):
584
 
        """Resolve the conflict.
585
 
 
586
 
        :param tt: The TreeTransform where the conflict is resolved.
587
 
        :param suffix_to_remove: Either 'THIS' or 'OTHER'
588
 
 
589
 
        The resolution is symmetric: when taking THIS, OTHER is deleted and
590
 
        item.THIS is renamed into item and vice-versa.
591
 
        """
592
 
        try:
593
 
            # Delete 'item.THIS' or 'item.OTHER' depending on
594
 
            # suffix_to_remove
595
 
            tt.delete_contents(
596
 
                tt.trans_id_tree_path(self.path + '.' + suffix_to_remove))
597
 
        except errors.NoSuchFile:
598
 
            # There are valid cases where 'item.suffix_to_remove' either
599
 
            # never existed or was already deleted (including the case
600
 
            # where the user deleted it)
601
 
            pass
602
 
        try:
603
 
            this_path = tt._tree.id2path(self.file_id)
604
 
        except errors.NoSuchId:
605
 
            # The file is not present anymore. This may happen if the user
606
 
            # deleted the file either manually or when resolving a conflict on
607
 
            # the parent.  We may raise some exception to indicate that the
608
 
            # conflict doesn't exist anymore and as such doesn't need to be
609
 
            # resolved ? -- vila 20110615 
610
 
            this_tid = None
611
 
        else:
612
 
            this_tid = tt.trans_id_tree_path(this_path)
613
 
        if this_tid is not None:
614
 
            # Rename 'item.suffix_to_remove' (note that if
615
 
            # 'item.suffix_to_remove' has been deleted, this is a no-op)
616
 
            parent_tid = tt.get_tree_parent(this_tid)
617
 
            tt.adjust_path(osutils.basename(self.path), parent_tid, this_tid)
618
 
            tt.apply()
619
 
 
620
 
    def action_take_this(self, tree):
621
 
        self._resolve_with_cleanups(tree, 'OTHER')
622
 
 
623
 
    def action_take_other(self, tree):
624
 
        self._resolve_with_cleanups(tree, 'THIS')
625
 
 
626
 
 
627
 
# TODO: There should be a base revid attribute to better inform the user about
628
 
# how the conflicts were generated.
629
 
class TextConflict(Conflict):
 
387
 
 
388
class TextConflict(PathConflict):
630
389
    """The merge algorithm could not resolve all differences encountered."""
631
390
 
632
391
    has_files = True
635
394
 
636
395
    format = 'Text conflict in %(path)s'
637
396
 
638
 
    rformat = '%(class)s(%(path)r, %(file_id)r)'
639
 
 
640
 
    def associated_filenames(self):
641
 
        return [self.path + suffix for suffix in CONFLICT_SUFFIXES]
642
 
 
643
 
    def _resolve(self, tt, winner_suffix):
644
 
        """Resolve the conflict by copying one of .THIS or .OTHER into file.
645
 
 
646
 
        :param tt: The TreeTransform where the conflict is resolved.
647
 
        :param winner_suffix: Either 'THIS' or 'OTHER'
648
 
 
649
 
        The resolution is symmetric, when taking THIS, item.THIS is renamed
650
 
        into item and vice-versa. This takes one of the files as a whole
651
 
        ignoring every difference that could have been merged cleanly.
652
 
        """
653
 
        # To avoid useless copies, we switch item and item.winner_suffix, only
654
 
        # item will exist after the conflict has been resolved anyway.
655
 
        item_tid = tt.trans_id_file_id(self.file_id)
656
 
        item_parent_tid = tt.get_tree_parent(item_tid)
657
 
        winner_path = self.path + '.' + winner_suffix
658
 
        winner_tid = tt.trans_id_tree_path(winner_path)
659
 
        winner_parent_tid = tt.get_tree_parent(winner_tid)
660
 
        # Switch the paths to preserve the content
661
 
        tt.adjust_path(osutils.basename(self.path),
662
 
                       winner_parent_tid, winner_tid)
663
 
        tt.adjust_path(osutils.basename(winner_path), item_parent_tid, item_tid)
664
 
        # Associate the file_id to the right content
665
 
        tt.unversion_file(item_tid)
666
 
        tt.version_file(self.file_id, winner_tid)
667
 
        tt.apply()
668
 
 
669
 
    def action_take_this(self, tree):
670
 
        self._resolve_with_cleanups(tree, 'THIS')
671
 
 
672
 
    def action_take_other(self, tree):
673
 
        self._resolve_with_cleanups(tree, 'OTHER')
674
 
 
675
397
 
676
398
class HandledConflict(Conflict):
677
399
    """A path problem that has been provisionally resolved.
692
414
        s.add('action', self.action)
693
415
        return s
694
416
 
695
 
    def associated_filenames(self):
696
 
        # Nothing has been generated here
697
 
        return []
698
 
 
699
417
 
700
418
class HandledPathConflict(HandledConflict):
701
419
    """A provisionally-resolved path problem involving two paths.
742
460
 
743
461
    format = 'Conflict adding file %(conflict_path)s.  %(action)s %(path)s.'
744
462
 
745
 
    def action_take_this(self, tree):
746
 
        tree.remove([self.conflict_path], force=True, keep_files=False)
747
 
        tree.rename_one(self.path, self.conflict_path)
748
 
 
749
 
    def action_take_other(self, tree):
750
 
        tree.remove([self.path], force=True, keep_files=False)
751
 
 
752
463
 
753
464
class ParentLoop(HandledPathConflict):
754
465
    """An attempt to create an infinitely-looping directory structure.
755
466
    This is rare, but can be produced like so:
756
467
 
757
468
    tree A:
758
 
      mv foo bar
 
469
      mv foo/bar
759
470
    tree B:
760
 
      mv bar foo
 
471
      mv bar/foo
761
472
    merge A and B
762
473
    """
763
474
 
764
475
    typestring = 'parent loop'
765
476
 
766
 
    format = 'Conflict moving %(path)s into %(conflict_path)s. %(action)s.'
767
 
 
768
 
    def action_take_this(self, tree):
769
 
        # just acccept bzr proposal
770
 
        pass
771
 
 
772
 
    def action_take_other(self, tree):
773
 
        tt = transform.TreeTransform(tree)
774
 
        try:
775
 
            p_tid = tt.trans_id_file_id(self.file_id)
776
 
            parent_tid = tt.get_tree_parent(p_tid)
777
 
            cp_tid = tt.trans_id_file_id(self.conflict_file_id)
778
 
            cparent_tid = tt.get_tree_parent(cp_tid)
779
 
            tt.adjust_path(osutils.basename(self.path), cparent_tid, cp_tid)
780
 
            tt.adjust_path(osutils.basename(self.conflict_path),
781
 
                           parent_tid, p_tid)
782
 
            tt.apply()
783
 
        finally:
784
 
            tt.finalize()
 
477
    format = 'Conflict moving %(conflict_path)s into %(path)s.  %(action)s.'
785
478
 
786
479
 
787
480
class UnversionedParent(HandledConflict):
795
488
    format = 'Conflict because %(path)s is not versioned, but has versioned'\
796
489
             ' children.  %(action)s.'
797
490
 
798
 
    # FIXME: We silently do nothing to make tests pass, but most probably the
799
 
    # conflict shouldn't exist (the long story is that the conflict is
800
 
    # generated with another one that can be resolved properly) -- vila 091224
801
 
    def action_take_this(self, tree):
802
 
        pass
803
 
 
804
 
    def action_take_other(self, tree):
805
 
        pass
806
 
 
807
491
 
808
492
class MissingParent(HandledConflict):
809
493
    """An attempt to add files to a directory that is not present.
810
494
    Typically, the result of a merge where THIS deleted the directory and
811
495
    the OTHER added a file to it.
812
 
    See also: DeletingParent (same situation, THIS and OTHER reversed)
 
496
    See also: DeletingParent (same situation, reversed THIS and OTHER)
813
497
    """
814
498
 
815
499
    typestring = 'missing parent'
816
500
 
817
501
    format = 'Conflict adding files to %(path)s.  %(action)s.'
818
502
 
819
 
    def action_take_this(self, tree):
820
 
        tree.remove([self.path], force=True, keep_files=False)
821
 
 
822
 
    def action_take_other(self, tree):
823
 
        # just acccept bzr proposal
824
 
        pass
825
 
 
826
503
 
827
504
class DeletingParent(HandledConflict):
828
505
    """An attempt to add files to a directory that is not present.
835
512
    format = "Conflict: can't delete %(path)s because it is not empty.  "\
836
513
             "%(action)s."
837
514
 
838
 
    # FIXME: It's a bit strange that the default action is not coherent with
839
 
    # MissingParent from the *user* pov.
840
 
 
841
 
    def action_take_this(self, tree):
842
 
        # just acccept bzr proposal
843
 
        pass
844
 
 
845
 
    def action_take_other(self, tree):
846
 
        tree.remove([self.path], force=True, keep_files=False)
847
 
 
848
515
 
849
516
class NonDirectoryParent(HandledConflict):
850
 
    """An attempt to add files to a directory that is not a directory or
 
517
    """An attempt to add files to a directory that is not a director or
851
518
    an attempt to change the kind of a directory with files.
852
519
    """
853
520
 
856
523
    format = "Conflict: %(path)s is not a directory, but has files in it."\
857
524
             "  %(action)s."
858
525
 
859
 
    # FIXME: .OTHER should be used instead of .new when the conflict is created
860
 
 
861
 
    def action_take_this(self, tree):
862
 
        # FIXME: we should preserve that path when the conflict is generated !
863
 
        if self.path.endswith('.new'):
864
 
            conflict_path = self.path[:-(len('.new'))]
865
 
            tree.remove([self.path], force=True, keep_files=False)
866
 
            tree.add(conflict_path)
867
 
        else:
868
 
            raise NotImplementedError(self.action_take_this)
869
 
 
870
 
    def action_take_other(self, tree):
871
 
        # FIXME: we should preserve that path when the conflict is generated !
872
 
        if self.path.endswith('.new'):
873
 
            conflict_path = self.path[:-(len('.new'))]
874
 
            tree.remove([conflict_path], force=True, keep_files=False)
875
 
            tree.rename_one(self.path, conflict_path)
876
 
        else:
877
 
            raise NotImplementedError(self.action_take_other)
878
 
 
879
 
 
880
526
ctype = {}
881
527
 
882
528
 
886
532
    for conflict_type in conflict_types:
887
533
        ctype[conflict_type.typestring] = conflict_type
888
534
 
 
535
 
889
536
register_types(ContentsConflict, TextConflict, PathConflict, DuplicateID,
890
537
               DuplicateEntry, ParentLoop, UnversionedParent, MissingParent,
891
538
               DeletingParent, NonDirectoryParent)