← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/branch.py

  • Committer: Andrew Bennetts
  • Date: 2009-07-16 07:10:45 UTC
  • mto: This revision was merged to the branch mainline in revision 4542.
  • Revision ID: andrew.bennetts@canonical.com-20090716071045-xdve2is0ydw65gu0
Improve error message in osutils.file_relpath.

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
35
35
        symbol_versioning,
36
36
        transport,
37
37
        tsort,
38
 
        ui,
39
38
        urlutils,
40
39
        )
41
40
from bzrlib.config import BranchConfig, TransportConfig
46
45
    )
47
46
""")
48
47
 
49
 
from bzrlib.decorators import needs_read_lock, needs_write_lock, only_raises
 
48
from bzrlib.decorators import needs_read_lock, needs_write_lock
50
49
from bzrlib.hooks import HookPoint, Hooks
51
50
from bzrlib.inter import InterObject
52
 
from bzrlib.lock import _RelockDebugMixin
53
51
from bzrlib import registry
54
52
from bzrlib.symbol_versioning import (
55
53
    deprecated_in,
63
61
BZR_BRANCH_FORMAT_6 = "Bazaar Branch Format 6 (bzr 0.15)\n"
64
62
 
65
63
 
66
 
class Branch(bzrdir.ControlComponent):
 
64
# TODO: Maybe include checks for common corruption of newlines, etc?
 
65
 
 
66
# TODO: Some operations like log might retrieve the same revisions
 
67
# repeatedly to calculate deltas.  We could perhaps have a weakref
 
68
# cache in memory to make this faster.  In general anything can be
 
69
# cached in memory between lock and unlock operations. .. nb thats
 
70
# what the transaction identity map provides
 
71
 
 
72
 
 
73
######################################################################
 
74
# branch objects
 
75
 
 
76
class Branch(object):
67
77
    """Branch holding a history of revisions.
68
78
 
69
 
    :ivar base:
70
 
        Base directory/url of the branch; using control_url and
71
 
        control_transport is more standardized.
 
79
    base
 
80
        Base directory/url of the branch.
72
81
 
73
82
    hooks: An instance of BranchHooks.
74
83
    """
76
85
    # - RBC 20060112
77
86
    base = None
78
87
 
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
88
    def __init__(self, *ignored, **ignored_too):
88
89
        self.tags = self._format.make_tags(self)
89
90
        self._revision_history_cache = None
104
105
        """Activate the branch/repository from url as a fallback repository."""
105
106
        repo = self._get_fallback_repository(url)
106
107
        if repo.has_same_location(self.repository):
107
 
            raise errors.UnstackableLocationError(self.user_url, url)
 
108
            raise errors.UnstackableLocationError(self.base, url)
108
109
        self.repository.add_fallback_repository(repo)
109
110
 
110
111
    def break_lock(self):
147
148
        if self._partial_revision_history_cache[-1] == _mod_revision.NULL_REVISION:
148
149
            self._partial_revision_history_cache.pop()
149
150
 
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
151
    @staticmethod
159
152
    def open(base, _unsupported=False, possible_transports=None):
160
153
        """Open the branch rooted at base.
164
157
        """
165
158
        control = bzrdir.BzrDir.open(base, _unsupported,
166
159
                                     possible_transports=possible_transports)
167
 
        return control.open_branch(unsupported=_unsupported)
 
160
        return control.open_branch(_unsupported)
168
161
 
169
162
    @staticmethod
170
 
    def open_from_transport(transport, name=None, _unsupported=False):
 
163
    def open_from_transport(transport, _unsupported=False):
171
164
        """Open the branch rooted at transport"""
172
165
        control = bzrdir.BzrDir.open_from_transport(transport, _unsupported)
173
 
        return control.open_branch(name=name, unsupported=_unsupported)
 
166
        return control.open_branch(_unsupported)
174
167
 
175
168
    @staticmethod
176
169
    def open_containing(url, possible_transports=None):
214
207
    def _get_fallback_repository(self, url):
215
208
        """Get the repository we fallback to at url."""
216
209
        url = urlutils.join(self.base, url)
217
 
        a_branch = Branch.open(url,
 
210
        a_bzrdir = bzrdir.BzrDir.open(url,
218
211
            possible_transports=[self.bzrdir.root_transport])
219
 
        return a_branch.repository
 
212
        return a_bzrdir.open_branch().repository
220
213
 
221
214
    def _get_tags_bytes(self):
222
215
        """Get the bytes of a serialised tags dict.
417
410
            * 'include' - the stop revision is the last item in the result
418
411
            * 'with-merges' - include the stop revision and all of its
419
412
              merged revisions in the result
420
 
            * 'with-merges-without-common-ancestry' - filter out revisions 
421
 
              that are in both ancestries
422
413
        :param direction: either 'reverse' or 'forward':
423
414
            * reverse means return the start_revision_id first, i.e.
424
415
              start at the most recent revision and go backwards in history
446
437
        # start_revision_id.
447
438
        if self._merge_sorted_revisions_cache is None:
448
439
            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)
 
440
            graph = self.repository.get_graph()
 
441
            parent_map = dict(((key, value) for key, value in
 
442
                     graph.iter_ancestry([last_revision]) if value is not None))
 
443
            revision_graph = repository._strip_NULL_ghosts(parent_map)
 
444
            revs = tsort.merge_sort(revision_graph, last_revision, None,
 
445
                generate_revno=True)
 
446
            # Drop the sequence # before caching
 
447
            self._merge_sorted_revisions_cache = [r[1:] for r in revs]
 
448
 
453
449
        filtered = self._filter_merge_sorted_revisions(
454
450
            self._merge_sorted_revisions_cache, start_revision_id,
455
451
            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
452
        if direction == 'reverse':
460
453
            return filtered
461
454
        if direction == 'forward':
468
461
        """Iterate over an inclusive range of sorted revisions."""
469
462
        rev_iter = iter(merge_sorted_revisions)
470
463
        if start_revision_id is not None:
471
 
            for node in rev_iter:
472
 
                rev_id = node.key[-1]
 
464
            for rev_id, depth, revno, end_of_merge in rev_iter:
473
465
                if rev_id != start_revision_id:
474
466
                    continue
475
467
                else:
476
468
                    # The decision to include the start or not
477
469
                    # 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)
 
470
                    rev_iter = chain(
 
471
                        iter([(rev_id, depth, revno, end_of_merge)]),
 
472
                        rev_iter)
480
473
                    break
481
474
        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)
 
475
            for rev_id, depth, revno, end_of_merge in rev_iter:
 
476
                yield rev_id, depth, revno, end_of_merge
487
477
        elif stop_rule == 'exclude':
488
 
            for node in rev_iter:
489
 
                rev_id = node.key[-1]
 
478
            for rev_id, depth, revno, end_of_merge in rev_iter:
490
479
                if rev_id == stop_revision_id:
491
480
                    return
492
 
                yield (rev_id, node.merge_depth, node.revno,
493
 
                       node.end_of_merge)
 
481
                yield rev_id, depth, revno, end_of_merge
494
482
        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)
 
483
            for rev_id, depth, revno, end_of_merge in rev_iter:
 
484
                yield rev_id, depth, revno, end_of_merge
499
485
                if rev_id == stop_revision_id:
500
486
                    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
487
        elif stop_rule == 'with-merges':
514
488
            stop_rev = self.repository.get_revision(stop_revision_id)
515
489
            if stop_rev.parent_ids:
516
490
                left_parent = stop_rev.parent_ids[0]
517
491
            else:
518
492
                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]
 
493
            for rev_id, depth, revno, end_of_merge in rev_iter:
525
494
                if rev_id == left_parent:
526
 
                    # reached the left parent after the stop_revision
527
495
                    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)
 
496
                yield rev_id, depth, revno, end_of_merge
538
497
        else:
539
498
            raise ValueError('invalid stop_rule %r' % stop_rule)
540
499
 
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
500
    def leave_lock_in_place(self):
587
501
        """Tell this branch object not to release the physical lock when this
588
502
        object is unlocked.
605
519
        :param other: The branch to bind to
606
520
        :type other: Branch
607
521
        """
608
 
        raise errors.UpgradeRequired(self.user_url)
 
522
        raise errors.UpgradeRequired(self.base)
609
523
 
610
524
    def set_append_revisions_only(self, enabled):
611
525
        if not self._format.supports_set_append_revisions_only():
612
 
            raise errors.UpgradeRequired(self.user_url)
 
526
            raise errors.UpgradeRequired(self.base)
613
527
        if enabled:
614
528
            value = 'True'
615
529
        else:
663
577
    def get_old_bound_location(self):
664
578
        """Return the URL of the branch we used to be bound to
665
579
        """
666
 
        raise errors.UpgradeRequired(self.user_url)
 
580
        raise errors.UpgradeRequired(self.base)
667
581
 
668
582
    def get_commit_builder(self, parents, config=None, timestamp=None,
669
583
                           timezone=None, committer=None, revprops=None,
747
661
            stacking.
748
662
        """
749
663
        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.
 
664
            raise errors.UnstackableBranchFormat(self._format, self.base)
754
665
        self._check_stackable_repo()
755
666
        if not url:
756
667
            try:
758
669
            except (errors.NotStacked, errors.UnstackableBranchFormat,
759
670
                errors.UnstackableRepositoryFormat):
760
671
                return
761
 
            self._unstack()
 
672
            url = ''
 
673
            # XXX: Lock correctness - should unlock our old repo if we were
 
674
            # locked.
 
675
            # repositories don't offer an interface to remove fallback
 
676
            # repositories today; take the conceptually simpler option and just
 
677
            # reopen it.
 
678
            self.repository = self.bzrdir.find_repository()
 
679
            self.repository.lock_write()
 
680
            # for every revision reference the branch has, ensure it is pulled
 
681
            # in.
 
682
            source_repository = self._get_fallback_repository(old_url)
 
683
            for revision_id in chain([self.last_revision()],
 
684
                self.tags.get_reverse_tag_dict()):
 
685
                self.repository.fetch(source_repository, revision_id,
 
686
                    find_ghosts=True)
762
687
        else:
763
688
            self._activate_fallback_location(url)
764
689
        # write this out after the repository is stacked to avoid setting a
765
690
        # stacked config that doesn't work.
766
691
        self._set_config_location('stacked_on_location', url)
767
692
 
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
693
 
823
694
    def _set_tags_bytes(self, bytes):
824
695
        """Mirror method for _get_tags_bytes.
904
775
 
905
776
    def unbind(self):
906
777
        """Older format branches cannot bind or unbind."""
907
 
        raise errors.UpgradeRequired(self.user_url)
 
778
        raise errors.UpgradeRequired(self.base)
908
779
 
909
780
    def last_revision(self):
910
781
        """Return last revision id, or NULL_REVISION."""
1070
941
        try:
1071
942
            return urlutils.join(self.base[:-1], parent)
1072
943
        except errors.InvalidURLJoin, e:
1073
 
            raise errors.InaccessibleParent(parent, self.user_url)
 
944
            raise errors.InaccessibleParent(parent, self.base)
1074
945
 
1075
946
    def _get_parent_location(self):
1076
947
        raise NotImplementedError(self._get_parent_location)
1161
1032
        params = ChangeBranchTipParams(
1162
1033
            self, old_revno, new_revno, old_revid, new_revid)
1163
1034
        for hook in hooks:
1164
 
            hook(params)
 
1035
            try:
 
1036
                hook(params)
 
1037
            except errors.TipChangeRejected:
 
1038
                raise
 
1039
            except Exception:
 
1040
                exc_info = sys.exc_info()
 
1041
                hook_name = Branch.hooks.get_hook_name(hook)
 
1042
                raise errors.HookFailed(
 
1043
                    'pre_change_branch_tip', hook_name, exc_info)
1165
1044
 
1166
1045
    @needs_write_lock
1167
1046
    def update(self):
1216
1095
        revision_id: if not None, the revision history in the new branch will
1217
1096
                     be truncated to end with revision_id.
1218
1097
        """
1219
 
        if (repository_policy is not None and
1220
 
            repository_policy.requires_stacking()):
1221
 
            to_bzrdir._format.require_stacking(_skip_repo=True)
1222
1098
        result = to_bzrdir.create_branch()
1223
1099
        result.lock_write()
1224
1100
        try:
1292
1168
        target._set_all_reference_info(target_reference_dict)
1293
1169
 
1294
1170
    @needs_read_lock
1295
 
    def check(self, refs):
 
1171
    def check(self):
1296
1172
        """Check consistency of the branch.
1297
1173
 
1298
1174
        In particular this checks that revisions given in the revision-history
1301
1177
 
1302
1178
        Callers will typically also want to check the repository.
1303
1179
 
1304
 
        :param refs: Calculated refs for this branch as specified by
1305
 
            branch._get_check_refs()
1306
1180
        :return: A BranchCheckResult.
1307
1181
        """
1308
 
        result = BranchCheckResult(self)
 
1182
        ret = BranchCheckResult(self)
 
1183
        mainline_parent_id = None
1309
1184
        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
 
1185
        real_rev_history = []
 
1186
        try:
 
1187
            for revid in self.repository.iter_reverse_revision_history(
 
1188
                last_revision_id):
 
1189
                real_rev_history.append(revid)
 
1190
        except errors.RevisionNotPresent:
 
1191
            ret.ghosts_in_mainline = True
 
1192
        else:
 
1193
            ret.ghosts_in_mainline = False
 
1194
        real_rev_history.reverse()
 
1195
        if len(real_rev_history) != last_revno:
 
1196
            raise errors.BzrCheckError('revno does not match len(mainline)'
 
1197
                ' %s != %s' % (last_revno, len(real_rev_history)))
 
1198
        # TODO: We should probably also check that real_rev_history actually
 
1199
        #       matches self.revision_history()
 
1200
        for revision_id in real_rev_history:
 
1201
            try:
 
1202
                revision = self.repository.get_revision(revision_id)
 
1203
            except errors.NoSuchRevision, e:
 
1204
                raise errors.BzrCheckError("mainline revision {%s} not in repository"
 
1205
                            % revision_id)
 
1206
            # In general the first entry on the revision history has no parents.
 
1207
            # But it's not illegal for it to have parents listed; this can happen
 
1208
            # in imports from Arch when the parents weren't reachable.
 
1209
            if mainline_parent_id is not None:
 
1210
                if mainline_parent_id not in revision.parent_ids:
 
1211
                    raise errors.BzrCheckError("previous revision {%s} not listed among "
 
1212
                                        "parents of {%s}"
 
1213
                                        % (mainline_parent_id, revision_id))
 
1214
            mainline_parent_id = revision_id
 
1215
        return ret
1321
1216
 
1322
1217
    def _get_checkout_format(self):
1323
1218
        """Return the most suitable metadir for a checkout of this branch.
1348
1243
        # clone call. Or something. 20090224 RBC/spiv.
1349
1244
        if revision_id is None:
1350
1245
            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)
 
1246
        try:
 
1247
            dir_to = self.bzrdir.clone_on_transport(to_transport,
 
1248
                revision_id=revision_id, stacked_on=stacked_on,
 
1249
                create_prefix=create_prefix, use_existing_dir=use_existing_dir)
 
1250
        except errors.FileExists:
 
1251
            if not use_existing_dir:
 
1252
                raise
 
1253
        except errors.NoSuchFile:
 
1254
            if not create_prefix:
 
1255
                raise
1354
1256
        return dir_to.open_branch()
1355
1257
 
1356
1258
    def create_checkout(self, to_location, revision_id=None,
1375
1277
        if lightweight:
1376
1278
            format = self._get_checkout_format()
1377
1279
            checkout = format.initialize_on_transport(t)
1378
 
            from_branch = BranchReferenceFormat().initialize(checkout, 
1379
 
                target_branch=self)
 
1280
            from_branch = BranchReferenceFormat().initialize(checkout, self)
1380
1281
        else:
1381
1282
            format = self._get_checkout_format()
1382
1283
            checkout_branch = bzrdir.BzrDir.create_branch_convenience(
1424
1325
    def supports_tags(self):
1425
1326
        return self._format.supports_tags()
1426
1327
 
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
1328
    def _check_if_descendant_or_diverged(self, revision_a, revision_b, graph,
1440
1329
                                         other_branch):
1441
1330
        """Ensure that revision_b is a descendant of revision_a.
1505
1394
        return not (self == other)
1506
1395
 
1507
1396
    @classmethod
1508
 
    def find_format(klass, a_bzrdir, name=None):
 
1397
    def find_format(klass, a_bzrdir):
1509
1398
        """Return the format for the branch object in a_bzrdir."""
1510
1399
        try:
1511
 
            transport = a_bzrdir.get_branch_transport(None, name=name)
1512
 
            format_string = transport.get_bytes("format")
 
1400
            transport = a_bzrdir.get_branch_transport(None)
 
1401
            format_string = transport.get("format").read()
1513
1402
            return klass._formats[format_string]
1514
1403
        except errors.NoSuchFile:
1515
 
            raise errors.NotBranchError(path=transport.base, bzrdir=a_bzrdir)
 
1404
            raise errors.NotBranchError(path=transport.base)
1516
1405
        except KeyError:
1517
1406
            raise errors.UnknownFormatError(format=format_string, kind='branch')
1518
1407
 
1554
1443
        """Return the short format description for this format."""
1555
1444
        raise NotImplementedError(self.get_format_description)
1556
1445
 
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):
 
1446
    def _initialize_helper(self, a_bzrdir, utf8_files, lock_type='metadir',
 
1447
                           set_format=True):
1567
1448
        """Initialize a branch in a bzrdir, with specified files
1568
1449
 
1569
1450
        :param a_bzrdir: The bzrdir to initialize the branch in
1570
1451
        :param utf8_files: The files to create as a list of
1571
1452
            (filename, content) tuples
1572
 
        :param name: Name of colocated branch to create, if any
1573
1453
        :param set_format: If True, set the format with
1574
1454
            self.get_format_string.  (BzrBranch4 has its format set
1575
1455
            elsewhere)
1576
1456
        :return: a branch in this format
1577
1457
        """
1578
 
        mutter('creating branch %r in %s', self, a_bzrdir.user_url)
1579
 
        branch_transport = a_bzrdir.get_branch_transport(self, name=name)
 
1458
        mutter('creating branch %r in %s', self, a_bzrdir.transport.base)
 
1459
        branch_transport = a_bzrdir.get_branch_transport(self)
1580
1460
        lock_map = {
1581
1461
            'metadir': ('lock', lockdir.LockDir),
1582
1462
            'branch4': ('branch-lock', lockable_files.TransportLock),
1603
1483
        finally:
1604
1484
            if lock_taken:
1605
1485
                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
 
1486
        return self.open(a_bzrdir, _found=True)
1609
1487
 
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
 
        """
 
1488
    def initialize(self, a_bzrdir):
 
1489
        """Create a branch of this format in a_bzrdir."""
1615
1490
        raise NotImplementedError(self.initialize)
1616
1491
 
1617
1492
    def is_supported(self):
1647
1522
        """
1648
1523
        raise NotImplementedError(self.network_name)
1649
1524
 
1650
 
    def open(self, a_bzrdir, name=None, _found=False, ignore_fallbacks=False):
 
1525
    def open(self, a_bzrdir, _found=False, ignore_fallbacks=False):
1651
1526
        """Return the branch object for a_bzrdir
1652
1527
 
1653
1528
        :param a_bzrdir: A BzrDir that contains a branch.
1654
 
        :param name: Name of colocated branch to open
1655
1529
        :param _found: a private parameter, do not use it. It is used to
1656
1530
            indicate if format probing has already be done.
1657
1531
        :param ignore_fallbacks: when set, no fallback branches will be opened
1765
1639
            "multiple hooks installed for transform_fallback_location, "
1766
1640
            "all are called with the url returned from the previous hook."
1767
1641
            "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
1642
 
1786
1643
 
1787
1644
# install the default hooks into the Branch class.
1826
1683
            self.old_revno, self.old_revid, self.new_revno, self.new_revid)
1827
1684
 
1828
1685
 
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
1686
class BzrBranchFormat4(BranchFormat):
1908
1687
    """Bzr branch format 4.
1909
1688
 
1916
1695
        """See BranchFormat.get_format_description()."""
1917
1696
        return "Branch format 4"
1918
1697
 
1919
 
    def initialize(self, a_bzrdir, name=None):
 
1698
    def initialize(self, a_bzrdir):
1920
1699
        """Create a branch of this format in a_bzrdir."""
1921
1700
        utf8_files = [('revision-history', ''),
1922
1701
                      ('branch-name', ''),
1923
1702
                      ]
1924
 
        return self._initialize_helper(a_bzrdir, utf8_files, name=name,
 
1703
        return self._initialize_helper(a_bzrdir, utf8_files,
1925
1704
                                       lock_type='branch4', set_format=False)
1926
1705
 
1927
1706
    def __init__(self):
1932
1711
        """The network name for this format is the control dirs disk label."""
1933
1712
        return self._matchingbzrdir.get_format_string()
1934
1713
 
1935
 
    def open(self, a_bzrdir, name=None, _found=False, ignore_fallbacks=False):
 
1714
    def open(self, a_bzrdir, _found=False, ignore_fallbacks=False):
1936
1715
        """See BranchFormat.open()."""
1937
1716
        if not _found:
1938
1717
            # we are being called directly and must probe.
1940
1719
        return BzrBranch(_format=self,
1941
1720
                         _control_files=a_bzrdir._control_files,
1942
1721
                         a_bzrdir=a_bzrdir,
1943
 
                         name=name,
1944
1722
                         _repository=a_bzrdir.open_repository())
1945
1723
 
1946
1724
    def __str__(self):
1961
1739
        """
1962
1740
        return self.get_format_string()
1963
1741
 
1964
 
    def open(self, a_bzrdir, name=None, _found=False, ignore_fallbacks=False):
 
1742
    def open(self, a_bzrdir, _found=False, ignore_fallbacks=False):
1965
1743
        """See BranchFormat.open()."""
1966
1744
        if not _found:
1967
 
            format = BranchFormat.find_format(a_bzrdir, name=name)
 
1745
            format = BranchFormat.find_format(a_bzrdir)
1968
1746
            if format.__class__ != self.__class__:
1969
1747
                raise AssertionError("wrong format %r found for %r" %
1970
1748
                    (format, self))
1971
 
        transport = a_bzrdir.get_branch_transport(None, name=name)
1972
1749
        try:
 
1750
            transport = a_bzrdir.get_branch_transport(None)
1973
1751
            control_files = lockable_files.LockableFiles(transport, 'lock',
1974
1752
                                                         lockdir.LockDir)
1975
1753
            return self._branch_class()(_format=self,
1976
1754
                              _control_files=control_files,
1977
 
                              name=name,
1978
1755
                              a_bzrdir=a_bzrdir,
1979
1756
                              _repository=a_bzrdir.find_repository(),
1980
1757
                              ignore_fallbacks=ignore_fallbacks)
1981
1758
        except errors.NoSuchFile:
1982
 
            raise errors.NotBranchError(path=transport.base, bzrdir=a_bzrdir)
 
1759
            raise errors.NotBranchError(path=transport.base)
1983
1760
 
1984
1761
    def __init__(self):
1985
1762
        super(BranchFormatMetadir, self).__init__()
2014
1791
        """See BranchFormat.get_format_description()."""
2015
1792
        return "Branch format 5"
2016
1793
 
2017
 
    def initialize(self, a_bzrdir, name=None):
 
1794
    def initialize(self, a_bzrdir):
2018
1795
        """Create a branch of this format in a_bzrdir."""
2019
1796
        utf8_files = [('revision-history', ''),
2020
1797
                      ('branch-name', ''),
2021
1798
                      ]
2022
 
        return self._initialize_helper(a_bzrdir, utf8_files, name)
 
1799
        return self._initialize_helper(a_bzrdir, utf8_files)
2023
1800
 
2024
1801
    def supports_tags(self):
2025
1802
        return False
2047
1824
        """See BranchFormat.get_format_description()."""
2048
1825
        return "Branch format 6"
2049
1826
 
2050
 
    def initialize(self, a_bzrdir, name=None):
 
1827
    def initialize(self, a_bzrdir):
2051
1828
        """Create a branch of this format in a_bzrdir."""
2052
1829
        utf8_files = [('last-revision', '0 null:\n'),
2053
1830
                      ('branch.conf', ''),
2054
1831
                      ('tags', ''),
2055
1832
                      ]
2056
 
        return self._initialize_helper(a_bzrdir, utf8_files, name)
 
1833
        return self._initialize_helper(a_bzrdir, utf8_files)
2057
1834
 
2058
1835
    def make_tags(self, branch):
2059
1836
        """See bzrlib.branch.BranchFormat.make_tags()."""
2077
1854
        """See BranchFormat.get_format_description()."""
2078
1855
        return "Branch format 8"
2079
1856
 
2080
 
    def initialize(self, a_bzrdir, name=None):
 
1857
    def initialize(self, a_bzrdir):
2081
1858
        """Create a branch of this format in a_bzrdir."""
2082
1859
        utf8_files = [('last-revision', '0 null:\n'),
2083
1860
                      ('branch.conf', ''),
2084
1861
                      ('tags', ''),
2085
1862
                      ('references', '')
2086
1863
                      ]
2087
 
        return self._initialize_helper(a_bzrdir, utf8_files, name)
 
1864
        return self._initialize_helper(a_bzrdir, utf8_files)
2088
1865
 
2089
1866
    def __init__(self):
2090
1867
        super(BzrBranchFormat8, self).__init__()
2113
1890
    This format was introduced in bzr 1.6.
2114
1891
    """
2115
1892
 
2116
 
    def initialize(self, a_bzrdir, name=None):
 
1893
    def initialize(self, a_bzrdir):
2117
1894
        """Create a branch of this format in a_bzrdir."""
2118
1895
        utf8_files = [('last-revision', '0 null:\n'),
2119
1896
                      ('branch.conf', ''),
2120
1897
                      ('tags', ''),
2121
1898
                      ]
2122
 
        return self._initialize_helper(a_bzrdir, utf8_files, name)
 
1899
        return self._initialize_helper(a_bzrdir, utf8_files)
2123
1900
 
2124
1901
    def _branch_class(self):
2125
1902
        return BzrBranch7
2160
1937
    def get_reference(self, a_bzrdir):
2161
1938
        """See BranchFormat.get_reference()."""
2162
1939
        transport = a_bzrdir.get_branch_transport(None)
2163
 
        return transport.get_bytes('location')
 
1940
        return transport.get('location').read()
2164
1941
 
2165
1942
    def set_reference(self, a_bzrdir, to_branch):
2166
1943
        """See BranchFormat.set_reference()."""
2167
1944
        transport = a_bzrdir.get_branch_transport(None)
2168
1945
        location = transport.put_bytes('location', to_branch.base)
2169
1946
 
2170
 
    def initialize(self, a_bzrdir, name=None, target_branch=None):
 
1947
    def initialize(self, a_bzrdir, target_branch=None):
2171
1948
        """Create a branch of this format in a_bzrdir."""
2172
1949
        if target_branch is None:
2173
1950
            # this format does not implement branch itself, thus the implicit
2174
1951
            # creation contract must see it as uninitializable
2175
1952
            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)
 
1953
        mutter('creating branch reference in %s', a_bzrdir.transport.base)
 
1954
        branch_transport = a_bzrdir.get_branch_transport(self)
2178
1955
        branch_transport.put_bytes('location',
2179
 
            target_branch.bzrdir.user_url)
 
1956
            target_branch.bzrdir.root_transport.base)
2180
1957
        branch_transport.put_bytes('format', self.get_format_string())
2181
 
        branch = self.open(
2182
 
            a_bzrdir, name, _found=True,
 
1958
        return self.open(
 
1959
            a_bzrdir, _found=True,
2183
1960
            possible_transports=[target_branch.bzrdir.root_transport])
2184
 
        self._run_post_branch_init_hooks(a_bzrdir, name, branch)
2185
 
        return branch
2186
1961
 
2187
1962
    def __init__(self):
2188
1963
        super(BranchReferenceFormat, self).__init__()
2194
1969
        def clone(to_bzrdir, revision_id=None,
2195
1970
            repository_policy=None):
2196
1971
            """See Branch.clone()."""
2197
 
            return format.initialize(to_bzrdir, target_branch=a_branch)
 
1972
            return format.initialize(to_bzrdir, a_branch)
2198
1973
            # cannot obey revision_id limits when cloning a reference ...
2199
1974
            # FIXME RBC 20060210 either nuke revision_id for clone, or
2200
1975
            # emit some sort of warning/error to the caller ?!
2201
1976
        return clone
2202
1977
 
2203
 
    def open(self, a_bzrdir, name=None, _found=False, location=None,
 
1978
    def open(self, a_bzrdir, _found=False, location=None,
2204
1979
             possible_transports=None, ignore_fallbacks=False):
2205
1980
        """Return the branch that the branch reference in a_bzrdir points at.
2206
1981
 
2207
1982
        :param a_bzrdir: A BzrDir that contains a branch.
2208
 
        :param name: Name of colocated branch to open, if any
2209
1983
        :param _found: a private parameter, do not use it. It is used to
2210
1984
            indicate if format probing has already be done.
2211
1985
        :param ignore_fallbacks: when set, no fallback branches will be opened
2216
1990
        :param possible_transports: An optional reusable transports list.
2217
1991
        """
2218
1992
        if not _found:
2219
 
            format = BranchFormat.find_format(a_bzrdir, name=name)
 
1993
            format = BranchFormat.find_format(a_bzrdir)
2220
1994
            if format.__class__ != self.__class__:
2221
1995
                raise AssertionError("wrong format %r found for %r" %
2222
1996
                    (format, self))
2224
1998
            location = self.get_reference(a_bzrdir)
2225
1999
        real_bzrdir = bzrdir.BzrDir.open(
2226
2000
            location, possible_transports=possible_transports)
2227
 
        result = real_bzrdir.open_branch(name=name, 
2228
 
            ignore_fallbacks=ignore_fallbacks)
 
2001
        result = real_bzrdir.open_branch(ignore_fallbacks=ignore_fallbacks)
2229
2002
        # this changes the behaviour of result.clone to create a new reference
2230
2003
        # rather than a copy of the content of the branch.
2231
2004
        # I did not use a proxy object because that needs much more extensive
2258
2031
BranchFormat.register_format(__format6)
2259
2032
BranchFormat.register_format(__format7)
2260
2033
BranchFormat.register_format(__format8)
2261
 
BranchFormat.set_default_format(__format7)
 
2034
BranchFormat.set_default_format(__format6)
2262
2035
_legacy_formats = [BzrBranchFormat4(),
2263
2036
    ]
2264
2037
network_format_registry.register(
2265
2038
    _legacy_formats[0].network_name(), _legacy_formats[0].__class__)
2266
2039
 
2267
2040
 
2268
 
class BzrBranch(Branch, _RelockDebugMixin):
 
2041
class BzrBranch(Branch):
2269
2042
    """A branch stored in the actual filesystem.
2270
2043
 
2271
2044
    Note that it's "local" in the context of the filesystem; it doesn't
2277
2050
    :ivar repository: Repository for this branch.
2278
2051
    :ivar base: The url of the base directory for this branch; the one
2279
2052
        containing the .bzr directory.
2280
 
    :ivar name: Optional colocated branch name as it exists in the control
2281
 
        directory.
2282
2053
    """
2283
2054
 
2284
2055
    def __init__(self, _format=None,
2285
 
                 _control_files=None, a_bzrdir=None, name=None,
2286
 
                 _repository=None, ignore_fallbacks=False):
 
2056
                 _control_files=None, a_bzrdir=None, _repository=None,
 
2057
                 ignore_fallbacks=False):
2287
2058
        """Create new branch object at a particular location."""
2288
2059
        if a_bzrdir is None:
2289
2060
            raise ValueError('a_bzrdir must be supplied')
2290
2061
        else:
2291
2062
            self.bzrdir = a_bzrdir
2292
2063
        self._base = self.bzrdir.transport.clone('..').base
2293
 
        self.name = name
2294
2064
        # XXX: We should be able to just do
2295
2065
        #   self.base = self.bzrdir.root_transport.base
2296
2066
        # but this does not quite work yet -- mbp 20080522
2303
2073
        Branch.__init__(self)
2304
2074
 
2305
2075
    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)
 
2076
        return '%s(%r)' % (self.__class__.__name__, self.base)
2311
2077
 
2312
2078
    __repr__ = __str__
2313
2079
 
2324
2090
        return self.control_files.is_locked()
2325
2091
 
2326
2092
    def lock_write(self, token=None):
2327
 
        if not self.is_locked():
2328
 
            self._note_lock('w')
2329
2093
        # All-in-one needs to always unlock/lock.
2330
2094
        repo_control = getattr(self.repository, 'control_files', None)
2331
2095
        if self.control_files == repo_control or not self.is_locked():
2332
 
            self.repository._warn_if_deprecated(self)
2333
2096
            self.repository.lock_write()
2334
2097
            took_lock = True
2335
2098
        else:
2342
2105
            raise
2343
2106
 
2344
2107
    def lock_read(self):
2345
 
        if not self.is_locked():
2346
 
            self._note_lock('r')
2347
2108
        # All-in-one needs to always unlock/lock.
2348
2109
        repo_control = getattr(self.repository, 'control_files', None)
2349
2110
        if self.control_files == repo_control or not self.is_locked():
2350
 
            self.repository._warn_if_deprecated(self)
2351
2111
            self.repository.lock_read()
2352
2112
            took_lock = True
2353
2113
        else:
2359
2119
                self.repository.unlock()
2360
2120
            raise
2361
2121
 
2362
 
    @only_raises(errors.LockNotHeld, errors.LockBroken)
2363
2122
    def unlock(self):
2364
2123
        try:
2365
2124
            self.control_files.unlock()
2528
2287
        return result
2529
2288
 
2530
2289
    def get_stacked_on_url(self):
2531
 
        raise errors.UnstackableBranchFormat(self._format, self.user_url)
 
2290
        raise errors.UnstackableBranchFormat(self._format, self.base)
2532
2291
 
2533
2292
    def set_push_location(self, location):
2534
2293
        """See Branch.set_push_location."""
2724
2483
        if _mod_revision.is_null(last_revision):
2725
2484
            return
2726
2485
        if last_revision not in self._lefthand_history(revision_id):
2727
 
            raise errors.AppendRevisionsOnlyViolation(self.user_url)
 
2486
            raise errors.AppendRevisionsOnlyViolation(self.base)
2728
2487
 
2729
2488
    def _gen_revision_history(self):
2730
2489
        """Generate the revision history from last revision
2830
2589
        if branch_location is None:
2831
2590
            return Branch.reference_parent(self, file_id, path,
2832
2591
                                           possible_transports)
2833
 
        branch_location = urlutils.join(self.user_url, branch_location)
 
2592
        branch_location = urlutils.join(self.base, branch_location)
2834
2593
        return Branch.open(branch_location,
2835
2594
                           possible_transports=possible_transports)
2836
2595
 
2882
2641
        return stacked_url
2883
2642
 
2884
2643
    def _get_append_revisions_only(self):
2885
 
        return self.get_config(
2886
 
            ).get_user_option_as_bool('append_revisions_only')
 
2644
        value = self.get_config().get_user_option('append_revisions_only')
 
2645
        return value == 'True'
2887
2646
 
2888
2647
    @needs_write_lock
2889
2648
    def generate_revision_history(self, revision_id, last_rev=None,
2951
2710
    """
2952
2711
 
2953
2712
    def get_stacked_on_url(self):
2954
 
        raise errors.UnstackableBranchFormat(self._format, self.user_url)
 
2713
        raise errors.UnstackableBranchFormat(self._format, self.base)
2955
2714
 
2956
2715
 
2957
2716
######################################################################
3036
2795
 
3037
2796
    def __init__(self, branch):
3038
2797
        self.branch = branch
3039
 
        self.errors = []
 
2798
        self.ghosts_in_mainline = False
3040
2799
 
3041
2800
    def report_results(self, verbose):
3042
2801
        """Report the check results via trace.note.
3044
2803
        :param verbose: Requests more detailed display of what was checked,
3045
2804
            if any.
3046
2805
        """
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)
 
2806
        note('checked branch %s format %s',
 
2807
             self.branch.base,
 
2808
             self.branch._format)
 
2809
        if self.ghosts_in_mainline:
 
2810
            note('branch contains ghosts in mainline')
3051
2811
 
3052
2812
 
3053
2813
class Converter5to6(object):
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()