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