~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/conflicts.py

  • Committer: Canonical.com Patch Queue Manager
  • Date: 2008-03-16 16:58:03 UTC
  • mfrom: (3224.3.1 news-typo)
  • Revision ID: pqm@pqm.ubuntu.com-20080316165803-tisoc9mpob9z544o
(Matt Nordhoff) Trivial NEWS typo fix

Show diffs side-by-side

added added

removed removed

Lines of Context:
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
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
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,
57
53
    instead.  (This is useful for editing all files with text conflicts.)
58
54
 
59
55
    Use bzr resolve when you have fixed a problem.
 
56
 
 
57
    See also bzr resolve.
60
58
    """
61
59
    takes_options = [
62
 
            'directory',
63
 
            option.Option('text',
64
 
                          help='List paths of files with text conflicts.'),
 
60
            Option('text',
 
61
                   help='List paths of files with text conflicts.'),
65
62
        ]
66
 
    _see_also = ['resolve', 'conflict-types']
67
63
 
68
 
    def run(self, text=False, directory=u'.'):
69
 
        wt = workingtree.WorkingTree.open_containing(directory)[0]
 
64
    def run(self, text=False):
 
65
        from bzrlib.workingtree import WorkingTree
 
66
        wt = WorkingTree.open_containing(u'.')[0]
70
67
        for conflict in wt.conflicts():
71
68
            if text:
72
69
                if conflict.typestring != 'text conflict':
73
70
                    continue
74
71
                self.outf.write(conflict.path + '\n')
75
72
            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)
 
73
                self.outf.write(str(conflict) + '\n')
99
74
 
100
75
 
101
76
class cmd_resolve(commands.Command):
102
 
    __doc__ = """Mark a conflict as resolved.
 
77
    """Mark a conflict as resolved.
103
78
 
104
79
    Merge will do its best to combine the changes in two branches, but there
105
80
    are some kinds of problems only a human can fix.  When it encounters those,
107
82
    before you should commit.
108
83
 
109
84
    Once you have fixed a problem, use "bzr resolve" to automatically mark
110
 
    text conflicts as fixed, "bzr resolve FILE" to mark a specific conflict as
 
85
    text conflicts as fixed, resolve FILE to mark a specific conflict as
111
86
    resolved, or "bzr resolve --all" to mark all conflicts as resolved.
 
87
 
 
88
    See also bzr conflicts.
112
89
    """
113
90
    aliases = ['resolved']
114
91
    takes_args = ['file*']
115
92
    takes_options = [
116
 
            'directory',
117
 
            option.Option('all', help='Resolve all conflicts in this tree.'),
118
 
            ResolveActionOption(),
 
93
            Option('all', help='Resolve all conflicts in this tree.'),
119
94
            ]
120
 
    _see_also = ['conflicts']
121
 
    def run(self, file_list=None, all=False, action=None, directory=None):
 
95
    def run(self, file_list=None, all=False):
 
96
        from bzrlib.workingtree import WorkingTree
122
97
        if all:
123
98
            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'
 
99
                raise errors.BzrCommandError("If --all is specified,"
 
100
                                             " no FILE may be provided")
 
101
            tree = WorkingTree.open_containing('.')[0]
 
102
            resolve(tree)
131
103
        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':
 
104
            tree, file_list = builtins.tree_files(file_list)
146
105
            if file_list is None:
147
106
                un_resolved, resolved = tree.auto_resolve()
148
107
                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:'))
 
108
                    trace.note('%d conflict(s) auto-resolved.', len(resolved))
 
109
                    trace.note('Remaining conflicts:')
153
110
                    for conflict in un_resolved:
154
 
                        trace.note(unicode(conflict))
 
111
                        trace.note(conflict)
155
112
                    return 1
156
113
                else:
157
 
                    trace.note(gettext('All conflicts resolved.'))
 
114
                    trace.note('All conflicts resolved.')
158
115
                    return 0
159
116
            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'):
 
117
                resolve(tree, file_list)
 
118
 
 
119
 
 
120
def resolve(tree, paths=None, ignore_misses=False, recursive=False):
173
121
    """Resolve some or all of the conflicts in a working tree.
174
122
 
175
123
    :param paths: If None, resolve all conflicts.  Otherwise, select only
179
127
        recursive commands like revert, this should be True.  For commands
180
128
        or applications wishing finer-grained control, like the resolve
181
129
        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,
 
130
    :ignore_misses: If False, warnings will be printed if the supplied paths
 
131
        do not have conflicts.
185
132
    """
186
133
    tree.lock_tree_write()
187
 
    nb_conflicts_after = None
188
134
    try:
189
135
        tree_conflicts = tree.conflicts()
190
 
        nb_conflicts_before = len(tree_conflicts)
191
136
        if paths is None:
192
137
            new_conflicts = ConflictList()
193
 
            to_process = tree_conflicts
 
138
            selected_conflicts = tree_conflicts
194
139
        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)
 
140
            new_conflicts, selected_conflicts = \
 
141
                tree_conflicts.select_conflicts(tree, paths, ignore_misses,
 
142
                    recursive)
203
143
        try:
204
 
            nb_conflicts_after = len(new_conflicts)
205
144
            tree.set_conflicts(new_conflicts)
206
145
        except errors.UnsupportedOperation:
207
146
            pass
 
147
        selected_conflicts.remove_files(tree)
208
148
    finally:
209
149
        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
150
 
214
151
 
215
152
def restore(filename):
216
 
    """Restore a conflicted file to the state it was in before merging.
217
 
 
218
 
    Only text restoration is supported at present.
 
153
    """\
 
154
    Restore a conflicted file to the state it was in before merging.
 
155
    Only text restoration supported at present.
219
156
    """
220
157
    conflicted = False
221
158
    try:
291
228
        """Generator of stanzas"""
292
229
        for conflict in self:
293
230
            yield conflict.as_stanza()
294
 
 
 
231
            
295
232
    def to_strings(self):
296
233
        """Generate strings for the provided conflicts"""
297
234
        for conflict in self:
298
 
            yield unicode(conflict)
 
235
            yield str(conflict)
299
236
 
300
237
    def remove_files(self, tree):
301
238
        """Remove the THIS, BASE and OTHER files for listed conflicts"""
302
239
        for conflict in self:
303
240
            if not conflict.has_files:
304
241
                continue
305
 
            conflict.cleanup(tree)
 
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
306
248
 
307
249
    def select_conflicts(self, tree, paths, ignore_misses=False,
308
250
                         recurse=False):
309
251
        """Select the conflicts associated with paths in a tree.
310
 
 
 
252
        
311
253
        File-ids are also used for this.
312
254
        :return: a pair of ConflictLists: (not_selected, selected)
313
255
        """
357
299
                    print "%s is not conflicted" % path
358
300
        return new_conflicts, selected_conflicts
359
301
 
360
 
 
 
302
 
361
303
class Conflict(object):
362
304
    """Base class for all types of conflict"""
363
305
 
364
 
    # FIXME: cleanup should take care of that ? -- vila 091229
365
306
    has_files = False
366
307
 
367
308
    def __init__(self, path, file_id=None):
394
335
    def __ne__(self, other):
395
336
        return not self.__eq__(other)
396
337
 
397
 
    def __unicode__(self):
 
338
    def __str__(self):
398
339
        return self.format % self.__dict__
399
340
 
400
341
    def __repr__(self):
416
357
        else:
417
358
            return None, conflict.typestring
418
359
 
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
360
 
461
361
class PathConflict(Conflict):
462
362
    """A conflict was encountered merging file paths"""
466
366
    format = 'Path conflict: %(path)s / %(conflict_path)s'
467
367
 
468
368
    rformat = '%(class)s(%(path)r, %(conflict_path)r, %(file_id)r)'
469
 
 
470
369
    def __init__(self, path, conflict_path=None, file_id=None):
471
370
        Conflict.__init__(self, path, file_id)
472
371
        self.conflict_path = conflict_path
477
376
            s.add('conflict_path', self.conflict_path)
478
377
        return s
479
378
 
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
379
 
571
380
class ContentsConflict(PathConflict):
572
 
    """The files are of different types (or both binary), or not present"""
 
381
    """The files are of different types, or not present"""
573
382
 
574
383
    has_files = True
575
384
 
577
386
 
578
387
    format = 'Contents conflict in %(path)s'
579
388
 
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):
 
389
 
 
390
class TextConflict(PathConflict):
630
391
    """The merge algorithm could not resolve all differences encountered."""
631
392
 
632
393
    has_files = True
635
396
 
636
397
    format = 'Text conflict in %(path)s'
637
398
 
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
399
 
676
400
class HandledConflict(Conflict):
677
401
    """A path problem that has been provisionally resolved.
679
403
    """
680
404
 
681
405
    rformat = "%(class)s(%(action)r, %(path)r, %(file_id)r)"
682
 
 
 
406
    
683
407
    def __init__(self, action, path, file_id=None):
684
408
        Conflict.__init__(self, path, file_id)
685
409
        self.action = action
692
416
        s.add('action', self.action)
693
417
        return s
694
418
 
695
 
    def associated_filenames(self):
696
 
        # Nothing has been generated here
697
 
        return []
698
 
 
699
419
 
700
420
class HandledPathConflict(HandledConflict):
701
421
    """A provisionally-resolved path problem involving two paths.
708
428
    def __init__(self, action, path, conflict_path, file_id=None,
709
429
                 conflict_file_id=None):
710
430
        HandledConflict.__init__(self, action, path, file_id)
711
 
        self.conflict_path = conflict_path
 
431
        self.conflict_path = conflict_path 
712
432
        # warn turned off, because the factory blindly transfers the Stanza
713
433
        # values to __init__.
714
434
        self.conflict_file_id = osutils.safe_file_id(conflict_file_id,
715
435
                                                     warn=False)
716
 
 
 
436
        
717
437
    def _cmp_list(self):
718
 
        return HandledConflict._cmp_list(self) + [self.conflict_path,
 
438
        return HandledConflict._cmp_list(self) + [self.conflict_path, 
719
439
                                                  self.conflict_file_id]
720
440
 
721
441
    def as_stanza(self):
723
443
        s.add('conflict_path', self.conflict_path)
724
444
        if self.conflict_file_id is not None:
725
445
            s.add('conflict_file_id', self.conflict_file_id.decode('utf8'))
726
 
 
 
446
            
727
447
        return s
728
448
 
729
449
 
742
462
 
743
463
    format = 'Conflict adding file %(conflict_path)s.  %(action)s %(path)s.'
744
464
 
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
465
 
753
466
class ParentLoop(HandledPathConflict):
754
467
    """An attempt to create an infinitely-looping directory structure.
755
468
    This is rare, but can be produced like so:
756
469
 
757
470
    tree A:
758
 
      mv foo bar
 
471
      mv foo/bar
759
472
    tree B:
760
 
      mv bar foo
 
473
      mv bar/foo
761
474
    merge A and B
762
475
    """
763
476
 
764
477
    typestring = 'parent loop'
765
478
 
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()
 
479
    format = 'Conflict moving %(conflict_path)s into %(path)s.  %(action)s.'
785
480
 
786
481
 
787
482
class UnversionedParent(HandledConflict):
788
 
    """An attempt to version a file whose parent directory is not versioned.
 
483
    """An attempt to version an file whose parent directory is not versioned.
789
484
    Typically, the result of a merge where one tree unversioned the directory
790
485
    and the other added a versioned file to it.
791
486
    """
795
490
    format = 'Conflict because %(path)s is not versioned, but has versioned'\
796
491
             ' children.  %(action)s.'
797
492
 
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
493
 
808
494
class MissingParent(HandledConflict):
809
495
    """An attempt to add files to a directory that is not present.
810
496
    Typically, the result of a merge where THIS deleted the directory and
811
497
    the OTHER added a file to it.
812
 
    See also: DeletingParent (same situation, THIS and OTHER reversed)
 
498
    See also: DeletingParent (same situation, reversed THIS and OTHER)
813
499
    """
814
500
 
815
501
    typestring = 'missing parent'
816
502
 
817
503
    format = 'Conflict adding files to %(path)s.  %(action)s.'
818
504
 
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
505
 
827
506
class DeletingParent(HandledConflict):
828
507
    """An attempt to add files to a directory that is not present.
835
514
    format = "Conflict: can't delete %(path)s because it is not empty.  "\
836
515
             "%(action)s."
837
516
 
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
517
 
849
518
class NonDirectoryParent(HandledConflict):
850
 
    """An attempt to add files to a directory that is not a directory or
 
519
    """An attempt to add files to a directory that is not a director or
851
520
    an attempt to change the kind of a directory with files.
852
521
    """
853
522
 
856
525
    format = "Conflict: %(path)s is not a directory, but has files in it."\
857
526
             "  %(action)s."
858
527
 
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
528
ctype = {}
881
529
 
882
530
 
886
534
    for conflict_type in conflict_types:
887
535
        ctype[conflict_type.typestring] = conflict_type
888
536
 
 
537
 
889
538
register_types(ContentsConflict, TextConflict, PathConflict, DuplicateID,
890
539
               DuplicateEntry, ParentLoop, UnversionedParent, MissingParent,
891
540
               DeletingParent, NonDirectoryParent)