← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/branch.py

  • Committer: Martin Pool
  • Date: 2009-06-19 09:06:56 UTC
  • mfrom: (4463 +trunk)
  • mto: This revision was merged to the branch mainline in revision 4464.
  • Revision ID: mbp@sourcefrog.net-20090619090656-d5weqeecyscv8kqp
merge news

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/branch.py
1
 
# Copyright (C) 2005-2010 Canonical Ltd
 
1
# Copyright (C) 2005, 2006, 2007, 2008, 2009 Canonical Ltd
2
2
#
3
3
# This program is free software; you can redistribute it and/or modify
4
4
# it under the terms of the GNU General Public License as published by
46
46
    )
47
47
""")
48
48
 
49
 
from bzrlib.decorators import needs_read_lock, needs_write_lock, only_raises
 
49
from bzrlib.decorators import needs_read_lock, needs_write_lock
50
50
from bzrlib.hooks import HookPoint, Hooks
51
51
from bzrlib.inter import InterObject
52
 
from bzrlib.lock import _RelockDebugMixin
53
52
from bzrlib import registry
54
53
from bzrlib.symbol_versioning import (
55
54
    deprecated_in,
63
62
BZR_BRANCH_FORMAT_6 = "Bazaar Branch Format 6 (bzr 0.15)\n"
64
63
 
65
64
 
66
 
class Branch(bzrdir.ControlComponent):
 
65
# TODO: Maybe include checks for common corruption of newlines, etc?
 
66
 
 
67
# TODO: Some operations like log might retrieve the same revisions
 
68
# repeatedly to calculate deltas.  We could perhaps have a weakref
 
69
# cache in memory to make this faster.  In general anything can be
 
70
# cached in memory between lock and unlock operations. .. nb thats
 
71
# what the transaction identity map provides
 
72
 
 
73
 
 
74
######################################################################
 
75
# branch objects
 
76
 
 
77
class Branch(object):
67
78
    """Branch holding a history of revisions.
68
79
 
69
 
    :ivar base:
70
 
        Base directory/url of the branch; using control_url and
71
 
        control_transport is more standardized.
 
80
    base
 
81
        Base directory/url of the branch.
72
82
 
73
83
    hooks: An instance of BranchHooks.
74
84
    """
76
86
    # - RBC 20060112
77
87
    base = None
78
88
 
79
 
    @property
80
 
    def control_transport(self):
81
 
        return self._transport
82
 
 
83
 
    @property
84
 
    def user_transport(self):
85
 
        return self.bzrdir.user_transport
86
 
 
87
89
    def __init__(self, *ignored, **ignored_too):
88
90
        self.tags = self._format.make_tags(self)
89
91
        self._revision_history_cache = None
103
105
    def _activate_fallback_location(self, url):
104
106
        """Activate the branch/repository from url as a fallback repository."""
105
107
        repo = self._get_fallback_repository(url)
106
 
        if repo.has_same_location(self.repository):
107
 
            raise errors.UnstackableLocationError(self.user_url, url)
108
108
        self.repository.add_fallback_repository(repo)
109
109
 
110
110
    def break_lock(self):
147
147
        if self._partial_revision_history_cache[-1] == _mod_revision.NULL_REVISION:
148
148
            self._partial_revision_history_cache.pop()
149
149
 
150
 
    def _get_check_refs(self):
151
 
        """Get the references needed for check().
152
 
 
153
 
        See bzrlib.check.
154
 
        """
155
 
        revid = self.last_revision()
156
 
        return [('revision-existence', revid), ('lefthand-distance', revid)]
157
 
 
158
150
    @staticmethod
159
151
    def open(base, _unsupported=False, possible_transports=None):
160
152
        """Open the branch rooted at base.
164
156
        """
165
157
        control = bzrdir.BzrDir.open(base, _unsupported,
166
158
                                     possible_transports=possible_transports)
167
 
        return control.open_branch(unsupported=_unsupported)
 
159
        return control.open_branch(_unsupported)
168
160
 
169
161
    @staticmethod
170
 
    def open_from_transport(transport, name=None, _unsupported=False):
 
162
    def open_from_transport(transport, _unsupported=False):
171
163
        """Open the branch rooted at transport"""
172
164
        control = bzrdir.BzrDir.open_from_transport(transport, _unsupported)
173
 
        return control.open_branch(name=name, unsupported=_unsupported)
 
165
        return control.open_branch(_unsupported)
174
166
 
175
167
    @staticmethod
176
168
    def open_containing(url, possible_transports=None):
214
206
    def _get_fallback_repository(self, url):
215
207
        """Get the repository we fallback to at url."""
216
208
        url = urlutils.join(self.base, url)
217
 
        a_branch = Branch.open(url,
 
209
        a_bzrdir = bzrdir.BzrDir.open(url,
218
210
            possible_transports=[self.bzrdir.root_transport])
219
 
        return a_branch.repository
 
211
        return a_bzrdir.open_branch().repository
220
212
 
221
213
    def _get_tags_bytes(self):
222
214
        """Get the bytes of a serialised tags dict.
417
409
            * 'include' - the stop revision is the last item in the result
418
410
            * 'with-merges' - include the stop revision and all of its
419
411
              merged revisions in the result
420
 
            * 'with-merges-without-common-ancestry' - filter out revisions 
421
 
              that are in both ancestries
422
412
        :param direction: either 'reverse' or 'forward':
423
413
            * reverse means return the start_revision_id first, i.e.
424
414
              start at the most recent revision and go backwards in history
446
436
        # start_revision_id.
447
437
        if self._merge_sorted_revisions_cache is None:
448
438
            last_revision = self.last_revision()
449
 
            known_graph = self.repository.get_known_graph_ancestry(
450
 
                [last_revision])
451
 
            self._merge_sorted_revisions_cache = known_graph.merge_sort(
452
 
                last_revision)
 
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,
 
444
                generate_revno=True)
 
445
            # Drop the sequence # before caching
 
446
            self._merge_sorted_revisions_cache = [r[1:] for r in revs]
 
447
 
453
448
        filtered = self._filter_merge_sorted_revisions(
454
449
            self._merge_sorted_revisions_cache, start_revision_id,
455
450
            stop_revision_id, stop_rule)
456
 
        # Make sure we don't return revisions that are not part of the
457
 
        # start_revision_id ancestry.
458
 
        filtered = self._filter_start_non_ancestors(filtered)
459
451
        if direction == 'reverse':
460
452
            return filtered
461
453
        if direction == 'forward':
468
460
        """Iterate over an inclusive range of sorted revisions."""
469
461
        rev_iter = iter(merge_sorted_revisions)
470
462
        if start_revision_id is not None:
471
 
            for node in rev_iter:
472
 
                rev_id = node.key[-1]
 
463
            for rev_id, depth, revno, end_of_merge in rev_iter:
473
464
                if rev_id != start_revision_id:
474
465
                    continue
475
466
                else:
476
467
                    # The decision to include the start or not
477
468
                    # depends on the stop_rule if a stop is provided
478
 
                    # so pop this node back into the iterator
479
 
                    rev_iter = chain(iter([node]), rev_iter)
 
469
                    rev_iter = chain(
 
470
                        iter([(rev_id, depth, revno, end_of_merge)]),
 
471
                        rev_iter)
480
472
                    break
481
473
        if stop_revision_id is None:
482
 
            # Yield everything
483
 
            for node in rev_iter:
484
 
                rev_id = node.key[-1]
485
 
                yield (rev_id, node.merge_depth, node.revno,
486
 
                       node.end_of_merge)
 
474
            for rev_id, depth, revno, end_of_merge in rev_iter:
 
475
                yield rev_id, depth, revno, end_of_merge
487
476
        elif stop_rule == 'exclude':
488
 
            for node in rev_iter:
489
 
                rev_id = node.key[-1]
 
477
            for rev_id, depth, revno, end_of_merge in rev_iter:
490
478
                if rev_id == stop_revision_id:
491
479
                    return
492
 
                yield (rev_id, node.merge_depth, node.revno,
493
 
                       node.end_of_merge)
 
480
                yield rev_id, depth, revno, end_of_merge
494
481
        elif stop_rule == 'include':
495
 
            for node in rev_iter:
496
 
                rev_id = node.key[-1]
497
 
                yield (rev_id, node.merge_depth, node.revno,
498
 
                       node.end_of_merge)
 
482
            for rev_id, depth, revno, end_of_merge in rev_iter:
 
483
                yield rev_id, depth, revno, end_of_merge
499
484
                if rev_id == stop_revision_id:
500
485
                    return
501
 
        elif stop_rule == 'with-merges-without-common-ancestry':
502
 
            # We want to exclude all revisions that are already part of the
503
 
            # stop_revision_id ancestry.
504
 
            graph = self.repository.get_graph()
505
 
            ancestors = graph.find_unique_ancestors(start_revision_id,
506
 
                                                    [stop_revision_id])
507
 
            for node in rev_iter:
508
 
                rev_id = node.key[-1]
509
 
                if rev_id not in ancestors:
510
 
                    continue
511
 
                yield (rev_id, node.merge_depth, node.revno,
512
 
                       node.end_of_merge)
513
486
        elif stop_rule == 'with-merges':
514
487
            stop_rev = self.repository.get_revision(stop_revision_id)
515
488
            if stop_rev.parent_ids:
516
489
                left_parent = stop_rev.parent_ids[0]
517
490
            else:
518
491
                left_parent = _mod_revision.NULL_REVISION
519
 
            # left_parent is the actual revision we want to stop logging at,
520
 
            # since we want to show the merged revisions after the stop_rev too
521
 
            reached_stop_revision_id = False
522
 
            revision_id_whitelist = []
523
 
            for node in rev_iter:
524
 
                rev_id = node.key[-1]
 
492
            for rev_id, depth, revno, end_of_merge in rev_iter:
525
493
                if rev_id == left_parent:
526
 
                    # reached the left parent after the stop_revision
527
494
                    return
528
 
                if (not reached_stop_revision_id or
529
 
                        rev_id in revision_id_whitelist):
530
 
                    yield (rev_id, node.merge_depth, node.revno,
531
 
                       node.end_of_merge)
532
 
                    if reached_stop_revision_id or rev_id == stop_revision_id:
533
 
                        # only do the merged revs of rev_id from now on
534
 
                        rev = self.repository.get_revision(rev_id)
535
 
                        if rev.parent_ids:
536
 
                            reached_stop_revision_id = True
537
 
                            revision_id_whitelist.extend(rev.parent_ids)
 
495
                yield rev_id, depth, revno, end_of_merge
538
496
        else:
539
497
            raise ValueError('invalid stop_rule %r' % stop_rule)
540
498
 
541
 
    def _filter_start_non_ancestors(self, rev_iter):
542
 
        # If we started from a dotted revno, we want to consider it as a tip
543
 
        # and don't want to yield revisions that are not part of its
544
 
        # ancestry. Given the order guaranteed by the merge sort, we will see
545
 
        # uninteresting descendants of the first parent of our tip before the
546
 
        # tip itself.
547
 
        first = rev_iter.next()
548
 
        (rev_id, merge_depth, revno, end_of_merge) = first
549
 
        yield first
550
 
        if not merge_depth:
551
 
            # We start at a mainline revision so by definition, all others
552
 
            # revisions in rev_iter are ancestors
553
 
            for node in rev_iter:
554
 
                yield node
555
 
 
556
 
        clean = False
557
 
        whitelist = set()
558
 
        pmap = self.repository.get_parent_map([rev_id])
559
 
        parents = pmap.get(rev_id, [])
560
 
        if parents:
561
 
            whitelist.update(parents)
562
 
        else:
563
 
            # If there is no parents, there is nothing of interest left
564
 
 
565
 
            # FIXME: It's hard to test this scenario here as this code is never
566
 
            # called in that case. -- vila 20100322
567
 
            return
568
 
 
569
 
        for (rev_id, merge_depth, revno, end_of_merge) in rev_iter:
570
 
            if not clean:
571
 
                if rev_id in whitelist:
572
 
                    pmap = self.repository.get_parent_map([rev_id])
573
 
                    parents = pmap.get(rev_id, [])
574
 
                    whitelist.remove(rev_id)
575
 
                    whitelist.update(parents)
576
 
                    if merge_depth == 0:
577
 
                        # We've reached the mainline, there is nothing left to
578
 
                        # filter
579
 
                        clean = True
580
 
                else:
581
 
                    # A revision that is not part of the ancestry of our
582
 
                    # starting revision.
583
 
                    continue
584
 
            yield (rev_id, merge_depth, revno, end_of_merge)
585
 
 
586
499
    def leave_lock_in_place(self):
587
500
        """Tell this branch object not to release the physical lock when this
588
501
        object is unlocked.
605
518
        :param other: The branch to bind to
606
519
        :type other: Branch
607
520
        """
608
 
        raise errors.UpgradeRequired(self.user_url)
 
521
        raise errors.UpgradeRequired(self.base)
609
522
 
610
523
    def set_append_revisions_only(self, enabled):
611
524
        if not self._format.supports_set_append_revisions_only():
612
 
            raise errors.UpgradeRequired(self.user_url)
 
525
            raise errors.UpgradeRequired(self.base)
613
526
        if enabled:
614
527
            value = 'True'
615
528
        else:
663
576
    def get_old_bound_location(self):
664
577
        """Return the URL of the branch we used to be bound to
665
578
        """
666
 
        raise errors.UpgradeRequired(self.user_url)
 
579
        raise errors.UpgradeRequired(self.base)
667
580
 
668
581
    def get_commit_builder(self, parents, config=None, timestamp=None,
669
582
                           timezone=None, committer=None, revprops=None,
747
660
            stacking.
748
661
        """
749
662
        if not self._format.supports_stacking():
750
 
            raise errors.UnstackableBranchFormat(self._format, self.user_url)
751
 
        # XXX: Changing from one fallback repository to another does not check
752
 
        # that all the data you need is present in the new fallback.
753
 
        # Possibly it should.
 
663
            raise errors.UnstackableBranchFormat(self._format, self.base)
754
664
        self._check_stackable_repo()
755
665
        if not url:
756
666
            try:
758
668
            except (errors.NotStacked, errors.UnstackableBranchFormat,
759
669
                errors.UnstackableRepositoryFormat):
760
670
                return
761
 
            self._unstack()
 
671
            url = ''
 
672
            # XXX: Lock correctness - should unlock our old repo if we were
 
673
            # locked.
 
674
            # repositories don't offer an interface to remove fallback
 
675
            # repositories today; take the conceptually simpler option and just
 
676
            # reopen it.
 
677
            self.repository = self.bzrdir.find_repository()
 
678
            self.repository.lock_write()
 
679
            # for every revision reference the branch has, ensure it is pulled
 
680
            # in.
 
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,
 
685
                    find_ghosts=True)
762
686
        else:
763
687
            self._activate_fallback_location(url)
764
688
        # write this out after the repository is stacked to avoid setting a
765
689
        # stacked config that doesn't work.
766
690
        self._set_config_location('stacked_on_location', url)
767
691
 
768
 
    def _unstack(self):
769
 
        """Change a branch to be unstacked, copying data as needed.
770
 
        
771
 
        Don't call this directly, use set_stacked_on_url(None).
772
 
        """
773
 
        pb = ui.ui_factory.nested_progress_bar()
774
 
        try:
775
 
            pb.update("Unstacking")
776
 
            # The basic approach here is to fetch the tip of the branch,
777
 
            # including all available ghosts, from the existing stacked
778
 
            # repository into a new repository object without the fallbacks. 
779
 
            #
780
 
            # XXX: See <https://launchpad.net/bugs/397286> - this may not be
781
 
            # correct for CHKMap repostiories
782
 
            old_repository = self.repository
783
 
            if len(old_repository._fallback_repositories) != 1:
784
 
                raise AssertionError("can't cope with fallback repositories "
785
 
                    "of %r" % (self.repository,))
786
 
            # unlock it, including unlocking the fallback
787
 
            old_repository.unlock()
788
 
            old_repository.lock_read()
789
 
            try:
790
 
                # Repositories don't offer an interface to remove fallback
791
 
                # repositories today; take the conceptually simpler option and just
792
 
                # reopen it.  We reopen it starting from the URL so that we
793
 
                # get a separate connection for RemoteRepositories and can
794
 
                # stream from one of them to the other.  This does mean doing
795
 
                # separate SSH connection setup, but unstacking is not a
796
 
                # common operation so it's tolerable.
797
 
                new_bzrdir = bzrdir.BzrDir.open(self.bzrdir.root_transport.base)
798
 
                new_repository = new_bzrdir.find_repository()
799
 
                self.repository = new_repository
800
 
                if self.repository._fallback_repositories:
801
 
                    raise AssertionError("didn't expect %r to have "
802
 
                        "fallback_repositories"
803
 
                        % (self.repository,))
804
 
                # this is not paired with an unlock because it's just restoring
805
 
                # the previous state; the lock's released when set_stacked_on_url
806
 
                # returns
807
 
                self.repository.lock_write()
808
 
                # XXX: If you unstack a branch while it has a working tree
809
 
                # with a pending merge, the pending-merged revisions will no
810
 
                # longer be present.  You can (probably) revert and remerge.
811
 
                #
812
 
                # XXX: This only fetches up to the tip of the repository; it
813
 
                # doesn't bring across any tags.  That's fairly consistent
814
 
                # with how branch works, but perhaps not ideal.
815
 
                self.repository.fetch(old_repository,
816
 
                    revision_id=self.last_revision(),
817
 
                    find_ghosts=True)
818
 
            finally:
819
 
                old_repository.unlock()
820
 
        finally:
821
 
            pb.finished()
822
692
 
823
693
    def _set_tags_bytes(self, bytes):
824
694
        """Mirror method for _get_tags_bytes.
904
774
 
905
775
    def unbind(self):
906
776
        """Older format branches cannot bind or unbind."""
907
 
        raise errors.UpgradeRequired(self.user_url)
 
777
        raise errors.UpgradeRequired(self.base)
908
778
 
909
779
    def last_revision(self):
910
780
        """Return last revision id, or NULL_REVISION."""
1070
940
        try:
1071
941
            return urlutils.join(self.base[:-1], parent)
1072
942
        except errors.InvalidURLJoin, e:
1073
 
            raise errors.InaccessibleParent(parent, self.user_url)
 
943
            raise errors.InaccessibleParent(parent, self.base)
1074
944
 
1075
945
    def _get_parent_location(self):
1076
946
        raise NotImplementedError(self._get_parent_location)
1161
1031
        params = ChangeBranchTipParams(
1162
1032
            self, old_revno, new_revno, old_revid, new_revid)
1163
1033
        for hook in hooks:
1164
 
            hook(params)
 
1034
            try:
 
1035
                hook(params)
 
1036
            except errors.TipChangeRejected:
 
1037
                raise
 
1038
            except Exception:
 
1039
                exc_info = sys.exc_info()
 
1040
                hook_name = Branch.hooks.get_hook_name(hook)
 
1041
                raise errors.HookFailed(
 
1042
                    'pre_change_branch_tip', hook_name, exc_info)
1165
1043
 
1166
1044
    @needs_write_lock
1167
1045
    def update(self):
1216
1094
        revision_id: if not None, the revision history in the new branch will
1217
1095
                     be truncated to end with revision_id.
1218
1096
        """
1219
 
        if (repository_policy is not None and
1220
 
            repository_policy.requires_stacking()):
1221
 
            to_bzrdir._format.require_stacking(_skip_repo=True)
1222
1097
        result = to_bzrdir.create_branch()
1223
1098
        result.lock_write()
1224
1099
        try:
1292
1167
        target._set_all_reference_info(target_reference_dict)
1293
1168
 
1294
1169
    @needs_read_lock
1295
 
    def check(self, refs):
 
1170
    def check(self):
1296
1171
        """Check consistency of the branch.
1297
1172
 
1298
1173
        In particular this checks that revisions given in the revision-history
1301
1176
 
1302
1177
        Callers will typically also want to check the repository.
1303
1178
 
1304
 
        :param refs: Calculated refs for this branch as specified by
1305
 
            branch._get_check_refs()
1306
1179
        :return: A BranchCheckResult.
1307
1180
        """
1308
 
        result = BranchCheckResult(self)
 
1181
        ret = BranchCheckResult(self)
 
1182
        mainline_parent_id = None
1309
1183
        last_revno, last_revision_id = self.last_revision_info()
1310
 
        actual_revno = refs[('lefthand-distance', last_revision_id)]
1311
 
        if actual_revno != last_revno:
1312
 
            result.errors.append(errors.BzrCheckError(
1313
 
                'revno does not match len(mainline) %s != %s' % (
1314
 
                last_revno, actual_revno)))
1315
 
        # TODO: We should probably also check that self.revision_history
1316
 
        # matches the repository for older branch formats.
1317
 
        # If looking for the code that cross-checks repository parents against
1318
 
        # the iter_reverse_revision_history output, that is now a repository
1319
 
        # specific check.
1320
 
        return result
 
1184
        real_rev_history = []
 
1185
        try:
 
1186
            for revid in self.repository.iter_reverse_revision_history(
 
1187
                last_revision_id):
 
1188
                real_rev_history.append(revid)
 
1189
        except errors.RevisionNotPresent:
 
1190
            ret.ghosts_in_mainline = True
 
1191
        else:
 
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:
 
1200
            try:
 
1201
                revision = self.repository.get_revision(revision_id)
 
1202
            except errors.NoSuchRevision, e:
 
1203
                raise errors.BzrCheckError("mainline revision {%s} not in repository"
 
1204
                            % revision_id)
 
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 "
 
1211
                                        "parents of {%s}"
 
1212
                                        % (mainline_parent_id, revision_id))
 
1213
            mainline_parent_id = revision_id
 
1214
        return ret
1321
1215
 
1322
1216
    def _get_checkout_format(self):
1323
1217
        """Return the most suitable metadir for a checkout of this branch.
1348
1242
        # clone call. Or something. 20090224 RBC/spiv.
1349
1243
        if revision_id is None:
1350
1244
            revision_id = self.last_revision()
1351
 
        dir_to = self.bzrdir.clone_on_transport(to_transport,
1352
 
            revision_id=revision_id, stacked_on=stacked_on,
1353
 
            create_prefix=create_prefix, use_existing_dir=use_existing_dir)
 
1245
        try:
 
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:
 
1251
                raise
 
1252
        except errors.NoSuchFile:
 
1253
            if not create_prefix:
 
1254
                raise
1354
1255
        return dir_to.open_branch()
1355
1256
 
1356
1257
    def create_checkout(self, to_location, revision_id=None,
1375
1276
        if lightweight:
1376
1277
            format = self._get_checkout_format()
1377
1278
            checkout = format.initialize_on_transport(t)
1378
 
            from_branch = BranchReferenceFormat().initialize(checkout, 
1379
 
                target_branch=self)
 
1279
            from_branch = BranchReferenceFormat().initialize(checkout, self)
1380
1280
        else:
1381
1281
            format = self._get_checkout_format()
1382
1282
            checkout_branch = bzrdir.BzrDir.create_branch_convenience(
1424
1324
    def supports_tags(self):
1425
1325
        return self._format.supports_tags()
1426
1326
 
1427
 
    def automatic_tag_name(self, revision_id):
1428
 
        """Try to automatically find the tag name for a revision.
1429
 
 
1430
 
        :param revision_id: Revision id of the revision.
1431
 
        :return: A tag name or None if no tag name could be determined.
1432
 
        """
1433
 
        for hook in Branch.hooks['automatic_tag_name']:
1434
 
            ret = hook(self, revision_id)
1435
 
            if ret is not None:
1436
 
                return ret
1437
 
        return None
1438
 
 
1439
1327
    def _check_if_descendant_or_diverged(self, revision_a, revision_b, graph,
1440
1328
                                         other_branch):
1441
1329
        """Ensure that revision_b is a descendant of revision_a.
1505
1393
        return not (self == other)
1506
1394
 
1507
1395
    @classmethod
1508
 
    def find_format(klass, a_bzrdir, name=None):
 
1396
    def find_format(klass, a_bzrdir):
1509
1397
        """Return the format for the branch object in a_bzrdir."""
1510
1398
        try:
1511
 
            transport = a_bzrdir.get_branch_transport(None, name=name)
1512
 
            format_string = transport.get_bytes("format")
 
1399
            transport = a_bzrdir.get_branch_transport(None)
 
1400
            format_string = transport.get("format").read()
1513
1401
            return klass._formats[format_string]
1514
1402
        except errors.NoSuchFile:
1515
 
            raise errors.NotBranchError(path=transport.base, bzrdir=a_bzrdir)
 
1403
            raise errors.NotBranchError(path=transport.base)
1516
1404
        except KeyError:
1517
1405
            raise errors.UnknownFormatError(format=format_string, kind='branch')
1518
1406
 
1554
1442
        """Return the short format description for this format."""
1555
1443
        raise NotImplementedError(self.get_format_description)
1556
1444
 
1557
 
    def _run_post_branch_init_hooks(self, a_bzrdir, name, branch):
1558
 
        hooks = Branch.hooks['post_branch_init']
1559
 
        if not hooks:
1560
 
            return
1561
 
        params = BranchInitHookParams(self, a_bzrdir, name, branch)
1562
 
        for hook in hooks:
1563
 
            hook(params)
1564
 
 
1565
 
    def _initialize_helper(self, a_bzrdir, utf8_files, name=None,
1566
 
                           lock_type='metadir', set_format=True):
 
1445
    def _initialize_helper(self, a_bzrdir, utf8_files, lock_type='metadir',
 
1446
                           set_format=True):
1567
1447
        """Initialize a branch in a bzrdir, with specified files
1568
1448
 
1569
1449
        :param a_bzrdir: The bzrdir to initialize the branch in
1570
1450
        :param utf8_files: The files to create as a list of
1571
1451
            (filename, content) tuples
1572
 
        :param name: Name of colocated branch to create, if any
1573
1452
        :param set_format: If True, set the format with
1574
1453
            self.get_format_string.  (BzrBranch4 has its format set
1575
1454
            elsewhere)
1576
1455
        :return: a branch in this format
1577
1456
        """
1578
 
        mutter('creating branch %r in %s', self, a_bzrdir.user_url)
1579
 
        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)
1580
1459
        lock_map = {
1581
1460
            'metadir': ('lock', lockdir.LockDir),
1582
1461
            'branch4': ('branch-lock', lockable_files.TransportLock),
1603
1482
        finally:
1604
1483
            if lock_taken:
1605
1484
                control_files.unlock()
1606
 
        branch = self.open(a_bzrdir, name, _found=True)
1607
 
        self._run_post_branch_init_hooks(a_bzrdir, name, branch)
1608
 
        return branch
 
1485
        return self.open(a_bzrdir, _found=True)
1609
1486
 
1610
 
    def initialize(self, a_bzrdir, name=None):
1611
 
        """Create a branch of this format in a_bzrdir.
1612
 
        
1613
 
        :param name: Name of the colocated branch to create.
1614
 
        """
 
1487
    def initialize(self, a_bzrdir):
 
1488
        """Create a branch of this format in a_bzrdir."""
1615
1489
        raise NotImplementedError(self.initialize)
1616
1490
 
1617
1491
    def is_supported(self):
1647
1521
        """
1648
1522
        raise NotImplementedError(self.network_name)
1649
1523
 
1650
 
    def open(self, a_bzrdir, name=None, _found=False, ignore_fallbacks=False):
 
1524
    def open(self, a_bzrdir, _found=False, ignore_fallbacks=False):
1651
1525
        """Return the branch object for a_bzrdir
1652
1526
 
1653
1527
        :param a_bzrdir: A BzrDir that contains a branch.
1654
 
        :param name: Name of colocated branch to open
1655
1528
        :param _found: a private parameter, do not use it. It is used to
1656
1529
            indicate if format probing has already be done.
1657
1530
        :param ignore_fallbacks: when set, no fallback branches will be opened
1765
1638
            "multiple hooks installed for transform_fallback_location, "
1766
1639
            "all are called with the url returned from the previous hook."
1767
1640
            "The order is however undefined.", (1, 9), None))
1768
 
        self.create_hook(HookPoint('automatic_tag_name',
1769
 
            "Called to determine an automatic tag name for a revision."
1770
 
            "automatic_tag_name is called with (branch, revision_id) and "
1771
 
            "should return a tag name or None if no tag name could be "
1772
 
            "determined. The first non-None tag name returned will be used.",
1773
 
            (2, 2), None))
1774
 
        self.create_hook(HookPoint('post_branch_init',
1775
 
            "Called after new branch initialization completes. "
1776
 
            "post_branch_init is called with a "
1777
 
            "bzrlib.branch.BranchInitHookParams. "
1778
 
            "Note that init, branch and checkout (both heavyweight and "
1779
 
            "lightweight) will all trigger this hook.", (2, 2), None))
1780
 
        self.create_hook(HookPoint('post_switch',
1781
 
            "Called after a checkout switches branch. "
1782
 
            "post_switch is called with a "
1783
 
            "bzrlib.branch.SwitchHookParams.", (2, 2), None))
1784
 
 
1785
1641
 
1786
1642
 
1787
1643
# install the default hooks into the Branch class.
1826
1682
            self.old_revno, self.old_revid, self.new_revno, self.new_revid)
1827
1683
 
1828
1684
 
1829
 
class BranchInitHookParams(object):
1830
 
    """Object holding parameters passed to *_branch_init hooks.
1831
 
 
1832
 
    There are 4 fields that hooks may wish to access:
1833
 
 
1834
 
    :ivar format: the branch format
1835
 
    :ivar bzrdir: the BzrDir where the branch will be/has been initialized
1836
 
    :ivar name: name of colocated branch, if any (or None)
1837
 
    :ivar branch: the branch created
1838
 
 
1839
 
    Note that for lightweight checkouts, the bzrdir and format fields refer to
1840
 
    the checkout, hence they are different from the corresponding fields in
1841
 
    branch, which refer to the original branch.
1842
 
    """
1843
 
 
1844
 
    def __init__(self, format, a_bzrdir, name, branch):
1845
 
        """Create a group of BranchInitHook parameters.
1846
 
 
1847
 
        :param format: the branch format
1848
 
        :param a_bzrdir: the BzrDir where the branch will be/has been
1849
 
            initialized
1850
 
        :param name: name of colocated branch, if any (or None)
1851
 
        :param branch: the branch created
1852
 
 
1853
 
        Note that for lightweight checkouts, the bzrdir and format fields refer
1854
 
        to the checkout, hence they are different from the corresponding fields
1855
 
        in branch, which refer to the original branch.
1856
 
        """
1857
 
        self.format = format
1858
 
        self.bzrdir = a_bzrdir
1859
 
        self.name = name
1860
 
        self.branch = branch
1861
 
 
1862
 
    def __eq__(self, other):
1863
 
        return self.__dict__ == other.__dict__
1864
 
 
1865
 
    def __repr__(self):
1866
 
        if self.branch:
1867
 
            return "<%s of %s>" % (self.__class__.__name__, self.branch)
1868
 
        else:
1869
 
            return "<%s of format:%s bzrdir:%s>" % (
1870
 
                self.__class__.__name__, self.branch,
1871
 
                self.format, self.bzrdir)
1872
 
 
1873
 
 
1874
 
class SwitchHookParams(object):
1875
 
    """Object holding parameters passed to *_switch hooks.
1876
 
 
1877
 
    There are 4 fields that hooks may wish to access:
1878
 
 
1879
 
    :ivar control_dir: BzrDir of the checkout to change
1880
 
    :ivar to_branch: branch that the checkout is to reference
1881
 
    :ivar force: skip the check for local commits in a heavy checkout
1882
 
    :ivar revision_id: revision ID to switch to (or None)
1883
 
    """
1884
 
 
1885
 
    def __init__(self, control_dir, to_branch, force, revision_id):
1886
 
        """Create a group of SwitchHook parameters.
1887
 
 
1888
 
        :param control_dir: BzrDir of the checkout to change
1889
 
        :param to_branch: branch that the checkout is to reference
1890
 
        :param force: skip the check for local commits in a heavy checkout
1891
 
        :param revision_id: revision ID to switch to (or None)
1892
 
        """
1893
 
        self.control_dir = control_dir
1894
 
        self.to_branch = to_branch
1895
 
        self.force = force
1896
 
        self.revision_id = revision_id
1897
 
 
1898
 
    def __eq__(self, other):
1899
 
        return self.__dict__ == other.__dict__
1900
 
 
1901
 
    def __repr__(self):
1902
 
        return "<%s for %s to (%s, %s)>" % (self.__class__.__name__,
1903
 
            self.control_dir, self.to_branch,
1904
 
            self.revision_id)
1905
 
 
1906
 
 
1907
1685
class BzrBranchFormat4(BranchFormat):
1908
1686
    """Bzr branch format 4.
1909
1687
 
1916
1694
        """See BranchFormat.get_format_description()."""
1917
1695
        return "Branch format 4"
1918
1696
 
1919
 
    def initialize(self, a_bzrdir, name=None):
 
1697
    def initialize(self, a_bzrdir):
1920
1698
        """Create a branch of this format in a_bzrdir."""
1921
1699
        utf8_files = [('revision-history', ''),
1922
1700
                      ('branch-name', ''),
1923
1701
                      ]
1924
 
        return self._initialize_helper(a_bzrdir, utf8_files, name=name,
 
1702
        return self._initialize_helper(a_bzrdir, utf8_files,
1925
1703
                                       lock_type='branch4', set_format=False)
1926
1704
 
1927
1705
    def __init__(self):
1932
1710
        """The network name for this format is the control dirs disk label."""
1933
1711
        return self._matchingbzrdir.get_format_string()
1934
1712
 
1935
 
    def open(self, a_bzrdir, name=None, _found=False, ignore_fallbacks=False):
 
1713
    def open(self, a_bzrdir, _found=False, ignore_fallbacks=False):
1936
1714
        """See BranchFormat.open()."""
1937
1715
        if not _found:
1938
1716
            # we are being called directly and must probe.
1940
1718
        return BzrBranch(_format=self,
1941
1719
                         _control_files=a_bzrdir._control_files,
1942
1720
                         a_bzrdir=a_bzrdir,
1943
 
                         name=name,
1944
1721
                         _repository=a_bzrdir.open_repository())
1945
1722
 
1946
1723
    def __str__(self):
1961
1738
        """
1962
1739
        return self.get_format_string()
1963
1740
 
1964
 
    def open(self, a_bzrdir, name=None, _found=False, ignore_fallbacks=False):
 
1741
    def open(self, a_bzrdir, _found=False, ignore_fallbacks=False):
1965
1742
        """See BranchFormat.open()."""
1966
1743
        if not _found:
1967
 
            format = BranchFormat.find_format(a_bzrdir, name=name)
 
1744
            format = BranchFormat.find_format(a_bzrdir)
1968
1745
            if format.__class__ != self.__class__:
1969
1746
                raise AssertionError("wrong format %r found for %r" %
1970
1747
                    (format, self))
1971
 
        transport = a_bzrdir.get_branch_transport(None, name=name)
1972
1748
        try:
 
1749
            transport = a_bzrdir.get_branch_transport(None)
1973
1750
            control_files = lockable_files.LockableFiles(transport, 'lock',
1974
1751
                                                         lockdir.LockDir)
1975
1752
            return self._branch_class()(_format=self,
1976
1753
                              _control_files=control_files,
1977
 
                              name=name,
1978
1754
                              a_bzrdir=a_bzrdir,
1979
1755
                              _repository=a_bzrdir.find_repository(),
1980
1756
                              ignore_fallbacks=ignore_fallbacks)
1981
1757
        except errors.NoSuchFile:
1982
 
            raise errors.NotBranchError(path=transport.base, bzrdir=a_bzrdir)
 
1758
            raise errors.NotBranchError(path=transport.base)
1983
1759
 
1984
1760
    def __init__(self):
1985
1761
        super(BranchFormatMetadir, self).__init__()
2014
1790
        """See BranchFormat.get_format_description()."""
2015
1791
        return "Branch format 5"
2016
1792
 
2017
 
    def initialize(self, a_bzrdir, name=None):
 
1793
    def initialize(self, a_bzrdir):
2018
1794
        """Create a branch of this format in a_bzrdir."""
2019
1795
        utf8_files = [('revision-history', ''),
2020
1796
                      ('branch-name', ''),
2021
1797
                      ]
2022
 
        return self._initialize_helper(a_bzrdir, utf8_files, name)
 
1798
        return self._initialize_helper(a_bzrdir, utf8_files)
2023
1799
 
2024
1800
    def supports_tags(self):
2025
1801
        return False
2047
1823
        """See BranchFormat.get_format_description()."""
2048
1824
        return "Branch format 6"
2049
1825
 
2050
 
    def initialize(self, a_bzrdir, name=None):
 
1826
    def initialize(self, a_bzrdir):
2051
1827
        """Create a branch of this format in a_bzrdir."""
2052
1828
        utf8_files = [('last-revision', '0 null:\n'),
2053
1829
                      ('branch.conf', ''),
2054
1830
                      ('tags', ''),
2055
1831
                      ]
2056
 
        return self._initialize_helper(a_bzrdir, utf8_files, name)
 
1832
        return self._initialize_helper(a_bzrdir, utf8_files)
2057
1833
 
2058
1834
    def make_tags(self, branch):
2059
1835
        """See bzrlib.branch.BranchFormat.make_tags()."""
2077
1853
        """See BranchFormat.get_format_description()."""
2078
1854
        return "Branch format 8"
2079
1855
 
2080
 
    def initialize(self, a_bzrdir, name=None):
 
1856
    def initialize(self, a_bzrdir):
2081
1857
        """Create a branch of this format in a_bzrdir."""
2082
1858
        utf8_files = [('last-revision', '0 null:\n'),
2083
1859
                      ('branch.conf', ''),
2084
1860
                      ('tags', ''),
2085
1861
                      ('references', '')
2086
1862
                      ]
2087
 
        return self._initialize_helper(a_bzrdir, utf8_files, name)
 
1863
        return self._initialize_helper(a_bzrdir, utf8_files)
2088
1864
 
2089
1865
    def __init__(self):
2090
1866
        super(BzrBranchFormat8, self).__init__()
2113
1889
    This format was introduced in bzr 1.6.
2114
1890
    """
2115
1891
 
2116
 
    def initialize(self, a_bzrdir, name=None):
 
1892
    def initialize(self, a_bzrdir):
2117
1893
        """Create a branch of this format in a_bzrdir."""
2118
1894
        utf8_files = [('last-revision', '0 null:\n'),
2119
1895
                      ('branch.conf', ''),
2120
1896
                      ('tags', ''),
2121
1897
                      ]
2122
 
        return self._initialize_helper(a_bzrdir, utf8_files, name)
 
1898
        return self._initialize_helper(a_bzrdir, utf8_files)
2123
1899
 
2124
1900
    def _branch_class(self):
2125
1901
        return BzrBranch7
2160
1936
    def get_reference(self, a_bzrdir):
2161
1937
        """See BranchFormat.get_reference()."""
2162
1938
        transport = a_bzrdir.get_branch_transport(None)
2163
 
        return transport.get_bytes('location')
 
1939
        return transport.get('location').read()
2164
1940
 
2165
1941
    def set_reference(self, a_bzrdir, to_branch):
2166
1942
        """See BranchFormat.set_reference()."""
2167
1943
        transport = a_bzrdir.get_branch_transport(None)
2168
1944
        location = transport.put_bytes('location', to_branch.base)
2169
1945
 
2170
 
    def initialize(self, a_bzrdir, name=None, target_branch=None):
 
1946
    def initialize(self, a_bzrdir, target_branch=None):
2171
1947
        """Create a branch of this format in a_bzrdir."""
2172
1948
        if target_branch is None:
2173
1949
            # this format does not implement branch itself, thus the implicit
2174
1950
            # creation contract must see it as uninitializable
2175
1951
            raise errors.UninitializableFormat(self)
2176
 
        mutter('creating branch reference in %s', a_bzrdir.user_url)
2177
 
        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)
2178
1954
        branch_transport.put_bytes('location',
2179
 
            target_branch.bzrdir.user_url)
 
1955
            target_branch.bzrdir.root_transport.base)
2180
1956
        branch_transport.put_bytes('format', self.get_format_string())
2181
 
        branch = self.open(
2182
 
            a_bzrdir, name, _found=True,
 
1957
        return self.open(
 
1958
            a_bzrdir, _found=True,
2183
1959
            possible_transports=[target_branch.bzrdir.root_transport])
2184
 
        self._run_post_branch_init_hooks(a_bzrdir, name, branch)
2185
 
        return branch
2186
1960
 
2187
1961
    def __init__(self):
2188
1962
        super(BranchReferenceFormat, self).__init__()
2194
1968
        def clone(to_bzrdir, revision_id=None,
2195
1969
            repository_policy=None):
2196
1970
            """See Branch.clone()."""
2197
 
            return format.initialize(to_bzrdir, target_branch=a_branch)
 
1971
            return format.initialize(to_bzrdir, a_branch)
2198
1972
            # cannot obey revision_id limits when cloning a reference ...
2199
1973
            # FIXME RBC 20060210 either nuke revision_id for clone, or
2200
1974
            # emit some sort of warning/error to the caller ?!
2201
1975
        return clone
2202
1976
 
2203
 
    def open(self, a_bzrdir, name=None, _found=False, location=None,
 
1977
    def open(self, a_bzrdir, _found=False, location=None,
2204
1978
             possible_transports=None, ignore_fallbacks=False):
2205
1979
        """Return the branch that the branch reference in a_bzrdir points at.
2206
1980
 
2207
1981
        :param a_bzrdir: A BzrDir that contains a branch.
2208
 
        :param name: Name of colocated branch to open, if any
2209
1982
        :param _found: a private parameter, do not use it. It is used to
2210
1983
            indicate if format probing has already be done.
2211
1984
        :param ignore_fallbacks: when set, no fallback branches will be opened
2216
1989
        :param possible_transports: An optional reusable transports list.
2217
1990
        """
2218
1991
        if not _found:
2219
 
            format = BranchFormat.find_format(a_bzrdir, name=name)
 
1992
            format = BranchFormat.find_format(a_bzrdir)
2220
1993
            if format.__class__ != self.__class__:
2221
1994
                raise AssertionError("wrong format %r found for %r" %
2222
1995
                    (format, self))
2224
1997
            location = self.get_reference(a_bzrdir)
2225
1998
        real_bzrdir = bzrdir.BzrDir.open(
2226
1999
            location, possible_transports=possible_transports)
2227
 
        result = real_bzrdir.open_branch(name=name, 
2228
 
            ignore_fallbacks=ignore_fallbacks)
 
2000
        result = real_bzrdir.open_branch(ignore_fallbacks=ignore_fallbacks)
2229
2001
        # this changes the behaviour of result.clone to create a new reference
2230
2002
        # rather than a copy of the content of the branch.
2231
2003
        # I did not use a proxy object because that needs much more extensive
2258
2030
BranchFormat.register_format(__format6)
2259
2031
BranchFormat.register_format(__format7)
2260
2032
BranchFormat.register_format(__format8)
2261
 
BranchFormat.set_default_format(__format7)
 
2033
BranchFormat.set_default_format(__format6)
2262
2034
_legacy_formats = [BzrBranchFormat4(),
2263
2035
    ]
2264
2036
network_format_registry.register(
2265
2037
    _legacy_formats[0].network_name(), _legacy_formats[0].__class__)
2266
2038
 
2267
2039
 
2268
 
class BzrBranch(Branch, _RelockDebugMixin):
 
2040
class BzrBranch(Branch):
2269
2041
    """A branch stored in the actual filesystem.
2270
2042
 
2271
2043
    Note that it's "local" in the context of the filesystem; it doesn't
2277
2049
    :ivar repository: Repository for this branch.
2278
2050
    :ivar base: The url of the base directory for this branch; the one
2279
2051
        containing the .bzr directory.
2280
 
    :ivar name: Optional colocated branch name as it exists in the control
2281
 
        directory.
2282
2052
    """
2283
2053
 
2284
2054
    def __init__(self, _format=None,
2285
 
                 _control_files=None, a_bzrdir=None, name=None,
2286
 
                 _repository=None, ignore_fallbacks=False):
 
2055
                 _control_files=None, a_bzrdir=None, _repository=None,
 
2056
                 ignore_fallbacks=False):
2287
2057
        """Create new branch object at a particular location."""
2288
2058
        if a_bzrdir is None:
2289
2059
            raise ValueError('a_bzrdir must be supplied')
2290
2060
        else:
2291
2061
            self.bzrdir = a_bzrdir
2292
2062
        self._base = self.bzrdir.transport.clone('..').base
2293
 
        self.name = name
2294
2063
        # XXX: We should be able to just do
2295
2064
        #   self.base = self.bzrdir.root_transport.base
2296
2065
        # but this does not quite work yet -- mbp 20080522
2303
2072
        Branch.__init__(self)
2304
2073
 
2305
2074
    def __str__(self):
2306
 
        if self.name is None:
2307
 
            return '%s(%s)' % (self.__class__.__name__, self.user_url)
2308
 
        else:
2309
 
            return '%s(%s,%s)' % (self.__class__.__name__, self.user_url,
2310
 
                self.name)
 
2075
        return '%s(%r)' % (self.__class__.__name__, self.base)
2311
2076
 
2312
2077
    __repr__ = __str__
2313
2078
 
2324
2089
        return self.control_files.is_locked()
2325
2090
 
2326
2091
    def lock_write(self, token=None):
2327
 
        if not self.is_locked():
2328
 
            self._note_lock('w')
2329
2092
        # All-in-one needs to always unlock/lock.
2330
2093
        repo_control = getattr(self.repository, 'control_files', None)
2331
2094
        if self.control_files == repo_control or not self.is_locked():
2332
 
            self.repository._warn_if_deprecated(self)
2333
2095
            self.repository.lock_write()
2334
2096
            took_lock = True
2335
2097
        else:
2342
2104
            raise
2343
2105
 
2344
2106
    def lock_read(self):
2345
 
        if not self.is_locked():
2346
 
            self._note_lock('r')
2347
2107
        # All-in-one needs to always unlock/lock.
2348
2108
        repo_control = getattr(self.repository, 'control_files', None)
2349
2109
        if self.control_files == repo_control or not self.is_locked():
2350
 
            self.repository._warn_if_deprecated(self)
2351
2110
            self.repository.lock_read()
2352
2111
            took_lock = True
2353
2112
        else:
2359
2118
                self.repository.unlock()
2360
2119
            raise
2361
2120
 
2362
 
    @only_raises(errors.LockNotHeld, errors.LockBroken)
2363
2121
    def unlock(self):
2364
2122
        try:
2365
2123
            self.control_files.unlock()
2528
2286
        return result
2529
2287
 
2530
2288
    def get_stacked_on_url(self):
2531
 
        raise errors.UnstackableBranchFormat(self._format, self.user_url)
 
2289
        raise errors.UnstackableBranchFormat(self._format, self.base)
2532
2290
 
2533
2291
    def set_push_location(self, location):
2534
2292
        """See Branch.set_push_location."""
2724
2482
        if _mod_revision.is_null(last_revision):
2725
2483
            return
2726
2484
        if last_revision not in self._lefthand_history(revision_id):
2727
 
            raise errors.AppendRevisionsOnlyViolation(self.user_url)
 
2485
            raise errors.AppendRevisionsOnlyViolation(self.base)
2728
2486
 
2729
2487
    def _gen_revision_history(self):
2730
2488
        """Generate the revision history from last revision
2830
2588
        if branch_location is None:
2831
2589
            return Branch.reference_parent(self, file_id, path,
2832
2590
                                           possible_transports)
2833
 
        branch_location = urlutils.join(self.user_url, branch_location)
 
2591
        branch_location = urlutils.join(self.base, branch_location)
2834
2592
        return Branch.open(branch_location,
2835
2593
                           possible_transports=possible_transports)
2836
2594
 
2882
2640
        return stacked_url
2883
2641
 
2884
2642
    def _get_append_revisions_only(self):
2885
 
        return self.get_config(
2886
 
            ).get_user_option_as_bool('append_revisions_only')
 
2643
        value = self.get_config().get_user_option('append_revisions_only')
 
2644
        return value == 'True'
2887
2645
 
2888
2646
    @needs_write_lock
2889
2647
    def generate_revision_history(self, revision_id, last_rev=None,
2951
2709
    """
2952
2710
 
2953
2711
    def get_stacked_on_url(self):
2954
 
        raise errors.UnstackableBranchFormat(self._format, self.user_url)
 
2712
        raise errors.UnstackableBranchFormat(self._format, self.base)
2955
2713
 
2956
2714
 
2957
2715
######################################################################
3036
2794
 
3037
2795
    def __init__(self, branch):
3038
2796
        self.branch = branch
3039
 
        self.errors = []
 
2797
        self.ghosts_in_mainline = False
3040
2798
 
3041
2799
    def report_results(self, verbose):
3042
2800
        """Report the check results via trace.note.
3044
2802
        :param verbose: Requests more detailed display of what was checked,
3045
2803
            if any.
3046
2804
        """
3047
 
        note('checked branch %s format %s', self.branch.user_url,
3048
 
            self.branch._format)
3049
 
        for error in self.errors:
3050
 
            note('found error:%s', error)
 
2805
        note('checked branch %s format %s',
 
2806
             self.branch.base,
 
2807
             self.branch._format)
 
2808
        if self.ghosts_in_mainline:
 
2809
            note('branch contains ghosts in mainline')
3051
2810
 
3052
2811
 
3053
2812
class Converter5to6(object):
3148
2907
    @staticmethod
3149
2908
    def _get_branch_formats_to_test():
3150
2909
        """Return a tuple with the Branch formats to use when testing."""
3151
 
        raise NotImplementedError(InterBranch._get_branch_formats_to_test)
 
2910
        raise NotImplementedError(self._get_branch_formats_to_test)
3152
2911
 
3153
2912
    def pull(self, overwrite=False, stop_revision=None,
3154
2913
             possible_transports=None, local=False):
3309
3068
                _override_hook_source_branch=_override_hook_source_branch)
3310
3069
        finally:
3311
3070
            self.source.unlock()
 
3071
        return result
3312
3072
 
3313
3073
    def _push_with_bound_branches(self, overwrite, stop_revision,
3314
3074
            _override_hook_source_branch=None):
3379
3139
        if local and not bound_location:
3380
3140
            raise errors.LocalRequiresBoundBranch()
3381
3141
        master_branch = None
3382
 
        if not local and bound_location and self.source.user_url != bound_location:
 
3142
        if not local and bound_location and self.source.base != bound_location:
3383
3143
            # not pulling from master, so we need to update master.
3384
3144
            master_branch = self.target.get_master_branch(possible_transports)
3385
3145
            master_branch.lock_write()