~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/config.py

add a clean target

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()