]> git.ipfire.org Git - thirdparty/openembedded/openembedded-core-contrib.git/commitdiff
bitbake: lib/bb/codeparser: ensure BufferedLogger respects target logging level
authorPaul Eggleton <paul.eggleton@linux.intel.com>
Tue, 28 Mar 2017 20:41:47 +0000 (09:41 +1300)
committerRichard Purdie <richard.purdie@linuxfoundation.org>
Wed, 29 Mar 2017 15:36:43 +0000 (16:36 +0100)
BufferedLogger was sending log records to the target logger without
checking if the logger is enabled for the level of the record - and
handle() doesn't check this either (it's normally checked earlier when
the relevant log function is called e.g. logger.debug()), leading for
example to debug messages from codeparser getting printed when the log
level for the main BitBake logger was set to logging.WARNING.

(Bitbake rev: 968a77388dd1a24c1dadec6ce49bf53ebb5b643f)

Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
bitbake/lib/bb/codeparser.py

index 5d2d44065ac584547865f6f1b9dde075396e85d1..89d24ab49aff8ff911632be9ca80bc95e54070aa 100644 (file)
@@ -186,7 +186,8 @@ class BufferedLogger(Logger):
 
     def flush(self):
         for record in self.buffer:
-            self.target.handle(record)
+            if self.target.isEnabledFor(record.levelno):
+                self.target.handle(record)
         self.buffer = []
 
 class PythonParser():