/brz/remove-bazaar

To get this branch, use:
bzr branch http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar

« back to all changes in this revision

Viewing changes to bzrlib/bzrdir.py

  • Committer: Jelmer Vernooij
  • Date: 2010-08-02 11:20:39 UTC
  • mto: This revision was merged to the branch mainline in revision 5389.
  • Revision ID: jelmer@samba.org-20100802112039-j96qqgswdytb4are
Rename per_bzrdir => per_controldir.

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/bzrdir.py
75
75
from bzrlib.weave import Weave
76
76
""")
77
77
 
 
78
from bzrlib.controldir import (
 
79
    ControlDir,
 
80
    )
 
81
 
78
82
from bzrlib.trace import (
79
83
    mutter,
80
84
    note,
86
90
    registry,
87
91
    symbol_versioning,
88
92
    )
89
 
    
90
 
    
 
93
 
 
94
 
91
95
class ControlComponent(object):
92
96
    """Abstract base class for control directory components.
93
 
    
94
 
    This provides interfaces that are common across bzrdirs, 
 
97
 
 
98
    This provides interfaces that are common across bzrdirs,
95
99
    repositories, branches, and workingtree control directories.
96
 
    
97
 
    They all expose two urls and transports: the *user* URL is the 
98
 
    one that stops above the control directory (eg .bzr) and that 
 
100
 
 
101
    They all expose two urls and transports: the *user* URL is the
 
102
    one that stops above the control directory (eg .bzr) and that
99
103
    should normally be used in messages, and the *control* URL is
100
104
    under that in eg .bzr/checkout and is used to read the control
101
105
    files.
102
 
    
103
 
    This can be used as a mixin and is intended to fit with 
 
106
 
 
107
    This can be used as a mixin and is intended to fit with
104
108
    foreign formats.
105
109
    """
106
 
    
 
110
 
107
111
    @property
108
112
    def control_transport(self):
109
113
        raise NotImplementedError
110
 
   
 
114
 
111
115
    @property
112
116
    def control_url(self):
113
117
        return self.control_transport.base
114
 
    
 
118
 
115
119
    @property
116
120
    def user_transport(self):
117
121
        raise NotImplementedError
118
 
        
 
122
 
119
123
    @property
120
124
    def user_url(self):
121
125
        return self.user_transport.base
122
 
    
123
 
 
124
 
class BzrDir(ControlComponent):
 
126
 
 
127
 
 
128
class BzrDir(ControlComponent,ControlDir):
125
129
    """A .bzr control diretory.
126
130
 
127
131
    BzrDir instances let you create or open any of the things that can be
158
162
                    return
159
163
        thing_to_unlock.break_lock()
160
164
 
161
 
    def can_convert_format(self):
162
 
        """Return true if this bzrdir is one whose format we can convert from."""
163
 
        return True
164
 
 
165
165
    def check_conversion_target(self, target_format):
166
166
        """Check that a bzrdir as a whole can be converted to a new format."""
167
167
        # The only current restriction is that the repository content can be 
241
241
        """
242
242
        # Overview: put together a broad description of what we want to end up
243
243
        # with; then make as few api calls as possible to do it.
244
 
        
 
244
 
245
245
        # We may want to create a repo/branch/tree, if we do so what format
246
246
        # would we want for each:
247
247
        require_stacking = (stacked_on is not None)
248
248
        format = self.cloning_metadir(require_stacking)
249
 
        
 
249
 
250
250
        # Figure out what objects we want:
251
251
        try:
252
252
            local_repo = self.find_repository()
375
375
            recurse = True
376
376
            try:
377
377
                bzrdir = BzrDir.open_from_transport(current_transport)
378
 
            except errors.NotBranchError:
 
378
            except (errors.NotBranchError, errors.PermissionDenied):
379
379
                pass
380
380
            else:
381
381
                recurse, value = evaluate(bzrdir)
382
382
                yield value
383
383
            try:
384
384
                subdirs = list_current(current_transport)
385
 
            except errors.NoSuchFile:
 
385
            except (errors.NoSuchFile, errors.PermissionDenied):
386
386
                continue
387
387
            if recurse:
388
388
                for subdir in sorted(subdirs, reverse=True):
389
389
                    pending.append(current_transport.clone(subdir))
390
390
 
391
 
    def list_branches(self):
392
 
        """Return a sequence of all branches local to this control directory.
393
 
 
394
 
        """
395
 
        try:
396
 
            return [self.open_branch()]
397
 
        except errors.NotBranchError:
398
 
            return []
399
 
 
400
391
    @staticmethod
401
392
    def find_branches(transport):
402
393
        """Find all branches under a transport.
425
416
                ret.extend(branches)
426
417
        return ret
427
418
 
428
 
    def destroy_repository(self):
429
 
        """Destroy the repository in this BzrDir"""
430
 
        raise NotImplementedError(self.destroy_repository)
431
 
 
432
 
    def create_branch(self, name=None):
433
 
        """Create a branch in this BzrDir.
434
 
 
435
 
        :param name: Name of the colocated branch to create, None for
436
 
            the default branch.
437
 
 
438
 
        The bzrdir's format will control what branch format is created.
439
 
        For more control see BranchFormatXX.create(a_bzrdir).
440
 
        """
441
 
        raise NotImplementedError(self.create_branch)
442
 
 
443
 
    def destroy_branch(self, name=None):
444
 
        """Destroy a branch in this BzrDir.
445
 
        
446
 
        :param name: Name of the branch to destroy, None for the default 
447
 
            branch.
448
 
        """
449
 
        raise NotImplementedError(self.destroy_branch)
450
 
 
451
419
    @staticmethod
452
420
    def create_branch_and_repo(base, force_new_repo=False, format=None):
453
421
        """Create a new BzrDir, Branch and Repository at the url 'base'.
602
570
                                               format=format).bzrdir
603
571
        return bzrdir.create_workingtree()
604
572
 
605
 
    def create_workingtree(self, revision_id=None, from_branch=None,
606
 
        accelerator_tree=None, hardlink=False):
607
 
        """Create a working tree at this BzrDir.
608
 
 
609
 
        :param revision_id: create it as of this revision id.
610
 
        :param from_branch: override bzrdir branch (for lightweight checkouts)
611
 
        :param accelerator_tree: A tree which can be used for retrieving file
612
 
            contents more quickly than the revision tree, i.e. a workingtree.
613
 
            The revision tree will be used for cases where accelerator_tree's
614
 
            content is different.
615
 
        """
616
 
        raise NotImplementedError(self.create_workingtree)
617
 
 
618
573
    def backup_bzrdir(self):
619
574
        """Backup this bzr control directory.
620
575
 
668
623
                else:
669
624
                    pass
670
625
 
671
 
    def destroy_workingtree(self):
672
 
        """Destroy the working tree at this BzrDir.
673
 
 
674
 
        Formats that do not support this may raise UnsupportedOperation.
675
 
        """
676
 
        raise NotImplementedError(self.destroy_workingtree)
677
 
 
678
 
    def destroy_workingtree_metadata(self):
679
 
        """Destroy the control files for the working tree at this BzrDir.
680
 
 
681
 
        The contents of working tree files are not affected.
682
 
        Formats that do not support this may raise UnsupportedOperation.
683
 
        """
684
 
        raise NotImplementedError(self.destroy_workingtree_metadata)
685
 
 
686
626
    def _find_containing(self, evaluate):
687
627
        """Find something in a containing control directory.
688
628
 
737
677
            raise errors.NoRepositoryPresent(self)
738
678
        return found_repo
739
679
 
740
 
    def get_branch_reference(self):
741
 
        """Return the referenced URL for the branch in this bzrdir.
742
 
 
743
 
        :raises NotBranchError: If there is no Branch.
744
 
        :return: The URL the branch in this bzrdir references if it is a
745
 
            reference branch, or None for regular branches.
746
 
        """
747
 
        return None
748
 
 
749
 
    def get_branch_transport(self, branch_format, name=None):
750
 
        """Get the transport for use by branch format in this BzrDir.
751
 
 
752
 
        Note that bzr dirs that do not support format strings will raise
753
 
        IncompatibleFormat if the branch format they are given has
754
 
        a format string, and vice versa.
755
 
 
756
 
        If branch_format is None, the transport is returned with no
757
 
        checking. If it is not None, then the returned transport is
758
 
        guaranteed to point to an existing directory ready for use.
759
 
        """
760
 
        raise NotImplementedError(self.get_branch_transport)
761
 
 
762
680
    def _find_creation_modes(self):
763
681
        """Determine the appropriate modes for files and directories.
764
682
 
803
721
            self._find_creation_modes()
804
722
        return self._dir_mode
805
723
 
806
 
    def get_repository_transport(self, repository_format):
807
 
        """Get the transport for use by repository format in this BzrDir.
808
 
 
809
 
        Note that bzr dirs that do not support format strings will raise
810
 
        IncompatibleFormat if the repository format they are given has
811
 
        a format string, and vice versa.
812
 
 
813
 
        If repository_format is None, the transport is returned with no
814
 
        checking. If it is not None, then the returned transport is
815
 
        guaranteed to point to an existing directory ready for use.
816
 
        """
817
 
        raise NotImplementedError(self.get_repository_transport)
818
 
 
819
 
    def get_workingtree_transport(self, tree_format):
820
 
        """Get the transport for use by workingtree format in this BzrDir.
821
 
 
822
 
        Note that bzr dirs that do not support format strings will raise
823
 
        IncompatibleFormat if the workingtree format they are given has a
824
 
        format string, and vice versa.
825
 
 
826
 
        If workingtree_format is None, the transport is returned with no
827
 
        checking. If it is not None, then the returned transport is
828
 
        guaranteed to point to an existing directory ready for use.
829
 
        """
830
 
        raise NotImplementedError(self.get_workingtree_transport)
831
 
 
832
724
    def get_config(self):
833
725
        """Get configuration for this BzrDir."""
834
726
        return config.BzrDirConfig(self)
852
744
        self.transport = _transport.clone('.bzr')
853
745
        self.root_transport = _transport
854
746
        self._mode_check_done = False
855
 
        
 
747
 
856
748
    @property 
857
749
    def user_transport(self):
858
750
        return self.root_transport
859
 
        
 
751
 
860
752
    @property
861
753
    def control_transport(self):
862
754
        return self.transport
868
760
 
869
761
        This is true IF and ONLY IF the filename is part of the namespace reserved
870
762
        for bzr control dirs. Currently this is the '.bzr' directory in the root
871
 
        of the root_transport. it is expected that plugins will need to extend
872
 
        this in the future - for instance to make bzr talk with svn working
873
 
        trees.
 
763
        of the root_transport. 
874
764
        """
875
765
        # this might be better on the BzrDirFormat class because it refers to
876
766
        # all the possible bzrdir disk formats.
880
770
        # add new tests for it to the appropriate place.
881
771
        return filename == '.bzr' or filename.startswith('.bzr/')
882
772
 
883
 
    def needs_format_conversion(self, format=None):
884
 
        """Return true if this bzrdir needs convert_format run on it.
885
 
 
886
 
        For instance, if the repository format is out of date but the
887
 
        branch and working tree are not, this should return True.
888
 
 
889
 
        :param format: Optional parameter indicating a specific desired
890
 
                       format we plan to arrive at.
891
 
        """
892
 
        raise NotImplementedError(self.needs_format_conversion)
893
 
 
894
773
    @staticmethod
895
774
    def open_unsupported(base):
896
775
        """Open a branch which is not supported."""
940
819
        BzrDir._check_supported(format, _unsupported)
941
820
        return format.open(transport, _found=True)
942
821
 
943
 
    def open_branch(self, name=None, unsupported=False,
944
 
                    ignore_fallbacks=False):
945
 
        """Open the branch object at this BzrDir if one is present.
946
 
 
947
 
        If unsupported is True, then no longer supported branch formats can
948
 
        still be opened.
949
 
 
950
 
        TODO: static convenience version of this?
951
 
        """
952
 
        raise NotImplementedError(self.open_branch)
953
 
 
954
822
    @staticmethod
955
823
    def open_containing(url, possible_transports=None):
956
824
        """Open an existing branch which contains url.
994
862
                raise errors.NotBranchError(path=url)
995
863
            a_transport = new_t
996
864
 
997
 
    def _get_tree_branch(self):
 
865
    def _get_tree_branch(self, name=None):
998
866
        """Return the branch and tree, if any, for this bzrdir.
999
867
 
 
868
        :param name: Name of colocated branch to open.
 
869
 
1000
870
        Return None for tree if not present or inaccessible.
1001
871
        Raise NotBranchError if no branch is present.
1002
872
        :return: (tree, branch)
1005
875
            tree = self.open_workingtree()
1006
876
        except (errors.NoWorkingTree, errors.NotLocalUrl):
1007
877
            tree = None
1008
 
            branch = self.open_branch()
 
878
            branch = self.open_branch(name=name)
1009
879
        else:
1010
 
            branch = tree.branch
 
880
            if name is not None:
 
881
                branch = self.open_branch(name=name)
 
882
            else:
 
883
                branch = tree.branch
1011
884
        return tree, branch
1012
885
 
1013
886
    @classmethod
1061
934
                raise errors.NotBranchError(location)
1062
935
        return tree, branch, branch.repository, relpath
1063
936
 
1064
 
    def open_repository(self, _unsupported=False):
1065
 
        """Open the repository object at this BzrDir if one is present.
1066
 
 
1067
 
        This will not follow the Branch object pointer - it's strictly a direct
1068
 
        open facility. Most client code should use open_branch().repository to
1069
 
        get at a repository.
1070
 
 
1071
 
        :param _unsupported: a private parameter, not part of the api.
1072
 
        TODO: static convenience version of this?
1073
 
        """
1074
 
        raise NotImplementedError(self.open_repository)
1075
 
 
1076
 
    def open_workingtree(self, _unsupported=False,
1077
 
                         recommend_upgrade=True, from_branch=None):
1078
 
        """Open the workingtree object at this BzrDir if one is present.
1079
 
 
1080
 
        :param recommend_upgrade: Optional keyword parameter, when True (the
1081
 
            default), emit through the ui module a recommendation that the user
1082
 
            upgrade the working tree when the workingtree being opened is old
1083
 
            (but still fully supported).
1084
 
        :param from_branch: override bzrdir branch (for lightweight checkouts)
1085
 
        """
1086
 
        raise NotImplementedError(self.open_workingtree)
1087
 
 
1088
 
    def has_branch(self, name=None):
1089
 
        """Tell if this bzrdir contains a branch.
1090
 
 
1091
 
        Note: if you're going to open the branch, you should just go ahead
1092
 
        and try, and not ask permission first.  (This method just opens the
1093
 
        branch and discards it, and that's somewhat expensive.)
1094
 
        """
1095
 
        try:
1096
 
            self.open_branch(name)
1097
 
            return True
1098
 
        except errors.NotBranchError:
1099
 
            return False
1100
 
 
1101
 
    def has_workingtree(self):
1102
 
        """Tell if this bzrdir contains a working tree.
1103
 
 
1104
 
        This will still raise an exception if the bzrdir has a workingtree that
1105
 
        is remote & inaccessible.
1106
 
 
1107
 
        Note: if you're going to open the working tree, you should just go ahead
1108
 
        and try, and not ask permission first.  (This method just opens the
1109
 
        workingtree and discards it, and that's somewhat expensive.)
1110
 
        """
1111
 
        try:
1112
 
            self.open_workingtree(recommend_upgrade=False)
1113
 
            return True
1114
 
        except errors.NoWorkingTree:
1115
 
            return False
1116
 
 
1117
937
    def _cloning_metadir(self):
1118
938
        """Produce a metadir suitable for cloning with.
1119
939
 
1177
997
            format.require_stacking()
1178
998
        return format
1179
999
 
1180
 
    def checkout_metadir(self):
1181
 
        return self.cloning_metadir()
1182
 
 
1183
 
    def sprout(self, url, revision_id=None, force_new_repo=False,
1184
 
               recurse='down', possible_transports=None,
1185
 
               accelerator_tree=None, hardlink=False, stacked=False,
1186
 
               source_branch=None, create_tree_if_local=True):
1187
 
        """Create a copy of this bzrdir prepared for use as a new line of
1188
 
        development.
1189
 
 
1190
 
        If url's last component does not exist, it will be created.
1191
 
 
1192
 
        Attributes related to the identity of the source branch like
1193
 
        branch nickname will be cleaned, a working tree is created
1194
 
        whether one existed before or not; and a local branch is always
1195
 
        created.
1196
 
 
1197
 
        if revision_id is not None, then the clone operation may tune
1198
 
            itself to download less data.
1199
 
        :param accelerator_tree: A tree which can be used for retrieving file
1200
 
            contents more quickly than the revision tree, i.e. a workingtree.
1201
 
            The revision tree will be used for cases where accelerator_tree's
1202
 
            content is different.
1203
 
        :param hardlink: If true, hard-link files from accelerator_tree,
1204
 
            where possible.
1205
 
        :param stacked: If true, create a stacked branch referring to the
1206
 
            location of this control directory.
1207
 
        :param create_tree_if_local: If true, a working-tree will be created
1208
 
            when working locally.
1209
 
        """
1210
 
        target_transport = get_transport(url, possible_transports)
1211
 
        target_transport.ensure_base()
1212
 
        cloning_format = self.cloning_metadir(stacked)
1213
 
        # Create/update the result branch
1214
 
        result = cloning_format.initialize_on_transport(target_transport)
1215
 
        # if a stacked branch wasn't requested, we don't create one
1216
 
        # even if the origin was stacked
1217
 
        stacked_branch_url = None
1218
 
        if source_branch is not None:
1219
 
            if stacked:
1220
 
                stacked_branch_url = self.root_transport.base
1221
 
            source_repository = source_branch.repository
1222
 
        else:
1223
 
            try:
1224
 
                source_branch = self.open_branch()
1225
 
                source_repository = source_branch.repository
1226
 
                if stacked:
1227
 
                    stacked_branch_url = self.root_transport.base
1228
 
            except errors.NotBranchError:
1229
 
                source_branch = None
1230
 
                try:
1231
 
                    source_repository = self.open_repository()
1232
 
                except errors.NoRepositoryPresent:
1233
 
                    source_repository = None
1234
 
        repository_policy = result.determine_repository_policy(
1235
 
            force_new_repo, stacked_branch_url, require_stacking=stacked)
1236
 
        result_repo, is_new_repo = repository_policy.acquire_repository()
1237
 
        if is_new_repo and revision_id is not None and not stacked:
1238
 
            fetch_spec = graph.PendingAncestryResult(
1239
 
                [revision_id], source_repository)
1240
 
        else:
1241
 
            fetch_spec = None
1242
 
        if source_repository is not None:
1243
 
            # Fetch while stacked to prevent unstacked fetch from
1244
 
            # Branch.sprout.
1245
 
            if fetch_spec is None:
1246
 
                result_repo.fetch(source_repository, revision_id=revision_id)
1247
 
            else:
1248
 
                result_repo.fetch(source_repository, fetch_spec=fetch_spec)
1249
 
 
1250
 
        if source_branch is None:
1251
 
            # this is for sprouting a bzrdir without a branch; is that
1252
 
            # actually useful?
1253
 
            # Not especially, but it's part of the contract.
1254
 
            result_branch = result.create_branch()
1255
 
        else:
1256
 
            result_branch = source_branch.sprout(result,
1257
 
                revision_id=revision_id, repository_policy=repository_policy)
1258
 
        mutter("created new branch %r" % (result_branch,))
1259
 
 
1260
 
        # Create/update the result working tree
1261
 
        if (create_tree_if_local and
1262
 
            isinstance(target_transport, local.LocalTransport) and
1263
 
            (result_repo is None or result_repo.make_working_trees())):
1264
 
            wt = result.create_workingtree(accelerator_tree=accelerator_tree,
1265
 
                hardlink=hardlink)
1266
 
            wt.lock_write()
1267
 
            try:
1268
 
                if wt.path2id('') is None:
1269
 
                    try:
1270
 
                        wt.set_root_id(self.open_workingtree.get_root_id())
1271
 
                    except errors.NoWorkingTree:
1272
 
                        pass
1273
 
            finally:
1274
 
                wt.unlock()
1275
 
        else:
1276
 
            wt = None
1277
 
        if recurse == 'down':
1278
 
            if wt is not None:
1279
 
                basis = wt.basis_tree()
1280
 
                basis.lock_read()
1281
 
                subtrees = basis.iter_references()
1282
 
            elif result_branch is not None:
1283
 
                basis = result_branch.basis_tree()
1284
 
                basis.lock_read()
1285
 
                subtrees = basis.iter_references()
1286
 
            elif source_branch is not None:
1287
 
                basis = source_branch.basis_tree()
1288
 
                basis.lock_read()
1289
 
                subtrees = basis.iter_references()
1290
 
            else:
1291
 
                subtrees = []
1292
 
                basis = None
1293
 
            try:
1294
 
                for path, file_id in subtrees:
1295
 
                    target = urlutils.join(url, urlutils.escape(path))
1296
 
                    sublocation = source_branch.reference_parent(file_id, path)
1297
 
                    sublocation.bzrdir.sprout(target,
1298
 
                        basis.get_reference_revision(file_id, path),
1299
 
                        force_new_repo=force_new_repo, recurse=recurse,
1300
 
                        stacked=stacked)
1301
 
            finally:
1302
 
                if basis is not None:
1303
 
                    basis.unlock()
1304
 
        return result
1305
 
 
1306
 
    def push_branch(self, source, revision_id=None, overwrite=False, 
1307
 
        remember=False, create_prefix=False):
1308
 
        """Push the source branch into this BzrDir."""
1309
 
        br_to = None
1310
 
        # If we can open a branch, use its direct repository, otherwise see
1311
 
        # if there is a repository without a branch.
1312
 
        try:
1313
 
            br_to = self.open_branch()
1314
 
        except errors.NotBranchError:
1315
 
            # Didn't find a branch, can we find a repository?
1316
 
            repository_to = self.find_repository()
1317
 
        else:
1318
 
            # Found a branch, so we must have found a repository
1319
 
            repository_to = br_to.repository
1320
 
 
1321
 
        push_result = PushResult()
1322
 
        push_result.source_branch = source
1323
 
        if br_to is None:
1324
 
            # We have a repository but no branch, copy the revisions, and then
1325
 
            # create a branch.
1326
 
            repository_to.fetch(source.repository, revision_id=revision_id)
1327
 
            br_to = source.clone(self, revision_id=revision_id)
1328
 
            if source.get_push_location() is None or remember:
1329
 
                source.set_push_location(br_to.base)
1330
 
            push_result.stacked_on = None
1331
 
            push_result.branch_push_result = None
1332
 
            push_result.old_revno = None
1333
 
            push_result.old_revid = _mod_revision.NULL_REVISION
1334
 
            push_result.target_branch = br_to
1335
 
            push_result.master_branch = None
1336
 
            push_result.workingtree_updated = False
1337
 
        else:
1338
 
            # We have successfully opened the branch, remember if necessary:
1339
 
            if source.get_push_location() is None or remember:
1340
 
                source.set_push_location(br_to.base)
1341
 
            try:
1342
 
                tree_to = self.open_workingtree()
1343
 
            except errors.NotLocalUrl:
1344
 
                push_result.branch_push_result = source.push(br_to, 
1345
 
                    overwrite, stop_revision=revision_id)
1346
 
                push_result.workingtree_updated = False
1347
 
            except errors.NoWorkingTree:
1348
 
                push_result.branch_push_result = source.push(br_to,
1349
 
                    overwrite, stop_revision=revision_id)
1350
 
                push_result.workingtree_updated = None # Not applicable
1351
 
            else:
1352
 
                tree_to.lock_write()
1353
 
                try:
1354
 
                    push_result.branch_push_result = source.push(
1355
 
                        tree_to.branch, overwrite, stop_revision=revision_id)
1356
 
                    tree_to.update()
1357
 
                finally:
1358
 
                    tree_to.unlock()
1359
 
                push_result.workingtree_updated = True
1360
 
            push_result.old_revno = push_result.branch_push_result.old_revno
1361
 
            push_result.old_revid = push_result.branch_push_result.old_revid
1362
 
            push_result.target_branch = \
1363
 
                push_result.branch_push_result.target_branch
1364
 
        return push_result
1365
 
 
1366
1000
 
1367
1001
class BzrDirHooks(hooks.Hooks):
1368
1002
    """Hooks for BzrDir operations."""
1736
1370
    def destroy_workingtree_metadata(self):
1737
1371
        self.transport.delete_tree('checkout')
1738
1372
 
1739
 
    def find_branch_format(self):
 
1373
    def find_branch_format(self, name=None):
1740
1374
        """Find the branch 'format' for this bzrdir.
1741
1375
 
1742
1376
        This might be a synthetic object for e.g. RemoteBranch and SVN.
1743
1377
        """
1744
1378
        from bzrlib.branch import BranchFormat
1745
 
        return BranchFormat.find_format(self)
 
1379
        return BranchFormat.find_format(self, name=name)
1746
1380
 
1747
1381
    def _get_mkdir_mode(self):
1748
1382
        """Figure out the mode to use when creating a bzrdir subdir."""
1750
1384
                                     lockable_files.TransportLock)
1751
1385
        return temp_control._dir_mode
1752
1386
 
1753
 
    def get_branch_reference(self):
 
1387
    def get_branch_reference(self, name=None):
1754
1388
        """See BzrDir.get_branch_reference()."""
1755
1389
        from bzrlib.branch import BranchFormat
1756
 
        format = BranchFormat.find_format(self)
1757
 
        return format.get_reference(self)
 
1390
        format = BranchFormat.find_format(self, name=name)
 
1391
        return format.get_reference(self, name=name)
1758
1392
 
1759
1393
    def get_branch_transport(self, branch_format, name=None):
1760
1394
        """See BzrDir.get_branch_transport()."""
1854
1488
    def open_branch(self, name=None, unsupported=False,
1855
1489
                    ignore_fallbacks=False):
1856
1490
        """See BzrDir.open_branch."""
1857
 
        format = self.find_branch_format()
 
1491
        format = self.find_branch_format(name=name)
1858
1492
        self._check_supported(format, unsupported)
1859
1493
        return format.open(self, name=name,
1860
1494
            _found=True, ignore_fallbacks=ignore_fallbacks)
1948
1582
            format_string = transport.get_bytes(".bzr/branch-format")
1949
1583
        except errors.NoSuchFile:
1950
1584
            raise errors.NotBranchError(path=transport.base)
1951
 
 
1952
1585
        try:
1953
1586
            return klass._formats[format_string]
1954
1587
        except KeyError:
2863
2496
            self.revisions[rev_id] = rev
2864
2497
 
2865
2498
    def _load_old_inventory(self, rev_id):
2866
 
        old_inv_xml = self.branch.repository.inventory_store.get(rev_id).read()
 
2499
        f = self.branch.repository.inventory_store.get(rev_id)
 
2500
        try:
 
2501
            old_inv_xml = f.read()
 
2502
        finally:
 
2503
            f.close()
2867
2504
        inv = xml4.serializer_v4.read_inventory_from_string(old_inv_xml)
2868
2505
        inv.revision_id = rev_id
2869
2506
        rev = self.revisions[rev_id]
2947
2584
                ie.revision = previous_ie.revision
2948
2585
                return
2949
2586
        if ie.has_text():
2950
 
            text = self.branch.repository._text_store.get(ie.text_id)
2951
 
            file_lines = text.readlines()
 
2587
            f = self.branch.repository._text_store.get(ie.text_id)
 
2588
            try:
 
2589
                file_lines = f.readlines()
 
2590
            finally:
 
2591
                f.close()
2952
2592
            w.add_lines(rev_id, previous_revisions, file_lines)
2953
2593
            self.text_count += 1
2954
2594
        else:
3235
2875
        # XXX: It's a bit ugly that the network name is here, because we'd
3236
2876
        # like to believe that format objects are stateless or at least
3237
2877
        # immutable,  However, we do at least avoid mutating the name after
3238
 
        # it's returned.  See <https://bugs.edge.launchpad.net/bzr/+bug/504102>
 
2878
        # it's returned.  See <https://bugs.launchpad.net/bzr/+bug/504102>
3239
2879
        self._network_name = None
3240
2880
 
3241
2881
    def __repr__(self):