456
436
# start_revision_id.
457
437
if self._merge_sorted_revisions_cache is None:
458
438
last_revision = self.last_revision()
459
known_graph = self.repository.get_known_graph_ancestry(
461
self._merge_sorted_revisions_cache = known_graph.merge_sort(
439
graph = self.repository.get_graph()
440
parent_map = dict(((key, value) for key, value in
441
graph.iter_ancestry([last_revision]) if value is not None))
442
revision_graph = repository._strip_NULL_ghosts(parent_map)
443
revs = tsort.merge_sort(revision_graph, last_revision, None,
445
# Drop the sequence # before caching
446
self._merge_sorted_revisions_cache = [r[1:] for r in revs]
463
448
filtered = self._filter_merge_sorted_revisions(
464
449
self._merge_sorted_revisions_cache, start_revision_id,
465
450
stop_revision_id, stop_rule)
466
# Make sure we don't return revisions that are not part of the
467
# start_revision_id ancestry.
468
filtered = self._filter_start_non_ancestors(filtered)
469
451
if direction == 'reverse':
471
453
if direction == 'forward':
478
460
"""Iterate over an inclusive range of sorted revisions."""
479
461
rev_iter = iter(merge_sorted_revisions)
480
462
if start_revision_id is not None:
481
for node in rev_iter:
482
rev_id = node.key[-1]
463
for rev_id, depth, revno, end_of_merge in rev_iter:
483
464
if rev_id != start_revision_id:
486
467
# The decision to include the start or not
487
468
# depends on the stop_rule if a stop is provided
488
# so pop this node back into the iterator
489
rev_iter = chain(iter([node]), rev_iter)
470
iter([(rev_id, depth, revno, end_of_merge)]),
491
473
if stop_revision_id is None:
493
for node in rev_iter:
494
rev_id = node.key[-1]
495
yield (rev_id, node.merge_depth, node.revno,
474
for rev_id, depth, revno, end_of_merge in rev_iter:
475
yield rev_id, depth, revno, end_of_merge
497
476
elif stop_rule == 'exclude':
498
for node in rev_iter:
499
rev_id = node.key[-1]
477
for rev_id, depth, revno, end_of_merge in rev_iter:
500
478
if rev_id == stop_revision_id:
502
yield (rev_id, node.merge_depth, node.revno,
480
yield rev_id, depth, revno, end_of_merge
504
481
elif stop_rule == 'include':
505
for node in rev_iter:
506
rev_id = node.key[-1]
507
yield (rev_id, node.merge_depth, node.revno,
482
for rev_id, depth, revno, end_of_merge in rev_iter:
483
yield rev_id, depth, revno, end_of_merge
509
484
if rev_id == stop_revision_id:
511
elif stop_rule == 'with-merges-without-common-ancestry':
512
# We want to exclude all revisions that are already part of the
513
# stop_revision_id ancestry.
514
graph = self.repository.get_graph()
515
ancestors = graph.find_unique_ancestors(start_revision_id,
517
for node in rev_iter:
518
rev_id = node.key[-1]
519
if rev_id not in ancestors:
521
yield (rev_id, node.merge_depth, node.revno,
523
486
elif stop_rule == 'with-merges':
524
487
stop_rev = self.repository.get_revision(stop_revision_id)
525
488
if stop_rev.parent_ids:
526
489
left_parent = stop_rev.parent_ids[0]
528
491
left_parent = _mod_revision.NULL_REVISION
529
# left_parent is the actual revision we want to stop logging at,
530
# since we want to show the merged revisions after the stop_rev too
531
reached_stop_revision_id = False
532
revision_id_whitelist = []
533
for node in rev_iter:
534
rev_id = node.key[-1]
492
for rev_id, depth, revno, end_of_merge in rev_iter:
535
493
if rev_id == left_parent:
536
# reached the left parent after the stop_revision
538
if (not reached_stop_revision_id or
539
rev_id in revision_id_whitelist):
540
yield (rev_id, node.merge_depth, node.revno,
542
if reached_stop_revision_id or rev_id == stop_revision_id:
543
# only do the merged revs of rev_id from now on
544
rev = self.repository.get_revision(rev_id)
546
reached_stop_revision_id = True
547
revision_id_whitelist.extend(rev.parent_ids)
495
yield rev_id, depth, revno, end_of_merge
549
497
raise ValueError('invalid stop_rule %r' % stop_rule)
551
def _filter_start_non_ancestors(self, rev_iter):
552
# If we started from a dotted revno, we want to consider it as a tip
553
# and don't want to yield revisions that are not part of its
554
# ancestry. Given the order guaranteed by the merge sort, we will see
555
# uninteresting descendants of the first parent of our tip before the
557
first = rev_iter.next()
558
(rev_id, merge_depth, revno, end_of_merge) = first
561
# We start at a mainline revision so by definition, all others
562
# revisions in rev_iter are ancestors
563
for node in rev_iter:
568
pmap = self.repository.get_parent_map([rev_id])
569
parents = pmap.get(rev_id, [])
571
whitelist.update(parents)
573
# If there is no parents, there is nothing of interest left
575
# FIXME: It's hard to test this scenario here as this code is never
576
# called in that case. -- vila 20100322
579
for (rev_id, merge_depth, revno, end_of_merge) in rev_iter:
581
if rev_id in whitelist:
582
pmap = self.repository.get_parent_map([rev_id])
583
parents = pmap.get(rev_id, [])
584
whitelist.remove(rev_id)
585
whitelist.update(parents)
587
# We've reached the mainline, there is nothing left to
591
# A revision that is not part of the ancestry of our
594
yield (rev_id, merge_depth, revno, end_of_merge)
596
499
def leave_lock_in_place(self):
597
500
"""Tell this branch object not to release the physical lock when this
598
501
object is unlocked.
768
668
except (errors.NotStacked, errors.UnstackableBranchFormat,
769
669
errors.UnstackableRepositoryFormat):
672
# XXX: Lock correctness - should unlock our old repo if we were
674
# repositories don't offer an interface to remove fallback
675
# repositories today; take the conceptually simpler option and just
677
self.repository = self.bzrdir.find_repository()
678
self.repository.lock_write()
679
# for every revision reference the branch has, ensure it is pulled
681
source_repository = self._get_fallback_repository(old_url)
682
for revision_id in chain([self.last_revision()],
683
self.tags.get_reverse_tag_dict()):
684
self.repository.fetch(source_repository, revision_id,
773
687
self._activate_fallback_location(url)
774
688
# write this out after the repository is stacked to avoid setting a
775
689
# stacked config that doesn't work.
776
690
self._set_config_location('stacked_on_location', url)
779
"""Change a branch to be unstacked, copying data as needed.
781
Don't call this directly, use set_stacked_on_url(None).
783
pb = ui.ui_factory.nested_progress_bar()
785
pb.update("Unstacking")
786
# The basic approach here is to fetch the tip of the branch,
787
# including all available ghosts, from the existing stacked
788
# repository into a new repository object without the fallbacks.
790
# XXX: See <https://launchpad.net/bugs/397286> - this may not be
791
# correct for CHKMap repostiories
792
old_repository = self.repository
793
if len(old_repository._fallback_repositories) != 1:
794
raise AssertionError("can't cope with fallback repositories "
795
"of %r" % (self.repository,))
796
# unlock it, including unlocking the fallback
797
old_repository.unlock()
798
old_repository.lock_read()
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
817
self.repository.lock_write()
818
# XXX: If you unstack a branch while it has a working tree
819
# with a pending merge, the pending-merged revisions will no
820
# longer be present. You can (probably) revert and remerge.
822
# XXX: This only fetches up to the tip of the repository; it
823
# doesn't bring across any tags. That's fairly consistent
824
# with how branch works, but perhaps not ideal.
825
self.repository.fetch(old_repository,
826
revision_id=self.last_revision(),
829
old_repository.unlock()
833
693
def _set_tags_bytes(self, bytes):
834
694
"""Mirror method for _get_tags_bytes.
1312
1177
Callers will typically also want to check the repository.
1314
:param refs: Calculated refs for this branch as specified by
1315
branch._get_check_refs()
1316
1179
:return: A BranchCheckResult.
1318
result = BranchCheckResult(self)
1181
ret = BranchCheckResult(self)
1182
mainline_parent_id = None
1319
1183
last_revno, last_revision_id = self.last_revision_info()
1320
actual_revno = refs[('lefthand-distance', last_revision_id)]
1321
if actual_revno != last_revno:
1322
result.errors.append(errors.BzrCheckError(
1323
'revno does not match len(mainline) %s != %s' % (
1324
last_revno, actual_revno)))
1325
# TODO: We should probably also check that self.revision_history
1326
# matches the repository for older branch formats.
1327
# If looking for the code that cross-checks repository parents against
1328
# the iter_reverse_revision_history output, that is now a repository
1184
real_rev_history = []
1186
for revid in self.repository.iter_reverse_revision_history(
1188
real_rev_history.append(revid)
1189
except errors.RevisionNotPresent:
1190
ret.ghosts_in_mainline = True
1192
ret.ghosts_in_mainline = False
1193
real_rev_history.reverse()
1194
if len(real_rev_history) != last_revno:
1195
raise errors.BzrCheckError('revno does not match len(mainline)'
1196
' %s != %s' % (last_revno, len(real_rev_history)))
1197
# TODO: We should probably also check that real_rev_history actually
1198
# matches self.revision_history()
1199
for revision_id in real_rev_history:
1201
revision = self.repository.get_revision(revision_id)
1202
except errors.NoSuchRevision, e:
1203
raise errors.BzrCheckError("mainline revision {%s} not in repository"
1205
# In general the first entry on the revision history has no parents.
1206
# But it's not illegal for it to have parents listed; this can happen
1207
# in imports from Arch when the parents weren't reachable.
1208
if mainline_parent_id is not None:
1209
if mainline_parent_id not in revision.parent_ids:
1210
raise errors.BzrCheckError("previous revision {%s} not listed among "
1212
% (mainline_parent_id, revision_id))
1213
mainline_parent_id = revision_id
1332
1216
def _get_checkout_format(self):
1333
1217
"""Return the most suitable metadir for a checkout of this branch.
1357
1241
# XXX: Fix the bzrdir API to allow getting the branch back from the
1358
1242
# clone call. Or something. 20090224 RBC/spiv.
1359
# XXX: Should this perhaps clone colocated branches as well,
1360
# rather than just the default branch? 20100319 JRV
1361
1243
if revision_id is None:
1362
1244
revision_id = self.last_revision()
1363
dir_to = self.bzrdir.clone_on_transport(to_transport,
1364
revision_id=revision_id, stacked_on=stacked_on,
1365
create_prefix=create_prefix, use_existing_dir=use_existing_dir)
1246
dir_to = self.bzrdir.clone_on_transport(to_transport,
1247
revision_id=revision_id, stacked_on=stacked_on,
1248
create_prefix=create_prefix, use_existing_dir=use_existing_dir)
1249
except errors.FileExists:
1250
if not use_existing_dir:
1252
except errors.NoSuchFile:
1253
if not create_prefix:
1366
1255
return dir_to.open_branch()
1368
1257
def create_checkout(self, to_location, revision_id=None,
1568
1442
"""Return the short format description for this format."""
1569
1443
raise NotImplementedError(self.get_format_description)
1571
def _run_post_branch_init_hooks(self, a_bzrdir, name, branch):
1572
hooks = Branch.hooks['post_branch_init']
1575
params = BranchInitHookParams(self, a_bzrdir, name, branch)
1579
def _initialize_helper(self, a_bzrdir, utf8_files, name=None,
1580
lock_type='metadir', set_format=True):
1445
def _initialize_helper(self, a_bzrdir, utf8_files, lock_type='metadir',
1581
1447
"""Initialize a branch in a bzrdir, with specified files
1583
1449
:param a_bzrdir: The bzrdir to initialize the branch in
1584
1450
:param utf8_files: The files to create as a list of
1585
1451
(filename, content) tuples
1586
:param name: Name of colocated branch to create, if any
1587
1452
:param set_format: If True, set the format with
1588
1453
self.get_format_string. (BzrBranch4 has its format set
1590
1455
:return: a branch in this format
1592
mutter('creating branch %r in %s', self, a_bzrdir.user_url)
1593
branch_transport = a_bzrdir.get_branch_transport(self, name=name)
1457
mutter('creating branch %r in %s', self, a_bzrdir.transport.base)
1458
branch_transport = a_bzrdir.get_branch_transport(self)
1595
1460
'metadir': ('lock', lockdir.LockDir),
1596
1461
'branch4': ('branch-lock', lockable_files.TransportLock),
1779
1638
"multiple hooks installed for transform_fallback_location, "
1780
1639
"all are called with the url returned from the previous hook."
1781
1640
"The order is however undefined.", (1, 9), None))
1782
self.create_hook(HookPoint('automatic_tag_name',
1783
"Called to determine an automatic tag name for a revision."
1784
"automatic_tag_name is called with (branch, revision_id) and "
1785
"should return a tag name or None if no tag name could be "
1786
"determined. The first non-None tag name returned will be used.",
1788
self.create_hook(HookPoint('post_branch_init',
1789
"Called after new branch initialization completes. "
1790
"post_branch_init is called with a "
1791
"bzrlib.branch.BranchInitHookParams. "
1792
"Note that init, branch and checkout (both heavyweight and "
1793
"lightweight) will all trigger this hook.", (2, 2), None))
1794
self.create_hook(HookPoint('post_switch',
1795
"Called after a checkout switches branch. "
1796
"post_switch is called with a "
1797
"bzrlib.branch.SwitchHookParams.", (2, 2), None))
1801
1643
# install the default hooks into the Branch class.
1840
1682
self.old_revno, self.old_revid, self.new_revno, self.new_revid)
1843
class BranchInitHookParams(object):
1844
"""Object holding parameters passed to *_branch_init hooks.
1846
There are 4 fields that hooks may wish to access:
1848
:ivar format: the branch format
1849
:ivar bzrdir: the BzrDir where the branch will be/has been initialized
1850
:ivar name: name of colocated branch, if any (or None)
1851
:ivar branch: the branch created
1853
Note that for lightweight checkouts, the bzrdir and format fields refer to
1854
the checkout, hence they are different from the corresponding fields in
1855
branch, which refer to the original branch.
1858
def __init__(self, format, a_bzrdir, name, branch):
1859
"""Create a group of BranchInitHook parameters.
1861
:param format: the branch format
1862
:param a_bzrdir: the BzrDir where the branch will be/has been
1864
:param name: name of colocated branch, if any (or None)
1865
:param branch: the branch created
1867
Note that for lightweight checkouts, the bzrdir and format fields refer
1868
to the checkout, hence they are different from the corresponding fields
1869
in branch, which refer to the original branch.
1871
self.format = format
1872
self.bzrdir = a_bzrdir
1874
self.branch = branch
1876
def __eq__(self, other):
1877
return self.__dict__ == other.__dict__
1881
return "<%s of %s>" % (self.__class__.__name__, self.branch)
1883
return "<%s of format:%s bzrdir:%s>" % (
1884
self.__class__.__name__, self.branch,
1885
self.format, self.bzrdir)
1888
class SwitchHookParams(object):
1889
"""Object holding parameters passed to *_switch hooks.
1891
There are 4 fields that hooks may wish to access:
1893
:ivar control_dir: BzrDir of the checkout to change
1894
:ivar to_branch: branch that the checkout is to reference
1895
:ivar force: skip the check for local commits in a heavy checkout
1896
:ivar revision_id: revision ID to switch to (or None)
1899
def __init__(self, control_dir, to_branch, force, revision_id):
1900
"""Create a group of SwitchHook parameters.
1902
:param control_dir: BzrDir of the checkout to change
1903
:param to_branch: branch that the checkout is to reference
1904
:param force: skip the check for local commits in a heavy checkout
1905
:param revision_id: revision ID to switch to (or None)
1907
self.control_dir = control_dir
1908
self.to_branch = to_branch
1910
self.revision_id = revision_id
1912
def __eq__(self, other):
1913
return self.__dict__ == other.__dict__
1916
return "<%s for %s to (%s, %s)>" % (self.__class__.__name__,
1917
self.control_dir, self.to_branch,
1921
1685
class BzrBranchFormat4(BranchFormat):
1922
1686
"""Bzr branch format 4.
1976
1739
return self.get_format_string()
1978
def open(self, a_bzrdir, name=None, _found=False, ignore_fallbacks=False):
1741
def open(self, a_bzrdir, _found=False, ignore_fallbacks=False):
1979
1742
"""See BranchFormat.open()."""
1981
format = BranchFormat.find_format(a_bzrdir, name=name)
1744
format = BranchFormat.find_format(a_bzrdir)
1982
1745
if format.__class__ != self.__class__:
1983
1746
raise AssertionError("wrong format %r found for %r" %
1984
1747
(format, self))
1985
transport = a_bzrdir.get_branch_transport(None, name=name)
1749
transport = a_bzrdir.get_branch_transport(None)
1987
1750
control_files = lockable_files.LockableFiles(transport, 'lock',
1988
1751
lockdir.LockDir)
1989
1752
return self._branch_class()(_format=self,
1990
1753
_control_files=control_files,
1992
1754
a_bzrdir=a_bzrdir,
1993
1755
_repository=a_bzrdir.find_repository(),
1994
1756
ignore_fallbacks=ignore_fallbacks)
1995
1757
except errors.NoSuchFile:
1996
raise errors.NotBranchError(path=transport.base, bzrdir=a_bzrdir)
1758
raise errors.NotBranchError(path=transport.base)
1998
1760
def __init__(self):
1999
1761
super(BranchFormatMetadir, self).__init__()
2171
1933
"""See BranchFormat.get_format_description()."""
2172
1934
return "Checkout reference format 1"
2174
def get_reference(self, a_bzrdir, name=None):
1936
def get_reference(self, a_bzrdir):
2175
1937
"""See BranchFormat.get_reference()."""
2176
transport = a_bzrdir.get_branch_transport(None, name=name)
2177
return transport.get_bytes('location')
1938
transport = a_bzrdir.get_branch_transport(None)
1939
return transport.get('location').read()
2179
def set_reference(self, a_bzrdir, name, to_branch):
1941
def set_reference(self, a_bzrdir, to_branch):
2180
1942
"""See BranchFormat.set_reference()."""
2181
transport = a_bzrdir.get_branch_transport(None, name=name)
1943
transport = a_bzrdir.get_branch_transport(None)
2182
1944
location = transport.put_bytes('location', to_branch.base)
2184
def initialize(self, a_bzrdir, name=None, target_branch=None):
1946
def initialize(self, a_bzrdir, target_branch=None):
2185
1947
"""Create a branch of this format in a_bzrdir."""
2186
1948
if target_branch is None:
2187
1949
# this format does not implement branch itself, thus the implicit
2188
1950
# creation contract must see it as uninitializable
2189
1951
raise errors.UninitializableFormat(self)
2190
mutter('creating branch reference in %s', a_bzrdir.user_url)
2191
branch_transport = a_bzrdir.get_branch_transport(self, name=name)
1952
mutter('creating branch reference in %s', a_bzrdir.transport.base)
1953
branch_transport = a_bzrdir.get_branch_transport(self)
2192
1954
branch_transport.put_bytes('location',
2193
target_branch.bzrdir.user_url)
1955
target_branch.bzrdir.root_transport.base)
2194
1956
branch_transport.put_bytes('format', self.get_format_string())
2196
a_bzrdir, name, _found=True,
1958
a_bzrdir, _found=True,
2197
1959
possible_transports=[target_branch.bzrdir.root_transport])
2198
self._run_post_branch_init_hooks(a_bzrdir, name, branch)
2201
1961
def __init__(self):
2202
1962
super(BranchReferenceFormat, self).__init__()
2208
1968
def clone(to_bzrdir, revision_id=None,
2209
1969
repository_policy=None):
2210
1970
"""See Branch.clone()."""
2211
return format.initialize(to_bzrdir, target_branch=a_branch)
1971
return format.initialize(to_bzrdir, a_branch)
2212
1972
# cannot obey revision_id limits when cloning a reference ...
2213
1973
# FIXME RBC 20060210 either nuke revision_id for clone, or
2214
1974
# emit some sort of warning/error to the caller ?!
2217
def open(self, a_bzrdir, name=None, _found=False, location=None,
1977
def open(self, a_bzrdir, _found=False, location=None,
2218
1978
possible_transports=None, ignore_fallbacks=False):
2219
1979
"""Return the branch that the branch reference in a_bzrdir points at.
2221
1981
:param a_bzrdir: A BzrDir that contains a branch.
2222
:param name: Name of colocated branch to open, if any
2223
1982
:param _found: a private parameter, do not use it. It is used to
2224
1983
indicate if format probing has already be done.
2225
1984
:param ignore_fallbacks: when set, no fallback branches will be opened
2272
2030
BranchFormat.register_format(__format6)
2273
2031
BranchFormat.register_format(__format7)
2274
2032
BranchFormat.register_format(__format8)
2275
BranchFormat.set_default_format(__format7)
2033
BranchFormat.set_default_format(__format6)
2276
2034
_legacy_formats = [BzrBranchFormat4(),
2278
2036
network_format_registry.register(
2279
2037
_legacy_formats[0].network_name(), _legacy_formats[0].__class__)
2282
class BranchWriteLockResult(LogicalLockResult):
2283
"""The result of write locking a branch.
2285
:ivar branch_token: The token obtained from the underlying branch lock, or
2287
:ivar unlock: A callable which will unlock the lock.
2290
def __init__(self, unlock, branch_token):
2291
LogicalLockResult.__init__(self, unlock)
2292
self.branch_token = branch_token
2295
return "BranchWriteLockResult(%s, %s)" % (self.branch_token,
2299
class BzrBranch(Branch, _RelockDebugMixin):
2040
class BzrBranch(Branch):
2300
2041
"""A branch stored in the actual filesystem.
2302
2043
Note that it's "local" in the context of the filesystem; it doesn't
2308
2049
:ivar repository: Repository for this branch.
2309
2050
:ivar base: The url of the base directory for this branch; the one
2310
2051
containing the .bzr directory.
2311
:ivar name: Optional colocated branch name as it exists in the control
2315
2054
def __init__(self, _format=None,
2316
_control_files=None, a_bzrdir=None, name=None,
2317
_repository=None, ignore_fallbacks=False):
2055
_control_files=None, a_bzrdir=None, _repository=None,
2056
ignore_fallbacks=False):
2318
2057
"""Create new branch object at a particular location."""
2319
2058
if a_bzrdir is None:
2320
2059
raise ValueError('a_bzrdir must be supplied')
2322
2061
self.bzrdir = a_bzrdir
2323
2062
self._base = self.bzrdir.transport.clone('..').base
2325
2063
# XXX: We should be able to just do
2326
2064
# self.base = self.bzrdir.root_transport.base
2327
2065
# but this does not quite work yet -- mbp 20080522
2355
2089
return self.control_files.is_locked()
2357
2091
def lock_write(self, token=None):
2358
"""Lock the branch for write operations.
2360
:param token: A token to permit reacquiring a previously held and
2362
:return: A BranchWriteLockResult.
2364
if not self.is_locked():
2365
self._note_lock('w')
2366
2092
# All-in-one needs to always unlock/lock.
2367
2093
repo_control = getattr(self.repository, 'control_files', None)
2368
2094
if self.control_files == repo_control or not self.is_locked():
2369
self.repository._warn_if_deprecated(self)
2370
2095
self.repository.lock_write()
2371
2096
took_lock = True
2373
2098
took_lock = False
2375
return BranchWriteLockResult(self.unlock,
2376
self.control_files.lock_write(token=token))
2100
return self.control_files.lock_write(token=token)
2379
2103
self.repository.unlock()
2382
2106
def lock_read(self):
2383
"""Lock the branch for read operations.
2385
:return: A bzrlib.lock.LogicalLockResult.
2387
if not self.is_locked():
2388
self._note_lock('r')
2389
2107
# All-in-one needs to always unlock/lock.
2390
2108
repo_control = getattr(self.repository, 'control_files', None)
2391
2109
if self.control_files == repo_control or not self.is_locked():
2392
self.repository._warn_if_deprecated(self)
2393
2110
self.repository.lock_read()
2394
2111
took_lock = True
2396
2113
took_lock = False
2398
2115
self.control_files.lock_read()
2399
return LogicalLockResult(self.unlock)
2402
2118
self.repository.unlock()
2405
@only_raises(errors.LockNotHeld, errors.LockBroken)
2406
2121
def unlock(self):
2408
2123
self.control_files.unlock()
added
removed
bzrlib/branch.py
