/brz/remove-bazaar

To get this branch, use:
bzr branch http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar

« back to all changes in this revision

Viewing changes to bzrlib/merge.py

Add a NEWS entry and prepare submission.

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2005-2010 Canonical Ltd
 
1
# Copyright (C) 2005, 2006, 2008 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
 
import warnings
18
17
 
19
 
from bzrlib.lazy_import import lazy_import
20
 
lazy_import(globals(), """
21
18
from bzrlib import (
22
19
    branch as _mod_branch,
23
20
    conflicts as _mod_conflicts,
27
24
    merge3,
28
25
    osutils,
29
26
    patiencediff,
 
27
    progress,
 
28
    registry,
30
29
    revision as _mod_revision,
31
30
    textfile,
32
31
    trace,
34
33
    tree as _mod_tree,
35
34
    tsort,
36
35
    ui,
37
 
    versionedfile,
38
 
    )
39
 
from bzrlib.cleanup import OperationWithCleanups
40
 
""")
41
 
from bzrlib import (
42
 
    decorators,
43
 
    hooks,
 
36
    versionedfile
44
37
    )
45
38
from bzrlib.symbol_versioning import (
46
39
    deprecated_in,
51
44
 
52
45
def transform_tree(from_tree, to_tree, interesting_ids=None):
53
46
    from_tree.lock_tree_write()
54
 
    operation = OperationWithCleanups(merge_inner)
55
 
    operation.add_cleanup(from_tree.unlock)
56
 
    operation.run_simple(from_tree.branch, to_tree, from_tree,
57
 
        ignore_zero=True, interesting_ids=interesting_ids, this_tree=from_tree)
58
 
 
59
 
 
60
 
class MergeHooks(hooks.Hooks):
61
 
 
62
 
    def __init__(self):
63
 
        hooks.Hooks.__init__(self)
64
 
        self.create_hook(hooks.HookPoint('merge_file_content',
65
 
            "Called with a bzrlib.merge.Merger object to create a per file "
66
 
            "merge object when starting a merge. "
67
 
            "Should return either None or a subclass of "
68
 
            "``bzrlib.merge.AbstractPerFileMerger``. "
69
 
            "Such objects will then be called per file "
70
 
            "that needs to be merged (including when one "
71
 
            "side has deleted the file and the other has changed it). "
72
 
            "See the AbstractPerFileMerger API docs for details on how it is "
73
 
            "used by merge.",
74
 
            (2, 1), None))
75
 
 
76
 
 
77
 
class AbstractPerFileMerger(object):
78
 
    """PerFileMerger objects are used by plugins extending merge for bzrlib.
79
 
 
80
 
    See ``bzrlib.plugins.news_merge.news_merge`` for an example concrete class.
81
 
    
82
 
    :ivar merger: The Merge3Merger performing the merge.
83
 
    """
84
 
 
85
 
    def __init__(self, merger):
86
 
        """Create a PerFileMerger for use with merger."""
87
 
        self.merger = merger
88
 
 
89
 
    def merge_contents(self, merge_params):
90
 
        """Attempt to merge the contents of a single file.
91
 
        
92
 
        :param merge_params: A bzrlib.merge.MergeHookParams
93
 
        :return : A tuple of (status, chunks), where status is one of
94
 
            'not_applicable', 'success', 'conflicted', or 'delete'.  If status
95
 
            is 'success' or 'conflicted', then chunks should be an iterable of
96
 
            strings for the new file contents.
97
 
        """
98
 
        return ('not applicable', None)
99
 
 
100
 
 
101
 
class PerFileMerger(AbstractPerFileMerger):
102
 
    """Merge individual files when self.file_matches returns True.
103
 
 
104
 
    This class is intended to be subclassed.  The file_matches and
105
 
    merge_matching methods should be overridden with concrete implementations.
106
 
    """
107
 
 
108
 
    def file_matches(self, params):
109
 
        """Return True if merge_matching should be called on this file.
110
 
 
111
 
        Only called with merges of plain files with no clear winner.
112
 
 
113
 
        Subclasses must override this.
114
 
        """
115
 
        raise NotImplementedError(self.file_matches)
116
 
 
117
 
    def get_filename(self, params, tree):
118
 
        """Lookup the filename (i.e. basename, not path), given a Tree (e.g.
119
 
        self.merger.this_tree) and a MergeHookParams.
120
 
        """
121
 
        return osutils.basename(tree.id2path(params.file_id))
122
 
 
123
 
    def get_filepath(self, params, tree):
124
 
        """Calculate the path to the file in a tree.
125
 
 
126
 
        :param params: A MergeHookParams describing the file to merge
127
 
        :param tree: a Tree, e.g. self.merger.this_tree.
128
 
        """
129
 
        return tree.id2path(params.file_id)
130
 
 
131
 
    def merge_contents(self, params):
132
 
        """Merge the contents of a single file."""
133
 
        # Check whether this custom merge logic should be used.
134
 
        if (
135
 
            # OTHER is a straight winner, rely on default merge.
136
 
            params.winner == 'other' or
137
 
            # THIS and OTHER aren't both files.
138
 
            not params.is_file_merge() or
139
 
            # The filename doesn't match *.xml
140
 
            not self.file_matches(params)):
141
 
            return 'not_applicable', None
142
 
        return self.merge_matching(params)
143
 
 
144
 
    def merge_matching(self, params):
145
 
        """Merge the contents of a single file that has matched the criteria
146
 
        in PerFileMerger.merge_contents (is a conflict, is a file,
147
 
        self.file_matches is True).
148
 
 
149
 
        Subclasses must override this.
150
 
        """
151
 
        raise NotImplementedError(self.merge_matching)
152
 
 
153
 
 
154
 
class ConfigurableFileMerger(PerFileMerger):
155
 
    """Merge individual files when configured via a .conf file.
156
 
 
157
 
    This is a base class for concrete custom file merging logic. Concrete
158
 
    classes should implement ``merge_text``.
159
 
 
160
 
    See ``bzrlib.plugins.news_merge.news_merge`` for an example concrete class.
161
 
    
162
 
    :ivar affected_files: The configured file paths to merge.
163
 
 
164
 
    :cvar name_prefix: The prefix to use when looking up configuration
165
 
        details. <name_prefix>_merge_files describes the files targeted by the
166
 
        hook for example.
167
 
        
168
 
    :cvar default_files: The default file paths to merge when no configuration
169
 
        is present.
170
 
    """
171
 
 
172
 
    name_prefix = None
173
 
    default_files = None
174
 
 
175
 
    def __init__(self, merger):
176
 
        super(ConfigurableFileMerger, self).__init__(merger)
177
 
        self.affected_files = None
178
 
        self.default_files = self.__class__.default_files or []
179
 
        self.name_prefix = self.__class__.name_prefix
180
 
        if self.name_prefix is None:
181
 
            raise ValueError("name_prefix must be set.")
182
 
 
183
 
    def file_matches(self, params):
184
 
        """Check whether the file should call the merge hook.
185
 
 
186
 
        <name_prefix>_merge_files configuration variable is a list of files
187
 
        that should use the hook.
188
 
        """
189
 
        affected_files = self.affected_files
190
 
        if affected_files is None:
191
 
            config = self.merger.this_branch.get_config()
192
 
            # Until bzr provides a better policy for caching the config, we
193
 
            # just add the part we're interested in to the params to avoid
194
 
            # reading the config files repeatedly (bazaar.conf, location.conf,
195
 
            # branch.conf).
196
 
            config_key = self.name_prefix + '_merge_files'
197
 
            affected_files = config.get_user_option_as_list(config_key)
198
 
            if affected_files is None:
199
 
                # If nothing was specified in the config, use the default.
200
 
                affected_files = self.default_files
201
 
            self.affected_files = affected_files
202
 
        if affected_files:
203
 
            filepath = self.get_filepath(params, self.merger.this_tree)
204
 
            if filepath in affected_files:
205
 
                return True
206
 
        return False
207
 
 
208
 
    def merge_matching(self, params):
209
 
        return self.merge_text(params)
210
 
 
211
 
    def merge_text(self, params):
212
 
        """Merge the byte contents of a single file.
213
 
 
214
 
        This is called after checking that the merge should be performed in
215
 
        merge_contents, and it should behave as per
216
 
        ``bzrlib.merge.AbstractPerFileMerger.merge_contents``.
217
 
        """
218
 
        raise NotImplementedError(self.merge_text)
219
 
 
220
 
 
221
 
class MergeHookParams(object):
222
 
    """Object holding parameters passed to merge_file_content hooks.
223
 
 
224
 
    There are some fields hooks can access:
225
 
 
226
 
    :ivar file_id: the file ID of the file being merged
227
 
    :ivar trans_id: the transform ID for the merge of this file
228
 
    :ivar this_kind: kind of file_id in 'this' tree
229
 
    :ivar other_kind: kind of file_id in 'other' tree
230
 
    :ivar winner: one of 'this', 'other', 'conflict'
231
 
    """
232
 
 
233
 
    def __init__(self, merger, file_id, trans_id, this_kind, other_kind,
234
 
            winner):
235
 
        self._merger = merger
236
 
        self.file_id = file_id
237
 
        self.trans_id = trans_id
238
 
        self.this_kind = this_kind
239
 
        self.other_kind = other_kind
240
 
        self.winner = winner
241
 
 
242
 
    def is_file_merge(self):
243
 
        """True if this_kind and other_kind are both 'file'."""
244
 
        return self.this_kind == 'file' and self.other_kind == 'file'
245
 
 
246
 
    @decorators.cachedproperty
247
 
    def base_lines(self):
248
 
        """The lines of the 'base' version of the file."""
249
 
        return self._merger.get_lines(self._merger.base_tree, self.file_id)
250
 
 
251
 
    @decorators.cachedproperty
252
 
    def this_lines(self):
253
 
        """The lines of the 'this' version of the file."""
254
 
        return self._merger.get_lines(self._merger.this_tree, self.file_id)
255
 
 
256
 
    @decorators.cachedproperty
257
 
    def other_lines(self):
258
 
        """The lines of the 'other' version of the file."""
259
 
        return self._merger.get_lines(self._merger.other_tree, self.file_id)
 
47
    try:
 
48
        merge_inner(from_tree.branch, to_tree, from_tree, ignore_zero=True,
 
49
                    interesting_ids=interesting_ids, this_tree=from_tree)
 
50
    finally:
 
51
        from_tree.unlock()
260
52
 
261
53
 
262
54
class Merger(object):
263
 
 
264
 
    hooks = MergeHooks()
265
 
 
266
55
    def __init__(self, this_branch, other_tree=None, base_tree=None,
267
56
                 this_tree=None, pb=None, change_reporter=None,
268
57
                 recurse='down', revision_graph=None):
283
72
        self.interesting_files = None
284
73
        self.show_base = False
285
74
        self.reprocess = False
286
 
        if pb is not None:
287
 
            warnings.warn("pb parameter to Merger() is deprecated and ignored")
 
75
        if pb is None:
 
76
            pb = progress.DummyProgress()
 
77
        self._pb = pb
288
78
        self.pp = None
289
79
        self.recurse = recurse
290
80
        self.change_reporter = change_reporter
450
240
        if self.other_rev_id is None:
451
241
            other_basis_tree = self.revision_tree(self.other_basis)
452
242
            if other_basis_tree.has_changes(self.other_tree):
453
 
                raise errors.WorkingTreeNotRevision(self.this_tree)
 
243
                raise WorkingTreeNotRevision(self.this_tree)
454
244
            other_rev_id = self.other_basis
455
245
            self.other_tree = other_basis_tree
456
246
 
500
290
    def _add_parent(self):
501
291
        new_parents = self.this_tree.get_parent_ids() + [self.other_rev_id]
502
292
        new_parent_trees = []
503
 
        operation = OperationWithCleanups(self.this_tree.set_parent_trees)
504
293
        for revision_id in new_parents:
505
294
            try:
506
295
                tree = self.revision_tree(revision_id)
508
297
                tree = None
509
298
            else:
510
299
                tree.lock_read()
511
 
                operation.add_cleanup(tree.unlock)
512
300
            new_parent_trees.append((revision_id, tree))
513
 
        operation.run_simple(new_parent_trees, allow_leftmost_as_ghost=True)
 
301
        try:
 
302
            self.this_tree.set_parent_trees(new_parent_trees,
 
303
                                            allow_leftmost_as_ghost=True)
 
304
        finally:
 
305
            for _revision_id, tree in new_parent_trees:
 
306
                if tree is not None:
 
307
                    tree.unlock()
514
308
 
515
309
    def set_other(self, other_revision, possible_transports=None):
516
310
        """Set the revision and tree to merge from.
639
433
                  'other_tree': self.other_tree,
640
434
                  'interesting_ids': self.interesting_ids,
641
435
                  'interesting_files': self.interesting_files,
642
 
                  'this_branch': self.this_branch,
 
436
                  'pp': self.pp,
643
437
                  'do_merge': False}
644
438
        if self.merge_type.requires_base:
645
439
            kwargs['base_tree'] = self.base_tree
663
457
        if self._is_criss_cross and getattr(self.merge_type,
664
458
                                            'supports_lca_trees', False):
665
459
            kwargs['lca_trees'] = self._lca_trees
666
 
        return self.merge_type(pb=None,
 
460
        return self.merge_type(pb=self._pb,
667
461
                               change_reporter=self.change_reporter,
668
462
                               **kwargs)
669
463
 
670
 
    def _do_merge_to(self):
671
 
        merge = self.make_merger()
 
464
    def _do_merge_to(self, merge):
672
465
        if self.other_branch is not None:
673
466
            self.other_branch.update_references(self.this_branch)
674
467
        merge.do_merge()
688
481
                    sub_tree.branch.repository.revision_tree(base_revision)
689
482
                sub_merge.base_rev_id = base_revision
690
483
                sub_merge.do_merge()
691
 
        return merge
692
484
 
693
485
    def do_merge(self):
694
 
        operation = OperationWithCleanups(self._do_merge_to)
695
486
        self.this_tree.lock_tree_write()
696
 
        operation.add_cleanup(self.this_tree.unlock)
697
 
        if self.base_tree is not None:
698
 
            self.base_tree.lock_read()
699
 
            operation.add_cleanup(self.base_tree.unlock)
700
 
        if self.other_tree is not None:
701
 
            self.other_tree.lock_read()
702
 
            operation.add_cleanup(self.other_tree.unlock)
703
 
        merge = operation.run_simple()
 
487
        try:
 
488
            if self.base_tree is not None:
 
489
                self.base_tree.lock_read()
 
490
            try:
 
491
                if self.other_tree is not None:
 
492
                    self.other_tree.lock_read()
 
493
                try:
 
494
                    merge = self.make_merger()
 
495
                    self._do_merge_to(merge)
 
496
                finally:
 
497
                    if self.other_tree is not None:
 
498
                        self.other_tree.unlock()
 
499
            finally:
 
500
                if self.base_tree is not None:
 
501
                    self.base_tree.unlock()
 
502
        finally:
 
503
            self.this_tree.unlock()
704
504
        if len(merge.cooked_conflicts) == 0:
705
505
            if not self.ignore_zero and not trace.is_quiet():
706
506
                trace.note("All changes applied successfully.")
741
541
 
742
542
    def __init__(self, working_tree, this_tree, base_tree, other_tree,
743
543
                 interesting_ids=None, reprocess=False, show_base=False,
744
 
                 pb=None, pp=None, change_reporter=None,
 
544
                 pb=progress.DummyProgress(), pp=None, change_reporter=None,
745
545
                 interesting_files=None, do_merge=True,
746
 
                 cherrypick=False, lca_trees=None, this_branch=None):
 
546
                 cherrypick=False, lca_trees=None):
747
547
        """Initialize the merger object and perform the merge.
748
548
 
749
549
        :param working_tree: The working tree to apply the merge to
750
550
        :param this_tree: The local tree in the merge operation
751
551
        :param base_tree: The common tree in the merge operation
752
552
        :param other_tree: The other tree to merge changes from
753
 
        :param this_branch: The branch associated with this_tree.  Defaults to
754
 
            this_tree.branch if not supplied.
755
553
        :param interesting_ids: The file_ids of files that should be
756
554
            participate in the merge.  May not be combined with
757
555
            interesting_files.
758
556
        :param: reprocess If True, perform conflict-reduction processing.
759
557
        :param show_base: If True, show the base revision in text conflicts.
760
558
            (incompatible with reprocess)
761
 
        :param pb: ignored
 
559
        :param pb: A Progress bar
762
560
        :param pp: A ProgressPhase object
763
561
        :param change_reporter: An object that should report changes made
764
562
        :param interesting_files: The tree-relative paths of files that should
775
573
        if interesting_files is not None and interesting_ids is not None:
776
574
            raise ValueError(
777
575
                'specify either interesting_ids or interesting_files')
778
 
        if this_branch is None:
779
 
            this_branch = this_tree.branch
780
576
        self.interesting_ids = interesting_ids
781
577
        self.interesting_files = interesting_files
782
578
        self.this_tree = working_tree
783
579
        self.base_tree = base_tree
784
580
        self.other_tree = other_tree
785
 
        self.this_branch = this_branch
786
581
        self._raw_conflicts = []
787
582
        self.cooked_conflicts = []
788
583
        self.reprocess = reprocess
793
588
        # making sure we haven't missed any corner cases.
794
589
        # if lca_trees is None:
795
590
        #     self._lca_trees = [self.base_tree]
 
591
        self.pb = pb
 
592
        self.pp = pp
796
593
        self.change_reporter = change_reporter
797
594
        self.cherrypick = cherrypick
 
595
        if self.pp is None:
 
596
            self.pp = progress.ProgressPhase("Merge phase", 3, self.pb)
798
597
        if do_merge:
799
598
            self.do_merge()
800
 
        if pp is not None:
801
 
            warnings.warn("pp argument to Merge3Merger is deprecated")
802
 
        if pb is not None:
803
 
            warnings.warn("pb argument to Merge3Merger is deprecated")
804
599
 
805
600
    def do_merge(self):
806
 
        operation = OperationWithCleanups(self._do_merge)
807
601
        self.this_tree.lock_tree_write()
808
 
        operation.add_cleanup(self.this_tree.unlock)
809
602
        self.base_tree.lock_read()
810
 
        operation.add_cleanup(self.base_tree.unlock)
811
603
        self.other_tree.lock_read()
812
 
        operation.add_cleanup(self.other_tree.unlock)
813
 
        operation.run()
814
 
 
815
 
    def _do_merge(self, operation):
816
 
        self.tt = transform.TreeTransform(self.this_tree, None)
817
 
        operation.add_cleanup(self.tt.finalize)
818
 
        self._compute_transform()
819
 
        results = self.tt.apply(no_conflicts=True)
820
 
        self.write_modified(results)
821
604
        try:
822
 
            self.this_tree.add_conflicts(self.cooked_conflicts)
823
 
        except errors.UnsupportedOperation:
824
 
            pass
 
605
            self.tt = transform.TreeTransform(self.this_tree, self.pb)
 
606
            try:
 
607
                self.pp.next_phase()
 
608
                self._compute_transform()
 
609
                self.pp.next_phase()
 
610
                results = self.tt.apply(no_conflicts=True)
 
611
                self.write_modified(results)
 
612
                try:
 
613
                    self.this_tree.add_conflicts(self.cooked_conflicts)
 
614
                except errors.UnsupportedOperation:
 
615
                    pass
 
616
            finally:
 
617
                self.tt.finalize()
 
618
        finally:
 
619
            self.other_tree.unlock()
 
620
            self.base_tree.unlock()
 
621
            self.this_tree.unlock()
 
622
            self.pb.clear()
825
623
 
826
624
    def make_preview_transform(self):
827
 
        operation = OperationWithCleanups(self._make_preview_transform)
828
625
        self.base_tree.lock_read()
829
 
        operation.add_cleanup(self.base_tree.unlock)
830
626
        self.other_tree.lock_read()
831
 
        operation.add_cleanup(self.other_tree.unlock)
832
 
        return operation.run_simple()
833
 
 
834
 
    def _make_preview_transform(self):
835
627
        self.tt = transform.TransformPreview(self.this_tree)
836
 
        self._compute_transform()
 
628
        try:
 
629
            self.pp.next_phase()
 
630
            self._compute_transform()
 
631
            self.pp.next_phase()
 
632
        finally:
 
633
            self.other_tree.unlock()
 
634
            self.base_tree.unlock()
 
635
            self.pb.clear()
837
636
        return self.tt
838
637
 
839
638
    def _compute_transform(self):
845
644
            resolver = self._lca_multi_way
846
645
        child_pb = ui.ui_factory.nested_progress_bar()
847
646
        try:
848
 
            factories = Merger.hooks['merge_file_content']
849
 
            hooks = [factory(self) for factory in factories] + [self]
850
 
            self.active_hooks = [hook for hook in hooks if hook is not None]
851
647
            for num, (file_id, changed, parents3, names3,
852
648
                      executable3) in enumerate(entries):
853
649
                child_pb.update('Preparing file merge', num, len(entries))
854
650
                self._merge_names(file_id, parents3, names3, resolver=resolver)
855
651
                if changed:
856
 
                    file_status = self._do_merge_contents(file_id)
 
652
                    file_status = self.merge_contents(file_id)
857
653
                else:
858
654
                    file_status = 'unmodified'
859
655
                self._merge_executable(file_id,
861
657
        finally:
862
658
            child_pb.finished()
863
659
        self.fix_root()
 
660
        self.pp.next_phase()
864
661
        child_pb = ui.ui_factory.nested_progress_bar()
865
662
        try:
866
663
            fs_conflicts = transform.resolve_conflicts(self.tt, child_pb,
1064
861
                        continue
1065
862
                else:
1066
863
                    raise AssertionError('unhandled kind: %s' % other_ie.kind)
 
864
                # XXX: We need to handle kind == 'symlink'
1067
865
 
1068
866
            # If we have gotten this far, that means something has changed
1069
867
            result.append((file_id, content_changed,
1091
889
        other_root = self.tt.trans_id_file_id(other_root_file_id)
1092
890
        if other_root == self.tt.root:
1093
891
            return
 
892
        try:
 
893
            self.tt.final_kind(other_root)
 
894
        except errors.NoSuchFile:
 
895
            return
1094
896
        if self.other_tree.inventory.root.file_id in self.this_tree.inventory:
1095
 
            # the other tree's root is a non-root in the current tree (as when
1096
 
            # a previously unrelated branch is merged into another)
 
897
            # the other tree's root is a non-root in the current tree
1097
898
            return
1098
 
        try:
1099
 
            self.tt.final_kind(other_root)
1100
 
            other_root_is_present = True
1101
 
        except errors.NoSuchFile:
1102
 
            # other_root doesn't have a physical representation. We still need
1103
 
            # to move any references to the actual root of the tree.
1104
 
            other_root_is_present = False
1105
 
        # 'other_tree.inventory.root' is not present in this tree. We are
1106
 
        # calling adjust_path for children which *want* to be present with a
1107
 
        # correct place to go.
1108
 
        for thing, child in self.other_tree.inventory.root.children.iteritems():
 
899
        self.reparent_children(self.other_tree.inventory.root, self.tt.root)
 
900
        self.tt.cancel_creation(other_root)
 
901
        self.tt.cancel_versioning(other_root)
 
902
 
 
903
    def reparent_children(self, ie, target):
 
904
        for thing, child in ie.children.iteritems():
1109
905
            trans_id = self.tt.trans_id_file_id(child.file_id)
1110
 
            if not other_root_is_present:
1111
 
                # FIXME: Make final_kind returns None instead of raising
1112
 
                # NoSuchFile to avoid the ugly construct below -- vila 20100402
1113
 
                try:
1114
 
                    self.tt.final_kind(trans_id)
1115
 
                    # The item exist in the final tree and has a defined place
1116
 
                    # to go already.
1117
 
                    continue
1118
 
                except errors.NoSuchFile, e:
1119
 
                    pass
1120
 
            # Move the item into the root
1121
 
            self.tt.adjust_path(self.tt.final_name(trans_id),
1122
 
                                self.tt.root, trans_id)
1123
 
        if other_root_is_present:
1124
 
            self.tt.cancel_creation(other_root)
1125
 
            self.tt.cancel_versioning(other_root)
 
906
            self.tt.adjust_path(self.tt.final_name(trans_id), target, trans_id)
1126
907
 
1127
908
    def write_modified(self, results):
1128
909
        modified_hashes = {}
1160
941
    @staticmethod
1161
942
    def executable(tree, file_id):
1162
943
        """Determine the executability of a file-id (used as a key method)."""
1163
 
        if not tree.has_id(file_id):
 
944
        if file_id not in tree:
1164
945
            return None
1165
946
        if tree.kind(file_id) != "file":
1166
947
            return False
1169
950
    @staticmethod
1170
951
    def kind(tree, file_id):
1171
952
        """Determine the kind of a file-id (used as a key method)."""
1172
 
        if not tree.has_id(file_id):
 
953
        if file_id not in tree:
1173
954
            return None
1174
955
        return tree.kind(file_id)
1175
956
 
1176
957
    @staticmethod
1177
958
    def _three_way(base, other, this):
 
959
        #if base == other, either they all agree, or only THIS has changed.
1178
960
        if base == other:
1179
 
            # if 'base == other', either they all agree, or only 'this' has
1180
 
            # changed.
1181
961
            return 'this'
1182
962
        elif this not in (base, other):
1183
 
            # 'this' is neither 'base' nor 'other', so both sides changed
1184
963
            return 'conflict'
 
964
        # "Ambiguous clean merge" -- both sides have made the same change.
1185
965
        elif this == other:
1186
 
            # "Ambiguous clean merge" -- both sides have made the same change.
1187
966
            return "this"
 
967
        # this == base: only other has changed.
1188
968
        else:
1189
 
            # this == base: only other has changed.
1190
969
            return "other"
1191
970
 
1192
971
    @staticmethod
1236
1015
                # only has an lca value
1237
1016
                return 'other'
1238
1017
 
1239
 
        # At this point, the lcas disagree, and the tip disagree
 
1018
        # At this point, the lcas disagree, and the tips disagree
1240
1019
        return 'conflict'
1241
1020
 
1242
1021
    @staticmethod
1243
 
    @deprecated_method(deprecated_in((2, 2, 0)))
1244
1022
    def scalar_three_way(this_tree, base_tree, other_tree, file_id, key):
1245
1023
        """Do a three-way test on a scalar.
1246
1024
        Return "this", "other" or "conflict", depending whether a value wins.
1261
1039
 
1262
1040
    def merge_names(self, file_id):
1263
1041
        def get_entry(tree):
1264
 
            if tree.has_id(file_id):
 
1042
            if file_id in tree.inventory:
1265
1043
                return tree.inventory[file_id]
1266
1044
            else:
1267
1045
                return None
1296
1074
                parent_id_winner = "other"
1297
1075
        if name_winner == "this" and parent_id_winner == "this":
1298
1076
            return
1299
 
        if name_winner == 'conflict' or parent_id_winner == 'conflict':
1300
 
            # Creating helpers (.OTHER or .THIS) here cause problems down the
1301
 
            # road if a ContentConflict needs to be created so we should not do
1302
 
            # that
1303
 
            trans_id = self.tt.trans_id_file_id(file_id)
1304
 
            self._raw_conflicts.append(('path conflict', trans_id, file_id,
1305
 
                                        this_parent, this_name,
1306
 
                                        other_parent, other_name))
 
1077
        if name_winner == "conflict":
 
1078
            trans_id = self.tt.trans_id_file_id(file_id)
 
1079
            self._raw_conflicts.append(('name conflict', trans_id,
 
1080
                                        this_name, other_name))
 
1081
        if parent_id_winner == "conflict":
 
1082
            trans_id = self.tt.trans_id_file_id(file_id)
 
1083
            self._raw_conflicts.append(('parent conflict', trans_id,
 
1084
                                        this_parent, other_parent))
1307
1085
        if other_name is None:
1308
1086
            # it doesn't matter whether the result was 'other' or
1309
1087
            # 'conflict'-- if there's no 'other', we leave it alone.
1310
1088
            return
 
1089
        # if we get here, name_winner and parent_winner are set to safe values.
 
1090
        trans_id = self.tt.trans_id_file_id(file_id)
1311
1091
        parent_id = parents[self.winner_idx[parent_id_winner]]
1312
1092
        if parent_id is not None:
1313
 
            # if we get here, name_winner and parent_winner are set to safe
1314
 
            # values.
 
1093
            parent_trans_id = self.tt.trans_id_file_id(parent_id)
1315
1094
            self.tt.adjust_path(names[self.winner_idx[name_winner]],
1316
 
                                self.tt.trans_id_file_id(parent_id),
1317
 
                                self.tt.trans_id_file_id(file_id))
 
1095
                                parent_trans_id, trans_id)
1318
1096
 
1319
 
    def _do_merge_contents(self, file_id):
 
1097
    def merge_contents(self, file_id):
1320
1098
        """Performs a merge on file_id contents."""
1321
1099
        def contents_pair(tree):
1322
1100
            if file_id not in tree:
1330
1108
                contents = None
1331
1109
            return kind, contents
1332
1110
 
 
1111
        def contents_conflict():
 
1112
            trans_id = self.tt.trans_id_file_id(file_id)
 
1113
            name = self.tt.final_name(trans_id)
 
1114
            parent_id = self.tt.final_parent(trans_id)
 
1115
            if file_id in self.this_tree.inventory:
 
1116
                self.tt.unversion_file(trans_id)
 
1117
                if file_id in self.this_tree:
 
1118
                    self.tt.delete_contents(trans_id)
 
1119
            file_group = self._dump_conflicts(name, parent_id, file_id,
 
1120
                                              set_version=True)
 
1121
            self._raw_conflicts.append(('contents conflict', file_group))
 
1122
 
1333
1123
        # See SPOT run.  run, SPOT, run.
1334
1124
        # So we're not QUITE repeating ourselves; we do tricky things with
1335
1125
        # file kind...
1351
1141
        if winner == 'this':
1352
1142
            # No interesting changes introduced by OTHER
1353
1143
            return "unmodified"
1354
 
        # We have a hypothetical conflict, but if we have files, then we
1355
 
        # can try to merge the content
1356
1144
        trans_id = self.tt.trans_id_file_id(file_id)
1357
 
        params = MergeHookParams(self, file_id, trans_id, this_pair[0],
1358
 
            other_pair[0], winner)
1359
 
        hooks = self.active_hooks
1360
 
        hook_status = 'not_applicable'
1361
 
        for hook in hooks:
1362
 
            hook_status, lines = hook.merge_contents(params)
1363
 
            if hook_status != 'not_applicable':
1364
 
                # Don't try any more hooks, this one applies.
1365
 
                break
1366
 
        result = "modified"
1367
 
        if hook_status == 'not_applicable':
1368
 
            # This is a contents conflict, because none of the available
1369
 
            # functions could merge it.
1370
 
            result = None
1371
 
            name = self.tt.final_name(trans_id)
1372
 
            parent_id = self.tt.final_parent(trans_id)
1373
 
            if self.this_tree.has_id(file_id):
1374
 
                self.tt.unversion_file(trans_id)
1375
 
            file_group = self._dump_conflicts(name, parent_id, file_id,
1376
 
                                              set_version=True)
1377
 
            self._raw_conflicts.append(('contents conflict', file_group))
1378
 
        elif hook_status == 'success':
1379
 
            self.tt.create_file(lines, trans_id)
1380
 
        elif hook_status == 'conflicted':
1381
 
            # XXX: perhaps the hook should be able to provide
1382
 
            # the BASE/THIS/OTHER files?
1383
 
            self.tt.create_file(lines, trans_id)
1384
 
            self._raw_conflicts.append(('text conflict', trans_id))
1385
 
            name = self.tt.final_name(trans_id)
1386
 
            parent_id = self.tt.final_parent(trans_id)
1387
 
            self._dump_conflicts(name, parent_id, file_id)
1388
 
        elif hook_status == 'delete':
1389
 
            self.tt.unversion_file(trans_id)
1390
 
            result = "deleted"
1391
 
        elif hook_status == 'done':
1392
 
            # The hook function did whatever it needs to do directly, no
1393
 
            # further action needed here.
1394
 
            pass
1395
 
        else:
1396
 
            raise AssertionError('unknown hook_status: %r' % (hook_status,))
1397
 
        if not self.this_tree.has_id(file_id) and result == "modified":
1398
 
            self.tt.version_file(file_id, trans_id)
1399
 
        # The merge has been performed, so the old contents should not be
1400
 
        # retained.
1401
 
        try:
1402
 
            self.tt.delete_contents(trans_id)
1403
 
        except errors.NoSuchFile:
1404
 
            pass
1405
 
        return result
1406
 
 
1407
 
    def _default_other_winner_merge(self, merge_hook_params):
1408
 
        """Replace this contents with other."""
1409
 
        file_id = merge_hook_params.file_id
1410
 
        trans_id = merge_hook_params.trans_id
1411
 
        file_in_this = self.this_tree.has_id(file_id)
1412
 
        if self.other_tree.has_id(file_id):
1413
 
            # OTHER changed the file
1414
 
            wt = self.this_tree
1415
 
            if wt.supports_content_filtering():
1416
 
                # We get the path from the working tree if it exists.
1417
 
                # That fails though when OTHER is adding a file, so
1418
 
                # we fall back to the other tree to find the path if
1419
 
                # it doesn't exist locally.
1420
 
                try:
1421
 
                    filter_tree_path = wt.id2path(file_id)
1422
 
                except errors.NoSuchId:
1423
 
                    filter_tree_path = self.other_tree.id2path(file_id)
1424
 
            else:
1425
 
                # Skip the id2path lookup for older formats
1426
 
                filter_tree_path = None
1427
 
            transform.create_from_tree(self.tt, trans_id,
1428
 
                             self.other_tree, file_id,
1429
 
                             filter_tree_path=filter_tree_path)
1430
 
            return 'done', None
1431
 
        elif file_in_this:
1432
 
            # OTHER deleted the file
1433
 
            return 'delete', None
1434
 
        else:
1435
 
            raise AssertionError(
1436
 
                'winner is OTHER, but file_id %r not in THIS or OTHER tree'
1437
 
                % (file_id,))
1438
 
 
1439
 
    def merge_contents(self, merge_hook_params):
1440
 
        """Fallback merge logic after user installed hooks."""
1441
 
        # This function is used in merge hooks as the fallback instance.
1442
 
        # Perhaps making this function and the functions it calls be a 
1443
 
        # a separate class would be better.
1444
 
        if merge_hook_params.winner == 'other':
 
1145
        if winner == 'other':
1445
1146
            # OTHER is a straight winner, so replace this contents with other
1446
 
            return self._default_other_winner_merge(merge_hook_params)
1447
 
        elif merge_hook_params.is_file_merge():
1448
 
            # THIS and OTHER are both files, so text merge.  Either
1449
 
            # BASE is a file, or both converted to files, so at least we
1450
 
            # have agreement that output should be a file.
1451
 
            try:
1452
 
                self.text_merge(merge_hook_params.file_id,
1453
 
                    merge_hook_params.trans_id)
1454
 
            except errors.BinaryFile:
1455
 
                return 'not_applicable', None
1456
 
            return 'done', None
 
1147
            file_in_this = file_id in self.this_tree
 
1148
            if file_in_this:
 
1149
                # Remove any existing contents
 
1150
                self.tt.delete_contents(trans_id)
 
1151
            if file_id in self.other_tree:
 
1152
                # OTHER changed the file
 
1153
                wt = self.this_tree
 
1154
                if wt.supports_content_filtering():
 
1155
                    # We get the path from the working tree if it exists.
 
1156
                    # That fails though when OTHER is adding a file, so
 
1157
                    # we fall back to the other tree to find the path if
 
1158
                    # it doesn't exist locally.
 
1159
                    try:
 
1160
                        filter_tree_path = wt.id2path(file_id)
 
1161
                    except errors.NoSuchId:
 
1162
                        filter_tree_path = self.other_tree.id2path(file_id)
 
1163
                else:
 
1164
                    # Skip the id2path lookup for older formats
 
1165
                    filter_tree_path = None
 
1166
                transform.create_from_tree(self.tt, trans_id,
 
1167
                                 self.other_tree, file_id,
 
1168
                                 filter_tree_path=filter_tree_path)
 
1169
                if not file_in_this:
 
1170
                    self.tt.version_file(file_id, trans_id)
 
1171
                return "modified"
 
1172
            elif file_in_this:
 
1173
                # OTHER deleted the file
 
1174
                self.tt.unversion_file(trans_id)
 
1175
                return "deleted"
1457
1176
        else:
1458
 
            return 'not_applicable', None
 
1177
            # We have a hypothetical conflict, but if we have files, then we
 
1178
            # can try to merge the content
 
1179
            if this_pair[0] == 'file' and other_pair[0] == 'file':
 
1180
                # THIS and OTHER are both files, so text merge.  Either
 
1181
                # BASE is a file, or both converted to files, so at least we
 
1182
                # have agreement that output should be a file.
 
1183
                try:
 
1184
                    self.text_merge(file_id, trans_id)
 
1185
                except errors.BinaryFile:
 
1186
                    return contents_conflict()
 
1187
                if file_id not in self.this_tree:
 
1188
                    self.tt.version_file(file_id, trans_id)
 
1189
                try:
 
1190
                    self.tt.tree_kind(trans_id)
 
1191
                    self.tt.delete_contents(trans_id)
 
1192
                except errors.NoSuchFile:
 
1193
                    pass
 
1194
                return "modified"
 
1195
            else:
 
1196
                return contents_conflict()
1459
1197
 
1460
1198
    def get_lines(self, tree, file_id):
1461
1199
        """Return the lines in a file, or an empty list."""
1462
 
        if tree.has_id(file_id):
1463
 
            return tree.get_file_lines(file_id)
 
1200
        if file_id in tree:
 
1201
            return tree.get_file(file_id).readlines()
1464
1202
        else:
1465
1203
            return []
1466
1204
 
1468
1206
        """Perform a three-way text merge on a file_id"""
1469
1207
        # it's possible that we got here with base as a different type.
1470
1208
        # if so, we just want two-way text conflicts.
1471
 
        if self.base_tree.has_id(file_id) and \
 
1209
        if file_id in self.base_tree and \
1472
1210
            self.base_tree.kind(file_id) == "file":
1473
1211
            base_lines = self.get_lines(self.base_tree, file_id)
1474
1212
        else:
1537
1275
        versioned = False
1538
1276
        file_group = []
1539
1277
        for suffix, tree, lines in data:
1540
 
            if tree.has_id(file_id):
 
1278
            if file_id in tree:
1541
1279
                trans_id = self._conflict_file(name, parent_id, tree, file_id,
1542
1280
                                               suffix, lines, filter_tree_path)
1543
1281
                file_group.append(trans_id)
1587
1325
        if winner == "this":
1588
1326
            executability = this_executable
1589
1327
        else:
1590
 
            if self.other_tree.has_id(file_id):
 
1328
            if file_id in self.other_tree:
1591
1329
                executability = other_executable
1592
 
            elif self.this_tree.has_id(file_id):
 
1330
            elif file_id in self.this_tree:
1593
1331
                executability = this_executable
1594
 
            elif self.base_tree_has_id(file_id):
 
1332
            elif file_id in self.base_tree:
1595
1333
                executability = base_executable
1596
1334
        if executability is not None:
1597
1335
            trans_id = self.tt.trans_id_file_id(file_id)
1599
1337
 
1600
1338
    def cook_conflicts(self, fs_conflicts):
1601
1339
        """Convert all conflicts into a form that doesn't depend on trans_id"""
 
1340
        name_conflicts = {}
1602
1341
        self.cooked_conflicts.extend(transform.cook_conflicts(
1603
1342
                fs_conflicts, self.tt))
1604
1343
        fp = transform.FinalPaths(self.tt)
1605
1344
        for conflict in self._raw_conflicts:
1606
1345
            conflict_type = conflict[0]
1607
 
            if conflict_type == 'path conflict':
1608
 
                (trans_id, file_id,
1609
 
                this_parent, this_name,
1610
 
                other_parent, other_name) = conflict[1:]
1611
 
                if this_parent is None or this_name is None:
1612
 
                    this_path = '<deleted>'
1613
 
                else:
1614
 
                    parent_path =  fp.get_path(
1615
 
                        self.tt.trans_id_file_id(this_parent))
1616
 
                    this_path = osutils.pathjoin(parent_path, this_name)
1617
 
                if other_parent is None or other_name is None:
1618
 
                    other_path = '<deleted>'
1619
 
                else:
1620
 
                    parent_path =  fp.get_path(
1621
 
                        self.tt.trans_id_file_id(other_parent))
1622
 
                    other_path = osutils.pathjoin(parent_path, other_name)
1623
 
                c = _mod_conflicts.Conflict.factory(
1624
 
                    'path conflict', path=this_path,
1625
 
                    conflict_path=other_path,
1626
 
                    file_id=file_id)
1627
 
            elif conflict_type == 'contents conflict':
 
1346
            if conflict_type in ('name conflict', 'parent conflict'):
 
1347
                trans_id = conflict[1]
 
1348
                conflict_args = conflict[2:]
 
1349
                if trans_id not in name_conflicts:
 
1350
                    name_conflicts[trans_id] = {}
 
1351
                transform.unique_add(name_conflicts[trans_id], conflict_type,
 
1352
                                     conflict_args)
 
1353
            if conflict_type == 'contents conflict':
1628
1354
                for trans_id in conflict[1]:
1629
1355
                    file_id = self.tt.final_file_id(trans_id)
1630
1356
                    if file_id is not None:
1636
1362
                        break
1637
1363
                c = _mod_conflicts.Conflict.factory(conflict_type,
1638
1364
                                                    path=path, file_id=file_id)
1639
 
            elif conflict_type == 'text conflict':
 
1365
                self.cooked_conflicts.append(c)
 
1366
            if conflict_type == 'text conflict':
1640
1367
                trans_id = conflict[1]
1641
1368
                path = fp.get_path(trans_id)
1642
1369
                file_id = self.tt.final_file_id(trans_id)
1643
1370
                c = _mod_conflicts.Conflict.factory(conflict_type,
1644
1371
                                                    path=path, file_id=file_id)
 
1372
                self.cooked_conflicts.append(c)
 
1373
 
 
1374
        for trans_id, conflicts in name_conflicts.iteritems():
 
1375
            try:
 
1376
                this_parent, other_parent = conflicts['parent conflict']
 
1377
                if this_parent == other_parent:
 
1378
                    raise AssertionError()
 
1379
            except KeyError:
 
1380
                this_parent = other_parent = \
 
1381
                    self.tt.final_file_id(self.tt.final_parent(trans_id))
 
1382
            try:
 
1383
                this_name, other_name = conflicts['name conflict']
 
1384
                if this_name == other_name:
 
1385
                    raise AssertionError()
 
1386
            except KeyError:
 
1387
                this_name = other_name = self.tt.final_name(trans_id)
 
1388
            other_path = fp.get_path(trans_id)
 
1389
            if this_parent is not None and this_name is not None:
 
1390
                this_parent_path = \
 
1391
                    fp.get_path(self.tt.trans_id_file_id(this_parent))
 
1392
                this_path = osutils.pathjoin(this_parent_path, this_name)
1645
1393
            else:
1646
 
                raise AssertionError('bad conflict type: %r' % (conflict,))
 
1394
                this_path = "<deleted>"
 
1395
            file_id = self.tt.final_file_id(trans_id)
 
1396
            c = _mod_conflicts.Conflict.factory('path conflict', path=this_path,
 
1397
                                                conflict_path=other_path,
 
1398
                                                file_id=file_id)
1647
1399
            self.cooked_conflicts.append(c)
1648
1400
        self.cooked_conflicts.sort(key=_mod_conflicts.Conflict.sort_key)
1649
1401
 
1764
1516
                other_rev_id=None,
1765
1517
                interesting_files=None,
1766
1518
                this_tree=None,
1767
 
                pb=None,
 
1519
                pb=progress.DummyProgress(),
1768
1520
                change_reporter=None):
1769
1521
    """Primary interface for merging.
1770
1522