~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/branch.py

  • Committer: Martin Pool
  • Date: 2010-08-13 08:09:53 UTC
  • mto: (5050.17.6 2.2)
  • mto: This revision was merged to the branch mainline in revision 5379.
  • Revision ID: mbp@sourcefrog.net-20100813080953-c00cm9l3qgu2flj9
Remove spuriously-resurrected test

Show diffs side-by-side

added added

removed removed

Lines of Context:
29
29
        errors,
30
30
        lockdir,
31
31
        lockable_files,
 
32
        remote,
32
33
        repository,
33
34
        revision as _mod_revision,
34
35
        rio,
49
50
from bzrlib.decorators import needs_read_lock, needs_write_lock, only_raises
50
51
from bzrlib.hooks import HookPoint, Hooks
51
52
from bzrlib.inter import InterObject
52
 
from bzrlib.lock import _RelockDebugMixin
 
53
from bzrlib.lock import _RelockDebugMixin, LogicalLockResult
53
54
from bzrlib import registry
54
55
from bzrlib.symbol_versioning import (
55
56
    deprecated_in,
197
198
        return self.supports_tags() and self.tags.get_tag_dict()
198
199
 
199
200
    def get_config(self):
 
201
        """Get a bzrlib.config.BranchConfig for this Branch.
 
202
 
 
203
        This can then be used to get and set configuration options for the
 
204
        branch.
 
205
 
 
206
        :return: A bzrlib.config.BranchConfig.
 
207
        """
200
208
        return BranchConfig(self)
201
209
 
202
210
    def _get_config(self):
238
246
        if not local and not config.has_explicit_nickname():
239
247
            try:
240
248
                master = self.get_master_branch(possible_transports)
 
249
                if master and self.user_url == master.user_url:
 
250
                    raise errors.RecursiveBind(self.user_url)
241
251
                if master is not None:
242
252
                    # return the master branch value
243
253
                    return master.nick
 
254
            except errors.RecursiveBind, e:
 
255
                raise e
244
256
            except errors.BzrError, e:
245
257
                # Silently fall back to local implicit nick if the master is
246
258
                # unavailable
283
295
        new_history.reverse()
284
296
        return new_history
285
297
 
286
 
    def lock_write(self):
 
298
    def lock_write(self, token=None):
 
299
        """Lock the branch for write operations.
 
300
 
 
301
        :param token: A token to permit reacquiring a previously held and
 
302
            preserved lock.
 
303
        :return: A BranchWriteLockResult.
 
304
        """
287
305
        raise NotImplementedError(self.lock_write)
288
306
 
289
307
    def lock_read(self):
 
308
        """Lock the branch for read operations.
 
309
 
 
310
        :return: A bzrlib.lock.LogicalLockResult.
 
311
        """
290
312
        raise NotImplementedError(self.lock_read)
291
313
 
292
314
    def unlock(self):
783
805
            if len(old_repository._fallback_repositories) != 1:
784
806
                raise AssertionError("can't cope with fallback repositories "
785
807
                    "of %r" % (self.repository,))
786
 
            # unlock it, including unlocking the fallback
 
808
            # Open the new repository object.
 
809
            # Repositories don't offer an interface to remove fallback
 
810
            # repositories today; take the conceptually simpler option and just
 
811
            # reopen it.  We reopen it starting from the URL so that we
 
812
            # get a separate connection for RemoteRepositories and can
 
813
            # stream from one of them to the other.  This does mean doing
 
814
            # separate SSH connection setup, but unstacking is not a
 
815
            # common operation so it's tolerable.
 
816
            new_bzrdir = bzrdir.BzrDir.open(self.bzrdir.root_transport.base)
 
817
            new_repository = new_bzrdir.find_repository()
 
818
            if new_repository._fallback_repositories:
 
819
                raise AssertionError("didn't expect %r to have "
 
820
                    "fallback_repositories"
 
821
                    % (self.repository,))
 
822
            # Replace self.repository with the new repository.
 
823
            # Do our best to transfer the lock state (i.e. lock-tokens and
 
824
            # lock count) of self.repository to the new repository.
 
825
            lock_token = old_repository.lock_write().repository_token
 
826
            self.repository = new_repository
 
827
            if isinstance(self, remote.RemoteBranch):
 
828
                # Remote branches can have a second reference to the old
 
829
                # repository that need to be replaced.
 
830
                if self._real_branch is not None:
 
831
                    self._real_branch.repository = new_repository
 
832
            self.repository.lock_write(token=lock_token)
 
833
            if lock_token is not None:
 
834
                old_repository.leave_lock_in_place()
787
835
            old_repository.unlock()
 
836
            if lock_token is not None:
 
837
                # XXX: self.repository.leave_lock_in_place() before this
 
838
                # function will not be preserved.  Fortunately that doesn't
 
839
                # affect the current default format (2a), and would be a
 
840
                # corner-case anyway.
 
841
                #  - Andrew Bennetts, 2010/06/30
 
842
                self.repository.dont_leave_lock_in_place()
 
843
            old_lock_count = 0
 
844
            while True:
 
845
                try:
 
846
                    old_repository.unlock()
 
847
                except errors.LockNotHeld:
 
848
                    break
 
849
                old_lock_count += 1
 
850
            if old_lock_count == 0:
 
851
                raise AssertionError(
 
852
                    'old_repository should have been locked at least once.')
 
853
            for i in range(old_lock_count-1):
 
854
                self.repository.lock_write()
 
855
            # Fetch from the old repository into the new.
788
856
            old_repository.lock_read()
789
857
            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
858
                # XXX: If you unstack a branch while it has a working tree
809
859
                # with a pending merge, the pending-merged revisions will no
810
860
                # longer be present.  You can (probably) revert and remerge.
951
1001
                raise errors.NoSuchRevision(self, stop_revision)
952
1002
        return other_history[self_len:stop_revision]
953
1003
 
954
 
    @needs_write_lock
955
1004
    def update_revisions(self, other, stop_revision=None, overwrite=False,
956
1005
                         graph=None):
957
1006
        """Pull in new perfect-fit revisions.
1006
1055
            self._extend_partial_history(distance_from_last)
1007
1056
        return self._partial_revision_history_cache[distance_from_last]
1008
1057
 
1009
 
    @needs_write_lock
1010
1058
    def pull(self, source, overwrite=False, stop_revision=None,
1011
1059
             possible_transports=None, *args, **kwargs):
1012
1060
        """Mirror source into this branch.
1255
1303
                revno = 1
1256
1304
        destination.set_last_revision_info(revno, revision_id)
1257
1305
 
1258
 
    @needs_read_lock
1259
1306
    def copy_content_into(self, destination, revision_id=None):
1260
1307
        """Copy the content of self into destination.
1261
1308
 
1262
1309
        revision_id: if not None, the revision history in the new branch will
1263
1310
                     be truncated to end with revision_id.
1264
1311
        """
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)
 
1312
        return InterBranch.get(self, destination).copy_content_into(
 
1313
            revision_id=revision_id)
1276
1314
 
1277
1315
    def update_references(self, target):
1278
1316
        if not getattr(self._format, 'supports_reference_locations', False):
1346
1384
        """
1347
1385
        # XXX: Fix the bzrdir API to allow getting the branch back from the
1348
1386
        # clone call. Or something. 20090224 RBC/spiv.
 
1387
        # XXX: Should this perhaps clone colocated branches as well, 
 
1388
        # rather than just the default branch? 20100319 JRV
1349
1389
        if revision_id is None:
1350
1390
            revision_id = self.last_revision()
1351
1391
        dir_to = self.bzrdir.clone_on_transport(to_transport,
1510
1550
        try:
1511
1551
            transport = a_bzrdir.get_branch_transport(None, name=name)
1512
1552
            format_string = transport.get_bytes("format")
1513
 
            return klass._formats[format_string]
 
1553
            format = klass._formats[format_string]
 
1554
            if isinstance(format, MetaDirBranchFormatFactory):
 
1555
                return format()
 
1556
            return format
1514
1557
        except errors.NoSuchFile:
1515
1558
            raise errors.NotBranchError(path=transport.base, bzrdir=a_bzrdir)
1516
1559
        except KeyError:
1521
1564
        """Return the current default format."""
1522
1565
        return klass._default_format
1523
1566
 
1524
 
    def get_reference(self, a_bzrdir):
 
1567
    @classmethod
 
1568
    def get_formats(klass):
 
1569
        """Get all the known formats.
 
1570
 
 
1571
        Warning: This triggers a load of all lazy registered formats: do not
 
1572
        use except when that is desireed.
 
1573
        """
 
1574
        result = []
 
1575
        for fmt in klass._formats.values():
 
1576
            if isinstance(fmt, MetaDirBranchFormatFactory):
 
1577
                fmt = fmt()
 
1578
            result.append(fmt)
 
1579
        return result
 
1580
 
 
1581
    def get_reference(self, a_bzrdir, name=None):
1525
1582
        """Get the target reference of the branch in a_bzrdir.
1526
1583
 
1527
1584
        format probing must have been completed before calling
1529
1586
        in a_bzrdir is correct.
1530
1587
 
1531
1588
        :param a_bzrdir: The bzrdir to get the branch data from.
 
1589
        :param name: Name of the colocated branch to fetch
1532
1590
        :return: None if the branch is not a reference branch.
1533
1591
        """
1534
1592
        return None
1535
1593
 
1536
1594
    @classmethod
1537
 
    def set_reference(self, a_bzrdir, to_branch):
 
1595
    def set_reference(self, a_bzrdir, name, to_branch):
1538
1596
        """Set the target reference of the branch in a_bzrdir.
1539
1597
 
1540
1598
        format probing must have been completed before calling
1542
1600
        in a_bzrdir is correct.
1543
1601
 
1544
1602
        :param a_bzrdir: The bzrdir to set the branch reference for.
 
1603
        :param name: Name of colocated branch to set, None for default
1545
1604
        :param to_branch: branch that the checkout is to reference
1546
1605
        """
1547
1606
        raise NotImplementedError(self.set_reference)
1661
1720
 
1662
1721
    @classmethod
1663
1722
    def register_format(klass, format):
1664
 
        """Register a metadir format."""
 
1723
        """Register a metadir format.
 
1724
        
 
1725
        See MetaDirBranchFormatFactory for the ability to register a format
 
1726
        without loading the code the format needs until it is actually used.
 
1727
        """
1665
1728
        klass._formats[format.get_format_string()] = format
1666
1729
        # 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__)
 
1730
        # registered as factories.
 
1731
        if isinstance(format, MetaDirBranchFormatFactory):
 
1732
            network_format_registry.register(format.get_format_string(), format)
 
1733
        else:
 
1734
            network_format_registry.register(format.get_format_string(),
 
1735
                format.__class__)
1669
1736
 
1670
1737
    @classmethod
1671
1738
    def set_default_format(klass, format):
1691
1758
        return False  # by default
1692
1759
 
1693
1760
 
 
1761
class MetaDirBranchFormatFactory(registry._LazyObjectGetter):
 
1762
    """A factory for a BranchFormat object, permitting simple lazy registration.
 
1763
    
 
1764
    While none of the built in BranchFormats are lazy registered yet,
 
1765
    bzrlib.tests.test_branch.TestMetaDirBranchFormatFactory demonstrates how to
 
1766
    use it, and the bzr-loom plugin uses it as well (see
 
1767
    bzrlib.plugins.loom.formats).
 
1768
    """
 
1769
 
 
1770
    def __init__(self, format_string, module_name, member_name):
 
1771
        """Create a MetaDirBranchFormatFactory.
 
1772
 
 
1773
        :param format_string: The format string the format has.
 
1774
        :param module_name: Module to load the format class from.
 
1775
        :param member_name: Attribute name within the module for the format class.
 
1776
        """
 
1777
        registry._LazyObjectGetter.__init__(self, module_name, member_name)
 
1778
        self._format_string = format_string
 
1779
        
 
1780
    def get_format_string(self):
 
1781
        """See BranchFormat.get_format_string."""
 
1782
        return self._format_string
 
1783
 
 
1784
    def __call__(self):
 
1785
        """Used for network_format_registry support."""
 
1786
        return self.get_obj()()
 
1787
 
 
1788
 
1694
1789
class BranchHooks(Hooks):
1695
1790
    """A dictionary mapping hook name to a list of callables for branch hooks.
1696
1791
 
1766
1861
            "all are called with the url returned from the previous hook."
1767
1862
            "The order is however undefined.", (1, 9), None))
1768
1863
        self.create_hook(HookPoint('automatic_tag_name',
1769
 
            "Called to determine an automatic tag name for a revision."
 
1864
            "Called to determine an automatic tag name for a revision. "
1770
1865
            "automatic_tag_name is called with (branch, revision_id) and "
1771
1866
            "should return a tag name or None if no tag name could be "
1772
1867
            "determined. The first non-None tag name returned will be used.",
2157
2252
        """See BranchFormat.get_format_description()."""
2158
2253
        return "Checkout reference format 1"
2159
2254
 
2160
 
    def get_reference(self, a_bzrdir):
 
2255
    def get_reference(self, a_bzrdir, name=None):
2161
2256
        """See BranchFormat.get_reference()."""
2162
 
        transport = a_bzrdir.get_branch_transport(None)
 
2257
        transport = a_bzrdir.get_branch_transport(None, name=name)
2163
2258
        return transport.get_bytes('location')
2164
2259
 
2165
 
    def set_reference(self, a_bzrdir, to_branch):
 
2260
    def set_reference(self, a_bzrdir, name, to_branch):
2166
2261
        """See BranchFormat.set_reference()."""
2167
 
        transport = a_bzrdir.get_branch_transport(None)
 
2262
        transport = a_bzrdir.get_branch_transport(None, name=name)
2168
2263
        location = transport.put_bytes('location', to_branch.base)
2169
2264
 
2170
2265
    def initialize(self, a_bzrdir, name=None, target_branch=None):
2221
2316
                raise AssertionError("wrong format %r found for %r" %
2222
2317
                    (format, self))
2223
2318
        if location is None:
2224
 
            location = self.get_reference(a_bzrdir)
 
2319
            location = self.get_reference(a_bzrdir, name)
2225
2320
        real_bzrdir = bzrdir.BzrDir.open(
2226
2321
            location, possible_transports=possible_transports)
2227
2322
        result = real_bzrdir.open_branch(name=name, 
2265
2360
    _legacy_formats[0].network_name(), _legacy_formats[0].__class__)
2266
2361
 
2267
2362
 
 
2363
class BranchWriteLockResult(LogicalLockResult):
 
2364
    """The result of write locking a branch.
 
2365
 
 
2366
    :ivar branch_token: The token obtained from the underlying branch lock, or
 
2367
        None.
 
2368
    :ivar unlock: A callable which will unlock the lock.
 
2369
    """
 
2370
 
 
2371
    def __init__(self, unlock, branch_token):
 
2372
        LogicalLockResult.__init__(self, unlock)
 
2373
        self.branch_token = branch_token
 
2374
 
 
2375
    def __repr__(self):
 
2376
        return "BranchWriteLockResult(%s, %s)" % (self.branch_token,
 
2377
            self.unlock)
 
2378
 
 
2379
 
2268
2380
class BzrBranch(Branch, _RelockDebugMixin):
2269
2381
    """A branch stored in the actual filesystem.
2270
2382
 
2324
2436
        return self.control_files.is_locked()
2325
2437
 
2326
2438
    def lock_write(self, token=None):
 
2439
        """Lock the branch for write operations.
 
2440
 
 
2441
        :param token: A token to permit reacquiring a previously held and
 
2442
            preserved lock.
 
2443
        :return: A BranchWriteLockResult.
 
2444
        """
2327
2445
        if not self.is_locked():
2328
2446
            self._note_lock('w')
2329
2447
        # All-in-one needs to always unlock/lock.
2335
2453
        else:
2336
2454
            took_lock = False
2337
2455
        try:
2338
 
            return self.control_files.lock_write(token=token)
 
2456
            return BranchWriteLockResult(self.unlock,
 
2457
                self.control_files.lock_write(token=token))
2339
2458
        except:
2340
2459
            if took_lock:
2341
2460
                self.repository.unlock()
2342
2461
            raise
2343
2462
 
2344
2463
    def lock_read(self):
 
2464
        """Lock the branch for read operations.
 
2465
 
 
2466
        :return: A bzrlib.lock.LogicalLockResult.
 
2467
        """
2345
2468
        if not self.is_locked():
2346
2469
            self._note_lock('r')
2347
2470
        # All-in-one needs to always unlock/lock.
2354
2477
            took_lock = False
2355
2478
        try:
2356
2479
            self.control_files.lock_read()
 
2480
            return LogicalLockResult(self.unlock)
2357
2481
        except:
2358
2482
            if took_lock:
2359
2483
                self.repository.unlock()
3145
3269
    _optimisers = []
3146
3270
    """The available optimised InterBranch types."""
3147
3271
 
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)
 
3272
    @classmethod
 
3273
    def _get_branch_formats_to_test(klass):
 
3274
        """Return an iterable of format tuples for testing.
 
3275
        
 
3276
        :return: An iterable of (from_format, to_format) to use when testing
 
3277
            this InterBranch class. Each InterBranch class should define this
 
3278
            method itself.
 
3279
        """
 
3280
        raise NotImplementedError(klass._get_branch_formats_to_test)
3152
3281
 
 
3282
    @needs_write_lock
3153
3283
    def pull(self, overwrite=False, stop_revision=None,
3154
3284
             possible_transports=None, local=False):
3155
3285
        """Mirror source into target branch.
3160
3290
        """
3161
3291
        raise NotImplementedError(self.pull)
3162
3292
 
 
3293
    @needs_write_lock
3163
3294
    def update_revisions(self, stop_revision=None, overwrite=False,
3164
3295
                         graph=None):
3165
3296
        """Pull in new perfect-fit revisions.
3173
3304
        """
3174
3305
        raise NotImplementedError(self.update_revisions)
3175
3306
 
 
3307
    @needs_write_lock
3176
3308
    def push(self, overwrite=False, stop_revision=None,
3177
3309
             _override_hook_source_branch=None):
3178
3310
        """Mirror the source branch into the target branch.
3183
3315
 
3184
3316
 
3185
3317
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
 
 
 
3318
    """InterBranch implementation that uses public Branch functions."""
 
3319
 
 
3320
    @classmethod
 
3321
    def is_compatible(klass, source, target):
 
3322
        # GenericBranch uses the public API, so always compatible
 
3323
        return True
 
3324
 
 
3325
    @classmethod
 
3326
    def _get_branch_formats_to_test(klass):
 
3327
        return [(BranchFormat._default_format, BranchFormat._default_format)]
 
3328
 
 
3329
    @classmethod
 
3330
    def unwrap_format(klass, format):
 
3331
        if isinstance(format, remote.RemoteBranchFormat):
 
3332
            format._ensure_real()
 
3333
            return format._custom_format
 
3334
        return format                                                                                                  
 
3335
 
 
3336
    @needs_write_lock
 
3337
    def copy_content_into(self, revision_id=None):
 
3338
        """Copy the content of source into target
 
3339
 
 
3340
        revision_id: if not None, the revision history in the new branch will
 
3341
                     be truncated to end with revision_id.
 
3342
        """
 
3343
        self.source.update_references(self.target)
 
3344
        self.source._synchronize_history(self.target, revision_id)
 
3345
        try:
 
3346
            parent = self.source.get_parent()
 
3347
        except errors.InaccessibleParent, e:
 
3348
            mutter('parent was not accessible to copy: %s', e)
 
3349
        else:
 
3350
            if parent:
 
3351
                self.target.set_parent(parent)
 
3352
        if self.source._push_should_merge_tags():
 
3353
            self.source.tags.merge_to(self.target.tags)
 
3354
 
 
3355
    @needs_write_lock
3193
3356
    def update_revisions(self, stop_revision=None, overwrite=False,
3194
3357
        graph=None):
3195
3358
        """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
 
 
 
3359
        other_revno, other_last_revision = self.source.last_revision_info()
 
3360
        stop_revno = None # unknown
 
3361
        if stop_revision is None:
 
3362
            stop_revision = other_last_revision
 
3363
            if _mod_revision.is_null(stop_revision):
 
3364
                # if there are no commits, we're done.
 
3365
                return
 
3366
            stop_revno = other_revno
 
3367
 
 
3368
        # what's the current last revision, before we fetch [and change it
 
3369
        # possibly]
 
3370
        last_rev = _mod_revision.ensure_null(self.target.last_revision())
 
3371
        # we fetch here so that we don't process data twice in the common
 
3372
        # case of having something to pull, and so that the check for
 
3373
        # already merged can operate on the just fetched graph, which will
 
3374
        # be cached in memory.
 
3375
        self.target.fetch(self.source, stop_revision)
 
3376
        # Check to see if one is an ancestor of the other
 
3377
        if not overwrite:
 
3378
            if graph is None:
 
3379
                graph = self.target.repository.get_graph()
 
3380
            if self.target._check_if_descendant_or_diverged(
 
3381
                    stop_revision, last_rev, graph, self.source):
 
3382
                # stop_revision is a descendant of last_rev, but we aren't
 
3383
                # overwriting, so we're done.
 
3384
                return
 
3385
        if stop_revno is None:
 
3386
            if graph is None:
 
3387
                graph = self.target.repository.get_graph()
 
3388
            this_revno, this_last_revision = \
 
3389
                    self.target.last_revision_info()
 
3390
            stop_revno = graph.find_distance_to_null(stop_revision,
 
3391
                            [(other_last_revision, other_revno),
 
3392
                             (this_last_revision, this_revno)])
 
3393
        self.target.set_last_revision_info(stop_revno, stop_revision)
 
3394
 
 
3395
    @needs_write_lock
3236
3396
    def pull(self, overwrite=False, stop_revision=None,
3237
 
             possible_transports=None, _hook_master=None, run_hooks=True,
 
3397
             possible_transports=None, run_hooks=True,
3238
3398
             _override_hook_target=None, local=False):
3239
 
        """See Branch.pull.
 
3399
        """Pull from source into self, updating my master if any.
3240
3400
 
3241
 
        :param _hook_master: Private parameter - set the branch to
3242
 
            be supplied as the master to pull hooks.
3243
3401
        :param run_hooks: Private parameter - if false, this branch
3244
3402
            is being called because it's the master of the primary branch,
3245
3403
            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
3404
        """
3250
 
        # This type of branch can't be bound.
3251
 
        if local:
 
3405
        bound_location = self.target.get_bound_location()
 
3406
        if local and not bound_location:
3252
3407
            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()
 
3408
        master_branch = None
 
3409
        if not local and bound_location and self.source.user_url != bound_location:
 
3410
            # not pulling from master, so we need to update master.
 
3411
            master_branch = self.target.get_master_branch(possible_transports)
 
3412
            master_branch.lock_write()
3260
3413
        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)
 
3414
            if master_branch:
 
3415
                # pull from source into master.
 
3416
                master_branch.pull(self.source, overwrite, stop_revision,
 
3417
                    run_hooks=False)
 
3418
            return self._pull(overwrite,
 
3419
                stop_revision, _hook_master=master_branch,
 
3420
                run_hooks=run_hooks,
 
3421
                _override_hook_target=_override_hook_target)
3287
3422
        finally:
3288
 
            self.source.unlock()
3289
 
        return result
 
3423
            if master_branch:
 
3424
                master_branch.unlock()
3290
3425
 
3291
3426
    def push(self, overwrite=False, stop_revision=None,
3292
3427
             _override_hook_source_branch=None):
3354
3489
            _run_hooks()
3355
3490
            return result
3356
3491
 
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,
 
3492
    def _pull(self, overwrite=False, stop_revision=None,
 
3493
             possible_transports=None, _hook_master=None, run_hooks=True,
3371
3494
             _override_hook_target=None, local=False):
3372
 
        """Pull from source into self, updating my master if any.
3373
 
 
 
3495
        """See Branch.pull.
 
3496
 
 
3497
        This function is the core worker, used by GenericInterBranch.pull to
 
3498
        avoid duplication when pulling source->master and source->local.
 
3499
 
 
3500
        :param _hook_master: Private parameter - set the branch to
 
3501
            be supplied as the master to pull hooks.
3374
3502
        :param run_hooks: Private parameter - if false, this branch
3375
3503
            is being called because it's the master of the primary branch,
3376
3504
            so it should not run its hooks.
 
3505
        :param _override_hook_target: Private parameter - set the branch to be
 
3506
            supplied as the target_branch to pull hooks.
 
3507
        :param local: Only update the local branch, and not the bound branch.
3377
3508
        """
3378
 
        bound_location = self.target.get_bound_location()
3379
 
        if local and not bound_location:
 
3509
        # This type of branch can't be bound.
 
3510
        if local:
3380
3511
            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()
 
3512
        result = PullResult()
 
3513
        result.source_branch = self.source
 
3514
        if _override_hook_target is None:
 
3515
            result.target_branch = self.target
 
3516
        else:
 
3517
            result.target_branch = _override_hook_target
 
3518
        self.source.lock_read()
3386
3519
        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)
 
3520
            # We assume that during 'pull' the target repository is closer than
 
3521
            # the source one.
 
3522
            self.source.update_references(self.target)
 
3523
            graph = self.target.repository.get_graph(self.source.repository)
 
3524
            # TODO: Branch formats should have a flag that indicates 
 
3525
            # that revno's are expensive, and pull() should honor that flag.
 
3526
            # -- JRV20090506
 
3527
            result.old_revno, result.old_revid = \
 
3528
                self.target.last_revision_info()
 
3529
            self.target.update_revisions(self.source, stop_revision,
 
3530
                overwrite=overwrite, graph=graph)
 
3531
            # TODO: The old revid should be specified when merging tags, 
 
3532
            # so a tags implementation that versions tags can only 
 
3533
            # pull in the most recent changes. -- JRV20090506
 
3534
            result.tag_conflicts = self.source.tags.merge_to(self.target.tags,
 
3535
                overwrite)
 
3536
            result.new_revno, result.new_revid = self.target.last_revision_info()
 
3537
            if _hook_master:
 
3538
                result.master_branch = _hook_master
 
3539
                result.local_branch = result.target_branch
 
3540
            else:
 
3541
                result.master_branch = result.target_branch
 
3542
                result.local_branch = None
 
3543
            if run_hooks:
 
3544
                for hook in Branch.hooks['post_pull']:
 
3545
                    hook(result)
3395
3546
        finally:
3396
 
            if master_branch:
3397
 
                master_branch.unlock()
 
3547
            self.source.unlock()
 
3548
        return result
3398
3549
 
3399
3550
 
3400
3551
InterBranch.register_optimiser(GenericInterBranch)
3401
 
InterBranch.register_optimiser(InterToBranch5)