/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/workingtree.py

  • Committer: Canonical.com Patch Queue Manager
  • Date: 2009-12-18 10:09:49 UTC
  • mfrom: (4871.5.4 admin-guide-submit)
  • Revision ID: pqm@pqm.ubuntu.com-20091218100949-2c1ityvnbqjtdf3g
(nmb) Add backup section to admin-guide

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/workingtree.py
1
 
# Copyright (C) 2005-2010 Canonical Ltd
 
1
# Copyright (C) 2005, 2006, 2007, 2008, 2009 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
29
29
WorkingTree.open(dir).
30
30
"""
31
31
 
 
32
# TODO: Give the workingtree sole responsibility for the working inventory;
 
33
# remove the variable and references to it from the branch.  This may require
 
34
# updating the commit code so as to update the inventory within the working
 
35
# copy, and making sure there's only one WorkingTree for any directory on disk.
 
36
# At the moment they may alias the inventory and have old copies of it in
 
37
# memory.  (Now done? -- mbp 20060309)
32
38
 
33
39
from cStringIO import StringIO
34
40
import os
61
67
    revisiontree,
62
68
    trace,
63
69
    transform,
64
 
    transport,
65
70
    ui,
66
71
    views,
67
72
    xml5,
68
73
    xml7,
69
74
    )
 
75
import bzrlib.branch
 
76
from bzrlib.transport import get_transport
70
77
from bzrlib.workingtree_4 import (
71
78
    WorkingTreeFormat4,
72
79
    WorkingTreeFormat5,
76
83
 
77
84
from bzrlib import symbol_versioning
78
85
from bzrlib.decorators import needs_read_lock, needs_write_lock
79
 
from bzrlib.lock import LogicalLockResult
80
86
from bzrlib.lockable_files import LockableFiles
81
87
from bzrlib.lockdir import LockDir
82
88
import bzrlib.mutabletree
95
101
from bzrlib.filters import filtered_input_file
96
102
from bzrlib.trace import mutter, note
97
103
from bzrlib.transport.local import LocalTransport
 
104
from bzrlib.progress import DummyProgress, ProgressPhase
98
105
from bzrlib.revision import CURRENT_REVISION
99
106
from bzrlib.rio import RioReader, rio_file, Stanza
100
107
from bzrlib.symbol_versioning import (
104
111
 
105
112
 
106
113
MERGE_MODIFIED_HEADER_1 = "BZR merge-modified list format 1"
107
 
# TODO: Modifying the conflict objects or their type is currently nearly
108
 
# impossible as there is no clear relationship between the working tree format
109
 
# and the conflict list file format.
110
114
CONFLICT_HEADER_1 = "BZR conflict list format 1"
111
115
 
112
116
ERROR_PATH_NOT_FOUND = 3    # WindowsError errno code, equivalent to ENOENT
167
171
        return ''
168
172
 
169
173
 
170
 
class WorkingTree(bzrlib.mutabletree.MutableTree,
171
 
    bzrdir.ControlComponent):
 
174
class WorkingTree(bzrlib.mutabletree.MutableTree):
172
175
    """Working copy tree.
173
176
 
174
177
    The inventory is held in the `Branch` working-inventory, and the
176
179
 
177
180
    It is possible for a `WorkingTree` to have a filename which is
178
181
    not listed in the Inventory and vice versa.
179
 
 
180
 
    :ivar basedir: The root of the tree on disk. This is a unicode path object
181
 
        (as opposed to a URL).
182
182
    """
183
183
 
184
184
    # override this to set the strategy for storing views
250
250
        self._rules_searcher = None
251
251
        self.views = self._make_views()
252
252
 
253
 
    @property
254
 
    def user_transport(self):
255
 
        return self.bzrdir.user_transport
256
 
 
257
 
    @property
258
 
    def control_transport(self):
259
 
        return self._transport
260
 
 
261
253
    def _detect_case_handling(self):
262
254
        wt_trans = self.bzrdir.get_workingtree_transport(None)
263
255
        try:
371
363
                return True, None
372
364
            else:
373
365
                return True, tree
374
 
        t = transport.get_transport(location)
375
 
        iterator = bzrdir.BzrDir.find_bzrdirs(t, evaluate=evaluate,
 
366
        transport = get_transport(location)
 
367
        iterator = bzrdir.BzrDir.find_bzrdirs(transport, evaluate=evaluate,
376
368
                                              list_current=list_current)
377
 
        return [tr for tr in iterator if tr is not None]
 
369
        return [t for t in iterator if t is not None]
378
370
 
379
371
    # should be deprecated - this is slow and in any case treating them as a
380
372
    # container is (we now know) bad style -- mbp 20070302
465
457
        return (file_obj, stat_value)
466
458
 
467
459
    def get_file_text(self, file_id, path=None, filtered=True):
468
 
        my_file = self.get_file(file_id, path=path, filtered=filtered)
469
 
        try:
470
 
            return my_file.read()
471
 
        finally:
472
 
            my_file.close()
 
460
        return self.get_file(file_id, path=path, filtered=filtered).read()
473
461
 
474
462
    def get_file_byname(self, filename, filtered=True):
475
463
        path = self.abspath(filename)
529
517
 
530
518
        # Now we have the parents of this content
531
519
        annotator = self.branch.repository.texts.get_annotator()
532
 
        text = self.get_file_text(file_id)
 
520
        text = self.get_file(file_id).read()
533
521
        this_key =(file_id, default_revision)
534
522
        annotator.add_special_text(this_key, file_parent_keys, text)
535
523
        annotations = [(key[-1], line)
921
909
            branch.last_revision().
922
910
        """
923
911
        from bzrlib.merge import Merger, Merge3Merger
924
 
        merger = Merger(self.branch, this_tree=self)
925
 
        # check that there are no local alterations
926
 
        if not force and self.has_changes():
927
 
            raise errors.UncommittedChanges(self)
928
 
        if to_revision is None:
929
 
            to_revision = _mod_revision.ensure_null(branch.last_revision())
930
 
        merger.other_rev_id = to_revision
931
 
        if _mod_revision.is_null(merger.other_rev_id):
932
 
            raise errors.NoCommits(branch)
933
 
        self.branch.fetch(branch, last_revision=merger.other_rev_id)
934
 
        merger.other_basis = merger.other_rev_id
935
 
        merger.other_tree = self.branch.repository.revision_tree(
936
 
            merger.other_rev_id)
937
 
        merger.other_branch = branch
938
 
        if from_revision is None:
939
 
            merger.find_base()
940
 
        else:
941
 
            merger.set_base_revision(from_revision, branch)
942
 
        if merger.base_rev_id == merger.other_rev_id:
943
 
            raise errors.PointlessMerge
944
 
        merger.backup_files = False
945
 
        if merge_type is None:
946
 
            merger.merge_type = Merge3Merger
947
 
        else:
948
 
            merger.merge_type = merge_type
949
 
        merger.set_interesting_files(None)
950
 
        merger.show_base = False
951
 
        merger.reprocess = False
952
 
        conflicts = merger.do_merge()
953
 
        merger.set_pending()
 
912
        pb = ui.ui_factory.nested_progress_bar()
 
913
        try:
 
914
            merger = Merger(self.branch, this_tree=self, pb=pb)
 
915
            merger.pp = ProgressPhase("Merge phase", 5, pb)
 
916
            merger.pp.next_phase()
 
917
            # check that there are no local alterations
 
918
            if not force and self.has_changes():
 
919
                raise errors.UncommittedChanges(self)
 
920
            if to_revision is None:
 
921
                to_revision = _mod_revision.ensure_null(branch.last_revision())
 
922
            merger.other_rev_id = to_revision
 
923
            if _mod_revision.is_null(merger.other_rev_id):
 
924
                raise errors.NoCommits(branch)
 
925
            self.branch.fetch(branch, last_revision=merger.other_rev_id)
 
926
            merger.other_basis = merger.other_rev_id
 
927
            merger.other_tree = self.branch.repository.revision_tree(
 
928
                merger.other_rev_id)
 
929
            merger.other_branch = branch
 
930
            merger.pp.next_phase()
 
931
            if from_revision is None:
 
932
                merger.find_base()
 
933
            else:
 
934
                merger.set_base_revision(from_revision, branch)
 
935
            if merger.base_rev_id == merger.other_rev_id:
 
936
                raise errors.PointlessMerge
 
937
            merger.backup_files = False
 
938
            if merge_type is None:
 
939
                merger.merge_type = Merge3Merger
 
940
            else:
 
941
                merger.merge_type = merge_type
 
942
            merger.set_interesting_files(None)
 
943
            merger.show_base = False
 
944
            merger.reprocess = False
 
945
            conflicts = merger.do_merge()
 
946
            merger.set_pending()
 
947
        finally:
 
948
            pb.finished()
954
949
        return conflicts
955
950
 
956
951
    @needs_read_lock
1103
1098
        tree_transport = self.bzrdir.root_transport.clone(sub_path)
1104
1099
        if tree_transport.base != branch_transport.base:
1105
1100
            tree_bzrdir = format.initialize_on_transport(tree_transport)
1106
 
            branch.BranchReferenceFormat().initialize(tree_bzrdir,
1107
 
                target_branch=new_branch)
 
1101
            branch.BranchReferenceFormat().initialize(tree_bzrdir, new_branch)
1108
1102
        else:
1109
1103
            tree_bzrdir = branch_bzrdir
1110
1104
        wt = tree_bzrdir.create_workingtree(_mod_revision.NULL_REVISION)
1148
1142
        This does not include files that have been deleted in this
1149
1143
        tree. Skips the control directory.
1150
1144
 
1151
 
        :param include_root: if True, return an entry for the root
 
1145
        :param include_root: if True, do not return an entry for the root
1152
1146
        :param from_dir: start from this directory or None for the root
1153
1147
        :param recursive: whether to recurse into subdirectories or not
1154
1148
        """
1608
1602
    @needs_write_lock
1609
1603
    def pull(self, source, overwrite=False, stop_revision=None,
1610
1604
             change_reporter=None, possible_transports=None, local=False):
 
1605
        top_pb = ui.ui_factory.nested_progress_bar()
1611
1606
        source.lock_read()
1612
1607
        try:
 
1608
            pp = ProgressPhase("Pull phase", 2, top_pb)
 
1609
            pp.next_phase()
1613
1610
            old_revision_info = self.branch.last_revision_info()
1614
1611
            basis_tree = self.basis_tree()
1615
1612
            count = self.branch.pull(source, overwrite, stop_revision,
1617
1614
                                     local=local)
1618
1615
            new_revision_info = self.branch.last_revision_info()
1619
1616
            if new_revision_info != old_revision_info:
 
1617
                pp.next_phase()
1620
1618
                repository = self.branch.repository
 
1619
                pb = ui.ui_factory.nested_progress_bar()
1621
1620
                basis_tree.lock_read()
1622
1621
                try:
1623
1622
                    new_basis_tree = self.branch.basis_tree()
1626
1625
                                new_basis_tree,
1627
1626
                                basis_tree,
1628
1627
                                this_tree=self,
1629
 
                                pb=None,
 
1628
                                pb=pb,
1630
1629
                                change_reporter=change_reporter)
1631
 
                    basis_root_id = basis_tree.get_root_id()
1632
 
                    new_root_id = new_basis_tree.get_root_id()
1633
 
                    if basis_root_id != new_root_id:
1634
 
                        self.set_root_id(new_root_id)
 
1630
                    if (basis_tree.inventory.root is None and
 
1631
                        new_basis_tree.inventory.root is not None):
 
1632
                        self.set_root_id(new_basis_tree.get_root_id())
1635
1633
                finally:
 
1634
                    pb.finished()
1636
1635
                    basis_tree.unlock()
1637
1636
                # TODO - dedup parents list with things merged by pull ?
1638
1637
                # reuse the revisiontree we merged against to set the new
1651
1650
            return count
1652
1651
        finally:
1653
1652
            source.unlock()
 
1653
            top_pb.finished()
1654
1654
 
1655
1655
    @needs_write_lock
1656
1656
    def put_file_bytes_non_atomic(self, file_id, bytes):
1741
1741
        r"""Check whether the filename matches an ignore pattern.
1742
1742
 
1743
1743
        Patterns containing '/' or '\' need to match the whole path;
1744
 
        others match against only the last component.  Patterns starting
1745
 
        with '!' are ignore exceptions.  Exceptions take precedence
1746
 
        over regular patterns and cause the filename to not be ignored.
 
1744
        others match against only the last component.
1747
1745
 
1748
1746
        If the file is ignored, returns the pattern which caused it to
1749
1747
        be ignored, otherwise None.  So this can simply be used as a
1750
1748
        boolean if desired."""
1751
1749
        if getattr(self, '_ignoreglobster', None) is None:
1752
 
            self._ignoreglobster = globbing.ExceptionGlobster(self.get_ignore_list())
 
1750
            self._ignoreglobster = globbing.Globster(self.get_ignore_list())
1753
1751
        return self._ignoreglobster.match(filename)
1754
1752
 
1755
1753
    def kind(self, file_id):
1805
1803
            raise errors.ObjectNotLocked(self)
1806
1804
 
1807
1805
    def lock_read(self):
1808
 
        """Lock the tree for reading.
1809
 
 
1810
 
        This also locks the branch, and can be unlocked via self.unlock().
1811
 
 
1812
 
        :return: A bzrlib.lock.LogicalLockResult.
1813
 
        """
 
1806
        """See Branch.lock_read, and WorkingTree.unlock."""
1814
1807
        if not self.is_locked():
1815
1808
            self._reset_data()
1816
1809
        self.branch.lock_read()
1817
1810
        try:
1818
 
            self._control_files.lock_read()
1819
 
            return LogicalLockResult(self.unlock)
 
1811
            return self._control_files.lock_read()
1820
1812
        except:
1821
1813
            self.branch.unlock()
1822
1814
            raise
1823
1815
 
1824
1816
    def lock_tree_write(self):
1825
 
        """See MutableTree.lock_tree_write, and WorkingTree.unlock.
1826
 
 
1827
 
        :return: A bzrlib.lock.LogicalLockResult.
1828
 
        """
 
1817
        """See MutableTree.lock_tree_write, and WorkingTree.unlock."""
1829
1818
        if not self.is_locked():
1830
1819
            self._reset_data()
1831
1820
        self.branch.lock_read()
1832
1821
        try:
1833
 
            self._control_files.lock_write()
1834
 
            return LogicalLockResult(self.unlock)
 
1822
            return self._control_files.lock_write()
1835
1823
        except:
1836
1824
            self.branch.unlock()
1837
1825
            raise
1838
1826
 
1839
1827
    def lock_write(self):
1840
 
        """See MutableTree.lock_write, and WorkingTree.unlock.
1841
 
 
1842
 
        :return: A bzrlib.lock.LogicalLockResult.
1843
 
        """
 
1828
        """See MutableTree.lock_write, and WorkingTree.unlock."""
1844
1829
        if not self.is_locked():
1845
1830
            self._reset_data()
1846
1831
        self.branch.lock_write()
1847
1832
        try:
1848
 
            self._control_files.lock_write()
1849
 
            return LogicalLockResult(self.unlock)
 
1833
            return self._control_files.lock_write()
1850
1834
        except:
1851
1835
            self.branch.unlock()
1852
1836
            raise
1917
1901
            # revision_id is set. We must check for this full string, because a
1918
1902
            # root node id can legitimately look like 'revision_id' but cannot
1919
1903
            # contain a '"'.
1920
 
            xml = self.branch.repository._get_inventory_xml(new_revision)
 
1904
            xml = self.branch.repository.get_inventory_xml(new_revision)
1921
1905
            firstline = xml.split('\n', 1)[0]
1922
1906
            if (not 'revision_id="' in firstline or
1923
1907
                'format="7"' not in firstline):
1977
1961
        def recurse_directory_to_add_files(directory):
1978
1962
            # Recurse directory and add all files
1979
1963
            # so we can check if they have changed.
1980
 
            for parent_info, file_infos in self.walkdirs(directory):
 
1964
            for parent_info, file_infos in\
 
1965
                self.walkdirs(directory):
1981
1966
                for relpath, basename, kind, lstat, fileid, kind in file_infos:
1982
1967
                    # Is it versioned or ignored?
1983
1968
                    if self.path2id(relpath) or self.is_ignored(relpath):
2018
2003
                            # ... but not ignored
2019
2004
                            has_changed_files = True
2020
2005
                            break
2021
 
                    elif (content_change and (kind[1] is not None) and
2022
 
                            osutils.is_inside_any(files, path[1])):
2023
 
                        # Versioned and changed, but not deleted, and still
2024
 
                        # in one of the dirs to be deleted.
 
2006
                    elif content_change and (kind[1] is not None):
 
2007
                        # Versioned and changed, but not deleted
2025
2008
                        has_changed_files = True
2026
2009
                        break
2027
2010
 
2080
2063
 
2081
2064
    @needs_tree_write_lock
2082
2065
    def revert(self, filenames=None, old_tree=None, backups=True,
2083
 
               pb=None, report_changes=False):
 
2066
               pb=DummyProgress(), report_changes=False):
2084
2067
        from bzrlib.conflicts import resolve
2085
2068
        if filenames == []:
2086
2069
            filenames = None
2208
2191
        """
2209
2192
        raise NotImplementedError(self.unlock)
2210
2193
 
2211
 
    _marker = object()
2212
 
 
2213
 
    def update(self, change_reporter=None, possible_transports=None,
2214
 
               revision=None, old_tip=_marker):
 
2194
    def update(self, change_reporter=None, possible_transports=None):
2215
2195
        """Update a working tree along its branch.
2216
2196
 
2217
2197
        This will update the branch if its bound too, which means we have
2235
2215
        - Merge current state -> basis tree of the master w.r.t. the old tree
2236
2216
          basis.
2237
2217
        - Do a 'normal' merge of the old branch basis if it is relevant.
2238
 
 
2239
 
        :param revision: The target revision to update to. Must be in the
2240
 
            revision history.
2241
 
        :param old_tip: If branch.update() has already been run, the value it
2242
 
            returned (old tip of the branch or None). _marker is used
2243
 
            otherwise.
2244
2218
        """
2245
2219
        if self.branch.get_bound_location() is not None:
2246
2220
            self.lock_write()
2247
 
            update_branch = (old_tip is self._marker)
 
2221
            update_branch = True
2248
2222
        else:
2249
2223
            self.lock_tree_write()
2250
2224
            update_branch = False
2252
2226
            if update_branch:
2253
2227
                old_tip = self.branch.update(possible_transports)
2254
2228
            else:
2255
 
                if old_tip is self._marker:
2256
 
                    old_tip = None
2257
 
            return self._update_tree(old_tip, change_reporter, revision)
 
2229
                old_tip = None
 
2230
            return self._update_tree(old_tip, change_reporter)
2258
2231
        finally:
2259
2232
            self.unlock()
2260
2233
 
2261
2234
    @needs_tree_write_lock
2262
 
    def _update_tree(self, old_tip=None, change_reporter=None, revision=None):
 
2235
    def _update_tree(self, old_tip=None, change_reporter=None):
2263
2236
        """Update a tree to the master branch.
2264
2237
 
2265
2238
        :param old_tip: if supplied, the previous tip revision the branch,
2275
2248
        # We MUST save it even if an error occurs, because otherwise the users
2276
2249
        # local work is unreferenced and will appear to have been lost.
2277
2250
        #
2278
 
        nb_conflicts = 0
 
2251
        result = 0
2279
2252
        try:
2280
2253
            last_rev = self.get_parent_ids()[0]
2281
2254
        except IndexError:
2282
2255
            last_rev = _mod_revision.NULL_REVISION
2283
 
        if revision is None:
2284
 
            revision = self.branch.last_revision()
2285
 
 
2286
 
        old_tip = old_tip or _mod_revision.NULL_REVISION
2287
 
 
2288
 
        if not _mod_revision.is_null(old_tip) and old_tip != last_rev:
2289
 
            # the branch we are bound to was updated
2290
 
            # merge those changes in first
2291
 
            base_tree  = self.basis_tree()
2292
 
            other_tree = self.branch.repository.revision_tree(old_tip)
2293
 
            nb_conflicts = merge.merge_inner(self.branch, other_tree,
2294
 
                                             base_tree, this_tree=self,
2295
 
                                             change_reporter=change_reporter)
2296
 
            if nb_conflicts:
2297
 
                self.add_parent_tree((old_tip, other_tree))
2298
 
                trace.note('Rerun update after fixing the conflicts.')
2299
 
                return nb_conflicts
2300
 
 
2301
 
        if last_rev != _mod_revision.ensure_null(revision):
2302
 
            # the working tree is up to date with the branch
2303
 
            # we can merge the specified revision from master
2304
 
            to_tree = self.branch.repository.revision_tree(revision)
2305
 
            to_root_id = to_tree.get_root_id()
2306
 
 
 
2256
        if last_rev != _mod_revision.ensure_null(self.branch.last_revision()):
 
2257
            # merge tree state up to new branch tip.
2307
2258
            basis = self.basis_tree()
2308
2259
            basis.lock_read()
2309
2260
            try:
2310
 
                if (basis.inventory.root is None
2311
 
                    or basis.inventory.root.file_id != to_root_id):
2312
 
                    self.set_root_id(to_root_id)
 
2261
                to_tree = self.branch.basis_tree()
 
2262
                if basis.inventory.root is None:
 
2263
                    self.set_root_id(to_tree.get_root_id())
2313
2264
                    self.flush()
 
2265
                result += merge.merge_inner(
 
2266
                                      self.branch,
 
2267
                                      to_tree,
 
2268
                                      basis,
 
2269
                                      this_tree=self,
 
2270
                                      change_reporter=change_reporter)
2314
2271
            finally:
2315
2272
                basis.unlock()
2316
 
 
2317
 
            # determine the branch point
2318
 
            graph = self.branch.repository.get_graph()
2319
 
            base_rev_id = graph.find_unique_lca(self.branch.last_revision(),
2320
 
                                                last_rev)
2321
 
            base_tree = self.branch.repository.revision_tree(base_rev_id)
2322
 
 
2323
 
            nb_conflicts = merge.merge_inner(self.branch, to_tree, base_tree,
2324
 
                                             this_tree=self,
2325
 
                                             change_reporter=change_reporter)
2326
 
            self.set_last_revision(revision)
2327
2273
            # TODO - dedup parents list with things merged by pull ?
2328
2274
            # reuse the tree we've updated to to set the basis:
2329
 
            parent_trees = [(revision, to_tree)]
 
2275
            parent_trees = [(self.branch.last_revision(), to_tree)]
2330
2276
            merges = self.get_parent_ids()[1:]
2331
2277
            # Ideally we ask the tree for the trees here, that way the working
2332
2278
            # tree can decide whether to give us the entire tree or give us a
2336
2282
            for parent in merges:
2337
2283
                parent_trees.append(
2338
2284
                    (parent, self.branch.repository.revision_tree(parent)))
2339
 
            if not _mod_revision.is_null(old_tip):
 
2285
            if (old_tip is not None and not _mod_revision.is_null(old_tip)):
2340
2286
                parent_trees.append(
2341
2287
                    (old_tip, self.branch.repository.revision_tree(old_tip)))
2342
2288
            self.set_parent_trees(parent_trees)
2343
2289
            last_rev = parent_trees[0][0]
2344
 
        return nb_conflicts
 
2290
        else:
 
2291
            # the working tree had the same last-revision as the master
 
2292
            # branch did. We may still have pivot local work from the local
 
2293
            # branch into old_tip:
 
2294
            if (old_tip is not None and not _mod_revision.is_null(old_tip)):
 
2295
                self.add_parent_tree_id(old_tip)
 
2296
        if (old_tip is not None and not _mod_revision.is_null(old_tip)
 
2297
            and old_tip != last_rev):
 
2298
            # our last revision was not the prior branch last revision
 
2299
            # and we have converted that last revision to a pending merge.
 
2300
            # base is somewhere between the branch tip now
 
2301
            # and the now pending merge
 
2302
 
 
2303
            # Since we just modified the working tree and inventory, flush out
 
2304
            # the current state, before we modify it again.
 
2305
            # TODO: jam 20070214 WorkingTree3 doesn't require this, dirstate
 
2306
            #       requires it only because TreeTransform directly munges the
 
2307
            #       inventory and calls tree._write_inventory(). Ultimately we
 
2308
            #       should be able to remove this extra flush.
 
2309
            self.flush()
 
2310
            graph = self.branch.repository.get_graph()
 
2311
            base_rev_id = graph.find_unique_lca(self.branch.last_revision(),
 
2312
                                                old_tip)
 
2313
            base_tree = self.branch.repository.revision_tree(base_rev_id)
 
2314
            other_tree = self.branch.repository.revision_tree(old_tip)
 
2315
            result += merge.merge_inner(
 
2316
                                  self.branch,
 
2317
                                  other_tree,
 
2318
                                  base_tree,
 
2319
                                  this_tree=self,
 
2320
                                  change_reporter=change_reporter)
 
2321
        return result
2345
2322
 
2346
2323
    def _write_hashcache_if_dirty(self):
2347
2324
        """Write out the hashcache if it is dirty."""
2658
2635
 
2659
2636
        In Format2 WorkingTrees we have a single lock for the branch and tree
2660
2637
        so lock_tree_write() degrades to lock_write().
2661
 
 
2662
 
        :return: An object with an unlock method which will release the lock
2663
 
            obtained.
2664
2638
        """
2665
2639
        self.branch.lock_write()
2666
2640
        try:
2667
 
            self._control_files.lock_write()
2668
 
            return self
 
2641
            return self._control_files.lock_write()
2669
2642
        except:
2670
2643
            self.branch.unlock()
2671
2644
            raise