diff options
authorRichard Purdie <richard.purdie@linuxfoundation.org>2012-12-14 12:30:27 +0000
committerRichard Purdie <richard.purdie@linuxfoundation.org>2012-12-14 12:35:38 +0000
commit3a1b75037fe1e5282ffd40b037e93353405faa45 (patch)
parentc6211291ae07410832031a5274690437cc2b09a6 (diff)
bin/bitbake, bb/__init__: Move BBDEBUG handling to bin/bitbake
The method this older code was accessing doesn't exist. A library shouldn't be changing behaviour based on the environment anyway when sepcific function calls exist to correctly setup the logging. Fix the crash if BBDEBUG was used by moving the code to bin/bitbake. [YOCTO #3319] Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
2 files changed, 5 insertions, 5 deletions
diff --git a/bin/bitbake b/bin/bitbake
index 324ee4fa..e86d59bd 100755
--- a/bin/bitbake
+++ b/bin/bitbake
@@ -214,6 +214,11 @@ Default BBFILES are the .bb files in the current directory.""")
if configuration.bind and configuration.servertype != "xmlrpc":
sys.exit("FATAL: If '-B' or '--bind' is defined, we must set the servertype as 'xmlrpc'.\n")
+ if "BBDEBUG" in os.environ:
+ level = int(os.environ["BBDEBUG"])
+ if level > configuration.debug:
+ configuration.debug = level
bb.msg.init_msgconfig(configuration.verbose, configuration.debug,
diff --git a/lib/bb/__init__.py b/lib/bb/__init__.py
index 660fd5bf..faa94e6c 100644
--- a/lib/bb/__init__.py
+++ b/lib/bb/__init__.py
@@ -74,11 +74,6 @@ logger.setLevel(logging.DEBUG - 2)
# can result in construction of the various loggers.
import bb.msg
-if "BBDEBUG" in os.environ:
- level = int(os.environ["BBDEBUG"])
- if level:
- bb.msg.set_debug_level(level)
from bb import fetch2 as fetch
sys.modules['bb.fetch'] = sys.modules['bb.fetch2']