/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/trace.py

  • Committer: Robert Collins
  • Date: 2010-06-26 01:07:16 UTC
  • mto: This revision was merged to the branch mainline in revision 5324.
  • Revision ID: robertc@robertcollins.net-20100626010716-jowzrldm4ntsaki2
Make bzrlib startup use a trace context manager.

Show diffs side-by-side

added added

removed removed

Lines of Context:
 
1
# Copyright (C) 2005-2010 Canonical Ltd
 
2
#
 
3
# This program is free software; you can redistribute it and/or modify
 
4
# it under the terms of the GNU General Public License as published by
 
5
# the Free Software Foundation; either version 2 of the License, or
 
6
# (at your option) any later version.
 
7
#
 
8
# This program is distributed in the hope that it will be useful,
 
9
# but WITHOUT ANY WARRANTY; without even the implied warranty of
 
10
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 
11
# GNU General Public License for more details.
 
12
#
 
13
# You should have received a copy of the GNU General Public License
 
14
# along with this program; if not, write to the Free Software
 
15
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
 
16
 
 
17
"""Messages and logging.
 
18
 
 
19
Messages are supplied by callers as a string-formatting template, plus values
 
20
to be inserted into it.  The actual %-formatting is deferred to the log
 
21
library so that it doesn't need to be done for messages that won't be emitted.
 
22
 
 
23
Messages are classified by severity levels: critical, error, warning, info,
 
24
and debug.
 
25
 
 
26
They can be sent to two places: to stderr, and to ~/.bzr.log.  For purposes
 
27
such as running the test suite, they can also be redirected away from both of
 
28
those two places to another location.
 
29
 
 
30
~/.bzr.log gets all messages, and full tracebacks for uncaught exceptions.
 
31
This trace file is always in UTF-8, regardless of the user's default encoding,
 
32
so that we can always rely on writing any message.
 
33
 
 
34
Output to stderr depends on the mode chosen by the user.  By default, messages
 
35
of info and above are sent out, which results in progress messages such as the
 
36
list of files processed by add and commit.  In debug mode, stderr gets debug messages too.
 
37
 
 
38
Errors that terminate an operation are generally passed back as exceptions;
 
39
others may be just emitted as messages.
 
40
 
 
41
Exceptions are reported in a brief form to stderr so as not to look scary.
 
42
BzrErrors are required to be able to format themselves into a properly
 
43
explanatory message.  This is not true for builtin exceptions such as
 
44
KeyError, which typically just str to "0".  They're printed in a different
 
45
form.
 
46
"""
 
47
 
 
48
# FIXME: Unfortunately it turns out that python's logging module
 
49
# is quite expensive, even when the message is not printed by any handlers.
 
50
# We should perhaps change back to just simply doing it here.
 
51
#
 
52
# On the other hand, as of 1.2 we generally only call the mutter() statement
 
53
# if (according to debug_flags) we actually intend to write it.  So the
 
54
# increased cost of logging.py is not so bad, and we could standardize on
 
55
# that.
 
56
 
 
57
import codecs
 
58
import logging
 
59
import os
 
60
import sys
 
61
import time
 
62
 
 
63
from bzrlib.lazy_import import lazy_import
 
64
lazy_import(globals(), """
 
65
from cStringIO import StringIO
 
66
import errno
 
67
import locale
 
68
import traceback
 
69
""")
 
70
 
 
71
import bzrlib
 
72
 
 
73
from bzrlib.symbol_versioning import (
 
74
    deprecated_function,
 
75
    deprecated_in,
 
76
    )
 
77
 
 
78
lazy_import(globals(), """
 
79
from bzrlib import (
 
80
    debug,
 
81
    errors,
 
82
    osutils,
 
83
    plugin,
 
84
    symbol_versioning,
 
85
    ui,
 
86
    )
 
87
""")
 
88
 
 
89
 
 
90
# global verbosity for bzrlib; controls the log level for stderr; 0=normal; <0
 
91
# is quiet; >0 is verbose.
 
92
_verbosity_level = 0
 
93
 
 
94
# File-like object where mutter/debug output is currently sent.  Can be
 
95
# changed by _push_log_file etc.  This is directly manipulated by some
 
96
# external code; maybe there should be functions to do that more precisely
 
97
# than push/pop_log_file.
 
98
_trace_file = None
 
99
 
 
100
# Absolute path for ~/.bzr.log.  Not changed even if the log/trace output is
 
101
# redirected elsewhere.  Used to show the location in --version.
 
102
_bzr_log_filename = None
 
103
 
 
104
# The time the first message was written to the trace file, so that we can
 
105
# show relative times since startup.
 
106
_bzr_log_start_time = bzrlib._start_time
 
107
 
 
108
 
 
109
# held in a global for quick reference
 
110
_bzr_logger = logging.getLogger('bzr')
 
111
 
 
112
 
 
113
def note(*args, **kwargs):
 
114
    """Output a note to the user.
 
115
 
 
116
    Takes the same parameters as logging.info.
 
117
 
 
118
    :return: None
 
119
    """
 
120
    # FIXME note always emits utf-8, regardless of the terminal encoding
 
121
    #
 
122
    # FIXME: clearing the ui and then going through the abstract logging
 
123
    # framework is whack; we should probably have a logging Handler that
 
124
    # deals with terminal output if needed.
 
125
    import bzrlib.ui
 
126
    bzrlib.ui.ui_factory.clear_term()
 
127
    _bzr_logger.info(*args, **kwargs)
 
128
 
 
129
 
 
130
def warning(*args, **kwargs):
 
131
    import bzrlib.ui
 
132
    bzrlib.ui.ui_factory.clear_term()
 
133
    _bzr_logger.warning(*args, **kwargs)
 
134
 
 
135
 
 
136
@deprecated_function(deprecated_in((2, 1, 0)))
 
137
def info(*args, **kwargs):
 
138
    """Deprecated: use trace.note instead."""
 
139
    note(*args, **kwargs)
 
140
 
 
141
 
 
142
@deprecated_function(deprecated_in((2, 1, 0)))
 
143
def log_error(*args, **kwargs):
 
144
    """Deprecated: use bzrlib.trace.show_error instead"""
 
145
    _bzr_logger.error(*args, **kwargs)
 
146
 
 
147
 
 
148
@deprecated_function(deprecated_in((2, 1, 0)))
 
149
def error(*args, **kwargs):
 
150
    """Deprecated: use bzrlib.trace.show_error instead"""
 
151
    _bzr_logger.error(*args, **kwargs)
 
152
 
 
153
 
 
154
def show_error(*args, **kwargs):
 
155
    """Show an error message to the user.
 
156
 
 
157
    Don't use this for exceptions, use report_exception instead.
 
158
    """
 
159
    _bzr_logger.error(*args, **kwargs)
 
160
 
 
161
 
 
162
def mutter(fmt, *args):
 
163
    if _trace_file is None:
 
164
        return
 
165
    # XXX: Don't check this every time; instead anyone who closes the file
 
166
    # ought to deregister it.  We can tolerate None.
 
167
    if (getattr(_trace_file, 'closed', None) is not None) and _trace_file.closed:
 
168
        return
 
169
 
 
170
    if isinstance(fmt, unicode):
 
171
        fmt = fmt.encode('utf8')
 
172
 
 
173
    if len(args) > 0:
 
174
        # It seems that if we do ascii % (unicode, ascii) we can
 
175
        # get a unicode cannot encode ascii error, so make sure that "fmt"
 
176
        # is a unicode string
 
177
        real_args = []
 
178
        for arg in args:
 
179
            if isinstance(arg, unicode):
 
180
                arg = arg.encode('utf8')
 
181
            real_args.append(arg)
 
182
        out = fmt % tuple(real_args)
 
183
    else:
 
184
        out = fmt
 
185
    now = time.time()
 
186
    timestamp = '%0.3f  ' % (now - _bzr_log_start_time,)
 
187
    out = timestamp + out + '\n'
 
188
    _trace_file.write(out)
 
189
    # there's no explicit flushing; the file is typically line buffered.
 
190
 
 
191
 
 
192
def mutter_callsite(stacklevel, fmt, *args):
 
193
    """Perform a mutter of fmt and args, logging the call trace.
 
194
 
 
195
    :param stacklevel: The number of frames to show. None will show all
 
196
        frames.
 
197
    :param fmt: The format string to pass to mutter.
 
198
    :param args: A list of substitution variables.
 
199
    """
 
200
    outf = StringIO()
 
201
    if stacklevel is None:
 
202
        limit = None
 
203
    else:
 
204
        limit = stacklevel + 1
 
205
    traceback.print_stack(limit=limit, file=outf)
 
206
    formatted_lines = outf.getvalue().splitlines()
 
207
    formatted_stack = '\n'.join(formatted_lines[:-2])
 
208
    mutter(fmt + "\nCalled from:\n%s", *(args + (formatted_stack,)))
 
209
 
 
210
 
 
211
def _rollover_trace_maybe(trace_fname):
 
212
    import stat
 
213
    try:
 
214
        size = os.stat(trace_fname)[stat.ST_SIZE]
 
215
        if size <= 4 << 20:
 
216
            return
 
217
        old_fname = trace_fname + '.old'
 
218
        osutils.rename(trace_fname, old_fname)
 
219
    except OSError:
 
220
        return
 
221
 
 
222
 
 
223
def _get_bzr_log_filename():
 
224
    bzr_log = os.environ.get('BZR_LOG')
 
225
    if bzr_log:
 
226
        return bzr_log
 
227
    home = os.environ.get('BZR_HOME')
 
228
    if home is None:
 
229
        if sys.platform == 'win32':
 
230
            from bzrlib import win32utils
 
231
            home = win32utils.get_home_location()
 
232
        else:
 
233
            home = os.path.expanduser('~')
 
234
    return os.path.join(home, '.bzr.log')
 
235
 
 
236
 
 
237
def _open_bzr_log():
 
238
    """Open the .bzr.log trace file.
 
239
 
 
240
    If the log is more than a particular length, the old file is renamed to
 
241
    .bzr.log.old and a new file is started.  Otherwise, we append to the
 
242
    existing file.
 
243
 
 
244
    This sets the global _bzr_log_filename.
 
245
    """
 
246
    global _bzr_log_filename
 
247
 
 
248
    def _open_or_create_log_file(filename):
 
249
        """Open existing log file, or create with ownership and permissions
 
250
 
 
251
        It inherits the ownership and permissions (masked by umask) from
 
252
        the containing directory to cope better with being run under sudo
 
253
        with $HOME still set to the user's homedir.
 
254
        """
 
255
        flags = os.O_WRONLY | os.O_APPEND | osutils.O_TEXT
 
256
        while True:
 
257
            try:
 
258
                fd = os.open(filename, flags)
 
259
                break
 
260
            except OSError, e:
 
261
                if e.errno != errno.ENOENT:
 
262
                    raise
 
263
            try:
 
264
                fd = os.open(filename, flags | os.O_CREAT | os.O_EXCL, 0666)
 
265
            except OSError, e:
 
266
                if e.errno != errno.EEXIST:
 
267
                    raise
 
268
            else:
 
269
                osutils.copy_ownership_from_path(filename)
 
270
                break
 
271
        return os.fdopen(fd, 'at', 0) # unbuffered
 
272
 
 
273
 
 
274
    _bzr_log_filename = _get_bzr_log_filename()
 
275
    _rollover_trace_maybe(_bzr_log_filename)
 
276
    try:
 
277
        bzr_log_file = _open_or_create_log_file(_bzr_log_filename)
 
278
        bzr_log_file.write('\n')
 
279
        if bzr_log_file.tell() <= 2:
 
280
            bzr_log_file.write("this is a debug log for diagnosing/reporting problems in bzr\n")
 
281
            bzr_log_file.write("you can delete or truncate this file, or include sections in\n")
 
282
            bzr_log_file.write("bug reports to https://bugs.launchpad.net/bzr/+filebug\n\n")
 
283
 
 
284
        return bzr_log_file
 
285
 
 
286
    except EnvironmentError, e:
 
287
        # If we are failing to open the log, then most likely logging has not
 
288
        # been set up yet. So we just write to stderr rather than using
 
289
        # 'warning()'. If we using warning(), users get the unhelpful 'no
 
290
        # handlers registered for "bzr"' when something goes wrong on the
 
291
        # server. (bug #503886)
 
292
        sys.stderr.write("failed to open trace file: %s\n" % (e,))
 
293
    # TODO: What should happen if we fail to open the trace file?  Maybe the
 
294
    # objects should be pointed at /dev/null or the equivalent?  Currently
 
295
    # returns None which will cause failures later.
 
296
    return None
 
297
 
 
298
 
 
299
def enable_default_logging():
 
300
    """Configure default logging: messages to stderr and debug to .bzr.log
 
301
 
 
302
    This should only be called once per process.
 
303
 
 
304
    Non-command-line programs embedding bzrlib do not need to call this.  They
 
305
    can instead either pass a file to _push_log_file, or act directly on
 
306
    logging.getLogger("bzr").
 
307
 
 
308
    Output can be redirected away by calling _push_log_file.
 
309
 
 
310
    :return: A memento from push_log_file for restoring the log state.
 
311
    """
 
312
    start_time = osutils.format_local_date(_bzr_log_start_time,
 
313
                                           timezone='local')
 
314
    # create encoded wrapper around stderr
 
315
    bzr_log_file = _open_bzr_log()
 
316
    if bzr_log_file is not None:
 
317
        bzr_log_file.write(start_time.encode('utf-8') + '\n')
 
318
    memento = push_log_file(bzr_log_file,
 
319
        r'[%(process)5d] %(asctime)s.%(msecs)03d %(levelname)s: %(message)s',
 
320
        r'%Y-%m-%d %H:%M:%S')
 
321
    # after hooking output into bzr_log, we also need to attach a stderr
 
322
    # handler, writing only at level info and with encoding
 
323
    term_encoding = osutils.get_terminal_encoding()
 
324
    writer_factory = codecs.getwriter(term_encoding)
 
325
    encoded_stderr = writer_factory(sys.stderr, errors='replace')
 
326
    stderr_handler = logging.StreamHandler(encoded_stderr)
 
327
    stderr_handler.setLevel(logging.INFO)
 
328
    logging.getLogger('bzr').addHandler(stderr_handler)
 
329
    return memento
 
330
 
 
331
 
 
332
def push_log_file(to_file, log_format=None, date_format=None):
 
333
    """Intercept log and trace messages and send them to a file.
 
334
 
 
335
    :param to_file: A file-like object to which messages will be sent.
 
336
 
 
337
    :returns: A memento that should be passed to _pop_log_file to restore the
 
338
    previously active logging.
 
339
    """
 
340
    global _trace_file
 
341
    # make a new handler
 
342
    new_handler = logging.StreamHandler(to_file)
 
343
    new_handler.setLevel(logging.DEBUG)
 
344
    if log_format is None:
 
345
        log_format = '%(levelname)8s  %(message)s'
 
346
    new_handler.setFormatter(logging.Formatter(log_format, date_format))
 
347
    # save and remove any existing log handlers
 
348
    bzr_logger = logging.getLogger('bzr')
 
349
    old_handlers = bzr_logger.handlers[:]
 
350
    del bzr_logger.handlers[:]
 
351
    # set that as the default logger
 
352
    bzr_logger.addHandler(new_handler)
 
353
    bzr_logger.setLevel(logging.DEBUG)
 
354
    # TODO: check if any changes are needed to the root logger
 
355
    #
 
356
    # TODO: also probably need to save and restore the level on bzr_logger.
 
357
    # but maybe we can avoid setting the logger level altogether, and just set
 
358
    # the level on the handler?
 
359
    #
 
360
    # save the old trace file
 
361
    old_trace_file = _trace_file
 
362
    # send traces to the new one
 
363
    _trace_file = to_file
 
364
    result = new_handler, _trace_file
 
365
    return ('log_memento', old_handlers, new_handler, old_trace_file, to_file)
 
366
 
 
367
 
 
368
def pop_log_file((magic, old_handlers, new_handler, old_trace_file, new_trace_file)):
 
369
    """Undo changes to logging/tracing done by _push_log_file.
 
370
 
 
371
    This flushes, but does not close the trace file.
 
372
 
 
373
    Takes the memento returned from _push_log_file."""
 
374
    global _trace_file
 
375
    _trace_file = old_trace_file
 
376
    bzr_logger = logging.getLogger('bzr')
 
377
    bzr_logger.removeHandler(new_handler)
 
378
    # must be closed, otherwise logging will try to close it at exit, and the
 
379
    # file will likely already be closed underneath.
 
380
    new_handler.close()
 
381
    bzr_logger.handlers = old_handlers
 
382
    new_trace_file.flush()
 
383
 
 
384
 
 
385
def log_exception_quietly():
 
386
    """Log the last exception to the trace file only.
 
387
 
 
388
    Used for exceptions that occur internally and that may be
 
389
    interesting to developers but not to users.  For example,
 
390
    errors loading plugins.
 
391
    """
 
392
    mutter(traceback.format_exc())
 
393
 
 
394
 
 
395
def set_verbosity_level(level):
 
396
    """Set the verbosity level.
 
397
 
 
398
    :param level: -ve for quiet, 0 for normal, +ve for verbose
 
399
    """
 
400
    global _verbosity_level
 
401
    _verbosity_level = level
 
402
    _update_logging_level(level < 0)
 
403
    ui.ui_factory.be_quiet(level < 0)
 
404
 
 
405
 
 
406
def get_verbosity_level():
 
407
    """Get the verbosity level.
 
408
 
 
409
    See set_verbosity_level() for values.
 
410
    """
 
411
    return _verbosity_level
 
412
 
 
413
 
 
414
def be_quiet(quiet=True):
 
415
    if quiet:
 
416
        set_verbosity_level(-1)
 
417
    else:
 
418
        set_verbosity_level(0)
 
419
 
 
420
 
 
421
def _update_logging_level(quiet=True):
 
422
    """Hide INFO messages if quiet."""
 
423
    if quiet:
 
424
        _bzr_logger.setLevel(logging.WARNING)
 
425
    else:
 
426
        _bzr_logger.setLevel(logging.INFO)
 
427
 
 
428
 
 
429
def is_quiet():
 
430
    """Is the verbosity level negative?"""
 
431
    return _verbosity_level < 0
 
432
 
 
433
 
 
434
def is_verbose():
 
435
    """Is the verbosity level positive?"""
 
436
    return _verbosity_level > 0
 
437
 
 
438
 
 
439
def debug_memory(message='', short=True):
 
440
    """Write out a memory dump."""
 
441
    if sys.platform == 'win32':
 
442
        from bzrlib import win32utils
 
443
        win32utils.debug_memory_win32api(message=message, short=short)
 
444
    else:
 
445
        _debug_memory_proc(message=message, short=short)
 
446
 
 
447
 
 
448
_short_fields = ('VmPeak', 'VmSize', 'VmRSS')
 
449
 
 
450
def _debug_memory_proc(message='', short=True):
 
451
    try:
 
452
        status_file = file('/proc/%s/status' % os.getpid(), 'rb')
 
453
    except IOError:
 
454
        return
 
455
    try:
 
456
        status = status_file.read()
 
457
    finally:
 
458
        status_file.close()
 
459
    if message:
 
460
        note(message)
 
461
    for line in status.splitlines():
 
462
        if not short:
 
463
            note(line)
 
464
        else:
 
465
            for field in _short_fields:
 
466
                if line.startswith(field):
 
467
                    note(line)
 
468
                    break
 
469
 
 
470
 
 
471
def report_exception(exc_info, err_file):
 
472
    """Report an exception to err_file (typically stderr) and to .bzr.log.
 
473
 
 
474
    This will show either a full traceback or a short message as appropriate.
 
475
 
 
476
    :return: The appropriate exit code for this error.
 
477
    """
 
478
    # Log the full traceback to ~/.bzr.log
 
479
    log_exception_quietly()
 
480
    if 'error' in debug.debug_flags:
 
481
        print_exception(exc_info, err_file)
 
482
        return errors.EXIT_ERROR
 
483
    exc_type, exc_object, exc_tb = exc_info
 
484
    if (isinstance(exc_object, IOError)
 
485
        and getattr(exc_object, 'errno', None) == errno.EPIPE):
 
486
        err_file.write("bzr: broken pipe\n")
 
487
        return errors.EXIT_ERROR
 
488
    elif isinstance(exc_object, KeyboardInterrupt):
 
489
        err_file.write("bzr: interrupted\n")
 
490
        return errors.EXIT_ERROR
 
491
    elif isinstance(exc_object, MemoryError):
 
492
        err_file.write("bzr: out of memory\n")
 
493
        return errors.EXIT_ERROR
 
494
    elif isinstance(exc_object, ImportError) \
 
495
        and str(exc_object).startswith("No module named "):
 
496
        report_user_error(exc_info, err_file,
 
497
            'You may need to install this Python library separately.')
 
498
        return errors.EXIT_ERROR
 
499
    elif not getattr(exc_object, 'internal_error', True):
 
500
        report_user_error(exc_info, err_file)
 
501
        return errors.EXIT_ERROR
 
502
    elif isinstance(exc_object, (OSError, IOError)) or (
 
503
        # GZ 2010-05-20: Like (exc_type is pywintypes.error) but avoid import
 
504
        exc_type.__name__ == "error" and exc_type.__module__ == "pywintypes"):
 
505
        # Might be nice to catch all of these and show them as something more
 
506
        # specific, but there are too many cases at the moment.
 
507
        report_user_error(exc_info, err_file)
 
508
        return errors.EXIT_ERROR
 
509
    else:
 
510
        report_bug(exc_info, err_file)
 
511
        return errors.EXIT_INTERNAL_ERROR
 
512
 
 
513
 
 
514
def print_exception(exc_info, err_file):
 
515
    exc_type, exc_object, exc_tb = exc_info
 
516
    err_file.write("bzr: ERROR: %s.%s: %s\n" % (
 
517
        exc_type.__module__, exc_type.__name__, exc_object))
 
518
    err_file.write('\n')
 
519
    traceback.print_exception(exc_type, exc_object, exc_tb, file=err_file)
 
520
 
 
521
 
 
522
# TODO: Should these be specially encoding the output?
 
523
def report_user_error(exc_info, err_file, advice=None):
 
524
    """Report to err_file an error that's not an internal error.
 
525
 
 
526
    These don't get a traceback unless -Derror was given.
 
527
 
 
528
    :param exc_info: 3-tuple from sys.exc_info()
 
529
    :param advice: Extra advice to the user to be printed following the
 
530
        exception.
 
531
    """
 
532
    err_file.write("bzr: ERROR: %s\n" % (exc_info[1],))
 
533
    if advice:
 
534
        err_file.write("%s\n" % (advice,))
 
535
 
 
536
 
 
537
def report_bug(exc_info, err_file):
 
538
    """Report an exception that probably indicates a bug in bzr"""
 
539
    from bzrlib.crash import report_bug
 
540
    report_bug(exc_info, err_file)
 
541
 
 
542
 
 
543
def _flush_stdout_stderr():
 
544
    # called from the bzrlib library finalizer returned by bzrlib.initialize()
 
545
    try:
 
546
        sys.stdout.flush()
 
547
        sys.stderr.flush()
 
548
    except IOError, e:
 
549
        import errno
 
550
        if e.errno in [errno.EINVAL, errno.EPIPE]:
 
551
            pass
 
552
        else:
 
553
            raise
 
554
 
 
555
 
 
556
def _flush_trace():
 
557
    # called from the bzrlib library finalizer returned by bzrlib.initialize()
 
558
    global _trace_file
 
559
    if _trace_file:
 
560
        _trace_file.flush()
 
561
 
 
562
 
 
563
class Config(object):
 
564
    """Configuration of message tracing in bzrlib.
 
565
 
 
566
    This implements the context manager protocol and should manage any global
 
567
    variables still used. The default config used is DefaultConfig, but
 
568
    embedded uses of bzrlib may wish to use a custom manager.
 
569
    """
 
570
 
 
571
    def __enter__(self):
 
572
        return self # This is bound to the 'as' clause in a with statement.
 
573
 
 
574
    def __exit__(self, exc_type, exc_val, exc_tb):
 
575
        return False # propogate exceptions.
 
576
 
 
577
 
 
578
class DefaultConfig(Config):
 
579
    """A default configuration for tracing of messages in bzrlib.
 
580
 
 
581
    This implements the context manager protocol.
 
582
    """
 
583
 
 
584
    def __enter__(self):
 
585
        self._original_filename = _bzr_log_filename
 
586
        self._original_state = enable_default_logging()
 
587
        return self # This is bound to the 'as' clause in a with statement.
 
588
 
 
589
    def __exit__(self, exc_type, exc_val, exc_tb):
 
590
        pop_log_file(self._original_state)
 
591
        global _bzr_log_filename
 
592
        _bzr_log_filename = self._original_filename
 
593
        return False # propogate exceptions.