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

  • Committer: John Arbash Meinel
  • Date: 2011-01-10 22:20:12 UTC
  • mfrom: (5582 +trunk)
  • mto: This revision was merged to the branch mainline in revision 5599.
  • Revision ID: john@arbash-meinel.com-20110110222012-mtcqudkvmzwiufuc
Merge in the bzr.dev 5582

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2005-2010 Canonical Ltd
 
1
# Copyright (C) 2005-2011 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
25
25
        bzrdir,
26
26
        cache_utf8,
27
27
        config as _mod_config,
 
28
        controldir,
28
29
        debug,
29
30
        errors,
30
31
        lockdir,
31
32
        lockable_files,
 
33
        remote,
32
34
        repository,
33
35
        revision as _mod_revision,
34
36
        rio,
49
51
from bzrlib.decorators import needs_read_lock, needs_write_lock, only_raises
50
52
from bzrlib.hooks import HookPoint, Hooks
51
53
from bzrlib.inter import InterObject
52
 
from bzrlib.lock import _RelockDebugMixin
 
54
from bzrlib.lock import _RelockDebugMixin, LogicalLockResult
53
55
from bzrlib import registry
54
56
from bzrlib.symbol_versioning import (
55
57
    deprecated_in,
63
65
BZR_BRANCH_FORMAT_6 = "Bazaar Branch Format 6 (bzr 0.15)\n"
64
66
 
65
67
 
66
 
class Branch(bzrdir.ControlComponent):
 
68
class Branch(controldir.ControlComponent):
67
69
    """Branch holding a history of revisions.
68
70
 
69
71
    :ivar base:
90
92
        self._revision_id_to_revno_cache = None
91
93
        self._partial_revision_id_to_revno_cache = {}
92
94
        self._partial_revision_history_cache = []
 
95
        self._tags_bytes = None
93
96
        self._last_revision_info_cache = None
94
97
        self._merge_sorted_revisions_cache = None
95
98
        self._open_hook()
102
105
 
103
106
    def _activate_fallback_location(self, url):
104
107
        """Activate the branch/repository from url as a fallback repository."""
 
108
        for existing_fallback_repo in self.repository._fallback_repositories:
 
109
            if existing_fallback_repo.user_url == url:
 
110
                # This fallback is already configured.  This probably only
 
111
                # happens because BzrDir.sprout is a horrible mess.  To avoid
 
112
                # confusing _unstack we don't add this a second time.
 
113
                mutter('duplicate activation of fallback %r on %r', url, self)
 
114
                return
105
115
        repo = self._get_fallback_repository(url)
106
116
        if repo.has_same_location(self.repository):
107
117
            raise errors.UnstackableLocationError(self.user_url, url)
197
207
        return self.supports_tags() and self.tags.get_tag_dict()
198
208
 
199
209
    def get_config(self):
 
210
        """Get a bzrlib.config.BranchConfig for this Branch.
 
211
 
 
212
        This can then be used to get and set configuration options for the
 
213
        branch.
 
214
 
 
215
        :return: A bzrlib.config.BranchConfig.
 
216
        """
200
217
        return BranchConfig(self)
201
218
 
202
219
    def _get_config(self):
218
235
            possible_transports=[self.bzrdir.root_transport])
219
236
        return a_branch.repository
220
237
 
 
238
    @needs_read_lock
221
239
    def _get_tags_bytes(self):
222
240
        """Get the bytes of a serialised tags dict.
223
241
 
230
248
        :return: The bytes of the tags file.
231
249
        :seealso: Branch._set_tags_bytes.
232
250
        """
233
 
        return self._transport.get_bytes('tags')
 
251
        if self._tags_bytes is None:
 
252
            self._tags_bytes = self._transport.get_bytes('tags')
 
253
        return self._tags_bytes
234
254
 
235
255
    def _get_nick(self, local=False, possible_transports=None):
236
256
        config = self.get_config()
238
258
        if not local and not config.has_explicit_nickname():
239
259
            try:
240
260
                master = self.get_master_branch(possible_transports)
 
261
                if master and self.user_url == master.user_url:
 
262
                    raise errors.RecursiveBind(self.user_url)
241
263
                if master is not None:
242
264
                    # return the master branch value
243
265
                    return master.nick
 
266
            except errors.RecursiveBind, e:
 
267
                raise e
244
268
            except errors.BzrError, e:
245
269
                # Silently fall back to local implicit nick if the master is
246
270
                # unavailable
283
307
        new_history.reverse()
284
308
        return new_history
285
309
 
286
 
    def lock_write(self):
 
310
    def lock_write(self, token=None):
 
311
        """Lock the branch for write operations.
 
312
 
 
313
        :param token: A token to permit reacquiring a previously held and
 
314
            preserved lock.
 
315
        :return: A BranchWriteLockResult.
 
316
        """
287
317
        raise NotImplementedError(self.lock_write)
288
318
 
289
319
    def lock_read(self):
 
320
        """Lock the branch for read operations.
 
321
 
 
322
        :return: A bzrlib.lock.LogicalLockResult.
 
323
        """
290
324
        raise NotImplementedError(self.lock_read)
291
325
 
292
326
    def unlock(self):
782
816
            old_repository = self.repository
783
817
            if len(old_repository._fallback_repositories) != 1:
784
818
                raise AssertionError("can't cope with fallback repositories "
785
 
                    "of %r" % (self.repository,))
786
 
            # unlock it, including unlocking the fallback
 
819
                    "of %r (fallbacks: %r)" % (old_repository,
 
820
                        old_repository._fallback_repositories))
 
821
            # Open the new repository object.
 
822
            # Repositories don't offer an interface to remove fallback
 
823
            # repositories today; take the conceptually simpler option and just
 
824
            # reopen it.  We reopen it starting from the URL so that we
 
825
            # get a separate connection for RemoteRepositories and can
 
826
            # stream from one of them to the other.  This does mean doing
 
827
            # separate SSH connection setup, but unstacking is not a
 
828
            # common operation so it's tolerable.
 
829
            new_bzrdir = bzrdir.BzrDir.open(self.bzrdir.root_transport.base)
 
830
            new_repository = new_bzrdir.find_repository()
 
831
            if new_repository._fallback_repositories:
 
832
                raise AssertionError("didn't expect %r to have "
 
833
                    "fallback_repositories"
 
834
                    % (self.repository,))
 
835
            # Replace self.repository with the new repository.
 
836
            # Do our best to transfer the lock state (i.e. lock-tokens and
 
837
            # lock count) of self.repository to the new repository.
 
838
            lock_token = old_repository.lock_write().repository_token
 
839
            self.repository = new_repository
 
840
            if isinstance(self, remote.RemoteBranch):
 
841
                # Remote branches can have a second reference to the old
 
842
                # repository that need to be replaced.
 
843
                if self._real_branch is not None:
 
844
                    self._real_branch.repository = new_repository
 
845
            self.repository.lock_write(token=lock_token)
 
846
            if lock_token is not None:
 
847
                old_repository.leave_lock_in_place()
787
848
            old_repository.unlock()
 
849
            if lock_token is not None:
 
850
                # XXX: self.repository.leave_lock_in_place() before this
 
851
                # function will not be preserved.  Fortunately that doesn't
 
852
                # affect the current default format (2a), and would be a
 
853
                # corner-case anyway.
 
854
                #  - Andrew Bennetts, 2010/06/30
 
855
                self.repository.dont_leave_lock_in_place()
 
856
            old_lock_count = 0
 
857
            while True:
 
858
                try:
 
859
                    old_repository.unlock()
 
860
                except errors.LockNotHeld:
 
861
                    break
 
862
                old_lock_count += 1
 
863
            if old_lock_count == 0:
 
864
                raise AssertionError(
 
865
                    'old_repository should have been locked at least once.')
 
866
            for i in range(old_lock_count-1):
 
867
                self.repository.lock_write()
 
868
            # Fetch from the old repository into the new.
788
869
            old_repository.lock_read()
789
870
            try:
790
 
                # Repositories don't offer an interface to remove fallback
791
 
                # repositories today; take the conceptually simpler option and just
792
 
                # reopen it.  We reopen it starting from the URL so that we
793
 
                # get a separate connection for RemoteRepositories and can
794
 
                # stream from one of them to the other.  This does mean doing
795
 
                # separate SSH connection setup, but unstacking is not a
796
 
                # common operation so it's tolerable.
797
 
                new_bzrdir = bzrdir.BzrDir.open(self.bzrdir.root_transport.base)
798
 
                new_repository = new_bzrdir.find_repository()
799
 
                self.repository = new_repository
800
 
                if self.repository._fallback_repositories:
801
 
                    raise AssertionError("didn't expect %r to have "
802
 
                        "fallback_repositories"
803
 
                        % (self.repository,))
804
 
                # this is not paired with an unlock because it's just restoring
805
 
                # the previous state; the lock's released when set_stacked_on_url
806
 
                # returns
807
 
                self.repository.lock_write()
808
871
                # XXX: If you unstack a branch while it has a working tree
809
872
                # with a pending merge, the pending-merged revisions will no
810
873
                # longer be present.  You can (probably) revert and remerge.
825
888
 
826
889
        :seealso: Branch._get_tags_bytes.
827
890
        """
828
 
        return _run_with_write_locked_target(self, self._transport.put_bytes,
829
 
            'tags', bytes)
 
891
        return _run_with_write_locked_target(self, self._set_tags_bytes_locked,
 
892
                bytes)
 
893
 
 
894
    def _set_tags_bytes_locked(self, bytes):
 
895
        self._tags_bytes = bytes
 
896
        return self._transport.put_bytes('tags', bytes)
830
897
 
831
898
    def _cache_revision_history(self, rev_history):
832
899
        """Set the cached revision history to rev_history.
862
929
        self._merge_sorted_revisions_cache = None
863
930
        self._partial_revision_history_cache = []
864
931
        self._partial_revision_id_to_revno_cache = {}
 
932
        self._tags_bytes = None
865
933
 
866
934
    def _gen_revision_history(self):
867
935
        """Return sequence of revision hashes on to this branch.
951
1019
                raise errors.NoSuchRevision(self, stop_revision)
952
1020
        return other_history[self_len:stop_revision]
953
1021
 
954
 
    @needs_write_lock
955
1022
    def update_revisions(self, other, stop_revision=None, overwrite=False,
956
1023
                         graph=None):
957
1024
        """Pull in new perfect-fit revisions.
1006
1073
            self._extend_partial_history(distance_from_last)
1007
1074
        return self._partial_revision_history_cache[distance_from_last]
1008
1075
 
1009
 
    @needs_write_lock
1010
1076
    def pull(self, source, overwrite=False, stop_revision=None,
1011
1077
             possible_transports=None, *args, **kwargs):
1012
1078
        """Mirror source into this branch.
1208
1274
        return result
1209
1275
 
1210
1276
    @needs_read_lock
1211
 
    def sprout(self, to_bzrdir, revision_id=None, repository_policy=None):
 
1277
    def sprout(self, to_bzrdir, revision_id=None, repository_policy=None,
 
1278
            repository=None):
1212
1279
        """Create a new line of development from the branch, into to_bzrdir.
1213
1280
 
1214
1281
        to_bzrdir controls the branch format.
1219
1286
        if (repository_policy is not None and
1220
1287
            repository_policy.requires_stacking()):
1221
1288
            to_bzrdir._format.require_stacking(_skip_repo=True)
1222
 
        result = to_bzrdir.create_branch()
 
1289
        result = to_bzrdir.create_branch(repository=repository)
1223
1290
        result.lock_write()
1224
1291
        try:
1225
1292
            if repository_policy is not None:
1255
1322
                revno = 1
1256
1323
        destination.set_last_revision_info(revno, revision_id)
1257
1324
 
1258
 
    @needs_read_lock
1259
1325
    def copy_content_into(self, destination, revision_id=None):
1260
1326
        """Copy the content of self into destination.
1261
1327
 
1262
1328
        revision_id: if not None, the revision history in the new branch will
1263
1329
                     be truncated to end with revision_id.
1264
1330
        """
1265
 
        self.update_references(destination)
1266
 
        self._synchronize_history(destination, revision_id)
1267
 
        try:
1268
 
            parent = self.get_parent()
1269
 
        except errors.InaccessibleParent, e:
1270
 
            mutter('parent was not accessible to copy: %s', e)
1271
 
        else:
1272
 
            if parent:
1273
 
                destination.set_parent(parent)
1274
 
        if self._push_should_merge_tags():
1275
 
            self.tags.merge_to(destination.tags)
 
1331
        return InterBranch.get(self, destination).copy_content_into(
 
1332
            revision_id=revision_id)
1276
1333
 
1277
1334
    def update_references(self, target):
1278
1335
        if not getattr(self._format, 'supports_reference_locations', False):
1333
1390
        return format
1334
1391
 
1335
1392
    def create_clone_on_transport(self, to_transport, revision_id=None,
1336
 
        stacked_on=None, create_prefix=False, use_existing_dir=False):
 
1393
        stacked_on=None, create_prefix=False, use_existing_dir=False,
 
1394
        no_tree=None):
1337
1395
        """Create a clone of this branch and its bzrdir.
1338
1396
 
1339
1397
        :param to_transport: The transport to clone onto.
1346
1404
        """
1347
1405
        # XXX: Fix the bzrdir API to allow getting the branch back from the
1348
1406
        # clone call. Or something. 20090224 RBC/spiv.
 
1407
        # XXX: Should this perhaps clone colocated branches as well, 
 
1408
        # rather than just the default branch? 20100319 JRV
1349
1409
        if revision_id is None:
1350
1410
            revision_id = self.last_revision()
1351
1411
        dir_to = self.bzrdir.clone_on_transport(to_transport,
1352
1412
            revision_id=revision_id, stacked_on=stacked_on,
1353
 
            create_prefix=create_prefix, use_existing_dir=use_existing_dir)
 
1413
            create_prefix=create_prefix, use_existing_dir=use_existing_dir,
 
1414
            no_tree=no_tree)
1354
1415
        return dir_to.open_branch()
1355
1416
 
1356
1417
    def create_checkout(self, to_location, revision_id=None,
1481
1542
     * an open routine.
1482
1543
 
1483
1544
    Formats are placed in an dict by their format string for reference
1484
 
    during branch opening. Its not required that these be instances, they
 
1545
    during branch opening. It's not required that these be instances, they
1485
1546
    can be classes themselves with class methods - it simply depends on
1486
1547
    whether state is needed for a given format or not.
1487
1548
 
1510
1571
        try:
1511
1572
            transport = a_bzrdir.get_branch_transport(None, name=name)
1512
1573
            format_string = transport.get_bytes("format")
1513
 
            return klass._formats[format_string]
 
1574
            format = klass._formats[format_string]
 
1575
            if isinstance(format, MetaDirBranchFormatFactory):
 
1576
                return format()
 
1577
            return format
1514
1578
        except errors.NoSuchFile:
1515
1579
            raise errors.NotBranchError(path=transport.base, bzrdir=a_bzrdir)
1516
1580
        except KeyError:
1521
1585
        """Return the current default format."""
1522
1586
        return klass._default_format
1523
1587
 
1524
 
    def get_reference(self, a_bzrdir):
 
1588
    @classmethod
 
1589
    def get_formats(klass):
 
1590
        """Get all the known formats.
 
1591
 
 
1592
        Warning: This triggers a load of all lazy registered formats: do not
 
1593
        use except when that is desireed.
 
1594
        """
 
1595
        result = []
 
1596
        for fmt in klass._formats.values():
 
1597
            if isinstance(fmt, MetaDirBranchFormatFactory):
 
1598
                fmt = fmt()
 
1599
            result.append(fmt)
 
1600
        return result
 
1601
 
 
1602
    def get_reference(self, a_bzrdir, name=None):
1525
1603
        """Get the target reference of the branch in a_bzrdir.
1526
1604
 
1527
1605
        format probing must have been completed before calling
1529
1607
        in a_bzrdir is correct.
1530
1608
 
1531
1609
        :param a_bzrdir: The bzrdir to get the branch data from.
 
1610
        :param name: Name of the colocated branch to fetch
1532
1611
        :return: None if the branch is not a reference branch.
1533
1612
        """
1534
1613
        return None
1535
1614
 
1536
1615
    @classmethod
1537
 
    def set_reference(self, a_bzrdir, to_branch):
 
1616
    def set_reference(self, a_bzrdir, name, to_branch):
1538
1617
        """Set the target reference of the branch in a_bzrdir.
1539
1618
 
1540
1619
        format probing must have been completed before calling
1542
1621
        in a_bzrdir is correct.
1543
1622
 
1544
1623
        :param a_bzrdir: The bzrdir to set the branch reference for.
 
1624
        :param name: Name of colocated branch to set, None for default
1545
1625
        :param to_branch: branch that the checkout is to reference
1546
1626
        """
1547
1627
        raise NotImplementedError(self.set_reference)
1563
1643
            hook(params)
1564
1644
 
1565
1645
    def _initialize_helper(self, a_bzrdir, utf8_files, name=None,
1566
 
                           lock_type='metadir', set_format=True):
 
1646
                           repository=None, lock_type='metadir',
 
1647
                           set_format=True):
1567
1648
        """Initialize a branch in a bzrdir, with specified files
1568
1649
 
1569
1650
        :param a_bzrdir: The bzrdir to initialize the branch in
1603
1684
        finally:
1604
1685
            if lock_taken:
1605
1686
                control_files.unlock()
1606
 
        branch = self.open(a_bzrdir, name, _found=True)
 
1687
        branch = self.open(a_bzrdir, name, _found=True,
 
1688
                found_repository=repository)
1607
1689
        self._run_post_branch_init_hooks(a_bzrdir, name, branch)
1608
1690
        return branch
1609
1691
 
1610
 
    def initialize(self, a_bzrdir, name=None):
 
1692
    def initialize(self, a_bzrdir, name=None, repository=None):
1611
1693
        """Create a branch of this format in a_bzrdir.
1612
1694
        
1613
1695
        :param name: Name of the colocated branch to create.
1647
1729
        """
1648
1730
        raise NotImplementedError(self.network_name)
1649
1731
 
1650
 
    def open(self, a_bzrdir, name=None, _found=False, ignore_fallbacks=False):
 
1732
    def open(self, a_bzrdir, name=None, _found=False, ignore_fallbacks=False,
 
1733
            found_repository=None):
1651
1734
        """Return the branch object for a_bzrdir
1652
1735
 
1653
1736
        :param a_bzrdir: A BzrDir that contains a branch.
1661
1744
 
1662
1745
    @classmethod
1663
1746
    def register_format(klass, format):
1664
 
        """Register a metadir format."""
 
1747
        """Register a metadir format.
 
1748
        
 
1749
        See MetaDirBranchFormatFactory for the ability to register a format
 
1750
        without loading the code the format needs until it is actually used.
 
1751
        """
1665
1752
        klass._formats[format.get_format_string()] = format
1666
1753
        # Metadir formats have a network name of their format string, and get
1667
 
        # registered as class factories.
1668
 
        network_format_registry.register(format.get_format_string(), format.__class__)
 
1754
        # registered as factories.
 
1755
        if isinstance(format, MetaDirBranchFormatFactory):
 
1756
            network_format_registry.register(format.get_format_string(), format)
 
1757
        else:
 
1758
            network_format_registry.register(format.get_format_string(),
 
1759
                format.__class__)
1669
1760
 
1670
1761
    @classmethod
1671
1762
    def set_default_format(klass, format):
1691
1782
        return False  # by default
1692
1783
 
1693
1784
 
 
1785
class MetaDirBranchFormatFactory(registry._LazyObjectGetter):
 
1786
    """A factory for a BranchFormat object, permitting simple lazy registration.
 
1787
    
 
1788
    While none of the built in BranchFormats are lazy registered yet,
 
1789
    bzrlib.tests.test_branch.TestMetaDirBranchFormatFactory demonstrates how to
 
1790
    use it, and the bzr-loom plugin uses it as well (see
 
1791
    bzrlib.plugins.loom.formats).
 
1792
    """
 
1793
 
 
1794
    def __init__(self, format_string, module_name, member_name):
 
1795
        """Create a MetaDirBranchFormatFactory.
 
1796
 
 
1797
        :param format_string: The format string the format has.
 
1798
        :param module_name: Module to load the format class from.
 
1799
        :param member_name: Attribute name within the module for the format class.
 
1800
        """
 
1801
        registry._LazyObjectGetter.__init__(self, module_name, member_name)
 
1802
        self._format_string = format_string
 
1803
        
 
1804
    def get_format_string(self):
 
1805
        """See BranchFormat.get_format_string."""
 
1806
        return self._format_string
 
1807
 
 
1808
    def __call__(self):
 
1809
        """Used for network_format_registry support."""
 
1810
        return self.get_obj()()
 
1811
 
 
1812
 
1694
1813
class BranchHooks(Hooks):
1695
1814
    """A dictionary mapping hook name to a list of callables for branch hooks.
1696
1815
 
1723
1842
            "with a bzrlib.branch.PullResult object and only runs in the "
1724
1843
            "bzr client.", (0, 15), None))
1725
1844
        self.create_hook(HookPoint('pre_commit',
1726
 
            "Called after a commit is calculated but before it is is "
 
1845
            "Called after a commit is calculated but before it is "
1727
1846
            "completed. pre_commit is called with (local, master, old_revno, "
1728
1847
            "old_revid, future_revno, future_revid, tree_delta, future_tree"
1729
1848
            "). old_revid is NULL_REVISION for the first commit to a branch, "
1766
1885
            "all are called with the url returned from the previous hook."
1767
1886
            "The order is however undefined.", (1, 9), None))
1768
1887
        self.create_hook(HookPoint('automatic_tag_name',
1769
 
            "Called to determine an automatic tag name for a revision."
 
1888
            "Called to determine an automatic tag name for a revision. "
1770
1889
            "automatic_tag_name is called with (branch, revision_id) and "
1771
1890
            "should return a tag name or None if no tag name could be "
1772
1891
            "determined. The first non-None tag name returned will be used.",
1863
1982
        return self.__dict__ == other.__dict__
1864
1983
 
1865
1984
    def __repr__(self):
1866
 
        if self.branch:
1867
 
            return "<%s of %s>" % (self.__class__.__name__, self.branch)
1868
 
        else:
1869
 
            return "<%s of format:%s bzrdir:%s>" % (
1870
 
                self.__class__.__name__, self.branch,
1871
 
                self.format, self.bzrdir)
 
1985
        return "<%s of %s>" % (self.__class__.__name__, self.branch)
1872
1986
 
1873
1987
 
1874
1988
class SwitchHookParams(object):
1916
2030
        """See BranchFormat.get_format_description()."""
1917
2031
        return "Branch format 4"
1918
2032
 
1919
 
    def initialize(self, a_bzrdir, name=None):
 
2033
    def initialize(self, a_bzrdir, name=None, repository=None):
1920
2034
        """Create a branch of this format in a_bzrdir."""
 
2035
        if repository is not None:
 
2036
            raise NotImplementedError(
 
2037
                "initialize(repository=<not None>) on %r" % (self,))
1921
2038
        utf8_files = [('revision-history', ''),
1922
2039
                      ('branch-name', ''),
1923
2040
                      ]
1932
2049
        """The network name for this format is the control dirs disk label."""
1933
2050
        return self._matchingbzrdir.get_format_string()
1934
2051
 
1935
 
    def open(self, a_bzrdir, name=None, _found=False, ignore_fallbacks=False):
 
2052
    def open(self, a_bzrdir, name=None, _found=False, ignore_fallbacks=False,
 
2053
            found_repository=None):
1936
2054
        """See BranchFormat.open()."""
1937
2055
        if not _found:
1938
2056
            # we are being called directly and must probe.
1939
2057
            raise NotImplementedError
 
2058
        if found_repository is None:
 
2059
            found_repository = a_bzrdir.open_repository()
1940
2060
        return BzrBranch(_format=self,
1941
2061
                         _control_files=a_bzrdir._control_files,
1942
2062
                         a_bzrdir=a_bzrdir,
1943
2063
                         name=name,
1944
 
                         _repository=a_bzrdir.open_repository())
 
2064
                         _repository=found_repository)
1945
2065
 
1946
2066
    def __str__(self):
1947
2067
        return "Bazaar-NG branch format 4"
1961
2081
        """
1962
2082
        return self.get_format_string()
1963
2083
 
1964
 
    def open(self, a_bzrdir, name=None, _found=False, ignore_fallbacks=False):
 
2084
    def open(self, a_bzrdir, name=None, _found=False, ignore_fallbacks=False,
 
2085
            found_repository=None):
1965
2086
        """See BranchFormat.open()."""
1966
2087
        if not _found:
1967
2088
            format = BranchFormat.find_format(a_bzrdir, name=name)
1972
2093
        try:
1973
2094
            control_files = lockable_files.LockableFiles(transport, 'lock',
1974
2095
                                                         lockdir.LockDir)
 
2096
            if found_repository is None:
 
2097
                found_repository = a_bzrdir.find_repository()
1975
2098
            return self._branch_class()(_format=self,
1976
2099
                              _control_files=control_files,
1977
2100
                              name=name,
1978
2101
                              a_bzrdir=a_bzrdir,
1979
 
                              _repository=a_bzrdir.find_repository(),
 
2102
                              _repository=found_repository,
1980
2103
                              ignore_fallbacks=ignore_fallbacks)
1981
2104
        except errors.NoSuchFile:
1982
2105
            raise errors.NotBranchError(path=transport.base, bzrdir=a_bzrdir)
2014
2137
        """See BranchFormat.get_format_description()."""
2015
2138
        return "Branch format 5"
2016
2139
 
2017
 
    def initialize(self, a_bzrdir, name=None):
 
2140
    def initialize(self, a_bzrdir, name=None, repository=None):
2018
2141
        """Create a branch of this format in a_bzrdir."""
2019
2142
        utf8_files = [('revision-history', ''),
2020
2143
                      ('branch-name', ''),
2021
2144
                      ]
2022
 
        return self._initialize_helper(a_bzrdir, utf8_files, name)
 
2145
        return self._initialize_helper(a_bzrdir, utf8_files, name, repository)
2023
2146
 
2024
2147
    def supports_tags(self):
2025
2148
        return False
2047
2170
        """See BranchFormat.get_format_description()."""
2048
2171
        return "Branch format 6"
2049
2172
 
2050
 
    def initialize(self, a_bzrdir, name=None):
 
2173
    def initialize(self, a_bzrdir, name=None, repository=None):
2051
2174
        """Create a branch of this format in a_bzrdir."""
2052
2175
        utf8_files = [('last-revision', '0 null:\n'),
2053
2176
                      ('branch.conf', ''),
2054
2177
                      ('tags', ''),
2055
2178
                      ]
2056
 
        return self._initialize_helper(a_bzrdir, utf8_files, name)
 
2179
        return self._initialize_helper(a_bzrdir, utf8_files, name, repository)
2057
2180
 
2058
2181
    def make_tags(self, branch):
2059
2182
        """See bzrlib.branch.BranchFormat.make_tags()."""
2077
2200
        """See BranchFormat.get_format_description()."""
2078
2201
        return "Branch format 8"
2079
2202
 
2080
 
    def initialize(self, a_bzrdir, name=None):
 
2203
    def initialize(self, a_bzrdir, name=None, repository=None):
2081
2204
        """Create a branch of this format in a_bzrdir."""
2082
2205
        utf8_files = [('last-revision', '0 null:\n'),
2083
2206
                      ('branch.conf', ''),
2084
2207
                      ('tags', ''),
2085
2208
                      ('references', '')
2086
2209
                      ]
2087
 
        return self._initialize_helper(a_bzrdir, utf8_files, name)
 
2210
        return self._initialize_helper(a_bzrdir, utf8_files, name, repository)
2088
2211
 
2089
2212
    def __init__(self):
2090
2213
        super(BzrBranchFormat8, self).__init__()
2113
2236
    This format was introduced in bzr 1.6.
2114
2237
    """
2115
2238
 
2116
 
    def initialize(self, a_bzrdir, name=None):
 
2239
    def initialize(self, a_bzrdir, name=None, repository=None):
2117
2240
        """Create a branch of this format in a_bzrdir."""
2118
2241
        utf8_files = [('last-revision', '0 null:\n'),
2119
2242
                      ('branch.conf', ''),
2120
2243
                      ('tags', ''),
2121
2244
                      ]
2122
 
        return self._initialize_helper(a_bzrdir, utf8_files, name)
 
2245
        return self._initialize_helper(a_bzrdir, utf8_files, name, repository)
2123
2246
 
2124
2247
    def _branch_class(self):
2125
2248
        return BzrBranch7
2157
2280
        """See BranchFormat.get_format_description()."""
2158
2281
        return "Checkout reference format 1"
2159
2282
 
2160
 
    def get_reference(self, a_bzrdir):
 
2283
    def get_reference(self, a_bzrdir, name=None):
2161
2284
        """See BranchFormat.get_reference()."""
2162
 
        transport = a_bzrdir.get_branch_transport(None)
 
2285
        transport = a_bzrdir.get_branch_transport(None, name=name)
2163
2286
        return transport.get_bytes('location')
2164
2287
 
2165
 
    def set_reference(self, a_bzrdir, to_branch):
 
2288
    def set_reference(self, a_bzrdir, name, to_branch):
2166
2289
        """See BranchFormat.set_reference()."""
2167
 
        transport = a_bzrdir.get_branch_transport(None)
 
2290
        transport = a_bzrdir.get_branch_transport(None, name=name)
2168
2291
        location = transport.put_bytes('location', to_branch.base)
2169
2292
 
2170
 
    def initialize(self, a_bzrdir, name=None, target_branch=None):
 
2293
    def initialize(self, a_bzrdir, name=None, target_branch=None,
 
2294
            repository=None):
2171
2295
        """Create a branch of this format in a_bzrdir."""
2172
2296
        if target_branch is None:
2173
2297
            # this format does not implement branch itself, thus the implicit
2201
2325
        return clone
2202
2326
 
2203
2327
    def open(self, a_bzrdir, name=None, _found=False, location=None,
2204
 
             possible_transports=None, ignore_fallbacks=False):
 
2328
             possible_transports=None, ignore_fallbacks=False,
 
2329
             found_repository=None):
2205
2330
        """Return the branch that the branch reference in a_bzrdir points at.
2206
2331
 
2207
2332
        :param a_bzrdir: A BzrDir that contains a branch.
2221
2346
                raise AssertionError("wrong format %r found for %r" %
2222
2347
                    (format, self))
2223
2348
        if location is None:
2224
 
            location = self.get_reference(a_bzrdir)
 
2349
            location = self.get_reference(a_bzrdir, name)
2225
2350
        real_bzrdir = bzrdir.BzrDir.open(
2226
2351
            location, possible_transports=possible_transports)
2227
2352
        result = real_bzrdir.open_branch(name=name, 
2265
2390
    _legacy_formats[0].network_name(), _legacy_formats[0].__class__)
2266
2391
 
2267
2392
 
 
2393
class BranchWriteLockResult(LogicalLockResult):
 
2394
    """The result of write locking a branch.
 
2395
 
 
2396
    :ivar branch_token: The token obtained from the underlying branch lock, or
 
2397
        None.
 
2398
    :ivar unlock: A callable which will unlock the lock.
 
2399
    """
 
2400
 
 
2401
    def __init__(self, unlock, branch_token):
 
2402
        LogicalLockResult.__init__(self, unlock)
 
2403
        self.branch_token = branch_token
 
2404
 
 
2405
    def __repr__(self):
 
2406
        return "BranchWriteLockResult(%s, %s)" % (self.branch_token,
 
2407
            self.unlock)
 
2408
 
 
2409
 
2268
2410
class BzrBranch(Branch, _RelockDebugMixin):
2269
2411
    """A branch stored in the actual filesystem.
2270
2412
 
2324
2466
        return self.control_files.is_locked()
2325
2467
 
2326
2468
    def lock_write(self, token=None):
 
2469
        """Lock the branch for write operations.
 
2470
 
 
2471
        :param token: A token to permit reacquiring a previously held and
 
2472
            preserved lock.
 
2473
        :return: A BranchWriteLockResult.
 
2474
        """
2327
2475
        if not self.is_locked():
2328
2476
            self._note_lock('w')
2329
2477
        # All-in-one needs to always unlock/lock.
2335
2483
        else:
2336
2484
            took_lock = False
2337
2485
        try:
2338
 
            return self.control_files.lock_write(token=token)
 
2486
            return BranchWriteLockResult(self.unlock,
 
2487
                self.control_files.lock_write(token=token))
2339
2488
        except:
2340
2489
            if took_lock:
2341
2490
                self.repository.unlock()
2342
2491
            raise
2343
2492
 
2344
2493
    def lock_read(self):
 
2494
        """Lock the branch for read operations.
 
2495
 
 
2496
        :return: A bzrlib.lock.LogicalLockResult.
 
2497
        """
2345
2498
        if not self.is_locked():
2346
2499
            self._note_lock('r')
2347
2500
        # All-in-one needs to always unlock/lock.
2354
2507
            took_lock = False
2355
2508
        try:
2356
2509
            self.control_files.lock_read()
 
2510
            return LogicalLockResult(self.unlock)
2357
2511
        except:
2358
2512
            if took_lock:
2359
2513
                self.repository.unlock()
2983
3137
    :ivar tag_conflicts: A list of tag conflicts, see BasicTags.merge_to
2984
3138
    """
2985
3139
 
 
3140
    @deprecated_method(deprecated_in((2, 3, 0)))
2986
3141
    def __int__(self):
2987
 
        # DEPRECATED: pull used to return the change in revno
 
3142
        """Return the relative change in revno.
 
3143
 
 
3144
        :deprecated: Use `new_revno` and `old_revno` instead.
 
3145
        """
2988
3146
        return self.new_revno - self.old_revno
2989
3147
 
2990
3148
    def report(self, to_file):
3015
3173
        target, otherwise it will be None.
3016
3174
    """
3017
3175
 
 
3176
    @deprecated_method(deprecated_in((2, 3, 0)))
3018
3177
    def __int__(self):
3019
 
        # DEPRECATED: push used to return the change in revno
 
3178
        """Return the relative change in revno.
 
3179
 
 
3180
        :deprecated: Use `new_revno` and `old_revno` instead.
 
3181
        """
3020
3182
        return self.new_revno - self.old_revno
3021
3183
 
3022
3184
    def report(self, to_file):
3145
3307
    _optimisers = []
3146
3308
    """The available optimised InterBranch types."""
3147
3309
 
3148
 
    @staticmethod
3149
 
    def _get_branch_formats_to_test():
3150
 
        """Return a tuple with the Branch formats to use when testing."""
3151
 
        raise NotImplementedError(InterBranch._get_branch_formats_to_test)
 
3310
    @classmethod
 
3311
    def _get_branch_formats_to_test(klass):
 
3312
        """Return an iterable of format tuples for testing.
 
3313
        
 
3314
        :return: An iterable of (from_format, to_format) to use when testing
 
3315
            this InterBranch class. Each InterBranch class should define this
 
3316
            method itself.
 
3317
        """
 
3318
        raise NotImplementedError(klass._get_branch_formats_to_test)
3152
3319
 
 
3320
    @needs_write_lock
3153
3321
    def pull(self, overwrite=False, stop_revision=None,
3154
3322
             possible_transports=None, local=False):
3155
3323
        """Mirror source into target branch.
3160
3328
        """
3161
3329
        raise NotImplementedError(self.pull)
3162
3330
 
 
3331
    @needs_write_lock
3163
3332
    def update_revisions(self, stop_revision=None, overwrite=False,
3164
3333
                         graph=None):
3165
3334
        """Pull in new perfect-fit revisions.
3173
3342
        """
3174
3343
        raise NotImplementedError(self.update_revisions)
3175
3344
 
 
3345
    @needs_write_lock
3176
3346
    def push(self, overwrite=False, stop_revision=None,
3177
3347
             _override_hook_source_branch=None):
3178
3348
        """Mirror the source branch into the target branch.
3181
3351
        """
3182
3352
        raise NotImplementedError(self.push)
3183
3353
 
 
3354
    @needs_write_lock
 
3355
    def copy_content_into(self, revision_id=None):
 
3356
        """Copy the content of source into target
 
3357
 
 
3358
        revision_id: if not None, the revision history in the new branch will
 
3359
                     be truncated to end with revision_id.
 
3360
        """
 
3361
        raise NotImplementedError(self.copy_content_into)
 
3362
 
3184
3363
 
3185
3364
class GenericInterBranch(InterBranch):
3186
 
    """InterBranch implementation that uses public Branch functions.
3187
 
    """
3188
 
 
3189
 
    @staticmethod
3190
 
    def _get_branch_formats_to_test():
3191
 
        return BranchFormat._default_format, BranchFormat._default_format
3192
 
 
 
3365
    """InterBranch implementation that uses public Branch functions."""
 
3366
 
 
3367
    @classmethod
 
3368
    def is_compatible(klass, source, target):
 
3369
        # GenericBranch uses the public API, so always compatible
 
3370
        return True
 
3371
 
 
3372
    @classmethod
 
3373
    def _get_branch_formats_to_test(klass):
 
3374
        return [(BranchFormat._default_format, BranchFormat._default_format)]
 
3375
 
 
3376
    @classmethod
 
3377
    def unwrap_format(klass, format):
 
3378
        if isinstance(format, remote.RemoteBranchFormat):
 
3379
            format._ensure_real()
 
3380
            return format._custom_format
 
3381
        return format
 
3382
 
 
3383
    @needs_write_lock
 
3384
    def copy_content_into(self, revision_id=None):
 
3385
        """Copy the content of source into target
 
3386
 
 
3387
        revision_id: if not None, the revision history in the new branch will
 
3388
                     be truncated to end with revision_id.
 
3389
        """
 
3390
        self.source.update_references(self.target)
 
3391
        self.source._synchronize_history(self.target, revision_id)
 
3392
        try:
 
3393
            parent = self.source.get_parent()
 
3394
        except errors.InaccessibleParent, e:
 
3395
            mutter('parent was not accessible to copy: %s', e)
 
3396
        else:
 
3397
            if parent:
 
3398
                self.target.set_parent(parent)
 
3399
        if self.source._push_should_merge_tags():
 
3400
            self.source.tags.merge_to(self.target.tags)
 
3401
 
 
3402
    @needs_write_lock
3193
3403
    def update_revisions(self, stop_revision=None, overwrite=False,
3194
3404
        graph=None):
3195
3405
        """See InterBranch.update_revisions()."""
3196
 
        self.source.lock_read()
3197
 
        try:
3198
 
            other_revno, other_last_revision = self.source.last_revision_info()
3199
 
            stop_revno = None # unknown
3200
 
            if stop_revision is None:
3201
 
                stop_revision = other_last_revision
3202
 
                if _mod_revision.is_null(stop_revision):
3203
 
                    # if there are no commits, we're done.
3204
 
                    return
3205
 
                stop_revno = other_revno
3206
 
 
3207
 
            # what's the current last revision, before we fetch [and change it
3208
 
            # possibly]
3209
 
            last_rev = _mod_revision.ensure_null(self.target.last_revision())
3210
 
            # we fetch here so that we don't process data twice in the common
3211
 
            # case of having something to pull, and so that the check for
3212
 
            # already merged can operate on the just fetched graph, which will
3213
 
            # be cached in memory.
3214
 
            self.target.fetch(self.source, stop_revision)
3215
 
            # Check to see if one is an ancestor of the other
3216
 
            if not overwrite:
3217
 
                if graph is None:
3218
 
                    graph = self.target.repository.get_graph()
3219
 
                if self.target._check_if_descendant_or_diverged(
3220
 
                        stop_revision, last_rev, graph, self.source):
3221
 
                    # stop_revision is a descendant of last_rev, but we aren't
3222
 
                    # overwriting, so we're done.
3223
 
                    return
3224
 
            if stop_revno is None:
3225
 
                if graph is None:
3226
 
                    graph = self.target.repository.get_graph()
3227
 
                this_revno, this_last_revision = \
3228
 
                        self.target.last_revision_info()
3229
 
                stop_revno = graph.find_distance_to_null(stop_revision,
3230
 
                                [(other_last_revision, other_revno),
3231
 
                                 (this_last_revision, this_revno)])
3232
 
            self.target.set_last_revision_info(stop_revno, stop_revision)
3233
 
        finally:
3234
 
            self.source.unlock()
3235
 
 
 
3406
        other_revno, other_last_revision = self.source.last_revision_info()
 
3407
        stop_revno = None # unknown
 
3408
        if stop_revision is None:
 
3409
            stop_revision = other_last_revision
 
3410
            if _mod_revision.is_null(stop_revision):
 
3411
                # if there are no commits, we're done.
 
3412
                return
 
3413
            stop_revno = other_revno
 
3414
 
 
3415
        # what's the current last revision, before we fetch [and change it
 
3416
        # possibly]
 
3417
        last_rev = _mod_revision.ensure_null(self.target.last_revision())
 
3418
        # we fetch here so that we don't process data twice in the common
 
3419
        # case of having something to pull, and so that the check for
 
3420
        # already merged can operate on the just fetched graph, which will
 
3421
        # be cached in memory.
 
3422
        self.target.fetch(self.source, stop_revision)
 
3423
        # Check to see if one is an ancestor of the other
 
3424
        if not overwrite:
 
3425
            if graph is None:
 
3426
                graph = self.target.repository.get_graph()
 
3427
            if self.target._check_if_descendant_or_diverged(
 
3428
                    stop_revision, last_rev, graph, self.source):
 
3429
                # stop_revision is a descendant of last_rev, but we aren't
 
3430
                # overwriting, so we're done.
 
3431
                return
 
3432
        if stop_revno is None:
 
3433
            if graph is None:
 
3434
                graph = self.target.repository.get_graph()
 
3435
            this_revno, this_last_revision = \
 
3436
                    self.target.last_revision_info()
 
3437
            stop_revno = graph.find_distance_to_null(stop_revision,
 
3438
                            [(other_last_revision, other_revno),
 
3439
                             (this_last_revision, this_revno)])
 
3440
        self.target.set_last_revision_info(stop_revno, stop_revision)
 
3441
 
 
3442
    @needs_write_lock
3236
3443
    def pull(self, overwrite=False, stop_revision=None,
3237
 
             possible_transports=None, _hook_master=None, run_hooks=True,
 
3444
             possible_transports=None, run_hooks=True,
3238
3445
             _override_hook_target=None, local=False):
3239
 
        """See Branch.pull.
 
3446
        """Pull from source into self, updating my master if any.
3240
3447
 
3241
 
        :param _hook_master: Private parameter - set the branch to
3242
 
            be supplied as the master to pull hooks.
3243
3448
        :param run_hooks: Private parameter - if false, this branch
3244
3449
            is being called because it's the master of the primary branch,
3245
3450
            so it should not run its hooks.
3246
 
        :param _override_hook_target: Private parameter - set the branch to be
3247
 
            supplied as the target_branch to pull hooks.
3248
 
        :param local: Only update the local branch, and not the bound branch.
3249
3451
        """
3250
 
        # This type of branch can't be bound.
3251
 
        if local:
 
3452
        bound_location = self.target.get_bound_location()
 
3453
        if local and not bound_location:
3252
3454
            raise errors.LocalRequiresBoundBranch()
3253
 
        result = PullResult()
3254
 
        result.source_branch = self.source
3255
 
        if _override_hook_target is None:
3256
 
            result.target_branch = self.target
3257
 
        else:
3258
 
            result.target_branch = _override_hook_target
3259
 
        self.source.lock_read()
 
3455
        master_branch = None
 
3456
        if not local and bound_location and self.source.user_url != bound_location:
 
3457
            # not pulling from master, so we need to update master.
 
3458
            master_branch = self.target.get_master_branch(possible_transports)
 
3459
            master_branch.lock_write()
3260
3460
        try:
3261
 
            # We assume that during 'pull' the target repository is closer than
3262
 
            # the source one.
3263
 
            self.source.update_references(self.target)
3264
 
            graph = self.target.repository.get_graph(self.source.repository)
3265
 
            # TODO: Branch formats should have a flag that indicates 
3266
 
            # that revno's are expensive, and pull() should honor that flag.
3267
 
            # -- JRV20090506
3268
 
            result.old_revno, result.old_revid = \
3269
 
                self.target.last_revision_info()
3270
 
            self.target.update_revisions(self.source, stop_revision,
3271
 
                overwrite=overwrite, graph=graph)
3272
 
            # TODO: The old revid should be specified when merging tags, 
3273
 
            # so a tags implementation that versions tags can only 
3274
 
            # pull in the most recent changes. -- JRV20090506
3275
 
            result.tag_conflicts = self.source.tags.merge_to(self.target.tags,
3276
 
                overwrite)
3277
 
            result.new_revno, result.new_revid = self.target.last_revision_info()
3278
 
            if _hook_master:
3279
 
                result.master_branch = _hook_master
3280
 
                result.local_branch = result.target_branch
3281
 
            else:
3282
 
                result.master_branch = result.target_branch
3283
 
                result.local_branch = None
3284
 
            if run_hooks:
3285
 
                for hook in Branch.hooks['post_pull']:
3286
 
                    hook(result)
 
3461
            if master_branch:
 
3462
                # pull from source into master.
 
3463
                master_branch.pull(self.source, overwrite, stop_revision,
 
3464
                    run_hooks=False)
 
3465
            return self._pull(overwrite,
 
3466
                stop_revision, _hook_master=master_branch,
 
3467
                run_hooks=run_hooks,
 
3468
                _override_hook_target=_override_hook_target)
3287
3469
        finally:
3288
 
            self.source.unlock()
3289
 
        return result
 
3470
            if master_branch:
 
3471
                master_branch.unlock()
3290
3472
 
3291
3473
    def push(self, overwrite=False, stop_revision=None,
3292
3474
             _override_hook_source_branch=None):
3332
3514
                # push into the master from the source branch.
3333
3515
                self.source._basic_push(master_branch, overwrite, stop_revision)
3334
3516
                # and push into the target branch from the source. Note that we
3335
 
                # push from the source branch again, because its considered the
 
3517
                # push from the source branch again, because it's considered the
3336
3518
                # highest bandwidth repository.
3337
3519
                result = self.source._basic_push(self.target, overwrite,
3338
3520
                    stop_revision)
3354
3536
            _run_hooks()
3355
3537
            return result
3356
3538
 
3357
 
    @classmethod
3358
 
    def is_compatible(self, source, target):
3359
 
        # GenericBranch uses the public API, so always compatible
3360
 
        return True
3361
 
 
3362
 
 
3363
 
class InterToBranch5(GenericInterBranch):
3364
 
 
3365
 
    @staticmethod
3366
 
    def _get_branch_formats_to_test():
3367
 
        return BranchFormat._default_format, BzrBranchFormat5()
3368
 
 
3369
 
    def pull(self, overwrite=False, stop_revision=None,
3370
 
             possible_transports=None, run_hooks=True,
 
3539
    def _pull(self, overwrite=False, stop_revision=None,
 
3540
             possible_transports=None, _hook_master=None, run_hooks=True,
3371
3541
             _override_hook_target=None, local=False):
3372
 
        """Pull from source into self, updating my master if any.
3373
 
 
 
3542
        """See Branch.pull.
 
3543
 
 
3544
        This function is the core worker, used by GenericInterBranch.pull to
 
3545
        avoid duplication when pulling source->master and source->local.
 
3546
 
 
3547
        :param _hook_master: Private parameter - set the branch to
 
3548
            be supplied as the master to pull hooks.
3374
3549
        :param run_hooks: Private parameter - if false, this branch
3375
3550
            is being called because it's the master of the primary branch,
3376
3551
            so it should not run its hooks.
 
3552
        :param _override_hook_target: Private parameter - set the branch to be
 
3553
            supplied as the target_branch to pull hooks.
 
3554
        :param local: Only update the local branch, and not the bound branch.
3377
3555
        """
3378
 
        bound_location = self.target.get_bound_location()
3379
 
        if local and not bound_location:
 
3556
        # This type of branch can't be bound.
 
3557
        if local:
3380
3558
            raise errors.LocalRequiresBoundBranch()
3381
 
        master_branch = None
3382
 
        if not local and bound_location and self.source.user_url != bound_location:
3383
 
            # not pulling from master, so we need to update master.
3384
 
            master_branch = self.target.get_master_branch(possible_transports)
3385
 
            master_branch.lock_write()
 
3559
        result = PullResult()
 
3560
        result.source_branch = self.source
 
3561
        if _override_hook_target is None:
 
3562
            result.target_branch = self.target
 
3563
        else:
 
3564
            result.target_branch = _override_hook_target
 
3565
        self.source.lock_read()
3386
3566
        try:
3387
 
            if master_branch:
3388
 
                # pull from source into master.
3389
 
                master_branch.pull(self.source, overwrite, stop_revision,
3390
 
                    run_hooks=False)
3391
 
            return super(InterToBranch5, self).pull(overwrite,
3392
 
                stop_revision, _hook_master=master_branch,
3393
 
                run_hooks=run_hooks,
3394
 
                _override_hook_target=_override_hook_target)
 
3567
            # We assume that during 'pull' the target repository is closer than
 
3568
            # the source one.
 
3569
            self.source.update_references(self.target)
 
3570
            graph = self.target.repository.get_graph(self.source.repository)
 
3571
            # TODO: Branch formats should have a flag that indicates 
 
3572
            # that revno's are expensive, and pull() should honor that flag.
 
3573
            # -- JRV20090506
 
3574
            result.old_revno, result.old_revid = \
 
3575
                self.target.last_revision_info()
 
3576
            self.target.update_revisions(self.source, stop_revision,
 
3577
                overwrite=overwrite, graph=graph)
 
3578
            # TODO: The old revid should be specified when merging tags, 
 
3579
            # so a tags implementation that versions tags can only 
 
3580
            # pull in the most recent changes. -- JRV20090506
 
3581
            result.tag_conflicts = self.source.tags.merge_to(self.target.tags,
 
3582
                overwrite)
 
3583
            result.new_revno, result.new_revid = self.target.last_revision_info()
 
3584
            if _hook_master:
 
3585
                result.master_branch = _hook_master
 
3586
                result.local_branch = result.target_branch
 
3587
            else:
 
3588
                result.master_branch = result.target_branch
 
3589
                result.local_branch = None
 
3590
            if run_hooks:
 
3591
                for hook in Branch.hooks['post_pull']:
 
3592
                    hook(result)
3395
3593
        finally:
3396
 
            if master_branch:
3397
 
                master_branch.unlock()
 
3594
            self.source.unlock()
 
3595
        return result
3398
3596
 
3399
3597
 
3400
3598
InterBranch.register_optimiser(GenericInterBranch)
3401
 
InterBranch.register_optimiser(InterToBranch5)