/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: 2010-08-10 20:03:44 UTC
  • mto: This revision was merged to the branch mainline in revision 5376.
  • Revision ID: john@arbash-meinel.com-20100810200344-6muerwvkafqu7w47
Rework things a bit so the logic can be shared.

It turns out that some of the peak memory is actually during the inventory
to string to bundle translations. So re-use the refcount logic there.
This actually does show a decrease in peak memory.
Specifically 'cd bzr.dev; bzr send ../2.2' drops from 221MB peak to 156MB.

We don't speed anything up (16.5s both ways), but peak memory is quite
a bit better.

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/branch.py
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
295
307
    def lock_read(self):
296
308
        """Lock the branch for read operations.
297
309
 
298
 
        :return: An object with an unlock method which will release the lock
299
 
            obtained.
 
310
        :return: A bzrlib.lock.LogicalLockResult.
300
311
        """
301
312
        raise NotImplementedError(self.lock_read)
302
313
 
794
805
            if len(old_repository._fallback_repositories) != 1:
795
806
                raise AssertionError("can't cope with fallback repositories "
796
807
                    "of %r" % (self.repository,))
797
 
            # 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()
798
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.
799
856
            old_repository.lock_read()
800
857
            try:
801
 
                # Repositories don't offer an interface to remove fallback
802
 
                # repositories today; take the conceptually simpler option and just
803
 
                # reopen it.  We reopen it starting from the URL so that we
804
 
                # get a separate connection for RemoteRepositories and can
805
 
                # stream from one of them to the other.  This does mean doing
806
 
                # separate SSH connection setup, but unstacking is not a
807
 
                # common operation so it's tolerable.
808
 
                new_bzrdir = bzrdir.BzrDir.open(self.bzrdir.root_transport.base)
809
 
                new_repository = new_bzrdir.find_repository()
810
 
                self.repository = new_repository
811
 
                if self.repository._fallback_repositories:
812
 
                    raise AssertionError("didn't expect %r to have "
813
 
                        "fallback_repositories"
814
 
                        % (self.repository,))
815
 
                # this is not paired with an unlock because it's just restoring
816
 
                # the previous state; the lock's released when set_stacked_on_url
817
 
                # returns
818
 
                self.repository.lock_write()
819
858
                # XXX: If you unstack a branch while it has a working tree
820
859
                # with a pending merge, the pending-merged revisions will no
821
860
                # longer be present.  You can (probably) revert and remerge.
962
1001
                raise errors.NoSuchRevision(self, stop_revision)
963
1002
        return other_history[self_len:stop_revision]
964
1003
 
965
 
    @needs_write_lock
966
1004
    def update_revisions(self, other, stop_revision=None, overwrite=False,
967
1005
                         graph=None):
968
1006
        """Pull in new perfect-fit revisions.
1017
1055
            self._extend_partial_history(distance_from_last)
1018
1056
        return self._partial_revision_history_cache[distance_from_last]
1019
1057
 
1020
 
    @needs_write_lock
1021
1058
    def pull(self, source, overwrite=False, stop_revision=None,
1022
1059
             possible_transports=None, *args, **kwargs):
1023
1060
        """Mirror source into this branch.
1266
1303
                revno = 1
1267
1304
        destination.set_last_revision_info(revno, revision_id)
1268
1305
 
1269
 
    @needs_read_lock
1270
1306
    def copy_content_into(self, destination, revision_id=None):
1271
1307
        """Copy the content of self into destination.
1272
1308
 
1273
1309
        revision_id: if not None, the revision history in the new branch will
1274
1310
                     be truncated to end with revision_id.
1275
1311
        """
1276
 
        self.update_references(destination)
1277
 
        self._synchronize_history(destination, revision_id)
1278
 
        try:
1279
 
            parent = self.get_parent()
1280
 
        except errors.InaccessibleParent, e:
1281
 
            mutter('parent was not accessible to copy: %s', e)
1282
 
        else:
1283
 
            if parent:
1284
 
                destination.set_parent(parent)
1285
 
        if self._push_should_merge_tags():
1286
 
            self.tags.merge_to(destination.tags)
 
1312
        return InterBranch.get(self, destination).copy_content_into(
 
1313
            revision_id=revision_id)
1287
1314
 
1288
1315
    def update_references(self, target):
1289
1316
        if not getattr(self._format, 'supports_reference_locations', False):
1357
1384
        """
1358
1385
        # XXX: Fix the bzrdir API to allow getting the branch back from the
1359
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
1360
1389
        if revision_id is None:
1361
1390
            revision_id = self.last_revision()
1362
1391
        dir_to = self.bzrdir.clone_on_transport(to_transport,
1521
1550
        try:
1522
1551
            transport = a_bzrdir.get_branch_transport(None, name=name)
1523
1552
            format_string = transport.get_bytes("format")
1524
 
            return klass._formats[format_string]
 
1553
            format = klass._formats[format_string]
 
1554
            if isinstance(format, MetaDirBranchFormatFactory):
 
1555
                return format()
 
1556
            return format
1525
1557
        except errors.NoSuchFile:
1526
1558
            raise errors.NotBranchError(path=transport.base, bzrdir=a_bzrdir)
1527
1559
        except KeyError:
1532
1564
        """Return the current default format."""
1533
1565
        return klass._default_format
1534
1566
 
1535
 
    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):
1536
1582
        """Get the target reference of the branch in a_bzrdir.
1537
1583
 
1538
1584
        format probing must have been completed before calling
1540
1586
        in a_bzrdir is correct.
1541
1587
 
1542
1588
        :param a_bzrdir: The bzrdir to get the branch data from.
 
1589
        :param name: Name of the colocated branch to fetch
1543
1590
        :return: None if the branch is not a reference branch.
1544
1591
        """
1545
1592
        return None
1546
1593
 
1547
1594
    @classmethod
1548
 
    def set_reference(self, a_bzrdir, to_branch):
 
1595
    def set_reference(self, a_bzrdir, name, to_branch):
1549
1596
        """Set the target reference of the branch in a_bzrdir.
1550
1597
 
1551
1598
        format probing must have been completed before calling
1553
1600
        in a_bzrdir is correct.
1554
1601
 
1555
1602
        :param a_bzrdir: The bzrdir to set the branch reference for.
 
1603
        :param name: Name of colocated branch to set, None for default
1556
1604
        :param to_branch: branch that the checkout is to reference
1557
1605
        """
1558
1606
        raise NotImplementedError(self.set_reference)
1672
1720
 
1673
1721
    @classmethod
1674
1722
    def register_format(klass, format):
1675
 
        """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
        """
1676
1728
        klass._formats[format.get_format_string()] = format
1677
1729
        # Metadir formats have a network name of their format string, and get
1678
 
        # registered as class factories.
1679
 
        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__)
1680
1736
 
1681
1737
    @classmethod
1682
1738
    def set_default_format(klass, format):
1702
1758
        return False  # by default
1703
1759
 
1704
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
 
1705
1789
class BranchHooks(Hooks):
1706
1790
    """A dictionary mapping hook name to a list of callables for branch hooks.
1707
1791
 
2168
2252
        """See BranchFormat.get_format_description()."""
2169
2253
        return "Checkout reference format 1"
2170
2254
 
2171
 
    def get_reference(self, a_bzrdir):
 
2255
    def get_reference(self, a_bzrdir, name=None):
2172
2256
        """See BranchFormat.get_reference()."""
2173
 
        transport = a_bzrdir.get_branch_transport(None)
 
2257
        transport = a_bzrdir.get_branch_transport(None, name=name)
2174
2258
        return transport.get_bytes('location')
2175
2259
 
2176
 
    def set_reference(self, a_bzrdir, to_branch):
 
2260
    def set_reference(self, a_bzrdir, name, to_branch):
2177
2261
        """See BranchFormat.set_reference()."""
2178
 
        transport = a_bzrdir.get_branch_transport(None)
 
2262
        transport = a_bzrdir.get_branch_transport(None, name=name)
2179
2263
        location = transport.put_bytes('location', to_branch.base)
2180
2264
 
2181
2265
    def initialize(self, a_bzrdir, name=None, target_branch=None):
2232
2316
                raise AssertionError("wrong format %r found for %r" %
2233
2317
                    (format, self))
2234
2318
        if location is None:
2235
 
            location = self.get_reference(a_bzrdir)
 
2319
            location = self.get_reference(a_bzrdir, name)
2236
2320
        real_bzrdir = bzrdir.BzrDir.open(
2237
2321
            location, possible_transports=possible_transports)
2238
2322
        result = real_bzrdir.open_branch(name=name, 
2276
2360
    _legacy_formats[0].network_name(), _legacy_formats[0].__class__)
2277
2361
 
2278
2362
 
2279
 
class BranchWriteLockResult(object):
 
2363
class BranchWriteLockResult(LogicalLockResult):
2280
2364
    """The result of write locking a branch.
2281
2365
 
2282
2366
    :ivar branch_token: The token obtained from the underlying branch lock, or
2285
2369
    """
2286
2370
 
2287
2371
    def __init__(self, unlock, branch_token):
 
2372
        LogicalLockResult.__init__(self, unlock)
2288
2373
        self.branch_token = branch_token
2289
 
        self.unlock = unlock
 
2374
 
 
2375
    def __repr__(self):
 
2376
        return "BranchWriteLockResult(%s, %s)" % (self.branch_token,
 
2377
            self.unlock)
2290
2378
 
2291
2379
 
2292
2380
class BzrBranch(Branch, _RelockDebugMixin):
2375
2463
    def lock_read(self):
2376
2464
        """Lock the branch for read operations.
2377
2465
 
2378
 
        :return: An object with an unlock method which will release the lock
2379
 
            obtained.
 
2466
        :return: A bzrlib.lock.LogicalLockResult.
2380
2467
        """
2381
2468
        if not self.is_locked():
2382
2469
            self._note_lock('r')
2390
2477
            took_lock = False
2391
2478
        try:
2392
2479
            self.control_files.lock_read()
2393
 
            return self
 
2480
            return LogicalLockResult(self.unlock)
2394
2481
        except:
2395
2482
            if took_lock:
2396
2483
                self.repository.unlock()
3182
3269
    _optimisers = []
3183
3270
    """The available optimised InterBranch types."""
3184
3271
 
3185
 
    @staticmethod
3186
 
    def _get_branch_formats_to_test():
3187
 
        """Return a tuple with the Branch formats to use when testing."""
3188
 
        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)
3189
3281
 
 
3282
    @needs_write_lock
3190
3283
    def pull(self, overwrite=False, stop_revision=None,
3191
3284
             possible_transports=None, local=False):
3192
3285
        """Mirror source into target branch.
3197
3290
        """
3198
3291
        raise NotImplementedError(self.pull)
3199
3292
 
 
3293
    @needs_write_lock
3200
3294
    def update_revisions(self, stop_revision=None, overwrite=False,
3201
3295
                         graph=None):
3202
3296
        """Pull in new perfect-fit revisions.
3210
3304
        """
3211
3305
        raise NotImplementedError(self.update_revisions)
3212
3306
 
 
3307
    @needs_write_lock
3213
3308
    def push(self, overwrite=False, stop_revision=None,
3214
3309
             _override_hook_source_branch=None):
3215
3310
        """Mirror the source branch into the target branch.
3218
3313
        """
3219
3314
        raise NotImplementedError(self.push)
3220
3315
 
 
3316
    @needs_write_lock
 
3317
    def copy_content_into(self, revision_id=None):
 
3318
        """Copy the content of source into target
 
3319
 
 
3320
        revision_id: if not None, the revision history in the new branch will
 
3321
                     be truncated to end with revision_id.
 
3322
        """
 
3323
        raise NotImplementedError(self.copy_content_into)
 
3324
 
3221
3325
 
3222
3326
class GenericInterBranch(InterBranch):
3223
 
    """InterBranch implementation that uses public Branch functions.
3224
 
    """
3225
 
 
3226
 
    @staticmethod
3227
 
    def _get_branch_formats_to_test():
3228
 
        return BranchFormat._default_format, BranchFormat._default_format
3229
 
 
 
3327
    """InterBranch implementation that uses public Branch functions."""
 
3328
 
 
3329
    @classmethod
 
3330
    def is_compatible(klass, source, target):
 
3331
        # GenericBranch uses the public API, so always compatible
 
3332
        return True
 
3333
 
 
3334
    @classmethod
 
3335
    def _get_branch_formats_to_test(klass):
 
3336
        return [(BranchFormat._default_format, BranchFormat._default_format)]
 
3337
 
 
3338
    @classmethod
 
3339
    def unwrap_format(klass, format):
 
3340
        if isinstance(format, remote.RemoteBranchFormat):
 
3341
            format._ensure_real()
 
3342
            return format._custom_format
 
3343
        return format                                                                                                  
 
3344
 
 
3345
    @needs_write_lock
 
3346
    def copy_content_into(self, revision_id=None):
 
3347
        """Copy the content of source into target
 
3348
 
 
3349
        revision_id: if not None, the revision history in the new branch will
 
3350
                     be truncated to end with revision_id.
 
3351
        """
 
3352
        self.source.update_references(self.target)
 
3353
        self.source._synchronize_history(self.target, revision_id)
 
3354
        try:
 
3355
            parent = self.source.get_parent()
 
3356
        except errors.InaccessibleParent, e:
 
3357
            mutter('parent was not accessible to copy: %s', e)
 
3358
        else:
 
3359
            if parent:
 
3360
                self.target.set_parent(parent)
 
3361
        if self.source._push_should_merge_tags():
 
3362
            self.source.tags.merge_to(self.target.tags)
 
3363
 
 
3364
    @needs_write_lock
3230
3365
    def update_revisions(self, stop_revision=None, overwrite=False,
3231
3366
        graph=None):
3232
3367
        """See InterBranch.update_revisions()."""
3233
 
        self.source.lock_read()
3234
 
        try:
3235
 
            other_revno, other_last_revision = self.source.last_revision_info()
3236
 
            stop_revno = None # unknown
3237
 
            if stop_revision is None:
3238
 
                stop_revision = other_last_revision
3239
 
                if _mod_revision.is_null(stop_revision):
3240
 
                    # if there are no commits, we're done.
3241
 
                    return
3242
 
                stop_revno = other_revno
3243
 
 
3244
 
            # what's the current last revision, before we fetch [and change it
3245
 
            # possibly]
3246
 
            last_rev = _mod_revision.ensure_null(self.target.last_revision())
3247
 
            # we fetch here so that we don't process data twice in the common
3248
 
            # case of having something to pull, and so that the check for
3249
 
            # already merged can operate on the just fetched graph, which will
3250
 
            # be cached in memory.
3251
 
            self.target.fetch(self.source, stop_revision)
3252
 
            # Check to see if one is an ancestor of the other
3253
 
            if not overwrite:
3254
 
                if graph is None:
3255
 
                    graph = self.target.repository.get_graph()
3256
 
                if self.target._check_if_descendant_or_diverged(
3257
 
                        stop_revision, last_rev, graph, self.source):
3258
 
                    # stop_revision is a descendant of last_rev, but we aren't
3259
 
                    # overwriting, so we're done.
3260
 
                    return
3261
 
            if stop_revno is None:
3262
 
                if graph is None:
3263
 
                    graph = self.target.repository.get_graph()
3264
 
                this_revno, this_last_revision = \
3265
 
                        self.target.last_revision_info()
3266
 
                stop_revno = graph.find_distance_to_null(stop_revision,
3267
 
                                [(other_last_revision, other_revno),
3268
 
                                 (this_last_revision, this_revno)])
3269
 
            self.target.set_last_revision_info(stop_revno, stop_revision)
3270
 
        finally:
3271
 
            self.source.unlock()
3272
 
 
 
3368
        other_revno, other_last_revision = self.source.last_revision_info()
 
3369
        stop_revno = None # unknown
 
3370
        if stop_revision is None:
 
3371
            stop_revision = other_last_revision
 
3372
            if _mod_revision.is_null(stop_revision):
 
3373
                # if there are no commits, we're done.
 
3374
                return
 
3375
            stop_revno = other_revno
 
3376
 
 
3377
        # what's the current last revision, before we fetch [and change it
 
3378
        # possibly]
 
3379
        last_rev = _mod_revision.ensure_null(self.target.last_revision())
 
3380
        # we fetch here so that we don't process data twice in the common
 
3381
        # case of having something to pull, and so that the check for
 
3382
        # already merged can operate on the just fetched graph, which will
 
3383
        # be cached in memory.
 
3384
        self.target.fetch(self.source, stop_revision)
 
3385
        # Check to see if one is an ancestor of the other
 
3386
        if not overwrite:
 
3387
            if graph is None:
 
3388
                graph = self.target.repository.get_graph()
 
3389
            if self.target._check_if_descendant_or_diverged(
 
3390
                    stop_revision, last_rev, graph, self.source):
 
3391
                # stop_revision is a descendant of last_rev, but we aren't
 
3392
                # overwriting, so we're done.
 
3393
                return
 
3394
        if stop_revno is None:
 
3395
            if graph is None:
 
3396
                graph = self.target.repository.get_graph()
 
3397
            this_revno, this_last_revision = \
 
3398
                    self.target.last_revision_info()
 
3399
            stop_revno = graph.find_distance_to_null(stop_revision,
 
3400
                            [(other_last_revision, other_revno),
 
3401
                             (this_last_revision, this_revno)])
 
3402
        self.target.set_last_revision_info(stop_revno, stop_revision)
 
3403
 
 
3404
    @needs_write_lock
3273
3405
    def pull(self, overwrite=False, stop_revision=None,
3274
 
             possible_transports=None, _hook_master=None, run_hooks=True,
 
3406
             possible_transports=None, run_hooks=True,
3275
3407
             _override_hook_target=None, local=False):
3276
 
        """See Branch.pull.
 
3408
        """Pull from source into self, updating my master if any.
3277
3409
 
3278
 
        :param _hook_master: Private parameter - set the branch to
3279
 
            be supplied as the master to pull hooks.
3280
3410
        :param run_hooks: Private parameter - if false, this branch
3281
3411
            is being called because it's the master of the primary branch,
3282
3412
            so it should not run its hooks.
3283
 
        :param _override_hook_target: Private parameter - set the branch to be
3284
 
            supplied as the target_branch to pull hooks.
3285
 
        :param local: Only update the local branch, and not the bound branch.
3286
3413
        """
3287
 
        # This type of branch can't be bound.
3288
 
        if local:
 
3414
        bound_location = self.target.get_bound_location()
 
3415
        if local and not bound_location:
3289
3416
            raise errors.LocalRequiresBoundBranch()
3290
 
        result = PullResult()
3291
 
        result.source_branch = self.source
3292
 
        if _override_hook_target is None:
3293
 
            result.target_branch = self.target
3294
 
        else:
3295
 
            result.target_branch = _override_hook_target
3296
 
        self.source.lock_read()
 
3417
        master_branch = None
 
3418
        if not local and bound_location and self.source.user_url != bound_location:
 
3419
            # not pulling from master, so we need to update master.
 
3420
            master_branch = self.target.get_master_branch(possible_transports)
 
3421
            master_branch.lock_write()
3297
3422
        try:
3298
 
            # We assume that during 'pull' the target repository is closer than
3299
 
            # the source one.
3300
 
            self.source.update_references(self.target)
3301
 
            graph = self.target.repository.get_graph(self.source.repository)
3302
 
            # TODO: Branch formats should have a flag that indicates 
3303
 
            # that revno's are expensive, and pull() should honor that flag.
3304
 
            # -- JRV20090506
3305
 
            result.old_revno, result.old_revid = \
3306
 
                self.target.last_revision_info()
3307
 
            self.target.update_revisions(self.source, stop_revision,
3308
 
                overwrite=overwrite, graph=graph)
3309
 
            # TODO: The old revid should be specified when merging tags, 
3310
 
            # so a tags implementation that versions tags can only 
3311
 
            # pull in the most recent changes. -- JRV20090506
3312
 
            result.tag_conflicts = self.source.tags.merge_to(self.target.tags,
3313
 
                overwrite)
3314
 
            result.new_revno, result.new_revid = self.target.last_revision_info()
3315
 
            if _hook_master:
3316
 
                result.master_branch = _hook_master
3317
 
                result.local_branch = result.target_branch
3318
 
            else:
3319
 
                result.master_branch = result.target_branch
3320
 
                result.local_branch = None
3321
 
            if run_hooks:
3322
 
                for hook in Branch.hooks['post_pull']:
3323
 
                    hook(result)
 
3423
            if master_branch:
 
3424
                # pull from source into master.
 
3425
                master_branch.pull(self.source, overwrite, stop_revision,
 
3426
                    run_hooks=False)
 
3427
            return self._pull(overwrite,
 
3428
                stop_revision, _hook_master=master_branch,
 
3429
                run_hooks=run_hooks,
 
3430
                _override_hook_target=_override_hook_target)
3324
3431
        finally:
3325
 
            self.source.unlock()
3326
 
        return result
 
3432
            if master_branch:
 
3433
                master_branch.unlock()
3327
3434
 
3328
3435
    def push(self, overwrite=False, stop_revision=None,
3329
3436
             _override_hook_source_branch=None):
3391
3498
            _run_hooks()
3392
3499
            return result
3393
3500
 
3394
 
    @classmethod
3395
 
    def is_compatible(self, source, target):
3396
 
        # GenericBranch uses the public API, so always compatible
3397
 
        return True
3398
 
 
3399
 
 
3400
 
class InterToBranch5(GenericInterBranch):
3401
 
 
3402
 
    @staticmethod
3403
 
    def _get_branch_formats_to_test():
3404
 
        return BranchFormat._default_format, BzrBranchFormat5()
3405
 
 
3406
 
    def pull(self, overwrite=False, stop_revision=None,
3407
 
             possible_transports=None, run_hooks=True,
 
3501
    def _pull(self, overwrite=False, stop_revision=None,
 
3502
             possible_transports=None, _hook_master=None, run_hooks=True,
3408
3503
             _override_hook_target=None, local=False):
3409
 
        """Pull from source into self, updating my master if any.
3410
 
 
 
3504
        """See Branch.pull.
 
3505
 
 
3506
        This function is the core worker, used by GenericInterBranch.pull to
 
3507
        avoid duplication when pulling source->master and source->local.
 
3508
 
 
3509
        :param _hook_master: Private parameter - set the branch to
 
3510
            be supplied as the master to pull hooks.
3411
3511
        :param run_hooks: Private parameter - if false, this branch
3412
3512
            is being called because it's the master of the primary branch,
3413
3513
            so it should not run its hooks.
 
3514
        :param _override_hook_target: Private parameter - set the branch to be
 
3515
            supplied as the target_branch to pull hooks.
 
3516
        :param local: Only update the local branch, and not the bound branch.
3414
3517
        """
3415
 
        bound_location = self.target.get_bound_location()
3416
 
        if local and not bound_location:
 
3518
        # This type of branch can't be bound.
 
3519
        if local:
3417
3520
            raise errors.LocalRequiresBoundBranch()
3418
 
        master_branch = None
3419
 
        if not local and bound_location and self.source.user_url != bound_location:
3420
 
            # not pulling from master, so we need to update master.
3421
 
            master_branch = self.target.get_master_branch(possible_transports)
3422
 
            master_branch.lock_write()
 
3521
        result = PullResult()
 
3522
        result.source_branch = self.source
 
3523
        if _override_hook_target is None:
 
3524
            result.target_branch = self.target
 
3525
        else:
 
3526
            result.target_branch = _override_hook_target
 
3527
        self.source.lock_read()
3423
3528
        try:
3424
 
            if master_branch:
3425
 
                # pull from source into master.
3426
 
                master_branch.pull(self.source, overwrite, stop_revision,
3427
 
                    run_hooks=False)
3428
 
            return super(InterToBranch5, self).pull(overwrite,
3429
 
                stop_revision, _hook_master=master_branch,
3430
 
                run_hooks=run_hooks,
3431
 
                _override_hook_target=_override_hook_target)
 
3529
            # We assume that during 'pull' the target repository is closer than
 
3530
            # the source one.
 
3531
            self.source.update_references(self.target)
 
3532
            graph = self.target.repository.get_graph(self.source.repository)
 
3533
            # TODO: Branch formats should have a flag that indicates 
 
3534
            # that revno's are expensive, and pull() should honor that flag.
 
3535
            # -- JRV20090506
 
3536
            result.old_revno, result.old_revid = \
 
3537
                self.target.last_revision_info()
 
3538
            self.target.update_revisions(self.source, stop_revision,
 
3539
                overwrite=overwrite, graph=graph)
 
3540
            # TODO: The old revid should be specified when merging tags, 
 
3541
            # so a tags implementation that versions tags can only 
 
3542
            # pull in the most recent changes. -- JRV20090506
 
3543
            result.tag_conflicts = self.source.tags.merge_to(self.target.tags,
 
3544
                overwrite)
 
3545
            result.new_revno, result.new_revid = self.target.last_revision_info()
 
3546
            if _hook_master:
 
3547
                result.master_branch = _hook_master
 
3548
                result.local_branch = result.target_branch
 
3549
            else:
 
3550
                result.master_branch = result.target_branch
 
3551
                result.local_branch = None
 
3552
            if run_hooks:
 
3553
                for hook in Branch.hooks['post_pull']:
 
3554
                    hook(result)
3432
3555
        finally:
3433
 
            if master_branch:
3434
 
                master_branch.unlock()
 
3556
            self.source.unlock()
 
3557
        return result
3435
3558
 
3436
3559
 
3437
3560
InterBranch.register_optimiser(GenericInterBranch)
3438
 
InterBranch.register_optimiser(InterToBranch5)