~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/branch.py

  • Committer: Martin Pool
  • Date: 2010-06-02 05:03:31 UTC
  • mto: This revision was merged to the branch mainline in revision 5279.
  • Revision ID: mbp@canonical.com-20100602050331-n2p1qt8hfsahspnv
Correct more sloppy use of the term 'Linux'

Show diffs side-by-side

added added

removed removed

Lines of Context:
25
25
        bzrdir,
26
26
        cache_utf8,
27
27
        config as _mod_config,
28
 
        controldir,
29
28
        debug,
30
29
        errors,
31
30
        lockdir,
32
31
        lockable_files,
33
 
        remote,
34
32
        repository,
35
33
        revision as _mod_revision,
36
34
        rio,
65
63
BZR_BRANCH_FORMAT_6 = "Bazaar Branch Format 6 (bzr 0.15)\n"
66
64
 
67
65
 
68
 
class Branch(controldir.ControlComponent):
 
66
class Branch(bzrdir.ControlComponent):
69
67
    """Branch holding a history of revisions.
70
68
 
71
69
    :ivar base:
199
197
        return self.supports_tags() and self.tags.get_tag_dict()
200
198
 
201
199
    def get_config(self):
202
 
        """Get a bzrlib.config.BranchConfig for this Branch.
203
 
 
204
 
        This can then be used to get and set configuration options for the
205
 
        branch.
206
 
 
207
 
        :return: A bzrlib.config.BranchConfig.
208
 
        """
209
200
        return BranchConfig(self)
210
201
 
211
202
    def _get_config(self):
247
238
        if not local and not config.has_explicit_nickname():
248
239
            try:
249
240
                master = self.get_master_branch(possible_transports)
250
 
                if master and self.user_url == master.user_url:
251
 
                    raise errors.RecursiveBind(self.user_url)
252
241
                if master is not None:
253
242
                    # return the master branch value
254
243
                    return master.nick
255
 
            except errors.RecursiveBind, e:
256
 
                raise e
257
244
            except errors.BzrError, e:
258
245
                # Silently fall back to local implicit nick if the master is
259
246
                # unavailable
806
793
            if len(old_repository._fallback_repositories) != 1:
807
794
                raise AssertionError("can't cope with fallback repositories "
808
795
                    "of %r" % (self.repository,))
809
 
            # Open the new repository object.
810
 
            # Repositories don't offer an interface to remove fallback
811
 
            # repositories today; take the conceptually simpler option and just
812
 
            # reopen it.  We reopen it starting from the URL so that we
813
 
            # get a separate connection for RemoteRepositories and can
814
 
            # stream from one of them to the other.  This does mean doing
815
 
            # separate SSH connection setup, but unstacking is not a
816
 
            # common operation so it's tolerable.
817
 
            new_bzrdir = bzrdir.BzrDir.open(self.bzrdir.root_transport.base)
818
 
            new_repository = new_bzrdir.find_repository()
819
 
            if new_repository._fallback_repositories:
820
 
                raise AssertionError("didn't expect %r to have "
821
 
                    "fallback_repositories"
822
 
                    % (self.repository,))
823
 
            # Replace self.repository with the new repository.
824
 
            # Do our best to transfer the lock state (i.e. lock-tokens and
825
 
            # lock count) of self.repository to the new repository.
826
 
            lock_token = old_repository.lock_write().repository_token
827
 
            self.repository = new_repository
828
 
            if isinstance(self, remote.RemoteBranch):
829
 
                # Remote branches can have a second reference to the old
830
 
                # repository that need to be replaced.
831
 
                if self._real_branch is not None:
832
 
                    self._real_branch.repository = new_repository
833
 
            self.repository.lock_write(token=lock_token)
834
 
            if lock_token is not None:
835
 
                old_repository.leave_lock_in_place()
 
796
            # unlock it, including unlocking the fallback
836
797
            old_repository.unlock()
837
 
            if lock_token is not None:
838
 
                # XXX: self.repository.leave_lock_in_place() before this
839
 
                # function will not be preserved.  Fortunately that doesn't
840
 
                # affect the current default format (2a), and would be a
841
 
                # corner-case anyway.
842
 
                #  - Andrew Bennetts, 2010/06/30
843
 
                self.repository.dont_leave_lock_in_place()
844
 
            old_lock_count = 0
845
 
            while True:
846
 
                try:
847
 
                    old_repository.unlock()
848
 
                except errors.LockNotHeld:
849
 
                    break
850
 
                old_lock_count += 1
851
 
            if old_lock_count == 0:
852
 
                raise AssertionError(
853
 
                    'old_repository should have been locked at least once.')
854
 
            for i in range(old_lock_count-1):
 
798
            old_repository.lock_read()
 
799
            try:
 
800
                # Repositories don't offer an interface to remove fallback
 
801
                # repositories today; take the conceptually simpler option and just
 
802
                # reopen it.  We reopen it starting from the URL so that we
 
803
                # get a separate connection for RemoteRepositories and can
 
804
                # stream from one of them to the other.  This does mean doing
 
805
                # separate SSH connection setup, but unstacking is not a
 
806
                # common operation so it's tolerable.
 
807
                new_bzrdir = bzrdir.BzrDir.open(self.bzrdir.root_transport.base)
 
808
                new_repository = new_bzrdir.find_repository()
 
809
                self.repository = new_repository
 
810
                if self.repository._fallback_repositories:
 
811
                    raise AssertionError("didn't expect %r to have "
 
812
                        "fallback_repositories"
 
813
                        % (self.repository,))
 
814
                # this is not paired with an unlock because it's just restoring
 
815
                # the previous state; the lock's released when set_stacked_on_url
 
816
                # returns
855
817
                self.repository.lock_write()
856
 
            # Fetch from the old repository into the new.
857
 
            old_repository.lock_read()
858
 
            try:
859
818
                # XXX: If you unstack a branch while it has a working tree
860
819
                # with a pending merge, the pending-merged revisions will no
861
820
                # longer be present.  You can (probably) revert and remerge.
1002
961
                raise errors.NoSuchRevision(self, stop_revision)
1003
962
        return other_history[self_len:stop_revision]
1004
963
 
 
964
    @needs_write_lock
1005
965
    def update_revisions(self, other, stop_revision=None, overwrite=False,
1006
966
                         graph=None):
1007
967
        """Pull in new perfect-fit revisions.
1056
1016
            self._extend_partial_history(distance_from_last)
1057
1017
        return self._partial_revision_history_cache[distance_from_last]
1058
1018
 
 
1019
    @needs_write_lock
1059
1020
    def pull(self, source, overwrite=False, stop_revision=None,
1060
1021
             possible_transports=None, *args, **kwargs):
1061
1022
        """Mirror source into this branch.
1304
1265
                revno = 1
1305
1266
        destination.set_last_revision_info(revno, revision_id)
1306
1267
 
 
1268
    @needs_read_lock
1307
1269
    def copy_content_into(self, destination, revision_id=None):
1308
1270
        """Copy the content of self into destination.
1309
1271
 
1310
1272
        revision_id: if not None, the revision history in the new branch will
1311
1273
                     be truncated to end with revision_id.
1312
1274
        """
1313
 
        return InterBranch.get(self, destination).copy_content_into(
1314
 
            revision_id=revision_id)
 
1275
        self.update_references(destination)
 
1276
        self._synchronize_history(destination, revision_id)
 
1277
        try:
 
1278
            parent = self.get_parent()
 
1279
        except errors.InaccessibleParent, e:
 
1280
            mutter('parent was not accessible to copy: %s', e)
 
1281
        else:
 
1282
            if parent:
 
1283
                destination.set_parent(parent)
 
1284
        if self._push_should_merge_tags():
 
1285
            self.tags.merge_to(destination.tags)
1315
1286
 
1316
1287
    def update_references(self, target):
1317
1288
        if not getattr(self._format, 'supports_reference_locations', False):
1522
1493
     * an open routine.
1523
1494
 
1524
1495
    Formats are placed in an dict by their format string for reference
1525
 
    during branch opening. It's not required that these be instances, they
 
1496
    during branch opening. Its not required that these be instances, they
1526
1497
    can be classes themselves with class methods - it simply depends on
1527
1498
    whether state is needed for a given format or not.
1528
1499
 
1551
1522
        try:
1552
1523
            transport = a_bzrdir.get_branch_transport(None, name=name)
1553
1524
            format_string = transport.get_bytes("format")
1554
 
            format = klass._formats[format_string]
1555
 
            if isinstance(format, MetaDirBranchFormatFactory):
1556
 
                return format()
1557
 
            return format
 
1525
            return klass._formats[format_string]
1558
1526
        except errors.NoSuchFile:
1559
1527
            raise errors.NotBranchError(path=transport.base, bzrdir=a_bzrdir)
1560
1528
        except KeyError:
1565
1533
        """Return the current default format."""
1566
1534
        return klass._default_format
1567
1535
 
1568
 
    @classmethod
1569
 
    def get_formats(klass):
1570
 
        """Get all the known formats.
1571
 
 
1572
 
        Warning: This triggers a load of all lazy registered formats: do not
1573
 
        use except when that is desireed.
1574
 
        """
1575
 
        result = []
1576
 
        for fmt in klass._formats.values():
1577
 
            if isinstance(fmt, MetaDirBranchFormatFactory):
1578
 
                fmt = fmt()
1579
 
            result.append(fmt)
1580
 
        return result
1581
 
 
1582
1536
    def get_reference(self, a_bzrdir, name=None):
1583
1537
        """Get the target reference of the branch in a_bzrdir.
1584
1538
 
1721
1675
 
1722
1676
    @classmethod
1723
1677
    def register_format(klass, format):
1724
 
        """Register a metadir format.
1725
 
        
1726
 
        See MetaDirBranchFormatFactory for the ability to register a format
1727
 
        without loading the code the format needs until it is actually used.
1728
 
        """
 
1678
        """Register a metadir format."""
1729
1679
        klass._formats[format.get_format_string()] = format
1730
1680
        # Metadir formats have a network name of their format string, and get
1731
 
        # registered as factories.
1732
 
        if isinstance(format, MetaDirBranchFormatFactory):
1733
 
            network_format_registry.register(format.get_format_string(), format)
1734
 
        else:
1735
 
            network_format_registry.register(format.get_format_string(),
1736
 
                format.__class__)
 
1681
        # registered as class factories.
 
1682
        network_format_registry.register(format.get_format_string(), format.__class__)
1737
1683
 
1738
1684
    @classmethod
1739
1685
    def set_default_format(klass, format):
1759
1705
        return False  # by default
1760
1706
 
1761
1707
 
1762
 
class MetaDirBranchFormatFactory(registry._LazyObjectGetter):
1763
 
    """A factory for a BranchFormat object, permitting simple lazy registration.
1764
 
    
1765
 
    While none of the built in BranchFormats are lazy registered yet,
1766
 
    bzrlib.tests.test_branch.TestMetaDirBranchFormatFactory demonstrates how to
1767
 
    use it, and the bzr-loom plugin uses it as well (see
1768
 
    bzrlib.plugins.loom.formats).
1769
 
    """
1770
 
 
1771
 
    def __init__(self, format_string, module_name, member_name):
1772
 
        """Create a MetaDirBranchFormatFactory.
1773
 
 
1774
 
        :param format_string: The format string the format has.
1775
 
        :param module_name: Module to load the format class from.
1776
 
        :param member_name: Attribute name within the module for the format class.
1777
 
        """
1778
 
        registry._LazyObjectGetter.__init__(self, module_name, member_name)
1779
 
        self._format_string = format_string
1780
 
        
1781
 
    def get_format_string(self):
1782
 
        """See BranchFormat.get_format_string."""
1783
 
        return self._format_string
1784
 
 
1785
 
    def __call__(self):
1786
 
        """Used for network_format_registry support."""
1787
 
        return self.get_obj()()
1788
 
 
1789
 
 
1790
1708
class BranchHooks(Hooks):
1791
1709
    """A dictionary mapping hook name to a list of callables for branch hooks.
1792
1710
 
1819
1737
            "with a bzrlib.branch.PullResult object and only runs in the "
1820
1738
            "bzr client.", (0, 15), None))
1821
1739
        self.create_hook(HookPoint('pre_commit',
1822
 
            "Called after a commit is calculated but before it is "
 
1740
            "Called after a commit is calculated but before it is is "
1823
1741
            "completed. pre_commit is called with (local, master, old_revno, "
1824
1742
            "old_revid, future_revno, future_revid, tree_delta, future_tree"
1825
1743
            "). old_revid is NULL_REVISION for the first commit to a branch, "
1862
1780
            "all are called with the url returned from the previous hook."
1863
1781
            "The order is however undefined.", (1, 9), None))
1864
1782
        self.create_hook(HookPoint('automatic_tag_name',
1865
 
            "Called to determine an automatic tag name for a revision. "
 
1783
            "Called to determine an automatic tag name for a revision."
1866
1784
            "automatic_tag_name is called with (branch, revision_id) and "
1867
1785
            "should return a tag name or None if no tag name could be "
1868
1786
            "determined. The first non-None tag name returned will be used.",
1959
1877
        return self.__dict__ == other.__dict__
1960
1878
 
1961
1879
    def __repr__(self):
1962
 
        return "<%s of %s>" % (self.__class__.__name__, self.branch)
 
1880
        if self.branch:
 
1881
            return "<%s of %s>" % (self.__class__.__name__, self.branch)
 
1882
        else:
 
1883
            return "<%s of format:%s bzrdir:%s>" % (
 
1884
                self.__class__.__name__, self.branch,
 
1885
                self.format, self.bzrdir)
1963
1886
 
1964
1887
 
1965
1888
class SwitchHookParams(object):
3103
3026
    :ivar tag_conflicts: A list of tag conflicts, see BasicTags.merge_to
3104
3027
    """
3105
3028
 
3106
 
    @deprecated_method(deprecated_in((2, 3, 0)))
3107
3029
    def __int__(self):
3108
 
        """Return the relative change in revno.
3109
 
 
3110
 
        :deprecated: Use `new_revno` and `old_revno` instead.
3111
 
        """
 
3030
        # DEPRECATED: pull used to return the change in revno
3112
3031
        return self.new_revno - self.old_revno
3113
3032
 
3114
3033
    def report(self, to_file):
3139
3058
        target, otherwise it will be None.
3140
3059
    """
3141
3060
 
3142
 
    @deprecated_method(deprecated_in((2, 3, 0)))
3143
3061
    def __int__(self):
3144
 
        """Return the relative change in revno.
3145
 
 
3146
 
        :deprecated: Use `new_revno` and `old_revno` instead.
3147
 
        """
 
3062
        # DEPRECATED: push used to return the change in revno
3148
3063
        return self.new_revno - self.old_revno
3149
3064
 
3150
3065
    def report(self, to_file):
3273
3188
    _optimisers = []
3274
3189
    """The available optimised InterBranch types."""
3275
3190
 
3276
 
    @classmethod
3277
 
    def _get_branch_formats_to_test(klass):
3278
 
        """Return an iterable of format tuples for testing.
3279
 
        
3280
 
        :return: An iterable of (from_format, to_format) to use when testing
3281
 
            this InterBranch class. Each InterBranch class should define this
3282
 
            method itself.
3283
 
        """
3284
 
        raise NotImplementedError(klass._get_branch_formats_to_test)
 
3191
    @staticmethod
 
3192
    def _get_branch_formats_to_test():
 
3193
        """Return a tuple with the Branch formats to use when testing."""
 
3194
        raise NotImplementedError(InterBranch._get_branch_formats_to_test)
3285
3195
 
3286
 
    @needs_write_lock
3287
3196
    def pull(self, overwrite=False, stop_revision=None,
3288
3197
             possible_transports=None, local=False):
3289
3198
        """Mirror source into target branch.
3294
3203
        """
3295
3204
        raise NotImplementedError(self.pull)
3296
3205
 
3297
 
    @needs_write_lock
3298
3206
    def update_revisions(self, stop_revision=None, overwrite=False,
3299
3207
                         graph=None):
3300
3208
        """Pull in new perfect-fit revisions.
3308
3216
        """
3309
3217
        raise NotImplementedError(self.update_revisions)
3310
3218
 
3311
 
    @needs_write_lock
3312
3219
    def push(self, overwrite=False, stop_revision=None,
3313
3220
             _override_hook_source_branch=None):
3314
3221
        """Mirror the source branch into the target branch.
3317
3224
        """
3318
3225
        raise NotImplementedError(self.push)
3319
3226
 
3320
 
    @needs_write_lock
3321
 
    def copy_content_into(self, revision_id=None):
3322
 
        """Copy the content of source into target
3323
 
 
3324
 
        revision_id: if not None, the revision history in the new branch will
3325
 
                     be truncated to end with revision_id.
3326
 
        """
3327
 
        raise NotImplementedError(self.copy_content_into)
3328
 
 
3329
3227
 
3330
3228
class GenericInterBranch(InterBranch):
3331
 
    """InterBranch implementation that uses public Branch functions."""
3332
 
 
3333
 
    @classmethod
3334
 
    def is_compatible(klass, source, target):
3335
 
        # GenericBranch uses the public API, so always compatible
3336
 
        return True
3337
 
 
3338
 
    @classmethod
3339
 
    def _get_branch_formats_to_test(klass):
3340
 
        return [(BranchFormat._default_format, BranchFormat._default_format)]
3341
 
 
3342
 
    @classmethod
3343
 
    def unwrap_format(klass, format):
3344
 
        if isinstance(format, remote.RemoteBranchFormat):
3345
 
            format._ensure_real()
3346
 
            return format._custom_format
3347
 
        return format                                                                                                  
3348
 
 
3349
 
    @needs_write_lock
3350
 
    def copy_content_into(self, revision_id=None):
3351
 
        """Copy the content of source into target
3352
 
 
3353
 
        revision_id: if not None, the revision history in the new branch will
3354
 
                     be truncated to end with revision_id.
3355
 
        """
3356
 
        self.source.update_references(self.target)
3357
 
        self.source._synchronize_history(self.target, revision_id)
3358
 
        try:
3359
 
            parent = self.source.get_parent()
3360
 
        except errors.InaccessibleParent, e:
3361
 
            mutter('parent was not accessible to copy: %s', e)
3362
 
        else:
3363
 
            if parent:
3364
 
                self.target.set_parent(parent)
3365
 
        if self.source._push_should_merge_tags():
3366
 
            self.source.tags.merge_to(self.target.tags)
3367
 
 
3368
 
    @needs_write_lock
 
3229
    """InterBranch implementation that uses public Branch functions.
 
3230
    """
 
3231
 
 
3232
    @staticmethod
 
3233
    def _get_branch_formats_to_test():
 
3234
        return BranchFormat._default_format, BranchFormat._default_format
 
3235
 
3369
3236
    def update_revisions(self, stop_revision=None, overwrite=False,
3370
3237
        graph=None):
3371
3238
        """See InterBranch.update_revisions()."""
3372
 
        other_revno, other_last_revision = self.source.last_revision_info()
3373
 
        stop_revno = None # unknown
3374
 
        if stop_revision is None:
3375
 
            stop_revision = other_last_revision
3376
 
            if _mod_revision.is_null(stop_revision):
3377
 
                # if there are no commits, we're done.
3378
 
                return
3379
 
            stop_revno = other_revno
3380
 
 
3381
 
        # what's the current last revision, before we fetch [and change it
3382
 
        # possibly]
3383
 
        last_rev = _mod_revision.ensure_null(self.target.last_revision())
3384
 
        # we fetch here so that we don't process data twice in the common
3385
 
        # case of having something to pull, and so that the check for
3386
 
        # already merged can operate on the just fetched graph, which will
3387
 
        # be cached in memory.
3388
 
        self.target.fetch(self.source, stop_revision)
3389
 
        # Check to see if one is an ancestor of the other
3390
 
        if not overwrite:
3391
 
            if graph is None:
3392
 
                graph = self.target.repository.get_graph()
3393
 
            if self.target._check_if_descendant_or_diverged(
3394
 
                    stop_revision, last_rev, graph, self.source):
3395
 
                # stop_revision is a descendant of last_rev, but we aren't
3396
 
                # overwriting, so we're done.
3397
 
                return
3398
 
        if stop_revno is None:
3399
 
            if graph is None:
3400
 
                graph = self.target.repository.get_graph()
3401
 
            this_revno, this_last_revision = \
3402
 
                    self.target.last_revision_info()
3403
 
            stop_revno = graph.find_distance_to_null(stop_revision,
3404
 
                            [(other_last_revision, other_revno),
3405
 
                             (this_last_revision, this_revno)])
3406
 
        self.target.set_last_revision_info(stop_revno, stop_revision)
3407
 
 
3408
 
    @needs_write_lock
 
3239
        self.source.lock_read()
 
3240
        try:
 
3241
            other_revno, other_last_revision = self.source.last_revision_info()
 
3242
            stop_revno = None # unknown
 
3243
            if stop_revision is None:
 
3244
                stop_revision = other_last_revision
 
3245
                if _mod_revision.is_null(stop_revision):
 
3246
                    # if there are no commits, we're done.
 
3247
                    return
 
3248
                stop_revno = other_revno
 
3249
 
 
3250
            # what's the current last revision, before we fetch [and change it
 
3251
            # possibly]
 
3252
            last_rev = _mod_revision.ensure_null(self.target.last_revision())
 
3253
            # we fetch here so that we don't process data twice in the common
 
3254
            # case of having something to pull, and so that the check for
 
3255
            # already merged can operate on the just fetched graph, which will
 
3256
            # be cached in memory.
 
3257
            self.target.fetch(self.source, stop_revision)
 
3258
            # Check to see if one is an ancestor of the other
 
3259
            if not overwrite:
 
3260
                if graph is None:
 
3261
                    graph = self.target.repository.get_graph()
 
3262
                if self.target._check_if_descendant_or_diverged(
 
3263
                        stop_revision, last_rev, graph, self.source):
 
3264
                    # stop_revision is a descendant of last_rev, but we aren't
 
3265
                    # overwriting, so we're done.
 
3266
                    return
 
3267
            if stop_revno is None:
 
3268
                if graph is None:
 
3269
                    graph = self.target.repository.get_graph()
 
3270
                this_revno, this_last_revision = \
 
3271
                        self.target.last_revision_info()
 
3272
                stop_revno = graph.find_distance_to_null(stop_revision,
 
3273
                                [(other_last_revision, other_revno),
 
3274
                                 (this_last_revision, this_revno)])
 
3275
            self.target.set_last_revision_info(stop_revno, stop_revision)
 
3276
        finally:
 
3277
            self.source.unlock()
 
3278
 
3409
3279
    def pull(self, overwrite=False, stop_revision=None,
3410
 
             possible_transports=None, run_hooks=True,
 
3280
             possible_transports=None, _hook_master=None, run_hooks=True,
3411
3281
             _override_hook_target=None, local=False):
3412
 
        """Pull from source into self, updating my master if any.
 
3282
        """See Branch.pull.
3413
3283
 
 
3284
        :param _hook_master: Private parameter - set the branch to
 
3285
            be supplied as the master to pull hooks.
3414
3286
        :param run_hooks: Private parameter - if false, this branch
3415
3287
            is being called because it's the master of the primary branch,
3416
3288
            so it should not run its hooks.
 
3289
        :param _override_hook_target: Private parameter - set the branch to be
 
3290
            supplied as the target_branch to pull hooks.
 
3291
        :param local: Only update the local branch, and not the bound branch.
3417
3292
        """
3418
 
        bound_location = self.target.get_bound_location()
3419
 
        if local and not bound_location:
 
3293
        # This type of branch can't be bound.
 
3294
        if local:
3420
3295
            raise errors.LocalRequiresBoundBranch()
3421
 
        master_branch = None
3422
 
        if not local and bound_location and self.source.user_url != bound_location:
3423
 
            # not pulling from master, so we need to update master.
3424
 
            master_branch = self.target.get_master_branch(possible_transports)
3425
 
            master_branch.lock_write()
 
3296
        result = PullResult()
 
3297
        result.source_branch = self.source
 
3298
        if _override_hook_target is None:
 
3299
            result.target_branch = self.target
 
3300
        else:
 
3301
            result.target_branch = _override_hook_target
 
3302
        self.source.lock_read()
3426
3303
        try:
3427
 
            if master_branch:
3428
 
                # pull from source into master.
3429
 
                master_branch.pull(self.source, overwrite, stop_revision,
3430
 
                    run_hooks=False)
3431
 
            return self._pull(overwrite,
3432
 
                stop_revision, _hook_master=master_branch,
3433
 
                run_hooks=run_hooks,
3434
 
                _override_hook_target=_override_hook_target)
 
3304
            # We assume that during 'pull' the target repository is closer than
 
3305
            # the source one.
 
3306
            self.source.update_references(self.target)
 
3307
            graph = self.target.repository.get_graph(self.source.repository)
 
3308
            # TODO: Branch formats should have a flag that indicates 
 
3309
            # that revno's are expensive, and pull() should honor that flag.
 
3310
            # -- JRV20090506
 
3311
            result.old_revno, result.old_revid = \
 
3312
                self.target.last_revision_info()
 
3313
            self.target.update_revisions(self.source, stop_revision,
 
3314
                overwrite=overwrite, graph=graph)
 
3315
            # TODO: The old revid should be specified when merging tags, 
 
3316
            # so a tags implementation that versions tags can only 
 
3317
            # pull in the most recent changes. -- JRV20090506
 
3318
            result.tag_conflicts = self.source.tags.merge_to(self.target.tags,
 
3319
                overwrite)
 
3320
            result.new_revno, result.new_revid = self.target.last_revision_info()
 
3321
            if _hook_master:
 
3322
                result.master_branch = _hook_master
 
3323
                result.local_branch = result.target_branch
 
3324
            else:
 
3325
                result.master_branch = result.target_branch
 
3326
                result.local_branch = None
 
3327
            if run_hooks:
 
3328
                for hook in Branch.hooks['post_pull']:
 
3329
                    hook(result)
3435
3330
        finally:
3436
 
            if master_branch:
3437
 
                master_branch.unlock()
 
3331
            self.source.unlock()
 
3332
        return result
3438
3333
 
3439
3334
    def push(self, overwrite=False, stop_revision=None,
3440
3335
             _override_hook_source_branch=None):
3480
3375
                # push into the master from the source branch.
3481
3376
                self.source._basic_push(master_branch, overwrite, stop_revision)
3482
3377
                # and push into the target branch from the source. Note that we
3483
 
                # push from the source branch again, because it's considered the
 
3378
                # push from the source branch again, because its considered the
3484
3379
                # highest bandwidth repository.
3485
3380
                result = self.source._basic_push(self.target, overwrite,
3486
3381
                    stop_revision)
3502
3397
            _run_hooks()
3503
3398
            return result
3504
3399
 
3505
 
    def _pull(self, overwrite=False, stop_revision=None,
3506
 
             possible_transports=None, _hook_master=None, run_hooks=True,
 
3400
    @classmethod
 
3401
    def is_compatible(self, source, target):
 
3402
        # GenericBranch uses the public API, so always compatible
 
3403
        return True
 
3404
 
 
3405
 
 
3406
class InterToBranch5(GenericInterBranch):
 
3407
 
 
3408
    @staticmethod
 
3409
    def _get_branch_formats_to_test():
 
3410
        return BranchFormat._default_format, BzrBranchFormat5()
 
3411
 
 
3412
    def pull(self, overwrite=False, stop_revision=None,
 
3413
             possible_transports=None, run_hooks=True,
3507
3414
             _override_hook_target=None, local=False):
3508
 
        """See Branch.pull.
3509
 
 
3510
 
        This function is the core worker, used by GenericInterBranch.pull to
3511
 
        avoid duplication when pulling source->master and source->local.
3512
 
 
3513
 
        :param _hook_master: Private parameter - set the branch to
3514
 
            be supplied as the master to pull hooks.
 
3415
        """Pull from source into self, updating my master if any.
 
3416
 
3515
3417
        :param run_hooks: Private parameter - if false, this branch
3516
3418
            is being called because it's the master of the primary branch,
3517
3419
            so it should not run its hooks.
3518
 
        :param _override_hook_target: Private parameter - set the branch to be
3519
 
            supplied as the target_branch to pull hooks.
3520
 
        :param local: Only update the local branch, and not the bound branch.
3521
3420
        """
3522
 
        # This type of branch can't be bound.
3523
 
        if local:
 
3421
        bound_location = self.target.get_bound_location()
 
3422
        if local and not bound_location:
3524
3423
            raise errors.LocalRequiresBoundBranch()
3525
 
        result = PullResult()
3526
 
        result.source_branch = self.source
3527
 
        if _override_hook_target is None:
3528
 
            result.target_branch = self.target
3529
 
        else:
3530
 
            result.target_branch = _override_hook_target
3531
 
        self.source.lock_read()
 
3424
        master_branch = None
 
3425
        if not local and bound_location and self.source.user_url != bound_location:
 
3426
            # not pulling from master, so we need to update master.
 
3427
            master_branch = self.target.get_master_branch(possible_transports)
 
3428
            master_branch.lock_write()
3532
3429
        try:
3533
 
            # We assume that during 'pull' the target repository is closer than
3534
 
            # the source one.
3535
 
            self.source.update_references(self.target)
3536
 
            graph = self.target.repository.get_graph(self.source.repository)
3537
 
            # TODO: Branch formats should have a flag that indicates 
3538
 
            # that revno's are expensive, and pull() should honor that flag.
3539
 
            # -- JRV20090506
3540
 
            result.old_revno, result.old_revid = \
3541
 
                self.target.last_revision_info()
3542
 
            self.target.update_revisions(self.source, stop_revision,
3543
 
                overwrite=overwrite, graph=graph)
3544
 
            # TODO: The old revid should be specified when merging tags, 
3545
 
            # so a tags implementation that versions tags can only 
3546
 
            # pull in the most recent changes. -- JRV20090506
3547
 
            result.tag_conflicts = self.source.tags.merge_to(self.target.tags,
3548
 
                overwrite)
3549
 
            result.new_revno, result.new_revid = self.target.last_revision_info()
3550
 
            if _hook_master:
3551
 
                result.master_branch = _hook_master
3552
 
                result.local_branch = result.target_branch
3553
 
            else:
3554
 
                result.master_branch = result.target_branch
3555
 
                result.local_branch = None
3556
 
            if run_hooks:
3557
 
                for hook in Branch.hooks['post_pull']:
3558
 
                    hook(result)
 
3430
            if master_branch:
 
3431
                # pull from source into master.
 
3432
                master_branch.pull(self.source, overwrite, stop_revision,
 
3433
                    run_hooks=False)
 
3434
            return super(InterToBranch5, self).pull(overwrite,
 
3435
                stop_revision, _hook_master=master_branch,
 
3436
                run_hooks=run_hooks,
 
3437
                _override_hook_target=_override_hook_target)
3559
3438
        finally:
3560
 
            self.source.unlock()
3561
 
        return result
 
3439
            if master_branch:
 
3440
                master_branch.unlock()
3562
3441
 
3563
3442
 
3564
3443
InterBranch.register_optimiser(GenericInterBranch)
 
3444
InterBranch.register_optimiser(InterToBranch5)