Rev 6169: Remove the warning. in file:///home/vila/src/bzr/bugs/863401-library-state/
Vincent Ladeuil
v.ladeuil+lp at free.fr
Mon Oct 3 14:34:45 UTC 2011
At file:///home/vila/src/bzr/bugs/863401-library-state/
------------------------------------------------------------
revno: 6169
revision-id: v.ladeuil+lp at free.fr-20111003143445-s382csvk1613x0sp
parent: v.ladeuil+lp at free.fr-20110930175225-03pqlfa6lf7tdnqz
committer: Vincent Ladeuil <v.ladeuil+lp at free.fr>
branch nick: 863401-library-state
timestamp: Mon 2011-10-03 16:34:45 +0200
message:
Remove the warning.
-------------- next part --------------
=== modified file 'bzrlib/config.py'
--- a/bzrlib/config.py 2011-09-30 17:52:25 +0000
+++ b/bzrlib/config.py 2011-10-03 14:34:45 +0000
@@ -3276,7 +3276,6 @@
# Hack around library_state.initialize never called
if bzrlib.global_state is not None:
return [bzrlib.global_state.cmdline_overrides]
- trace.warning("You forgot to use 'with bzrlib.initialize():'")
return []
=== modified file 'bzrlib/tests/test_library_state.py'
--- a/bzrlib/tests/test_library_state.py 2011-09-30 17:52:25 +0000
+++ b/bzrlib/tests/test_library_state.py 2011-10-03 14:34:45 +0000
@@ -16,12 +16,9 @@
"""Tests for BzrLibraryState."""
-import bzrlib
from bzrlib import (
- config,
library_state,
tests,
- trace,
ui as _mod_ui
)
from bzrlib.tests import fixtures
@@ -52,17 +49,3 @@
finally:
state.__exit__(None, None, None)
self.assertEqual(['__enter__', '__exit__'], tracer._calls)
-
- def test_warns_if_not_called(self):
- self.overrideAttr(bzrlib, 'global_state', None)
- warnings = []
- def warning(*args):
- warnings.append(args[0] % args[1:])
- self.overrideAttr(trace, 'warning', warning)
- # Querying for a an option requires a real global_state or emits a
- # warning
- c = config.GlobalStack()
- v = c.get('whatever')
- self.assertLength(1, warnings)
- self.assertEquals("You forgot to use 'with bzrlib.initialize():'",
- warnings[0])
=== modified file 'doc/en/release-notes/bzr-2.5.txt'
--- a/doc/en/release-notes/bzr-2.5.txt 2011-09-30 17:52:25 +0000
+++ b/doc/en/release-notes/bzr-2.5.txt 2011-10-03 14:34:45 +0000
@@ -55,8 +55,8 @@
turned into a line-basde one when ``INSIDE_EMACS`` is set (which is the
case for all recent emacs versions). (Vincent Ladeuil, #856261)
-* Emit a warning but don't crash if bzrlib.initialize() has not been called.
- (Vincent Ladeuil, #863401)
+* Don't crash if bzrlib.initialize() has not been called while accessing
+ configs. (Vincent Ladeuil, #863401)
* Redirects between http and https no longer discard path information
in some cases. (Jelmer Vernooij, #853765)
More information about the bazaar-commits
mailing list