/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: Vincent Ladeuil
  • Date: 2010-06-17 09:23:19 UTC
  • mfrom: (5301 +trunk)
  • mto: (5247.1.8 first-try)
  • mto: This revision was merged to the branch mainline in revision 5326.
  • Revision ID: v.ladeuil+lp@free.fr-20100617092319-da2bzdtf3j0voynf
Merge bzr.dev into cleanup

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):
283
291
        new_history.reverse()
284
292
        return new_history
285
293
 
286
 
    def lock_write(self):
 
294
    def lock_write(self, token=None):
 
295
        """Lock the branch for write operations.
 
296
 
 
297
        :param token: A token to permit reacquiring a previously held and
 
298
            preserved lock.
 
299
        :return: A BranchWriteLockResult.
 
300
        """
287
301
        raise NotImplementedError(self.lock_write)
288
302
 
289
303
    def lock_read(self):
 
304
        """Lock the branch for read operations.
 
305
 
 
306
        :return: A bzrlib.lock.LogicalLockResult.
 
307
        """
290
308
        raise NotImplementedError(self.lock_read)
291
309
 
292
310
    def unlock(self):
951
969
                raise errors.NoSuchRevision(self, stop_revision)
952
970
        return other_history[self_len:stop_revision]
953
971
 
954
 
    @needs_write_lock
955
972
    def update_revisions(self, other, stop_revision=None, overwrite=False,
956
973
                         graph=None):
957
974
        """Pull in new perfect-fit revisions.
1006
1023
            self._extend_partial_history(distance_from_last)
1007
1024
        return self._partial_revision_history_cache[distance_from_last]
1008
1025
 
1009
 
    @needs_write_lock
1010
1026
    def pull(self, source, overwrite=False, stop_revision=None,
1011
1027
             possible_transports=None, *args, **kwargs):
1012
1028
        """Mirror source into this branch.
1255
1271
                revno = 1
1256
1272
        destination.set_last_revision_info(revno, revision_id)
1257
1273
 
1258
 
    @needs_read_lock
1259
1274
    def copy_content_into(self, destination, revision_id=None):
1260
1275
        """Copy the content of self into destination.
1261
1276
 
1262
1277
        revision_id: if not None, the revision history in the new branch will
1263
1278
                     be truncated to end with revision_id.
1264
1279
        """
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)
 
1280
        return InterBranch.get(self, destination).copy_content_into(
 
1281
            revision_id=revision_id)
1276
1282
 
1277
1283
    def update_references(self, target):
1278
1284
        if not getattr(self._format, 'supports_reference_locations', False):
1346
1352
        """
1347
1353
        # XXX: Fix the bzrdir API to allow getting the branch back from the
1348
1354
        # clone call. Or something. 20090224 RBC/spiv.
 
1355
        # XXX: Should this perhaps clone colocated branches as well, 
 
1356
        # rather than just the default branch? 20100319 JRV
1349
1357
        if revision_id is None:
1350
1358
            revision_id = self.last_revision()
1351
1359
        dir_to = self.bzrdir.clone_on_transport(to_transport,
1521
1529
        """Return the current default format."""
1522
1530
        return klass._default_format
1523
1531
 
1524
 
    def get_reference(self, a_bzrdir):
 
1532
    def get_reference(self, a_bzrdir, name=None):
1525
1533
        """Get the target reference of the branch in a_bzrdir.
1526
1534
 
1527
1535
        format probing must have been completed before calling
1529
1537
        in a_bzrdir is correct.
1530
1538
 
1531
1539
        :param a_bzrdir: The bzrdir to get the branch data from.
 
1540
        :param name: Name of the colocated branch to fetch
1532
1541
        :return: None if the branch is not a reference branch.
1533
1542
        """
1534
1543
        return None
1535
1544
 
1536
1545
    @classmethod
1537
 
    def set_reference(self, a_bzrdir, to_branch):
 
1546
    def set_reference(self, a_bzrdir, name, to_branch):
1538
1547
        """Set the target reference of the branch in a_bzrdir.
1539
1548
 
1540
1549
        format probing must have been completed before calling
1542
1551
        in a_bzrdir is correct.
1543
1552
 
1544
1553
        :param a_bzrdir: The bzrdir to set the branch reference for.
 
1554
        :param name: Name of colocated branch to set, None for default
1545
1555
        :param to_branch: branch that the checkout is to reference
1546
1556
        """
1547
1557
        raise NotImplementedError(self.set_reference)
2157
2167
        """See BranchFormat.get_format_description()."""
2158
2168
        return "Checkout reference format 1"
2159
2169
 
2160
 
    def get_reference(self, a_bzrdir):
 
2170
    def get_reference(self, a_bzrdir, name=None):
2161
2171
        """See BranchFormat.get_reference()."""
2162
 
        transport = a_bzrdir.get_branch_transport(None)
 
2172
        transport = a_bzrdir.get_branch_transport(None, name=name)
2163
2173
        return transport.get_bytes('location')
2164
2174
 
2165
 
    def set_reference(self, a_bzrdir, to_branch):
 
2175
    def set_reference(self, a_bzrdir, name, to_branch):
2166
2176
        """See BranchFormat.set_reference()."""
2167
 
        transport = a_bzrdir.get_branch_transport(None)
 
2177
        transport = a_bzrdir.get_branch_transport(None, name=name)
2168
2178
        location = transport.put_bytes('location', to_branch.base)
2169
2179
 
2170
2180
    def initialize(self, a_bzrdir, name=None, target_branch=None):
2221
2231
                raise AssertionError("wrong format %r found for %r" %
2222
2232
                    (format, self))
2223
2233
        if location is None:
2224
 
            location = self.get_reference(a_bzrdir)
 
2234
            location = self.get_reference(a_bzrdir, name)
2225
2235
        real_bzrdir = bzrdir.BzrDir.open(
2226
2236
            location, possible_transports=possible_transports)
2227
2237
        result = real_bzrdir.open_branch(name=name, 
2265
2275
    _legacy_formats[0].network_name(), _legacy_formats[0].__class__)
2266
2276
 
2267
2277
 
 
2278
class BranchWriteLockResult(LogicalLockResult):
 
2279
    """The result of write locking a branch.
 
2280
 
 
2281
    :ivar branch_token: The token obtained from the underlying branch lock, or
 
2282
        None.
 
2283
    :ivar unlock: A callable which will unlock the lock.
 
2284
    """
 
2285
 
 
2286
    def __init__(self, unlock, branch_token):
 
2287
        LogicalLockResult.__init__(self, unlock)
 
2288
        self.branch_token = branch_token
 
2289
 
 
2290
    def __repr__(self):
 
2291
        return "BranchWriteLockResult(%s, %s)" % (self.branch_token,
 
2292
            self.unlock)
 
2293
 
 
2294
 
2268
2295
class BzrBranch(Branch, _RelockDebugMixin):
2269
2296
    """A branch stored in the actual filesystem.
2270
2297
 
2324
2351
        return self.control_files.is_locked()
2325
2352
 
2326
2353
    def lock_write(self, token=None):
 
2354
        """Lock the branch for write operations.
 
2355
 
 
2356
        :param token: A token to permit reacquiring a previously held and
 
2357
            preserved lock.
 
2358
        :return: A BranchWriteLockResult.
 
2359
        """
2327
2360
        if not self.is_locked():
2328
2361
            self._note_lock('w')
2329
2362
        # All-in-one needs to always unlock/lock.
2335
2368
        else:
2336
2369
            took_lock = False
2337
2370
        try:
2338
 
            return self.control_files.lock_write(token=token)
 
2371
            return BranchWriteLockResult(self.unlock,
 
2372
                self.control_files.lock_write(token=token))
2339
2373
        except:
2340
2374
            if took_lock:
2341
2375
                self.repository.unlock()
2342
2376
            raise
2343
2377
 
2344
2378
    def lock_read(self):
 
2379
        """Lock the branch for read operations.
 
2380
 
 
2381
        :return: A bzrlib.lock.LogicalLockResult.
 
2382
        """
2345
2383
        if not self.is_locked():
2346
2384
            self._note_lock('r')
2347
2385
        # All-in-one needs to always unlock/lock.
2354
2392
            took_lock = False
2355
2393
        try:
2356
2394
            self.control_files.lock_read()
 
2395
            return LogicalLockResult(self.unlock)
2357
2396
        except:
2358
2397
            if took_lock:
2359
2398
                self.repository.unlock()
3145
3184
    _optimisers = []
3146
3185
    """The available optimised InterBranch types."""
3147
3186
 
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)
 
3187
    @classmethod
 
3188
    def _get_branch_formats_to_test(klass):
 
3189
        """Return an iterable of format tuples for testing.
 
3190
        
 
3191
        :return: An iterable of (from_format, to_format) to use when testing
 
3192
            this InterBranch class. Each InterBranch class should define this
 
3193
            method itself.
 
3194
        """
 
3195
        raise NotImplementedError(klass._get_branch_formats_to_test)
3152
3196
 
 
3197
    @needs_write_lock
3153
3198
    def pull(self, overwrite=False, stop_revision=None,
3154
3199
             possible_transports=None, local=False):
3155
3200
        """Mirror source into target branch.
3160
3205
        """
3161
3206
        raise NotImplementedError(self.pull)
3162
3207
 
 
3208
    @needs_write_lock
3163
3209
    def update_revisions(self, stop_revision=None, overwrite=False,
3164
3210
                         graph=None):
3165
3211
        """Pull in new perfect-fit revisions.
3173
3219
        """
3174
3220
        raise NotImplementedError(self.update_revisions)
3175
3221
 
 
3222
    @needs_write_lock
3176
3223
    def push(self, overwrite=False, stop_revision=None,
3177
3224
             _override_hook_source_branch=None):
3178
3225
        """Mirror the source branch into the target branch.
3183
3230
 
3184
3231
 
3185
3232
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
 
 
 
3233
    """InterBranch implementation that uses public Branch functions."""
 
3234
 
 
3235
    @classmethod
 
3236
    def is_compatible(klass, source, target):
 
3237
        # GenericBranch uses the public API, so always compatible
 
3238
        return True
 
3239
 
 
3240
    @classmethod
 
3241
    def _get_branch_formats_to_test(klass):
 
3242
        return [(BranchFormat._default_format, BranchFormat._default_format)]
 
3243
 
 
3244
    @classmethod
 
3245
    def unwrap_format(klass, format):
 
3246
        if isinstance(format, remote.RemoteBranchFormat):
 
3247
            format._ensure_real()
 
3248
            return format._custom_format
 
3249
        return format                                                                                                  
 
3250
 
 
3251
    @needs_write_lock
 
3252
    def copy_content_into(self, revision_id=None):
 
3253
        """Copy the content of source into target
 
3254
 
 
3255
        revision_id: if not None, the revision history in the new branch will
 
3256
                     be truncated to end with revision_id.
 
3257
        """
 
3258
        self.source.update_references(self.target)
 
3259
        self.source._synchronize_history(self.target, revision_id)
 
3260
        try:
 
3261
            parent = self.source.get_parent()
 
3262
        except errors.InaccessibleParent, e:
 
3263
            mutter('parent was not accessible to copy: %s', e)
 
3264
        else:
 
3265
            if parent:
 
3266
                self.target.set_parent(parent)
 
3267
        if self.source._push_should_merge_tags():
 
3268
            self.source.tags.merge_to(self.target.tags)
 
3269
 
 
3270
    @needs_write_lock
3193
3271
    def update_revisions(self, stop_revision=None, overwrite=False,
3194
3272
        graph=None):
3195
3273
        """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
 
 
 
3274
        other_revno, other_last_revision = self.source.last_revision_info()
 
3275
        stop_revno = None # unknown
 
3276
        if stop_revision is None:
 
3277
            stop_revision = other_last_revision
 
3278
            if _mod_revision.is_null(stop_revision):
 
3279
                # if there are no commits, we're done.
 
3280
                return
 
3281
            stop_revno = other_revno
 
3282
 
 
3283
        # what's the current last revision, before we fetch [and change it
 
3284
        # possibly]
 
3285
        last_rev = _mod_revision.ensure_null(self.target.last_revision())
 
3286
        # we fetch here so that we don't process data twice in the common
 
3287
        # case of having something to pull, and so that the check for
 
3288
        # already merged can operate on the just fetched graph, which will
 
3289
        # be cached in memory.
 
3290
        self.target.fetch(self.source, stop_revision)
 
3291
        # Check to see if one is an ancestor of the other
 
3292
        if not overwrite:
 
3293
            if graph is None:
 
3294
                graph = self.target.repository.get_graph()
 
3295
            if self.target._check_if_descendant_or_diverged(
 
3296
                    stop_revision, last_rev, graph, self.source):
 
3297
                # stop_revision is a descendant of last_rev, but we aren't
 
3298
                # overwriting, so we're done.
 
3299
                return
 
3300
        if stop_revno is None:
 
3301
            if graph is None:
 
3302
                graph = self.target.repository.get_graph()
 
3303
            this_revno, this_last_revision = \
 
3304
                    self.target.last_revision_info()
 
3305
            stop_revno = graph.find_distance_to_null(stop_revision,
 
3306
                            [(other_last_revision, other_revno),
 
3307
                             (this_last_revision, this_revno)])
 
3308
        self.target.set_last_revision_info(stop_revno, stop_revision)
 
3309
 
 
3310
    @needs_write_lock
3236
3311
    def pull(self, overwrite=False, stop_revision=None,
3237
 
             possible_transports=None, _hook_master=None, run_hooks=True,
 
3312
             possible_transports=None, run_hooks=True,
3238
3313
             _override_hook_target=None, local=False):
3239
 
        """See Branch.pull.
 
3314
        """Pull from source into self, updating my master if any.
3240
3315
 
3241
 
        :param _hook_master: Private parameter - set the branch to
3242
 
            be supplied as the master to pull hooks.
3243
3316
        :param run_hooks: Private parameter - if false, this branch
3244
3317
            is being called because it's the master of the primary branch,
3245
3318
            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
3319
        """
3250
 
        # This type of branch can't be bound.
3251
 
        if local:
 
3320
        bound_location = self.target.get_bound_location()
 
3321
        if local and not bound_location:
3252
3322
            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()
 
3323
        master_branch = None
 
3324
        if not local and bound_location and self.source.user_url != bound_location:
 
3325
            # not pulling from master, so we need to update master.
 
3326
            master_branch = self.target.get_master_branch(possible_transports)
 
3327
            master_branch.lock_write()
3260
3328
        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)
 
3329
            if master_branch:
 
3330
                # pull from source into master.
 
3331
                master_branch.pull(self.source, overwrite, stop_revision,
 
3332
                    run_hooks=False)
 
3333
            return self._pull(overwrite,
 
3334
                stop_revision, _hook_master=master_branch,
 
3335
                run_hooks=run_hooks,
 
3336
                _override_hook_target=_override_hook_target)
3287
3337
        finally:
3288
 
            self.source.unlock()
3289
 
        return result
 
3338
            if master_branch:
 
3339
                master_branch.unlock()
3290
3340
 
3291
3341
    def push(self, overwrite=False, stop_revision=None,
3292
3342
             _override_hook_source_branch=None):
3354
3404
            _run_hooks()
3355
3405
            return result
3356
3406
 
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,
 
3407
    def _pull(self, overwrite=False, stop_revision=None,
 
3408
             possible_transports=None, _hook_master=None, run_hooks=True,
3371
3409
             _override_hook_target=None, local=False):
3372
 
        """Pull from source into self, updating my master if any.
3373
 
 
 
3410
        """See Branch.pull.
 
3411
 
 
3412
        This function is the core worker, used by GenericInterBranch.pull to
 
3413
        avoid duplication when pulling source->master and source->local.
 
3414
 
 
3415
        :param _hook_master: Private parameter - set the branch to
 
3416
            be supplied as the master to pull hooks.
3374
3417
        :param run_hooks: Private parameter - if false, this branch
3375
3418
            is being called because it's the master of the primary branch,
3376
3419
            so it should not run its hooks.
 
3420
        :param _override_hook_target: Private parameter - set the branch to be
 
3421
            supplied as the target_branch to pull hooks.
 
3422
        :param local: Only update the local branch, and not the bound branch.
3377
3423
        """
3378
 
        bound_location = self.target.get_bound_location()
3379
 
        if local and not bound_location:
 
3424
        # This type of branch can't be bound.
 
3425
        if local:
3380
3426
            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()
 
3427
        result = PullResult()
 
3428
        result.source_branch = self.source
 
3429
        if _override_hook_target is None:
 
3430
            result.target_branch = self.target
 
3431
        else:
 
3432
            result.target_branch = _override_hook_target
 
3433
        self.source.lock_read()
3386
3434
        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)
 
3435
            # We assume that during 'pull' the target repository is closer than
 
3436
            # the source one.
 
3437
            self.source.update_references(self.target)
 
3438
            graph = self.target.repository.get_graph(self.source.repository)
 
3439
            # TODO: Branch formats should have a flag that indicates 
 
3440
            # that revno's are expensive, and pull() should honor that flag.
 
3441
            # -- JRV20090506
 
3442
            result.old_revno, result.old_revid = \
 
3443
                self.target.last_revision_info()
 
3444
            self.target.update_revisions(self.source, stop_revision,
 
3445
                overwrite=overwrite, graph=graph)
 
3446
            # TODO: The old revid should be specified when merging tags, 
 
3447
            # so a tags implementation that versions tags can only 
 
3448
            # pull in the most recent changes. -- JRV20090506
 
3449
            result.tag_conflicts = self.source.tags.merge_to(self.target.tags,
 
3450
                overwrite)
 
3451
            result.new_revno, result.new_revid = self.target.last_revision_info()
 
3452
            if _hook_master:
 
3453
                result.master_branch = _hook_master
 
3454
                result.local_branch = result.target_branch
 
3455
            else:
 
3456
                result.master_branch = result.target_branch
 
3457
                result.local_branch = None
 
3458
            if run_hooks:
 
3459
                for hook in Branch.hooks['post_pull']:
 
3460
                    hook(result)
3395
3461
        finally:
3396
 
            if master_branch:
3397
 
                master_branch.unlock()
 
3462
            self.source.unlock()
 
3463
        return result
3398
3464
 
3399
3465
 
3400
3466
InterBranch.register_optimiser(GenericInterBranch)
3401
 
InterBranch.register_optimiser(InterToBranch5)