~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/config.py

  • Committer: Canonical.com Patch Queue Manager
  • Date: 2006-06-16 14:33:42 UTC
  • mfrom: (1770.2.1 config)
  • Revision ID: pqm@pqm.ubuntu.com-20060616143342-8f7f4a4f77c1e4c8
Use create_signature for signing policy, deprecate check_signatures for this

Show diffs side-by-side

added added

removed removed

Lines of Context:
 
1
# Copyright (C) 2005 by Canonical Ltd
 
2
#   Authors: Robert Collins <robert.collins@canonical.com>
 
3
#
 
4
# This program is free software; you can redistribute it and/or modify
 
5
# it under the terms of the GNU General Public License as published by
 
6
# the Free Software Foundation; either version 2 of the License, or
 
7
# (at your option) any later version.
 
8
#
 
9
# This program is distributed in the hope that it will be useful,
 
10
# but WITHOUT ANY WARRANTY; without even the implied warranty of
 
11
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 
12
# GNU General Public License for more details.
 
13
#
 
14
# You should have received a copy of the GNU General Public License
 
15
# along with this program; if not, write to the Free Software
 
16
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
 
17
 
 
18
"""Configuration that affects the behaviour of Bazaar.
 
19
 
 
20
Currently this configuration resides in ~/.bazaar/bazaar.conf
 
21
and ~/.bazaar/branches.conf, which is written to by bzr.
 
22
 
 
23
In bazaar.conf the following options may be set:
 
24
[DEFAULT]
 
25
editor=name-of-program
 
26
email=Your Name <your@email.address>
 
27
check_signatures=require|ignore|check-available(default)
 
28
create_signatures=always|never|when-required(default)
 
29
gpg_signing_command=name-of-program
 
30
log_format=name-of-format
 
31
 
 
32
in branches.conf, you specify the url of a branch and options for it.
 
33
Wildcards may be used - * and ? as normal in shell completion. Options
 
34
set in both bazaar.conf and branches.conf are overridden by the branches.conf
 
35
setting.
 
36
[/home/robertc/source]
 
37
recurse=False|True(default)
 
38
email= as above
 
39
check_signatures= as above 
 
40
create_signatures= as above.
 
41
 
 
42
explanation of options
 
43
----------------------
 
44
editor - this option sets the pop up editor to use during commits.
 
45
email - this option sets the user id bzr will use when committing.
 
46
check_signatures - this option controls whether bzr will require good gpg
 
47
                   signatures, ignore them, or check them if they are 
 
48
                   present.
 
49
create_signatures - this option controls whether bzr will always create 
 
50
                    gpg signatures, never create them, or create them if the
 
51
                    branch is configured to require them.
 
52
log_format - This options set the default log format.  Options are long, 
 
53
             short, line, or a plugin can register new formats
 
54
 
 
55
In bazaar.conf you can also define aliases in the ALIASES sections, example
 
56
 
 
57
[ALIASES]
 
58
lastlog=log --line -r-10..-1
 
59
ll=log --line -r-10..-1
 
60
h=help
 
61
up=pull
 
62
"""
 
63
 
 
64
 
 
65
import errno
 
66
from fnmatch import fnmatch
 
67
import os
 
68
import re
 
69
import sys
 
70
from StringIO import StringIO
 
71
 
 
72
import bzrlib
 
73
import bzrlib.errors as errors
 
74
from bzrlib.osutils import pathjoin
 
75
from bzrlib.trace import mutter, warning
 
76
import bzrlib.util.configobj.configobj as configobj
 
77
 
 
78
 
 
79
CHECK_IF_POSSIBLE=0
 
80
CHECK_ALWAYS=1
 
81
CHECK_NEVER=2
 
82
 
 
83
 
 
84
SIGN_WHEN_REQUIRED=0
 
85
SIGN_ALWAYS=1
 
86
SIGN_NEVER=2
 
87
 
 
88
 
 
89
class ConfigObj(configobj.ConfigObj):
 
90
 
 
91
    def get_bool(self, section, key):
 
92
        return self[section].as_bool(key)
 
93
 
 
94
    def get_value(self, section, name):
 
95
        # Try [] for the old DEFAULT section.
 
96
        if section == "DEFAULT":
 
97
            try:
 
98
                return self[name]
 
99
            except KeyError:
 
100
                pass
 
101
        return self[section][name]
 
102
 
 
103
 
 
104
class Config(object):
 
105
    """A configuration policy - what username, editor, gpg needs etc."""
 
106
 
 
107
    def get_editor(self):
 
108
        """Get the users pop up editor."""
 
109
        raise NotImplementedError
 
110
 
 
111
    def _get_signature_checking(self):
 
112
        """Template method to override signature checking policy."""
 
113
 
 
114
    def _get_signing_policy(self):
 
115
        """Template method to override signature creation policy."""
 
116
 
 
117
    def _get_user_option(self, option_name):
 
118
        """Template method to provide a user option."""
 
119
        return None
 
120
 
 
121
    def get_user_option(self, option_name):
 
122
        """Get a generic option - no special process, no default."""
 
123
        return self._get_user_option(option_name)
 
124
 
 
125
    def gpg_signing_command(self):
 
126
        """What program should be used to sign signatures?"""
 
127
        result = self._gpg_signing_command()
 
128
        if result is None:
 
129
            result = "gpg"
 
130
        return result
 
131
 
 
132
    def _gpg_signing_command(self):
 
133
        """See gpg_signing_command()."""
 
134
        return None
 
135
 
 
136
    def log_format(self):
 
137
        """What log format should be used"""
 
138
        result = self._log_format()
 
139
        if result is None:
 
140
            result = "long"
 
141
        return result
 
142
 
 
143
    def _log_format(self):
 
144
        """See log_format()."""
 
145
        return None
 
146
 
 
147
    def __init__(self):
 
148
        super(Config, self).__init__()
 
149
 
 
150
    def post_commit(self):
 
151
        """An ordered list of python functions to call.
 
152
 
 
153
        Each function takes branch, rev_id as parameters.
 
154
        """
 
155
        return self._post_commit()
 
156
 
 
157
    def _post_commit(self):
 
158
        """See Config.post_commit."""
 
159
        return None
 
160
 
 
161
    def user_email(self):
 
162
        """Return just the email component of a username."""
 
163
        return extract_email_address(self.username())
 
164
 
 
165
    def username(self):
 
166
        """Return email-style username.
 
167
    
 
168
        Something similar to 'Martin Pool <mbp@sourcefrog.net>'
 
169
        
 
170
        $BZREMAIL can be set to override this, then
 
171
        the concrete policy type is checked, and finally
 
172
        $EMAIL is examined.
 
173
        If none is found, a reasonable default is (hopefully)
 
174
        created.
 
175
    
 
176
        TODO: Check it's reasonably well-formed.
 
177
        """
 
178
        v = os.environ.get('BZREMAIL')
 
179
        if v:
 
180
            return v.decode(bzrlib.user_encoding)
 
181
    
 
182
        v = self._get_user_id()
 
183
        if v:
 
184
            return v
 
185
        
 
186
        v = os.environ.get('EMAIL')
 
187
        if v:
 
188
            return v.decode(bzrlib.user_encoding)
 
189
 
 
190
        name, email = _auto_user_id()
 
191
        if name:
 
192
            return '%s <%s>' % (name, email)
 
193
        else:
 
194
            return email
 
195
 
 
196
    def signature_checking(self):
 
197
        """What is the current policy for signature checking?."""
 
198
        policy = self._get_signature_checking()
 
199
        if policy is not None:
 
200
            return policy
 
201
        return CHECK_IF_POSSIBLE
 
202
 
 
203
    def signing_policy(self):
 
204
        """What is the current policy for signature checking?."""
 
205
        policy = self._get_signing_policy()
 
206
        if policy is not None:
 
207
            return policy
 
208
        return SIGN_WHEN_REQUIRED
 
209
 
 
210
    def signature_needed(self):
 
211
        """Is a signature needed when committing ?."""
 
212
        policy = self._get_signing_policy()
 
213
        if policy is None:
 
214
            policy = self._get_signature_checking()
 
215
            if policy is not None:
 
216
                warning("Please use create_signatures, not check_signatures "
 
217
                        "to set signing policy.")
 
218
            if policy == CHECK_ALWAYS:
 
219
                return True
 
220
        elif policy == SIGN_ALWAYS:
 
221
            return True
 
222
        return False
 
223
 
 
224
    def get_alias(self, value):
 
225
        return self._get_alias(value)
 
226
 
 
227
    def _get_alias(self, value):
 
228
        pass
 
229
 
 
230
 
 
231
class IniBasedConfig(Config):
 
232
    """A configuration policy that draws from ini files."""
 
233
 
 
234
    def _get_parser(self, file=None):
 
235
        if self._parser is not None:
 
236
            return self._parser
 
237
        if file is None:
 
238
            input = self._get_filename()
 
239
        else:
 
240
            input = file
 
241
        try:
 
242
            self._parser = ConfigObj(input, encoding='utf-8')
 
243
        except configobj.ConfigObjError, e:
 
244
            raise errors.ParseConfigError(e.errors, e.config.filename)
 
245
        return self._parser
 
246
 
 
247
    def _get_section(self):
 
248
        """Override this to define the section used by the config."""
 
249
        return "DEFAULT"
 
250
 
 
251
    def _get_signature_checking(self):
 
252
        """See Config._get_signature_checking."""
 
253
        policy = self._get_user_option('check_signatures')
 
254
        if policy:
 
255
            return self._string_to_signature_policy(policy)
 
256
 
 
257
    def _get_signing_policy(self):
 
258
        """See Config._get_signature_checking."""
 
259
        policy = self._get_user_option('create_signatures')
 
260
        if policy:
 
261
            return self._string_to_signing_policy(policy)
 
262
 
 
263
    def _get_user_id(self):
 
264
        """Get the user id from the 'email' key in the current section."""
 
265
        return self._get_user_option('email')
 
266
 
 
267
    def _get_user_option(self, option_name):
 
268
        """See Config._get_user_option."""
 
269
        try:
 
270
            return self._get_parser().get_value(self._get_section(),
 
271
                                                option_name)
 
272
        except KeyError:
 
273
            pass
 
274
 
 
275
    def _gpg_signing_command(self):
 
276
        """See Config.gpg_signing_command."""
 
277
        return self._get_user_option('gpg_signing_command')
 
278
 
 
279
    def _log_format(self):
 
280
        """See Config.log_format."""
 
281
        return self._get_user_option('log_format')
 
282
 
 
283
    def __init__(self, get_filename):
 
284
        super(IniBasedConfig, self).__init__()
 
285
        self._get_filename = get_filename
 
286
        self._parser = None
 
287
        
 
288
    def _post_commit(self):
 
289
        """See Config.post_commit."""
 
290
        return self._get_user_option('post_commit')
 
291
 
 
292
    def _string_to_signature_policy(self, signature_string):
 
293
        """Convert a string to a signing policy."""
 
294
        if signature_string.lower() == 'check-available':
 
295
            return CHECK_IF_POSSIBLE
 
296
        if signature_string.lower() == 'ignore':
 
297
            return CHECK_NEVER
 
298
        if signature_string.lower() == 'require':
 
299
            return CHECK_ALWAYS
 
300
        raise errors.BzrError("Invalid signatures policy '%s'"
 
301
                              % signature_string)
 
302
 
 
303
    def _string_to_signing_policy(self, signature_string):
 
304
        """Convert a string to a signing policy."""
 
305
        if signature_string.lower() == 'when-required':
 
306
            return SIGN_WHEN_REQUIRED
 
307
        if signature_string.lower() == 'never':
 
308
            return SIGN_NEVER
 
309
        if signature_string.lower() == 'always':
 
310
            return SIGN_ALWAYS
 
311
        raise errors.BzrError("Invalid signing policy '%s'"
 
312
                              % signature_string)
 
313
 
 
314
    def _get_alias(self, value):
 
315
        try:
 
316
            return self._get_parser().get_value("ALIASES", 
 
317
                                                value)
 
318
        except KeyError:
 
319
            pass
 
320
 
 
321
 
 
322
class GlobalConfig(IniBasedConfig):
 
323
    """The configuration that should be used for a specific location."""
 
324
 
 
325
    def get_editor(self):
 
326
        return self._get_user_option('editor')
 
327
 
 
328
    def __init__(self):
 
329
        super(GlobalConfig, self).__init__(config_filename)
 
330
 
 
331
 
 
332
class LocationConfig(IniBasedConfig):
 
333
    """A configuration object that gives the policy for a location."""
 
334
 
 
335
    def __init__(self, location):
 
336
        super(LocationConfig, self).__init__(branches_config_filename)
 
337
        self._global_config = None
 
338
        self.location = location
 
339
 
 
340
    def _get_global_config(self):
 
341
        if self._global_config is None:
 
342
            self._global_config = GlobalConfig()
 
343
        return self._global_config
 
344
 
 
345
    def _get_section(self):
 
346
        """Get the section we should look in for config items.
 
347
 
 
348
        Returns None if none exists. 
 
349
        TODO: perhaps return a NullSection that thunks through to the 
 
350
              global config.
 
351
        """
 
352
        sections = self._get_parser()
 
353
        location_names = self.location.split('/')
 
354
        if self.location.endswith('/'):
 
355
            del location_names[-1]
 
356
        matches=[]
 
357
        for section in sections:
 
358
            section_names = section.split('/')
 
359
            if section.endswith('/'):
 
360
                del section_names[-1]
 
361
            names = zip(location_names, section_names)
 
362
            matched = True
 
363
            for name in names:
 
364
                if not fnmatch(name[0], name[1]):
 
365
                    matched = False
 
366
                    break
 
367
            if not matched:
 
368
                continue
 
369
            # so, for the common prefix they matched.
 
370
            # if section is longer, no match.
 
371
            if len(section_names) > len(location_names):
 
372
                continue
 
373
            # if path is longer, and recurse is not true, no match
 
374
            if len(section_names) < len(location_names):
 
375
                try:
 
376
                    if not self._get_parser()[section].as_bool('recurse'):
 
377
                        continue
 
378
                except KeyError:
 
379
                    pass
 
380
            matches.append((len(section_names), section))
 
381
        if not len(matches):
 
382
            return None
 
383
        matches.sort(reverse=True)
 
384
        return matches[0][1]
 
385
 
 
386
    def _gpg_signing_command(self):
 
387
        """See Config.gpg_signing_command."""
 
388
        command = super(LocationConfig, self)._gpg_signing_command()
 
389
        if command is not None:
 
390
            return command
 
391
        return self._get_global_config()._gpg_signing_command()
 
392
 
 
393
    def _log_format(self):
 
394
        """See Config.log_format."""
 
395
        command = super(LocationConfig, self)._log_format()
 
396
        if command is not None:
 
397
            return command
 
398
        return self._get_global_config()._log_format()
 
399
 
 
400
    def _get_user_id(self):
 
401
        user_id = super(LocationConfig, self)._get_user_id()
 
402
        if user_id is not None:
 
403
            return user_id
 
404
        return self._get_global_config()._get_user_id()
 
405
 
 
406
    def _get_user_option(self, option_name):
 
407
        """See Config._get_user_option."""
 
408
        option_value = super(LocationConfig, 
 
409
                             self)._get_user_option(option_name)
 
410
        if option_value is not None:
 
411
            return option_value
 
412
        return self._get_global_config()._get_user_option(option_name)
 
413
 
 
414
    def _get_signature_checking(self):
 
415
        """See Config._get_signature_checking."""
 
416
        check = super(LocationConfig, self)._get_signature_checking()
 
417
        if check is not None:
 
418
            return check
 
419
        return self._get_global_config()._get_signature_checking()
 
420
 
 
421
    def _get_signing_policy(self):
 
422
        """See Config._get_signing_policy."""
 
423
        sign = super(LocationConfig, self)._get_signing_policy()
 
424
        if sign is not None:
 
425
            return sign
 
426
        return self._get_global_config()._get_signing_policy()
 
427
 
 
428
    def _post_commit(self):
 
429
        """See Config.post_commit."""
 
430
        hook = self._get_user_option('post_commit')
 
431
        if hook is not None:
 
432
            return hook
 
433
        return self._get_global_config()._post_commit()
 
434
 
 
435
    def set_user_option(self, option, value):
 
436
        """Save option and its value in the configuration."""
 
437
        # FIXME: RBC 20051029 This should refresh the parser and also take a
 
438
        # file lock on branches.conf.
 
439
        conf_dir = os.path.dirname(self._get_filename())
 
440
        ensure_config_dir_exists(conf_dir)
 
441
        location = self.location
 
442
        if location.endswith('/'):
 
443
            location = location[:-1]
 
444
        if (not location in self._get_parser() and
 
445
            not location + '/' in self._get_parser()):
 
446
            self._get_parser()[location]={}
 
447
        elif location + '/' in self._get_parser():
 
448
            location = location + '/'
 
449
        self._get_parser()[location][option]=value
 
450
        self._get_parser().write(file(self._get_filename(), 'wb'))
 
451
 
 
452
 
 
453
class BranchConfig(Config):
 
454
    """A configuration object giving the policy for a branch."""
 
455
 
 
456
    def _get_location_config(self):
 
457
        if self._location_config is None:
 
458
            self._location_config = LocationConfig(self.branch.base)
 
459
        return self._location_config
 
460
 
 
461
    def _get_user_id(self):
 
462
        """Return the full user id for the branch.
 
463
    
 
464
        e.g. "John Hacker <jhacker@foo.org>"
 
465
        This is looked up in the email controlfile for the branch.
 
466
        """
 
467
        try:
 
468
            return (self.branch.control_files.get_utf8("email") 
 
469
                    .read()
 
470
                    .decode(bzrlib.user_encoding)
 
471
                    .rstrip("\r\n"))
 
472
        except errors.NoSuchFile, e:
 
473
            pass
 
474
        
 
475
        return self._get_location_config()._get_user_id()
 
476
 
 
477
    def _get_signature_checking(self):
 
478
        """See Config._get_signature_checking."""
 
479
        return self._get_location_config()._get_signature_checking()
 
480
 
 
481
    def _get_signing_policy(self):
 
482
        """See Config._get_signing_policy."""
 
483
        return self._get_location_config()._get_signing_policy()
 
484
 
 
485
    def _get_user_option(self, option_name):
 
486
        """See Config._get_user_option."""
 
487
        return self._get_location_config()._get_user_option(option_name)
 
488
 
 
489
    def _gpg_signing_command(self):
 
490
        """See Config.gpg_signing_command."""
 
491
        return self._get_location_config()._gpg_signing_command()
 
492
        
 
493
    def __init__(self, branch):
 
494
        super(BranchConfig, self).__init__()
 
495
        self._location_config = None
 
496
        self.branch = branch
 
497
 
 
498
    def _post_commit(self):
 
499
        """See Config.post_commit."""
 
500
        return self._get_location_config()._post_commit()
 
501
 
 
502
    def _log_format(self):
 
503
        """See Config.log_format."""
 
504
        return self._get_location_config()._log_format()
 
505
 
 
506
 
 
507
def ensure_config_dir_exists(path=None):
 
508
    """Make sure a configuration directory exists.
 
509
    This makes sure that the directory exists.
 
510
    On windows, since configuration directories are 2 levels deep,
 
511
    it makes sure both the directory and the parent directory exists.
 
512
    """
 
513
    if path is None:
 
514
        path = config_dir()
 
515
    if not os.path.isdir(path):
 
516
        if sys.platform == 'win32':
 
517
            parent_dir = os.path.dirname(path)
 
518
            if not os.path.isdir(parent_dir):
 
519
                mutter('creating config parent directory: %r', parent_dir)
 
520
            os.mkdir(parent_dir)
 
521
        mutter('creating config directory: %r', path)
 
522
        os.mkdir(path)
 
523
 
 
524
 
 
525
def config_dir():
 
526
    """Return per-user configuration directory.
 
527
 
 
528
    By default this is ~/.bazaar/
 
529
    
 
530
    TODO: Global option --config-dir to override this.
 
531
    """
 
532
    base = os.environ.get('BZR_HOME', None)
 
533
    if sys.platform == 'win32':
 
534
        if base is None:
 
535
            base = os.environ.get('APPDATA', None)
 
536
        if base is None:
 
537
            base = os.environ.get('HOME', None)
 
538
        if base is None:
 
539
            raise BzrError('You must have one of BZR_HOME, APPDATA, or HOME set')
 
540
        return pathjoin(base, 'bazaar', '2.0')
 
541
    else:
 
542
        # cygwin, linux, and darwin all have a $HOME directory
 
543
        if base is None:
 
544
            base = os.path.expanduser("~")
 
545
        return pathjoin(base, ".bazaar")
 
546
 
 
547
 
 
548
def config_filename():
 
549
    """Return per-user configuration ini file filename."""
 
550
    return pathjoin(config_dir(), 'bazaar.conf')
 
551
 
 
552
 
 
553
def branches_config_filename():
 
554
    """Return per-user configuration ini file filename."""
 
555
    return pathjoin(config_dir(), 'branches.conf')
 
556
 
 
557
 
 
558
def _auto_user_id():
 
559
    """Calculate automatic user identification.
 
560
 
 
561
    Returns (realname, email).
 
562
 
 
563
    Only used when none is set in the environment or the id file.
 
564
 
 
565
    This previously used the FQDN as the default domain, but that can
 
566
    be very slow on machines where DNS is broken.  So now we simply
 
567
    use the hostname.
 
568
    """
 
569
    import socket
 
570
 
 
571
    # XXX: Any good way to get real user name on win32?
 
572
 
 
573
    try:
 
574
        import pwd
 
575
        uid = os.getuid()
 
576
        w = pwd.getpwuid(uid)
 
577
 
 
578
        try:
 
579
            gecos = w.pw_gecos.decode(bzrlib.user_encoding)
 
580
            username = w.pw_name.decode(bzrlib.user_encoding)
 
581
        except UnicodeDecodeError:
 
582
            # We're using pwd, therefore we're on Unix, so /etc/passwd is ok.
 
583
            raise errors.BzrError("Can't decode username in " \
 
584
                    "/etc/passwd as %s." % bzrlib.user_encoding)
 
585
 
 
586
        comma = gecos.find(',')
 
587
        if comma == -1:
 
588
            realname = gecos
 
589
        else:
 
590
            realname = gecos[:comma]
 
591
        if not realname:
 
592
            realname = username
 
593
 
 
594
    except ImportError:
 
595
        import getpass
 
596
        try:
 
597
            realname = username = getpass.getuser().decode(bzrlib.user_encoding)
 
598
        except UnicodeDecodeError:
 
599
            raise errors.BzrError("Can't decode username as %s." % \
 
600
                    bzrlib.user_encoding)
 
601
 
 
602
    return realname, (username + '@' + socket.gethostname())
 
603
 
 
604
 
 
605
def extract_email_address(e):
 
606
    """Return just the address part of an email string.
 
607
    
 
608
    That is just the user@domain part, nothing else. 
 
609
    This part is required to contain only ascii characters.
 
610
    If it can't be extracted, raises an error.
 
611
    
 
612
    >>> extract_email_address('Jane Tester <jane@test.com>')
 
613
    "jane@test.com"
 
614
    """
 
615
    m = re.search(r'[\w+.-]+@[\w+.-]+', e)
 
616
    if not m:
 
617
        raise errors.BzrError("%r doesn't seem to contain "
 
618
                              "a reasonable email address" % e)
 
619
    return m.group(0)
 
620
 
 
621
 
 
622
class TreeConfig(object):
 
623
    """Branch configuration data associated with its contents, not location"""
 
624
    def __init__(self, branch):
 
625
        self.branch = branch
 
626
 
 
627
    def _get_config(self):
 
628
        try:
 
629
            obj = ConfigObj(self.branch.control_files.get('branch.conf'), 
 
630
                            encoding='utf-8')
 
631
        except errors.NoSuchFile:
 
632
            obj = ConfigObj(encoding='utf=8')
 
633
        return obj
 
634
 
 
635
    def get_option(self, name, section=None, default=None):
 
636
        self.branch.lock_read()
 
637
        try:
 
638
            obj = self._get_config()
 
639
            try:
 
640
                if section is not None:
 
641
                    obj[section]
 
642
                result = obj[name]
 
643
            except KeyError:
 
644
                result = default
 
645
        finally:
 
646
            self.branch.unlock()
 
647
        return result
 
648
 
 
649
    def set_option(self, value, name, section=None):
 
650
        """Set a per-branch configuration option"""
 
651
        self.branch.lock_write()
 
652
        try:
 
653
            cfg_obj = self._get_config()
 
654
            if section is None:
 
655
                obj = cfg_obj
 
656
            else:
 
657
                try:
 
658
                    obj = cfg_obj[section]
 
659
                except KeyError:
 
660
                    cfg_obj[section] = {}
 
661
                    obj = cfg_obj[section]
 
662
            obj[name] = value
 
663
            out_file = StringIO()
 
664
            cfg_obj.write(out_file)
 
665
            out_file.seek(0)
 
666
            self.branch.control_files.put('branch.conf', out_file)
 
667
        finally:
 
668
            self.branch.unlock()