5752.3.8
by John Arbash Meinel
Merge bzr.dev 5764 to resolve release-notes (aka NEWS) conflicts |
1 |
# Copyright (C) 2005-2011 Canonical Ltd
|
1740.5.1
by Martin Pool
When an unhandled exception occurs, write the traceback to stderr. |
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
|
|
4183.7.1
by Sabin Iacob
update FSF mailing address |
15 |
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
|
1
by mbp at sourcefrog
import from baz patch-364 |
16 |
|
4961.1.1
by Martin Pool
doc |
17 |
"""Messages and logging.
|
1185.33.6
by Martin Pool
Code and tests for shorter formatting of error messages |
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.
|
|
1185.33.51
by Martin Pool
Fix trace of non-ascii messages, and add test. |
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.
|
|
1185.33.6
by Martin Pool
Code and tests for shorter formatting of error messages |
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
|
|
4961.1.1
by Martin Pool
doc |
36 |
list of files processed by add and commit. In debug mode, stderr gets debug messages too.
|
1185.33.6
by Martin Pool
Code and tests for shorter formatting of error messages |
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
|
|
1759.2.1
by Jelmer Vernooij
Fix some types (found using aspell). |
43 |
explanatory message. This is not true for builtin exceptions such as
|
1185.33.6
by Martin Pool
Code and tests for shorter formatting of error messages |
44 |
KeyError, which typically just str to "0". They're printed in a different
|
45 |
form.
|
|
344
by Martin Pool
- It's not an error to use the library without |
46 |
"""
|
47 |
||
6379.6.7
by Jelmer Vernooij
Move importing from future until after doc string, otherwise the doc string will disappear. |
48 |
from __future__ import absolute_import |
49 |
||
1185.16.74
by Martin Pool
doc |
50 |
# FIXME: Unfortunately it turns out that python's logging module
|
51 |
# is quite expensive, even when the message is not printed by any handlers.
|
|
52 |
# We should perhaps change back to just simply doing it here.
|
|
3173.1.6
by Martin Pool
Remove _stderr_handler global |
53 |
#
|
54 |
# On the other hand, as of 1.2 we generally only call the mutter() statement
|
|
55 |
# if (according to debug_flags) we actually intend to write it. So the
|
|
56 |
# increased cost of logging.py is not so bad, and we could standardize on
|
|
57 |
# that.
|
|
1185.16.74
by Martin Pool
doc |
58 |
|
2978.1.1
by Alexander Belchenko
Output to stderr with trace module now going through wrapper that takes care about terminal encoding. (#54173) |
59 |
import logging |
1740.5.3
by Martin Pool
Cleanup more exception-formatting code |
60 |
import os |
1551.9.3
by Aaron Bentley
Revert buggy apport changes |
61 |
import sys |
3170.1.1
by Andrew Bennetts
Add -Dtimes, which adds a timestamp to mutters (using the time since the first mutter). |
62 |
import time |
1996.3.29
by John Arbash Meinel
don't import errors, and try to lazy import logging in bzrlib.trace |
63 |
|
64 |
from bzrlib.lazy_import import lazy_import |
|
65 |
lazy_import(globals(), """ |
|
2725.1.1
by Robert Collins
Add -Devil flag to highlight the use of problematic API calls. |
66 |
from cStringIO import StringIO
|
1996.3.29
by John Arbash Meinel
don't import errors, and try to lazy import logging in bzrlib.trace |
67 |
import errno
|
2842.1.1
by Martin Pool
Report locale, encodings and plugins in traceback |
68 |
import locale
|
5753.2.2
by Jelmer Vernooij
Remove some unnecessary imports, clean up lazy imports. |
69 |
import tempfile
|
2725.1.1
by Robert Collins
Add -Devil flag to highlight the use of problematic API calls. |
70 |
import traceback
|
1996.3.29
by John Arbash Meinel
don't import errors, and try to lazy import logging in bzrlib.trace |
71 |
""") |
1097
by Martin Pool
- send trace messages out through python logging module |
72 |
|
1185.33.8
by Martin Pool
Cleanup exception formatting stuff |
73 |
import bzrlib |
1097
by Martin Pool
- send trace messages out through python logging module |
74 |
|
2095.5.2
by Martin Pool
save commit message if commit fails (#32054) |
75 |
lazy_import(globals(), """ |
2713.2.1
by Martin Pool
Return exitcode 4 if an internal error occurs |
76 |
from bzrlib import (
|
77 |
debug,
|
|
78 |
errors,
|
|
2842.1.1
by Martin Pool
Report locale, encodings and plugins in traceback |
79 |
osutils,
|
4961.1.3
by Martin Pool
trace quietness now controls whether the progress bar appears |
80 |
ui,
|
2713.2.1
by Martin Pool
Return exitcode 4 if an internal error occurs |
81 |
)
|
2095.5.2
by Martin Pool
save commit message if commit fails (#32054) |
82 |
""") |
83 |
||
3173.1.4
by Martin Pool
Rearrange and clean up bzr trace/logging |
84 |
|
85 |
# global verbosity for bzrlib; controls the log level for stderr; 0=normal; <0
|
|
86 |
# is quiet; >0 is verbose.
|
|
2768.1.2
by Ian Clatworthy
Make noise levels a scale, not just a boolean in trace.py |
87 |
_verbosity_level = 0 |
3173.1.2
by Martin Pool
bzrlib.trace docs and cleanups |
88 |
|
89 |
# File-like object where mutter/debug output is currently sent. Can be
|
|
3173.1.4
by Martin Pool
Rearrange and clean up bzr trace/logging |
90 |
# changed by _push_log_file etc. This is directly manipulated by some
|
91 |
# external code; maybe there should be functions to do that more precisely
|
|
92 |
# than push/pop_log_file.
|
|
1185.43.7
by Martin Pool
Do debug messages not through python logging |
93 |
_trace_file = None |
3173.1.2
by Martin Pool
bzrlib.trace docs and cleanups |
94 |
|
3173.1.5
by Martin Pool
Remove global _bzr_log_file |
95 |
# Absolute path for ~/.bzr.log. Not changed even if the log/trace output is
|
96 |
# redirected elsewhere. Used to show the location in --version.
|
|
2566.1.1
by Martin Pool
Show bzr.log location in bzr --version output. |
97 |
_bzr_log_filename = None |
3173.1.2
by Martin Pool
bzrlib.trace docs and cleanups |
98 |
|
99 |
# The time the first message was written to the trace file, so that we can
|
|
100 |
# show relative times since startup.
|
|
3173.1.14
by Martin Pool
Merge trunk |
101 |
_bzr_log_start_time = bzrlib._start_time |
3173.1.2
by Martin Pool
bzrlib.trace docs and cleanups |
102 |
|
1097
by Martin Pool
- send trace messages out through python logging module |
103 |
|
3173.1.5
by Martin Pool
Remove global _bzr_log_file |
104 |
# held in a global for quick reference
|
1097
by Martin Pool
- send trace messages out through python logging module |
105 |
_bzr_logger = logging.getLogger('bzr') |
106 |
||
2978.1.1
by Alexander Belchenko
Output to stderr with trace module now going through wrapper that takes care about terminal encoding. (#54173) |
107 |
|
1558.8.1
by Aaron Bentley
Fix overall progress bar's interaction with 'note' and 'warning' |
108 |
def note(*args, **kwargs): |
5200.2.2
by Robert Collins
Document what note does a little. |
109 |
"""Output a note to the user.
|
110 |
||
111 |
Takes the same parameters as logging.info.
|
|
5200.2.4
by Robert Collins
Review feedback. |
112 |
|
5200.2.2
by Robert Collins
Document what note does a little. |
113 |
:return: None
|
114 |
"""
|
|
3173.1.7
by Martin Pool
Rename push/pop_log_file, and deprecate enable/disable_test_log |
115 |
# FIXME: clearing the ui and then going through the abstract logging
|
116 |
# framework is whack; we should probably have a logging Handler that
|
|
117 |
# deals with terminal output if needed.
|
|
5582.10.6
by Jelmer Vernooij
Fix ui imports. |
118 |
ui.ui_factory.clear_term() |
1558.8.1
by Aaron Bentley
Fix overall progress bar's interaction with 'note' and 'warning' |
119 |
_bzr_logger.info(*args, **kwargs) |
120 |
||
3173.1.4
by Martin Pool
Rearrange and clean up bzr trace/logging |
121 |
|
1558.8.1
by Aaron Bentley
Fix overall progress bar's interaction with 'note' and 'warning' |
122 |
def warning(*args, **kwargs): |
5582.10.6
by Jelmer Vernooij
Fix ui imports. |
123 |
ui.ui_factory.clear_term() |
1558.8.1
by Aaron Bentley
Fix overall progress bar's interaction with 'note' and 'warning' |
124 |
_bzr_logger.warning(*args, **kwargs) |
125 |
||
3173.1.4
by Martin Pool
Rearrange and clean up bzr trace/logging |
126 |
|
5055.4.1
by Gordon Tyler
Fixed show_error args and added test for show_error. |
127 |
def show_error(*args, **kwargs): |
4695.5.9
by Martin Pool
Restore info, log_error and error, as deprecated functions |
128 |
"""Show an error message to the user.
|
129 |
||
130 |
Don't use this for exceptions, use report_exception instead.
|
|
131 |
"""
|
|
132 |
_bzr_logger.error(*args, **kwargs) |
|
133 |
||
134 |
||
1185.43.7
by Martin Pool
Do debug messages not through python logging |
135 |
def mutter(fmt, *args): |
1185.43.10
by Martin Pool
Remove need for BZR_DEBUG to get debug info in trace file. |
136 |
if _trace_file is None: |
137 |
return
|
|
4792.6.1
by Martin Pool
Avoid time-based flushing of trace file, it's line buffered anyhow |
138 |
# XXX: Don't check this every time; instead anyone who closes the file
|
139 |
# ought to deregister it. We can tolerate None.
|
|
1963.2.6
by Robey Pointer
pychecker is on crack; go back to using 'is None'. |
140 |
if (getattr(_trace_file, 'closed', None) is not None) and _trace_file.closed: |
1185.43.10
by Martin Pool
Remove need for BZR_DEBUG to get debug info in trace file. |
141 |
return
|
1948.1.9
by John Arbash Meinel
Change mutter() so that it doesn't try so hard to write out perfect utf8, instead, rather than using a utf8 file, it changes unicode to utf8 manually |
142 |
|
143 |
if isinstance(fmt, unicode): |
|
144 |
fmt = fmt.encode('utf8') |
|
145 |
||
1185.43.10
by Martin Pool
Remove need for BZR_DEBUG to get debug info in trace file. |
146 |
if len(args) > 0: |
1685.1.2
by John Arbash Meinel
Re-enabling the non_ascii tests after fixing trace.py, bzr ignore also does the right thing now |
147 |
# It seems that if we do ascii % (unicode, ascii) we can
|
148 |
# get a unicode cannot encode ascii error, so make sure that "fmt"
|
|
149 |
# is a unicode string
|
|
1948.1.9
by John Arbash Meinel
Change mutter() so that it doesn't try so hard to write out perfect utf8, instead, rather than using a utf8 file, it changes unicode to utf8 manually |
150 |
real_args = [] |
151 |
for arg in args: |
|
152 |
if isinstance(arg, unicode): |
|
153 |
arg = arg.encode('utf8') |
|
154 |
real_args.append(arg) |
|
155 |
out = fmt % tuple(real_args) |
|
1185.43.10
by Martin Pool
Remove need for BZR_DEBUG to get debug info in trace file. |
156 |
else: |
1185.33.51
by Martin Pool
Fix trace of non-ascii messages, and add test. |
157 |
out = fmt |
3928.2.1
by John Arbash Meinel
Update mutter() to flush occasionally. |
158 |
now = time.time() |
159 |
timestamp = '%0.3f ' % (now - _bzr_log_start_time,) |
|
3195.1.1
by Andrew Bennetts
Always include timestamps in the trace file (i.e. remove -Dtimes in favour of having it switched on permanently) |
160 |
out = timestamp + out + '\n' |
1948.1.9
by John Arbash Meinel
Change mutter() so that it doesn't try so hard to write out perfect utf8, instead, rather than using a utf8 file, it changes unicode to utf8 manually |
161 |
_trace_file.write(out) |
4792.6.1
by Martin Pool
Avoid time-based flushing of trace file, it's line buffered anyhow |
162 |
# there's no explicit flushing; the file is typically line buffered.
|
1097
by Martin Pool
- send trace messages out through python logging module |
163 |
|
1185.43.10
by Martin Pool
Remove need for BZR_DEBUG to get debug info in trace file. |
164 |
|
2725.1.1
by Robert Collins
Add -Devil flag to highlight the use of problematic API calls. |
165 |
def mutter_callsite(stacklevel, fmt, *args): |
166 |
"""Perform a mutter of fmt and args, logging the call trace.
|
|
167 |
||
168 |
:param stacklevel: The number of frames to show. None will show all
|
|
169 |
frames.
|
|
170 |
:param fmt: The format string to pass to mutter.
|
|
171 |
:param args: A list of substitution variables.
|
|
172 |
"""
|
|
173 |
outf = StringIO() |
|
3441.5.1
by Andrew Bennetts
Avoid necessarily calling get_parent_map when pushing. |
174 |
if stacklevel is None: |
175 |
limit = None |
|
176 |
else: |
|
177 |
limit = stacklevel + 1 |
|
178 |
traceback.print_stack(limit=limit, file=outf) |
|
2725.1.1
by Robert Collins
Add -Devil flag to highlight the use of problematic API calls. |
179 |
formatted_lines = outf.getvalue().splitlines() |
180 |
formatted_stack = '\n'.join(formatted_lines[:-2]) |
|
181 |
mutter(fmt + "\nCalled from:\n%s", *(args + (formatted_stack,))) |
|
182 |
||
183 |
||
261
by Martin Pool
- auto-rollover of .bzr.log |
184 |
def _rollover_trace_maybe(trace_fname): |
359
by Martin Pool
- pychecker fixups |
185 |
import stat |
261
by Martin Pool
- auto-rollover of .bzr.log |
186 |
try: |
187 |
size = os.stat(trace_fname)[stat.ST_SIZE] |
|
333
by Martin Pool
- allow trace file to grow up to 4MB |
188 |
if size <= 4 << 20: |
261
by Martin Pool
- auto-rollover of .bzr.log |
189 |
return
|
190 |
old_fname = trace_fname + '.old' |
|
2851.3.1
by Martin Pool
Add unit test for _rollover_trace_maybe |
191 |
osutils.rename(trace_fname, old_fname) |
261
by Martin Pool
- auto-rollover of .bzr.log |
192 |
except OSError: |
193 |
return
|
|
194 |
||
195 |
||
3173.1.4
by Martin Pool
Rearrange and clean up bzr trace/logging |
196 |
def _get_bzr_log_filename(): |
6437.26.4
by Martin Packman
Treat path for .bzr.log as unicode |
197 |
bzr_log = osutils.path_from_environ('BZR_LOG') |
3193.6.8
by Alexander Belchenko
merge bzr.dev; update for 1.3; $BZR_LOG used in trace.py module (again), not in the main bzr script (req. by Robert Collins) |
198 |
if bzr_log: |
199 |
return bzr_log |
|
6437.26.4
by Martin Packman
Treat path for .bzr.log as unicode |
200 |
home = osutils.path_from_environ('BZR_HOME') |
3193.6.8
by Alexander Belchenko
merge bzr.dev; update for 1.3; $BZR_LOG used in trace.py module (again), not in the main bzr script (req. by Robert Collins) |
201 |
if home is None: |
6437.26.4
by Martin Packman
Treat path for .bzr.log as unicode |
202 |
# GZ 2012-02-01: Logging to the home dir is bad, but XDG is unclear
|
203 |
# over what would be better. On windows, bug 240550
|
|
204 |
# suggests LOCALAPPDATA be used instead.
|
|
205 |
home = osutils._get_home_dir() |
|
3173.1.4
by Martin Pool
Rearrange and clean up bzr trace/logging |
206 |
return os.path.join(home, '.bzr.log') |
207 |
||
208 |
||
209 |
def _open_bzr_log(): |
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
210 |
"""Open the .bzr.log trace file.
|
3173.1.2
by Martin Pool
bzrlib.trace docs and cleanups |
211 |
|
212 |
If the log is more than a particular length, the old file is renamed to
|
|
213 |
.bzr.log.old and a new file is started. Otherwise, we append to the
|
|
214 |
existing file.
|
|
215 |
||
3173.1.5
by Martin Pool
Remove global _bzr_log_file |
216 |
This sets the global _bzr_log_filename.
|
3173.1.2
by Martin Pool
bzrlib.trace docs and cleanups |
217 |
"""
|
3173.1.5
by Martin Pool
Remove global _bzr_log_file |
218 |
global _bzr_log_filename |
5116.2.3
by Parth Malwankar
updated log file creation to avoid race based on implementation by Martin [gz] |
219 |
|
5116.3.1
by Martin
Rearrange log opening helper function and fix a couple of bugs |
220 |
def _open_or_create_log_file(filename): |
221 |
"""Open existing log file, or create with ownership and permissions
|
|
5116.2.6
by Parth Malwankar
renamed copy_ownership to copy_ownership_from_path. |
222 |
|
223 |
It inherits the ownership and permissions (masked by umask) from
|
|
224 |
the containing directory to cope better with being run under sudo
|
|
225 |
with $HOME still set to the user's homedir.
|
|
5116.2.3
by Parth Malwankar
updated log file creation to avoid race based on implementation by Martin [gz] |
226 |
"""
|
5116.2.6
by Parth Malwankar
renamed copy_ownership to copy_ownership_from_path. |
227 |
flags = os.O_WRONLY | os.O_APPEND | osutils.O_TEXT |
5116.2.5
by Parth Malwankar
added 'while True:' so multiple process can spin on log file creation |
228 |
while True: |
229 |
try: |
|
5116.2.6
by Parth Malwankar
renamed copy_ownership to copy_ownership_from_path. |
230 |
fd = os.open(filename, flags) |
5116.3.1
by Martin
Rearrange log opening helper function and fix a couple of bugs |
231 |
break
|
5116.2.5
by Parth Malwankar
added 'while True:' so multiple process can spin on log file creation |
232 |
except OSError, e: |
233 |
if e.errno != errno.ENOENT: |
|
234 |
raise
|
|
235 |
try: |
|
5116.3.1
by Martin
Rearrange log opening helper function and fix a couple of bugs |
236 |
fd = os.open(filename, flags | os.O_CREAT | os.O_EXCL, 0666) |
5116.2.5
by Parth Malwankar
added 'while True:' so multiple process can spin on log file creation |
237 |
except OSError, e: |
238 |
if e.errno != errno.EEXIST: |
|
239 |
raise
|
|
240 |
else: |
|
5116.2.6
by Parth Malwankar
renamed copy_ownership to copy_ownership_from_path. |
241 |
osutils.copy_ownership_from_path(filename) |
5116.3.1
by Martin
Rearrange log opening helper function and fix a couple of bugs |
242 |
break
|
243 |
return os.fdopen(fd, 'at', 0) # unbuffered |
|
5116.2.3
by Parth Malwankar
updated log file creation to avoid race based on implementation by Martin [gz] |
244 |
|
245 |
||
3173.1.4
by Martin Pool
Rearrange and clean up bzr trace/logging |
246 |
_bzr_log_filename = _get_bzr_log_filename() |
2566.1.1
by Martin Pool
Show bzr.log location in bzr --version output. |
247 |
_rollover_trace_maybe(_bzr_log_filename) |
403
by Martin Pool
- Don't give an error if the trace file can't be opened |
248 |
try: |
5116.3.1
by Martin
Rearrange log opening helper function and fix a couple of bugs |
249 |
bzr_log_file = _open_or_create_log_file(_bzr_log_filename) |
3173.1.5
by Martin Pool
Remove global _bzr_log_file |
250 |
bzr_log_file.write('\n') |
251 |
if bzr_log_file.tell() <= 2: |
|
252 |
bzr_log_file.write("this is a debug log for diagnosing/reporting problems in bzr\n") |
|
253 |
bzr_log_file.write("you can delete or truncate this file, or include sections in\n") |
|
254 |
bzr_log_file.write("bug reports to https://bugs.launchpad.net/bzr/+filebug\n\n") |
|
4634.143.2
by Parth Malwankar
fixed test cases to handle special .bzr.log files |
255 |
|
3173.1.5
by Martin Pool
Remove global _bzr_log_file |
256 |
return bzr_log_file |
4634.143.4
by Parth Malwankar
added parent_dir and mkdir to osutils. osutils.mkdir optionally |
257 |
|
5116.3.1
by Martin
Rearrange log opening helper function and fix a couple of bugs |
258 |
except EnvironmentError, e: |
4634.118.1
by John Arbash Meinel
Fix bug #503886, errors setting up logging go to stderr. |
259 |
# If we are failing to open the log, then most likely logging has not
|
260 |
# been set up yet. So we just write to stderr rather than using
|
|
261 |
# 'warning()'. If we using warning(), users get the unhelpful 'no
|
|
262 |
# handlers registered for "bzr"' when something goes wrong on the
|
|
263 |
# server. (bug #503886)
|
|
264 |
sys.stderr.write("failed to open trace file: %s\n" % (e,)) |
|
3173.1.5
by Martin Pool
Remove global _bzr_log_file |
265 |
# TODO: What should happen if we fail to open the trace file? Maybe the
|
266 |
# objects should be pointed at /dev/null or the equivalent? Currently
|
|
267 |
# returns None which will cause failures later.
|
|
3805.2.4
by John Arbash Meinel
Handle when _open_bzr_log returns None. |
268 |
return None |
260
by Martin Pool
- remove atexit() dependency for writing out execution times |
269 |
|
1097
by Martin Pool
- send trace messages out through python logging module |
270 |
|
1111
by Martin Pool
- add functions to enable and disable default logging, so that we can |
271 |
def enable_default_logging(): |
3173.1.4
by Martin Pool
Rearrange and clean up bzr trace/logging |
272 |
"""Configure default logging: messages to stderr and debug to .bzr.log
|
3805.2.1
by John Arbash Meinel
Update trace to log when the process started. |
273 |
|
3173.1.4
by Martin Pool
Rearrange and clean up bzr trace/logging |
274 |
This should only be called once per process.
|
3173.1.6
by Martin Pool
Remove _stderr_handler global |
275 |
|
276 |
Non-command-line programs embedding bzrlib do not need to call this. They
|
|
277 |
can instead either pass a file to _push_log_file, or act directly on
|
|
278 |
logging.getLogger("bzr").
|
|
3805.2.1
by John Arbash Meinel
Update trace to log when the process started. |
279 |
|
3173.1.4
by Martin Pool
Rearrange and clean up bzr trace/logging |
280 |
Output can be redirected away by calling _push_log_file.
|
5320.2.5
by Robert Collins
Make bzrlib startup use a trace context manager. |
281 |
|
282 |
:return: A memento from push_log_file for restoring the log state.
|
|
3173.1.4
by Martin Pool
Rearrange and clean up bzr trace/logging |
283 |
"""
|
3805.2.3
by John Arbash Meinel
Use osutils.format_local_date() instead of time.strftime() |
284 |
start_time = osutils.format_local_date(_bzr_log_start_time, |
285 |
timezone='local') |
|
3173.1.5
by Martin Pool
Remove global _bzr_log_file |
286 |
bzr_log_file = _open_bzr_log() |
3805.2.4
by John Arbash Meinel
Handle when _open_bzr_log returns None. |
287 |
if bzr_log_file is not None: |
288 |
bzr_log_file.write(start_time.encode('utf-8') + '\n') |
|
5320.2.5
by Robert Collins
Make bzrlib startup use a trace context manager. |
289 |
memento = push_log_file(bzr_log_file, |
3173.1.4
by Martin Pool
Rearrange and clean up bzr trace/logging |
290 |
r'[%(process)5d] %(asctime)s.%(msecs)03d %(levelname)s: %(message)s', |
291 |
r'%Y-%m-%d %H:%M:%S') |
|
292 |
# after hooking output into bzr_log, we also need to attach a stderr
|
|
3173.1.6
by Martin Pool
Remove _stderr_handler global |
293 |
# handler, writing only at level info and with encoding
|
6015.52.2
by Martin Packman
Add EncodedStreamHandler as more robust logging class |
294 |
stderr_handler = EncodedStreamHandler(sys.stderr, |
295 |
osutils.get_terminal_encoding(), 'replace', level=logging.INFO) |
|
3173.1.6
by Martin Pool
Remove _stderr_handler global |
296 |
logging.getLogger('bzr').addHandler(stderr_handler) |
5320.2.5
by Robert Collins
Make bzrlib startup use a trace context manager. |
297 |
return memento |
3173.1.4
by Martin Pool
Rearrange and clean up bzr trace/logging |
298 |
|
299 |
||
3173.1.7
by Martin Pool
Rename push/pop_log_file, and deprecate enable/disable_test_log |
300 |
def push_log_file(to_file, log_format=None, date_format=None): |
3173.1.1
by Martin Pool
New _push_log_file and _pop_log_file as a cleaner stack, towards a fix for #124153 |
301 |
"""Intercept log and trace messages and send them to a file.
|
302 |
||
3173.1.11
by Martin Pool
Fix up renaming of push_log_file |
303 |
:param to_file: A file-like object to which messages will be sent.
|
304 |
||
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
305 |
:returns: A memento that should be passed to _pop_log_file to restore the
|
5891.1.3
by Andrew Bennetts
Move docstring formatting fixes. |
306 |
previously active logging.
|
3173.1.1
by Martin Pool
New _push_log_file and _pop_log_file as a cleaner stack, towards a fix for #124153 |
307 |
"""
|
308 |
global _trace_file |
|
309 |
# make a new handler
|
|
6015.52.2
by Martin Packman
Add EncodedStreamHandler as more robust logging class |
310 |
new_handler = EncodedStreamHandler(to_file, "utf-8", level=logging.DEBUG) |
3173.1.4
by Martin Pool
Rearrange and clean up bzr trace/logging |
311 |
if log_format is None: |
312 |
log_format = '%(levelname)8s %(message)s' |
|
313 |
new_handler.setFormatter(logging.Formatter(log_format, date_format)) |
|
3173.1.1
by Martin Pool
New _push_log_file and _pop_log_file as a cleaner stack, towards a fix for #124153 |
314 |
# save and remove any existing log handlers
|
315 |
bzr_logger = logging.getLogger('bzr') |
|
316 |
old_handlers = bzr_logger.handlers[:] |
|
317 |
del bzr_logger.handlers[:] |
|
318 |
# set that as the default logger
|
|
319 |
bzr_logger.addHandler(new_handler) |
|
320 |
bzr_logger.setLevel(logging.DEBUG) |
|
321 |
# TODO: check if any changes are needed to the root logger
|
|
322 |
#
|
|
323 |
# TODO: also probably need to save and restore the level on bzr_logger.
|
|
324 |
# but maybe we can avoid setting the logger level altogether, and just set
|
|
325 |
# the level on the handler?
|
|
326 |
#
|
|
327 |
# save the old trace file
|
|
328 |
old_trace_file = _trace_file |
|
329 |
# send traces to the new one
|
|
330 |
_trace_file = to_file |
|
331 |
result = new_handler, _trace_file |
|
332 |
return ('log_memento', old_handlers, new_handler, old_trace_file, to_file) |
|
333 |
||
334 |
||
3173.1.7
by Martin Pool
Rename push/pop_log_file, and deprecate enable/disable_test_log |
335 |
def pop_log_file((magic, old_handlers, new_handler, old_trace_file, new_trace_file)): |
3173.1.1
by Martin Pool
New _push_log_file and _pop_log_file as a cleaner stack, towards a fix for #124153 |
336 |
"""Undo changes to logging/tracing done by _push_log_file.
|
337 |
||
5320.2.7
by Robert Collins
Sanity check that new_trace_file in pop_log_file is valid, and also fix a test that monkey patched get_terminal_encoding. |
338 |
This flushes, but does not close the trace file (so that anything that was
|
339 |
in it is output.
|
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
340 |
|
3173.1.1
by Martin Pool
New _push_log_file and _pop_log_file as a cleaner stack, towards a fix for #124153 |
341 |
Takes the memento returned from _push_log_file."""
|
342 |
global _trace_file |
|
343 |
_trace_file = old_trace_file |
|
344 |
bzr_logger = logging.getLogger('bzr') |
|
345 |
bzr_logger.removeHandler(new_handler) |
|
5222.2.7
by Robert Collins
Document bzrlib.initialize a little better, and explicitly propogate exceptions in the new __exit__ methods. |
346 |
# must be closed, otherwise logging will try to close it at exit, and the
|
3173.1.1
by Martin Pool
New _push_log_file and _pop_log_file as a cleaner stack, towards a fix for #124153 |
347 |
# file will likely already be closed underneath.
|
348 |
new_handler.close() |
|
349 |
bzr_logger.handlers = old_handlers |
|
5320.2.7
by Robert Collins
Sanity check that new_trace_file in pop_log_file is valid, and also fix a test that monkey patched get_terminal_encoding. |
350 |
if new_trace_file is not None: |
351 |
new_trace_file.flush() |
|
3173.1.1
by Martin Pool
New _push_log_file and _pop_log_file as a cleaner stack, towards a fix for #124153 |
352 |
|
353 |
||
3427.2.2
by James Westby
Just print the exception, keeping the API of log_exception_quietly the same. |
354 |
def log_exception_quietly(): |
3173.1.4
by Martin Pool
Rearrange and clean up bzr trace/logging |
355 |
"""Log the last exception to the trace file only.
|
356 |
||
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
357 |
Used for exceptions that occur internally and that may be
|
358 |
interesting to developers but not to users. For example,
|
|
3173.1.4
by Martin Pool
Rearrange and clean up bzr trace/logging |
359 |
errors loading plugins.
|
360 |
"""
|
|
3427.2.2
by James Westby
Just print the exception, keeping the API of log_exception_quietly the same. |
361 |
mutter(traceback.format_exc()) |
3173.1.4
by Martin Pool
Rearrange and clean up bzr trace/logging |
362 |
|
363 |
||
364 |
def set_verbosity_level(level): |
|
365 |
"""Set the verbosity level.
|
|
366 |
||
367 |
:param level: -ve for quiet, 0 for normal, +ve for verbose
|
|
368 |
"""
|
|
369 |
global _verbosity_level |
|
370 |
_verbosity_level = level |
|
371 |
_update_logging_level(level < 0) |
|
4961.1.3
by Martin Pool
trace quietness now controls whether the progress bar appears |
372 |
ui.ui_factory.be_quiet(level < 0) |
3173.1.4
by Martin Pool
Rearrange and clean up bzr trace/logging |
373 |
|
374 |
||
375 |
def get_verbosity_level(): |
|
376 |
"""Get the verbosity level.
|
|
377 |
||
378 |
See set_verbosity_level() for values.
|
|
379 |
"""
|
|
380 |
return _verbosity_level |
|
381 |
||
382 |
||
383 |
def be_quiet(quiet=True): |
|
384 |
if quiet: |
|
385 |
set_verbosity_level(-1) |
|
386 |
else: |
|
387 |
set_verbosity_level(0) |
|
388 |
||
389 |
||
390 |
def _update_logging_level(quiet=True): |
|
391 |
"""Hide INFO messages if quiet."""
|
|
392 |
if quiet: |
|
3173.1.6
by Martin Pool
Remove _stderr_handler global |
393 |
_bzr_logger.setLevel(logging.WARNING) |
3173.1.4
by Martin Pool
Rearrange and clean up bzr trace/logging |
394 |
else: |
3173.1.6
by Martin Pool
Remove _stderr_handler global |
395 |
_bzr_logger.setLevel(logging.INFO) |
3173.1.4
by Martin Pool
Rearrange and clean up bzr trace/logging |
396 |
|
397 |
||
398 |
def is_quiet(): |
|
399 |
"""Is the verbosity level negative?"""
|
|
400 |
return _verbosity_level < 0 |
|
401 |
||
402 |
||
403 |
def is_verbose(): |
|
404 |
"""Is the verbosity level positive?"""
|
|
405 |
return _verbosity_level > 0 |
|
406 |
||
407 |
||
3698.1.2
by John Arbash Meinel
Make message optional, don't check the memory flag directly. |
408 |
def debug_memory(message='', short=True): |
3698.1.1
by John Arbash Meinel
Move the memory debugging into a helper function. |
409 |
"""Write out a memory dump."""
|
4011.1.1
by John Arbash Meinel
Implement -Dmemory for win32 |
410 |
if sys.platform == 'win32': |
411 |
from bzrlib import win32utils |
|
412 |
win32utils.debug_memory_win32api(message=message, short=short) |
|
413 |
else: |
|
414 |
_debug_memory_proc(message=message, short=short) |
|
415 |
||
416 |
||
417 |
_short_fields = ('VmPeak', 'VmSize', 'VmRSS') |
|
418 |
||
419 |
def _debug_memory_proc(message='', short=True): |
|
3698.1.1
by John Arbash Meinel
Move the memory debugging into a helper function. |
420 |
try: |
421 |
status_file = file('/proc/%s/status' % os.getpid(), 'rb') |
|
422 |
except IOError: |
|
423 |
return
|
|
424 |
try: |
|
425 |
status = status_file.read() |
|
426 |
finally: |
|
427 |
status_file.close() |
|
3698.1.2
by John Arbash Meinel
Make message optional, don't check the memory flag directly. |
428 |
if message: |
429 |
note(message) |
|
3698.1.1
by John Arbash Meinel
Move the memory debugging into a helper function. |
430 |
for line in status.splitlines(): |
431 |
if not short: |
|
432 |
note(line) |
|
433 |
else: |
|
434 |
for field in _short_fields: |
|
435 |
if line.startswith(field): |
|
436 |
note(line) |
|
437 |
break
|
|
438 |
||
5448.5.2
by Karl Bielefeldt
Incorporated suggestions from code review: |
439 |
def _dump_memory_usage(err_file): |
5448.5.1
by Karl Bielefeldt
Use meliae to dump memory to a file upon MemoryError. |
440 |
try: |
5448.5.4
by Karl Bielefeldt
Fix Python 2.4 incompatibilities |
441 |
try: |
442 |
fd, name = tempfile.mkstemp(prefix="bzr_memdump", suffix=".json") |
|
443 |
dump_file = os.fdopen(fd, 'w') |
|
444 |
from meliae import scanner |
|
445 |
scanner.dump_gc_objects(dump_file) |
|
446 |
err_file.write("Memory dumped to %s\n" % name) |
|
447 |
except ImportError: |
|
448 |
err_file.write("Dumping memory requires meliae module.\n") |
|
449 |
log_exception_quietly() |
|
450 |
except: |
|
451 |
err_file.write("Exception while dumping memory.\n") |
|
452 |
log_exception_quietly() |
|
5448.5.2
by Karl Bielefeldt
Incorporated suggestions from code review: |
453 |
finally: |
5448.5.4
by Karl Bielefeldt
Fix Python 2.4 incompatibilities |
454 |
if dump_file is not None: |
455 |
dump_file.close() |
|
456 |
elif fd is not None: |
|
457 |
os.close(fd) |
|
3698.1.1
by John Arbash Meinel
Move the memory debugging into a helper function. |
458 |
|
5677.3.1
by Martin
Add helper for getting qualified exception name to bzrlib.trace |
459 |
|
460 |
def _qualified_exception_name(eclass, unqualified_bzrlib_errors=False): |
|
461 |
"""Give name of error class including module for non-builtin exceptions
|
|
462 |
||
463 |
If `unqualified_bzrlib_errors` is True, errors specific to bzrlib will
|
|
464 |
also omit the module prefix.
|
|
465 |
"""
|
|
466 |
class_name = eclass.__name__ |
|
467 |
module_name = eclass.__module__ |
|
5677.2.10
by Martin
Omit '__main__' from exception names as well now they're purely decorative |
468 |
if module_name in ("exceptions", "__main__") or ( |
5677.3.1
by Martin
Add helper for getting qualified exception name to bzrlib.trace |
469 |
unqualified_bzrlib_errors and module_name == "bzrlib.errors"): |
470 |
return class_name |
|
471 |
return "%s.%s" % (module_name, class_name) |
|
472 |
||
473 |
||
1740.5.2
by Martin Pool
Improved tests for display of exceptions. |
474 |
def report_exception(exc_info, err_file): |
2830.2.1
by Martin Pool
If TestCase.run_bzr hits an internal exception, don't catch it but rather propagate up into the test suite |
475 |
"""Report an exception to err_file (typically stderr) and to .bzr.log.
|
2713.2.1
by Martin Pool
Return exitcode 4 if an internal error occurs |
476 |
|
477 |
This will show either a full traceback or a short message as appropriate.
|
|
478 |
||
479 |
:return: The appropriate exit code for this error.
|
|
480 |
"""
|
|
4805.3.2
by Benoît Pierre
Always log the full traceback to ~/.bzr.log, even when -Derror is used. |
481 |
# Log the full traceback to ~/.bzr.log
|
482 |
log_exception_quietly() |
|
4805.3.1
by Benoît Pierre
Always honor 'error' debug flag. |
483 |
if 'error' in debug.debug_flags: |
484 |
print_exception(exc_info, err_file) |
|
485 |
return errors.EXIT_ERROR |
|
1740.5.3
by Martin Pool
Cleanup more exception-formatting code |
486 |
exc_type, exc_object, exc_tb = exc_info |
6336.2.1
by Martin Packman
Add is_environment_error() and switch trace to using it |
487 |
if isinstance(exc_object, KeyboardInterrupt): |
2911.6.1
by Blake Winton
Change 'print >> f,'s to 'f.write('s. |
488 |
err_file.write("bzr: interrupted\n") |
2713.2.1
by Martin Pool
Return exitcode 4 if an internal error occurs |
489 |
return errors.EXIT_ERROR |
4634.26.1
by Martin Pool
Cleaner message when out of memory |
490 |
elif isinstance(exc_object, MemoryError): |
491 |
err_file.write("bzr: out of memory\n") |
|
5448.5.2
by Karl Bielefeldt
Incorporated suggestions from code review: |
492 |
if 'mem_dump' in debug.debug_flags: |
493 |
_dump_memory_usage(err_file) |
|
494 |
else: |
|
495 |
err_file.write("Use -Dmem_dump to dump memory to a file.\n") |
|
4634.26.1
by Martin Pool
Cleaner message when out of memory |
496 |
return errors.EXIT_ERROR |
3497.3.2
by Martin Pool
Show short error for missing libraries |
497 |
elif isinstance(exc_object, ImportError) \ |
498 |
and str(exc_object).startswith("No module named "): |
|
499 |
report_user_error(exc_info, err_file, |
|
500 |
'You may need to install this Python library separately.') |
|
501 |
return errors.EXIT_ERROR |
|
2067.3.2
by Martin Pool
Error cleanup review comments: |
502 |
elif not getattr(exc_object, 'internal_error', True): |
1740.5.6
by Martin Pool
Clean up many exception classes. |
503 |
report_user_error(exc_info, err_file) |
2713.2.1
by Martin Pool
Return exitcode 4 if an internal error occurs |
504 |
return errors.EXIT_ERROR |
6336.2.1
by Martin Packman
Add is_environment_error() and switch trace to using it |
505 |
elif osutils.is_environment_error(exc_object): |
506 |
if getattr(exc_object, 'errno', None) == errno.EPIPE: |
|
507 |
err_file.write("bzr: broken pipe\n") |
|
508 |
return errors.EXIT_ERROR |
|
1740.5.6
by Martin Pool
Clean up many exception classes. |
509 |
# Might be nice to catch all of these and show them as something more
|
510 |
# specific, but there are too many cases at the moment.
|
|
1740.5.1
by Martin Pool
When an unhandled exception occurs, write the traceback to stderr. |
511 |
report_user_error(exc_info, err_file) |
2713.2.1
by Martin Pool
Return exitcode 4 if an internal error occurs |
512 |
return errors.EXIT_ERROR |
1740.5.1
by Martin Pool
When an unhandled exception occurs, write the traceback to stderr. |
513 |
else: |
1551.9.3
by Aaron Bentley
Revert buggy apport changes |
514 |
report_bug(exc_info, err_file) |
2713.2.1
by Martin Pool
Return exitcode 4 if an internal error occurs |
515 |
return errors.EXIT_INTERNAL_ERROR |
1740.5.1
by Martin Pool
When an unhandled exception occurs, write the traceback to stderr. |
516 |
|
517 |
||
3427.2.2
by James Westby
Just print the exception, keeping the API of log_exception_quietly the same. |
518 |
def print_exception(exc_info, err_file): |
519 |
exc_type, exc_object, exc_tb = exc_info |
|
520 |
err_file.write("bzr: ERROR: %s.%s: %s\n" % ( |
|
521 |
exc_type.__module__, exc_type.__name__, exc_object)) |
|
522 |
err_file.write('\n') |
|
523 |
traceback.print_exception(exc_type, exc_object, exc_tb, file=err_file) |
|
524 |
||
525 |
||
1740.5.1
by Martin Pool
When an unhandled exception occurs, write the traceback to stderr. |
526 |
# TODO: Should these be specially encoding the output?
|
3497.3.2
by Martin Pool
Show short error for missing libraries |
527 |
def report_user_error(exc_info, err_file, advice=None): |
2095.5.1
by Martin Pool
Add new -Derror global option to always display tracebacks |
528 |
"""Report to err_file an error that's not an internal error.
|
529 |
||
530 |
These don't get a traceback unless -Derror was given.
|
|
3497.3.2
by Martin Pool
Show short error for missing libraries |
531 |
|
532 |
:param exc_info: 3-tuple from sys.exc_info()
|
|
533 |
:param advice: Extra advice to the user to be printed following the
|
|
534 |
exception.
|
|
2095.5.1
by Martin Pool
Add new -Derror global option to always display tracebacks |
535 |
"""
|
2911.6.3
by Blake Winton
Implemented suggestions from John Arbash Meinel. |
536 |
err_file.write("bzr: ERROR: %s\n" % (exc_info[1],)) |
3497.3.2
by Martin Pool
Show short error for missing libraries |
537 |
if advice: |
538 |
err_file.write("%s\n" % (advice,)) |
|
1740.5.1
by Martin Pool
When an unhandled exception occurs, write the traceback to stderr. |
539 |
|
540 |
||
541 |
def report_bug(exc_info, err_file): |
|
1740.5.2
by Martin Pool
Improved tests for display of exceptions. |
542 |
"""Report an exception that probably indicates a bug in bzr"""
|
4584.3.6
by Martin Pool
Move apport integration to bzrlib.crash and send output to a file. |
543 |
from bzrlib.crash import report_bug |
544 |
report_bug(exc_info, err_file) |
|
5017.1.1
by Martin Pool
Add bzrlib.initialize |
545 |
|
546 |
||
547 |
def _flush_stdout_stderr(): |
|
5222.2.3
by Robert Collins
Fix comments up in trace.py. |
548 |
# called from the bzrlib library finalizer returned by bzrlib.initialize()
|
5017.1.1
by Martin Pool
Add bzrlib.initialize |
549 |
try: |
550 |
sys.stdout.flush() |
|
551 |
sys.stderr.flush() |
|
6133.4.62
by John Arbash Meinel
Add a nicer repr for shutting down. |
552 |
except ValueError, e: |
553 |
# On Windows, I get ValueError calling stdout.flush() on a closed
|
|
554 |
# handle
|
|
555 |
pass
|
|
5017.1.1
by Martin Pool
Add bzrlib.initialize |
556 |
except IOError, e: |
557 |
import errno |
|
558 |
if e.errno in [errno.EINVAL, errno.EPIPE]: |
|
559 |
pass
|
|
560 |
else: |
|
561 |
raise
|
|
562 |
||
563 |
||
564 |
def _flush_trace(): |
|
5222.2.3
by Robert Collins
Fix comments up in trace.py. |
565 |
# called from the bzrlib library finalizer returned by bzrlib.initialize()
|
5017.1.1
by Martin Pool
Add bzrlib.initialize |
566 |
global _trace_file |
567 |
if _trace_file: |
|
568 |
_trace_file.flush() |
|
5320.2.5
by Robert Collins
Make bzrlib startup use a trace context manager. |
569 |
|
570 |
||
6015.52.2
by Martin Packman
Add EncodedStreamHandler as more robust logging class |
571 |
class EncodedStreamHandler(logging.Handler): |
572 |
"""Robustly write logging events to a stream using the specified encoding
|
|
573 |
||
574 |
Messages are expected to be formatted to unicode, but UTF-8 byte strings
|
|
575 |
are also accepted. An error during formatting or a str message in another
|
|
576 |
encoding will be quitely noted as an error in the Bazaar log file.
|
|
577 |
||
578 |
The stream is not closed so sys.stdout or sys.stderr may be passed.
|
|
579 |
"""
|
|
580 |
||
581 |
def __init__(self, stream, encoding=None, errors='strict', level=0): |
|
582 |
logging.Handler.__init__(self, level) |
|
583 |
self.stream = stream |
|
584 |
if encoding is None: |
|
585 |
encoding = getattr(stream, "encoding", "ascii") |
|
586 |
self.encoding = encoding |
|
587 |
self.errors = errors |
|
588 |
||
589 |
def flush(self): |
|
590 |
flush = getattr(self.stream, "flush", None) |
|
591 |
if flush is not None: |
|
592 |
flush() |
|
593 |
||
594 |
def emit(self, record): |
|
595 |
try: |
|
596 |
line = self.format(record) |
|
597 |
if not isinstance(line, unicode): |
|
598 |
line = line.decode("utf-8") |
|
599 |
self.stream.write(line.encode(self.encoding, self.errors) + "\n") |
|
600 |
except Exception: |
|
601 |
log_exception_quietly() |
|
602 |
# Try saving the details that would have been logged in some form
|
|
603 |
msg = args = "<Unformattable>" |
|
604 |
try: |
|
605 |
msg = repr(record.msg).encode("ascii") |
|
606 |
args = repr(record.args).encode("ascii") |
|
607 |
except Exception: |
|
608 |
pass
|
|
609 |
# Using mutter() bypasses the logging module and writes directly
|
|
610 |
# to the file so there's no danger of getting into a loop here.
|
|
611 |
mutter("Logging record unformattable: %s %% %s", msg, args) |
|
612 |
||
613 |
||
5320.2.5
by Robert Collins
Make bzrlib startup use a trace context manager. |
614 |
class Config(object): |
615 |
"""Configuration of message tracing in bzrlib.
|
|
616 |
||
617 |
This implements the context manager protocol and should manage any global
|
|
618 |
variables still used. The default config used is DefaultConfig, but
|
|
619 |
embedded uses of bzrlib may wish to use a custom manager.
|
|
620 |
"""
|
|
621 |
||
622 |
def __enter__(self): |
|
623 |
return self # This is bound to the 'as' clause in a with statement. |
|
624 |
||
625 |
def __exit__(self, exc_type, exc_val, exc_tb): |
|
626 |
return False # propogate exceptions. |
|
627 |
||
628 |
||
629 |
class DefaultConfig(Config): |
|
630 |
"""A default configuration for tracing of messages in bzrlib.
|
|
631 |
||
632 |
This implements the context manager protocol.
|
|
633 |
"""
|
|
634 |
||
635 |
def __enter__(self): |
|
636 |
self._original_filename = _bzr_log_filename |
|
637 |
self._original_state = enable_default_logging() |
|
638 |
return self # This is bound to the 'as' clause in a with statement. |
|
639 |
||
640 |
def __exit__(self, exc_type, exc_val, exc_tb): |
|
641 |
pop_log_file(self._original_state) |
|
642 |
global _bzr_log_filename |
|
643 |
_bzr_log_filename = self._original_filename |
|
644 |
return False # propogate exceptions. |