~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/config.py

  • Committer: Andrew Bennetts
  • Date: 2008-12-12 01:17:04 UTC
  • mto: This revision was merged to the branch mainline in revision 3899.
  • Revision ID: andrew.bennetts@canonical.com-20081212011704-m5208objzpfsjar7
Remove broken debugging cruft, and some unused imports.

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2005-2010 Canonical Ltd
 
1
# Copyright (C) 2005, 2007, 2008 Canonical Ltd
2
2
#   Authors: Robert Collins <robert.collins@canonical.com>
3
3
#            and others
4
4
#
14
14
#
15
15
# You should have received a copy of the GNU General Public License
16
16
# along with this program; if not, write to the Free Software
17
 
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
 
17
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
18
18
 
19
19
"""Configuration that affects the behaviour of Bazaar.
20
20
 
37
37
[/home/robertc/source]
38
38
recurse=False|True(default)
39
39
email= as above
40
 
check_signatures= as above
 
40
check_signatures= as above 
41
41
create_signatures= as above.
42
42
 
43
43
explanation of options
45
45
editor - this option sets the pop up editor to use during commits.
46
46
email - this option sets the user id bzr will use when committing.
47
47
check_signatures - this option controls whether bzr will require good gpg
48
 
                   signatures, ignore them, or check them if they are
 
48
                   signatures, ignore them, or check them if they are 
49
49
                   present.
50
 
create_signatures - this option controls whether bzr will always create
 
50
create_signatures - this option controls whether bzr will always create 
51
51
                    gpg signatures, never create them, or create them if the
52
52
                    branch is configured to require them.
53
53
log_format - this option sets the default log format.  Possible values are
78
78
    errors,
79
79
    mail_client,
80
80
    osutils,
81
 
    registry,
82
81
    symbol_versioning,
83
82
    trace,
84
83
    ui,
146
145
class Config(object):
147
146
    """A configuration policy - what username, editor, gpg needs etc."""
148
147
 
149
 
    def __init__(self):
150
 
        super(Config, self).__init__()
151
 
 
152
148
    def get_editor(self):
153
149
        """Get the users pop up editor."""
154
150
        raise NotImplementedError
155
151
 
156
 
    def get_change_editor(self, old_tree, new_tree):
157
 
        from bzrlib import diff
158
 
        cmd = self._get_change_editor()
159
 
        if cmd is None:
160
 
            return None
161
 
        return diff.DiffFromTool.from_string(cmd, old_tree, new_tree,
162
 
                                             sys.stdout)
163
 
 
164
 
 
165
152
    def get_mail_client(self):
166
153
        """Get a mail client to use"""
167
154
        selected_client = self.get_user_option('mail_client')
186
173
        """Get a generic option - no special process, no default."""
187
174
        return self._get_user_option(option_name)
188
175
 
189
 
    def get_user_option_as_bool(self, option_name):
190
 
        """Get a generic option as a boolean - no special process, no default.
191
 
 
192
 
        :return None if the option doesn't exist or its value can't be
193
 
            interpreted as a boolean. Returns True or False otherwise.
194
 
        """
195
 
        s = self._get_user_option(option_name)
196
 
        if s is None:
197
 
            # The option doesn't exist
198
 
            return None
199
 
        val = ui.bool_from_string(s)
200
 
        if val is None:
201
 
            # The value can't be interpreted as a boolean
202
 
            trace.warning('Value "%s" is not a boolean for "%s"',
203
 
                          s, option_name)
204
 
        return val
205
 
 
206
 
    def get_user_option_as_list(self, option_name):
207
 
        """Get a generic option as a list - no special process, no default.
208
 
 
209
 
        :return None if the option doesn't exist. Returns the value as a list
210
 
            otherwise.
211
 
        """
212
 
        l = self._get_user_option(option_name)
213
 
        if isinstance(l, (str, unicode)):
214
 
            # A single value, most probably the user forgot the final ','
215
 
            l = [l]
216
 
        return l
217
 
 
218
176
    def gpg_signing_command(self):
219
177
        """What program should be used to sign signatures?"""
220
178
        result = self._gpg_signing_command()
237
195
        """See log_format()."""
238
196
        return None
239
197
 
 
198
    def __init__(self):
 
199
        super(Config, self).__init__()
 
200
 
240
201
    def post_commit(self):
241
202
        """An ordered list of python functions to call.
242
203
 
254
215
 
255
216
    def username(self):
256
217
        """Return email-style username.
257
 
 
 
218
    
258
219
        Something similar to 'Martin Pool <mbp@sourcefrog.net>'
259
 
 
 
220
        
260
221
        $BZR_EMAIL can be set to override this (as well as the
261
222
        deprecated $BZREMAIL), then
262
223
        the concrete policy type is checked, and finally
263
224
        $EMAIL is examined.
264
225
        If none is found, a reasonable default is (hopefully)
265
226
        created.
266
 
 
 
227
    
267
228
        TODO: Check it's reasonably well-formed.
268
229
        """
269
230
        v = os.environ.get('BZR_EMAIL')
333
294
                path = 'bzr'
334
295
            return path
335
296
 
336
 
    def suppress_warning(self, warning):
337
 
        """Should the warning be suppressed or emitted.
338
 
 
339
 
        :param warning: The name of the warning being tested.
340
 
 
341
 
        :returns: True if the warning should be suppressed, False otherwise.
342
 
        """
343
 
        warnings = self.get_user_option_as_list('suppress_warnings')
344
 
        if warnings is None or warning not in warnings:
345
 
            return False
346
 
        else:
347
 
            return True
348
 
 
349
297
 
350
298
class IniBasedConfig(Config):
351
299
    """A configuration policy that draws from ini files."""
352
300
 
353
 
    def __init__(self, get_filename):
354
 
        super(IniBasedConfig, self).__init__()
355
 
        self._get_filename = get_filename
356
 
        self._parser = None
357
 
 
358
301
    def _get_parser(self, file=None):
359
302
        if self._parser is not None:
360
303
            return self._parser
388
331
        """Return the policy for the given (section, option_name) pair."""
389
332
        return POLICY_NONE
390
333
 
391
 
    def _get_change_editor(self):
392
 
        return self.get_user_option('change_editor')
393
 
 
394
334
    def _get_signature_checking(self):
395
335
        """See Config._get_signature_checking."""
396
336
        policy = self._get_user_option('check_signatures')
440
380
        """See Config.log_format."""
441
381
        return self._get_user_option('log_format')
442
382
 
 
383
    def __init__(self, get_filename):
 
384
        super(IniBasedConfig, self).__init__()
 
385
        self._get_filename = get_filename
 
386
        self._parser = None
 
387
        
443
388
    def _post_commit(self):
444
389
        """See Config.post_commit."""
445
390
        return self._get_user_option('post_commit')
468
413
 
469
414
    def _get_alias(self, value):
470
415
        try:
471
 
            return self._get_parser().get_value("ALIASES",
 
416
            return self._get_parser().get_value("ALIASES", 
472
417
                                                value)
473
418
        except KeyError:
474
419
            pass
518
463
        self._write_config_file()
519
464
 
520
465
    def _write_config_file(self):
521
 
        path = self._get_filename()
522
 
        f = open(path, 'wb')
523
 
        osutils.copy_ownership_from_path(path)
 
466
        f = open(self._get_filename(), 'wb')
524
467
        self._get_parser().write(f)
525
468
        f.close()
526
469
 
698
641
 
699
642
    def _get_safe_value(self, option_name):
700
643
        """This variant of get_best_value never returns untrusted values.
701
 
 
 
644
        
702
645
        It does not return values from the branch data, because the branch may
703
646
        not be controlled by the user.
704
647
 
713
656
 
714
657
    def _get_user_id(self):
715
658
        """Return the full user id for the branch.
716
 
 
 
659
    
717
660
        e.g. "John Hacker <jhacker@example.com>"
718
661
        This is looked up in the email controlfile for the branch.
719
662
        """
723
666
                    .rstrip("\r\n"))
724
667
        except errors.NoSuchFile, e:
725
668
            pass
726
 
 
 
669
        
727
670
        return self._get_best_value('_get_user_id')
728
671
 
729
 
    def _get_change_editor(self):
730
 
        return self._get_best_value('_get_change_editor')
731
 
 
732
672
    def _get_signature_checking(self):
733
673
        """See Config._get_signature_checking."""
734
674
        return self._get_best_value('_get_signature_checking')
768
708
                        trace.warning('Value "%s" is masked by "%s" from'
769
709
                                      ' branch.conf', value, mask_value)
770
710
 
 
711
 
771
712
    def _gpg_signing_command(self):
772
713
        """See Config.gpg_signing_command."""
773
714
        return self._get_safe_value('_gpg_signing_command')
774
 
 
 
715
        
775
716
    def __init__(self, branch):
776
717
        super(BranchConfig, self).__init__()
777
718
        self._location_config = None
778
719
        self._branch_data_config = None
779
720
        self._global_config = None
780
721
        self.branch = branch
781
 
        self.option_sources = (self._get_location_config,
 
722
        self.option_sources = (self._get_location_config, 
782
723
                               self._get_branch_data_config,
783
724
                               self._get_global_config)
784
725
 
820
761
            os.mkdir(parent_dir)
821
762
        trace.mutter('creating config directory: %r', path)
822
763
        os.mkdir(path)
823
 
        osutils.copy_ownership_from_path(path)
824
764
 
825
765
 
826
766
def config_dir():
827
767
    """Return per-user configuration directory.
828
768
 
829
769
    By default this is ~/.bazaar/
830
 
 
 
770
    
831
771
    TODO: Global option --config-dir to override this.
832
772
    """
833
773
    base = os.environ.get('BZR_HOME', None)
872
812
    return osutils.pathjoin(config_dir(), 'ignore')
873
813
 
874
814
 
875
 
def crash_dir():
876
 
    """Return the directory name to store crash files.
877
 
 
878
 
    This doesn't implicitly create it.
879
 
 
880
 
    On Windows it's in the config directory; elsewhere it's /var/crash
881
 
    which may be monitored by apport.  It can be overridden by
882
 
    $APPORT_CRASH_DIR.
883
 
    """
884
 
    if sys.platform == 'win32':
885
 
        return osutils.pathjoin(config_dir(), 'Crash')
886
 
    else:
887
 
        # XXX: hardcoded in apport_python_hook.py; therefore here too -- mbp
888
 
        # 2010-01-31
889
 
        return os.environ.get('APPORT_CRASH_DIR', '/var/crash')
890
 
 
891
 
 
892
 
def xdg_cache_dir():
893
 
    # See http://standards.freedesktop.org/basedir-spec/latest/ar01s03.html
894
 
    # Possibly this should be different on Windows?
895
 
    e = os.environ.get('XDG_CACHE_DIR', None)
896
 
    if e:
897
 
        return e
898
 
    else:
899
 
        return os.path.expanduser('~/.cache')
900
 
 
901
 
 
902
815
def _auto_user_id():
903
816
    """Calculate automatic user identification.
904
817
 
984
897
def extract_email_address(e):
985
898
    """Return just the address part of an email string.
986
899
 
987
 
    That is just the user@domain part, nothing else.
 
900
    That is just the user@domain part, nothing else. 
988
901
    This part is required to contain only ascii characters.
989
902
    If it can't be extracted, raises an error.
990
903
 
1003
916
    # XXX: Really needs a better name, as this is not part of the tree! -- mbp 20080507
1004
917
 
1005
918
    def __init__(self, branch):
1006
 
        self._config = branch._get_config()
 
919
        # XXX: Really this should be asking the branch for its configuration
 
920
        # data, rather than relying on a Transport, so that it can work 
 
921
        # more cleanly with a RemoteBranch that has no transport.
 
922
        self._config = TransportConfig(branch._transport, 'branch.conf')
1007
923
        self.branch = branch
1008
924
 
1009
925
    def _get_parser(self, file=None):
1017
933
            return self._config.get_option(name, section, default)
1018
934
        finally:
1019
935
            self.branch.unlock()
 
936
        return result
1020
937
 
1021
938
    def set_option(self, value, name, section=None):
1022
939
        """Set a per-branch configuration option"""
1075
992
        section[option_name] = value
1076
993
        self._save()
1077
994
 
1078
 
    def get_credentials(self, scheme, host, port=None, user=None, path=None, 
1079
 
                        realm=None):
 
995
    def get_credentials(self, scheme, host, port=None, user=None, path=None):
1080
996
        """Returns the matching credentials from authentication.conf file.
1081
997
 
1082
998
        :param scheme: protocol
1088
1004
        :param user: login (optional)
1089
1005
 
1090
1006
        :param path: the absolute path on the server (optional)
1091
 
        
1092
 
        :param realm: the http authentication realm (optional)
1093
1007
 
1094
1008
        :return: A dict containing the matching credentials or None.
1095
1009
           This includes:
1096
1010
           - name: the section name of the credentials in the
1097
1011
             authentication.conf file,
1098
 
           - user: can't be different from the provided user if any,
1099
 
           - scheme: the server protocol,
1100
 
           - host: the server address,
1101
 
           - port: the server port (can be None),
1102
 
           - path: the absolute server path (can be None),
1103
 
           - realm: the http specific authentication realm (can be None),
 
1012
           - user: can't de different from the provided user if any,
1104
1013
           - password: the decoded password, could be None if the credential
1105
1014
             defines only the user
1106
1015
           - verify_certificates: https specific, True if the server
1147
1056
            if a_user is None:
1148
1057
                # Can't find a user
1149
1058
                continue
1150
 
            # Prepare a credentials dictionary with additional keys
1151
 
            # for the credential providers
1152
1059
            credentials = dict(name=auth_def_name,
1153
1060
                               user=a_user,
1154
 
                               scheme=a_scheme,
1155
 
                               host=host,
1156
 
                               port=port,
1157
 
                               path=path,
1158
 
                               realm=realm,
1159
1061
                               password=auth_def.get('password', None),
1160
1062
                               verify_certificates=a_verify_certificates)
1161
 
            # Decode the password in the credentials (or get one)
1162
1063
            self.decode_password(credentials,
1163
1064
                                 auth_def.get('password_encoding', None))
1164
1065
            if 'auth' in debug.debug_flags:
1165
1066
                trace.mutter("Using authentication section: %r", auth_def_name)
1166
1067
            break
1167
1068
 
1168
 
        if credentials is None:
1169
 
            # No credentials were found in authentication.conf, try the fallback
1170
 
            # credentials stores.
1171
 
            credentials = credential_store_registry.get_fallback_credentials(
1172
 
                scheme, host, port, user, path, realm)
1173
 
 
1174
1069
        return credentials
1175
1070
 
1176
1071
    def set_credentials(self, name, host, user, scheme=None, password=None,
1177
 
                        port=None, path=None, verify_certificates=None,
1178
 
                        realm=None):
 
1072
                        port=None, path=None, verify_certificates=None):
1179
1073
        """Set authentication credentials for a host.
1180
1074
 
1181
1075
        Any existing credentials with matching scheme, host, port and path
1192
1086
            apply to.
1193
1087
        :param verify_certificates: On https, verify server certificates if
1194
1088
            True.
1195
 
        :param realm: The http authentication realm (optional).
1196
1089
        """
1197
1090
        values = {'host': host, 'user': user}
1198
1091
        if password is not None:
1205
1098
            values['path'] = path
1206
1099
        if verify_certificates is not None:
1207
1100
            values['verify_certificates'] = str(verify_certificates)
1208
 
        if realm is not None:
1209
 
            values['realm'] = realm
1210
1101
        config = self._get_config()
1211
1102
        for_deletion = []
1212
1103
        for section, existing_values in config.items():
1213
 
            for key in ('scheme', 'host', 'port', 'path', 'realm'):
 
1104
            for key in ('scheme', 'host', 'port', 'path'):
1214
1105
                if existing_values.get(key) != values.get(key):
1215
1106
                    break
1216
1107
            else:
1218
1109
        config.update({name: values})
1219
1110
        self._save()
1220
1111
 
1221
 
    def get_user(self, scheme, host, port=None, realm=None, path=None,
1222
 
                 prompt=None, ask=False, default=None):
 
1112
    def get_user(self, scheme, host, port=None,
 
1113
                 realm=None, path=None, prompt=None):
1223
1114
        """Get a user from authentication file.
1224
1115
 
1225
1116
        :param scheme: protocol
1232
1123
 
1233
1124
        :param path: the absolute path on the server (optional)
1234
1125
 
1235
 
        :param ask: Ask the user if there is no explicitly configured username 
1236
 
                    (optional)
1237
 
 
1238
 
        :param default: The username returned if none is defined (optional).
1239
 
 
1240
1126
        :return: The found user.
1241
1127
        """
1242
1128
        credentials = self.get_credentials(scheme, host, port, user=None,
1243
 
                                           path=path, realm=realm)
 
1129
                                           path=path)
1244
1130
        if credentials is not None:
1245
1131
            user = credentials['user']
1246
1132
        else:
1247
1133
            user = None
1248
 
        if user is None:
1249
 
            if ask:
1250
 
                if prompt is None:
1251
 
                    # Create a default prompt suitable for most cases
1252
 
                    prompt = scheme.upper() + ' %(host)s username'
1253
 
                # Special handling for optional fields in the prompt
1254
 
                if port is not None:
1255
 
                    prompt_host = '%s:%d' % (host, port)
1256
 
                else:
1257
 
                    prompt_host = host
1258
 
                user = ui.ui_factory.get_username(prompt, host=prompt_host)
1259
 
            else:
1260
 
                user = default
1261
1134
        return user
1262
1135
 
1263
1136
    def get_password(self, scheme, host, user, port=None,
1278
1151
 
1279
1152
        :return: The found password or the one entered by the user.
1280
1153
        """
1281
 
        credentials = self.get_credentials(scheme, host, port, user, path,
1282
 
                                           realm)
 
1154
        credentials = self.get_credentials(scheme, host, port, user, path)
1283
1155
        if credentials is not None:
1284
1156
            password = credentials['password']
1285
1157
            if password is not None and scheme is 'ssh':
1304
1176
        return password
1305
1177
 
1306
1178
    def decode_password(self, credentials, encoding):
1307
 
        try:
1308
 
            cs = credential_store_registry.get_credential_store(encoding)
1309
 
        except KeyError:
1310
 
            raise ValueError('%r is not a known password_encoding' % encoding)
1311
 
        credentials['password'] = cs.decode_password(credentials)
1312
 
        return credentials
1313
 
 
1314
 
 
1315
 
class CredentialStoreRegistry(registry.Registry):
1316
 
    """A class that registers credential stores.
1317
 
 
1318
 
    A credential store provides access to credentials via the password_encoding
1319
 
    field in authentication.conf sections.
1320
 
 
1321
 
    Except for stores provided by bzr itself, most stores are expected to be
1322
 
    provided by plugins that will therefore use
1323
 
    register_lazy(password_encoding, module_name, member_name, help=help,
1324
 
    fallback=fallback) to install themselves.
1325
 
 
1326
 
    A fallback credential store is one that is queried if no credentials can be
1327
 
    found via authentication.conf.
1328
 
    """
1329
 
 
1330
 
    def get_credential_store(self, encoding=None):
1331
 
        cs = self.get(encoding)
1332
 
        if callable(cs):
1333
 
            cs = cs()
1334
 
        return cs
1335
 
 
1336
 
    def is_fallback(self, name):
1337
 
        """Check if the named credentials store should be used as fallback."""
1338
 
        return self.get_info(name)
1339
 
 
1340
 
    def get_fallback_credentials(self, scheme, host, port=None, user=None,
1341
 
                                 path=None, realm=None):
1342
 
        """Request credentials from all fallback credentials stores.
1343
 
 
1344
 
        The first credentials store that can provide credentials wins.
1345
 
        """
1346
 
        credentials = None
1347
 
        for name in self.keys():
1348
 
            if not self.is_fallback(name):
1349
 
                continue
1350
 
            cs = self.get_credential_store(name)
1351
 
            credentials = cs.get_credentials(scheme, host, port, user,
1352
 
                                             path, realm)
1353
 
            if credentials is not None:
1354
 
                # We found some credentials
1355
 
                break
1356
 
        return credentials
1357
 
 
1358
 
    def register(self, key, obj, help=None, override_existing=False,
1359
 
                 fallback=False):
1360
 
        """Register a new object to a name.
1361
 
 
1362
 
        :param key: This is the key to use to request the object later.
1363
 
        :param obj: The object to register.
1364
 
        :param help: Help text for this entry. This may be a string or
1365
 
                a callable. If it is a callable, it should take two
1366
 
                parameters (registry, key): this registry and the key that
1367
 
                the help was registered under.
1368
 
        :param override_existing: Raise KeyErorr if False and something has
1369
 
                already been registered for that key. If True, ignore if there
1370
 
                is an existing key (always register the new value).
1371
 
        :param fallback: Whether this credential store should be 
1372
 
                used as fallback.
1373
 
        """
1374
 
        return super(CredentialStoreRegistry,
1375
 
                     self).register(key, obj, help, info=fallback,
1376
 
                                    override_existing=override_existing)
1377
 
 
1378
 
    def register_lazy(self, key, module_name, member_name,
1379
 
                      help=None, override_existing=False,
1380
 
                      fallback=False):
1381
 
        """Register a new credential store to be loaded on request.
1382
 
 
1383
 
        :param module_name: The python path to the module. Such as 'os.path'.
1384
 
        :param member_name: The member of the module to return.  If empty or
1385
 
                None, get() will return the module itself.
1386
 
        :param help: Help text for this entry. This may be a string or
1387
 
                a callable.
1388
 
        :param override_existing: If True, replace the existing object
1389
 
                with the new one. If False, if there is already something
1390
 
                registered with the same key, raise a KeyError
1391
 
        :param fallback: Whether this credential store should be 
1392
 
                used as fallback.
1393
 
        """
1394
 
        return super(CredentialStoreRegistry, self).register_lazy(
1395
 
            key, module_name, member_name, help,
1396
 
            info=fallback, override_existing=override_existing)
1397
 
 
1398
 
 
1399
 
credential_store_registry = CredentialStoreRegistry()
1400
 
 
1401
 
 
1402
 
class CredentialStore(object):
1403
 
    """An abstract class to implement storage for credentials"""
1404
 
 
1405
 
    def decode_password(self, credentials):
1406
 
        """Returns a clear text password for the provided credentials."""
1407
 
        raise NotImplementedError(self.decode_password)
1408
 
 
1409
 
    def get_credentials(self, scheme, host, port=None, user=None, path=None,
1410
 
                        realm=None):
1411
 
        """Return the matching credentials from this credential store.
1412
 
 
1413
 
        This method is only called on fallback credential stores.
1414
 
        """
1415
 
        raise NotImplementedError(self.get_credentials)
1416
 
 
1417
 
 
1418
 
 
1419
 
class PlainTextCredentialStore(CredentialStore):
1420
 
    __doc__ = """Plain text credential store for the authentication.conf file"""
1421
 
 
1422
 
    def decode_password(self, credentials):
1423
 
        """See CredentialStore.decode_password."""
1424
 
        return credentials['password']
1425
 
 
1426
 
 
1427
 
credential_store_registry.register('plain', PlainTextCredentialStore,
1428
 
                                   help=PlainTextCredentialStore.__doc__)
1429
 
credential_store_registry.default_key = 'plain'
 
1179
        return credentials
1430
1180
 
1431
1181
 
1432
1182
class BzrDirConfig(object):
1433
1183
 
1434
 
    def __init__(self, bzrdir):
1435
 
        self._bzrdir = bzrdir
1436
 
        self._config = bzrdir._get_config()
 
1184
    def __init__(self, transport):
 
1185
        self._config = TransportConfig(transport, 'control.conf')
1437
1186
 
1438
1187
    def set_default_stack_on(self, value):
1439
1188
        """Set the default stacking location.
1443
1192
        This policy affects all branches contained by this bzrdir, except for
1444
1193
        those under repositories.
1445
1194
        """
1446
 
        if self._config is None:
1447
 
            raise errors.BzrError("Cannot set configuration in %s" % self._bzrdir)
1448
1195
        if value is None:
1449
1196
            self._config.set_option('', 'default_stack_on')
1450
1197
        else:
1458
1205
        This policy affects all branches contained by this bzrdir, except for
1459
1206
        those under repositories.
1460
1207
        """
1461
 
        if self._config is None:
1462
 
            return None
1463
1208
        value = self._config.get_option('default_stack_on')
1464
1209
        if value == '':
1465
1210
            value = None
1510
1255
            configobj.setdefault(section, {})[name] = value
1511
1256
        self._set_configobj(configobj)
1512
1257
 
1513
 
    def _get_config_file(self):
 
1258
    def _get_configobj(self):
1514
1259
        try:
1515
 
            return StringIO(self._transport.get_bytes(self._filename))
 
1260
            return ConfigObj(self._transport.get(self._filename),
 
1261
                             encoding='utf-8')
1516
1262
        except errors.NoSuchFile:
1517
 
            return StringIO()
1518
 
 
1519
 
    def _get_configobj(self):
1520
 
        return ConfigObj(self._get_config_file(), encoding='utf-8')
 
1263
            return ConfigObj(encoding='utf-8')
1521
1264
 
1522
1265
    def _set_configobj(self, configobj):
1523
1266
        out_file = StringIO()