[oe-commits] [bitbake] 07/45: toasterui: fix brbe reporting

[git at git.openembedded.org]

rpurdie pushed a commit to branch master
in repository bitbake.

commit 5d7cce0d0ed70f6b3ebd6cbad300d86964a13398
Author: Ed Bartosh <ed.bartosh at linux.intel.com>
AuthorDate: Wed Apr 6 17:46:17 2016 +0100

    toasterui: fix brbe reporting
    
    buildinfohelper.brbe is lost when buildinfohelper is closed.
    This causes incorrect report of brbe when build is done.
    
    Saved brbe attribute before closing buildinfohelper and used
    it to report correct brbe.
    
    Got rid of useless and confusing 'ToasterUI build done 1'
    log message.
    
    Signed-off-by: Ed Bartosh <ed.bartosh at linux.intel.com>
    Signed-off-by: Michael Wood <michael.g.wood at intel.com>
    Signed-off-by: Richard Purdie <richard.purdie at linuxfoundation.org>
---
 lib/bb/ui/toasterui.py | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/lib/bb/ui/toasterui.py b/lib/bb/ui/toasterui.py
index 06a5f17..39a4fe5 100644
--- a/lib/bb/ui/toasterui.py
+++ b/lib/bb/ui/toasterui.py
@@ -372,9 +372,9 @@ def main(server, eventHandler, params):
 
                 # update the build info helper on BuildCompleted, not on CommandXXX
                 buildinfohelper.update_build_information(event, errors, warnings, taskfailures)
+
+                brbe = buildinfohelper.brbe
                 buildinfohelper.close(errorcode)
-                # mark the log output; controllers may kill the toasterUI after seeing this log
-                logger.info("ToasterUI build done 1, brbe: %s", buildinfohelper.brbe )
 
                 # we start a new build info
                 logger.debug("ToasterUI prepared for new build")
@@ -383,7 +383,7 @@ def main(server, eventHandler, params):
                 taskfailures = []
                 buildinfohelper = BuildInfoHelper(server, build_history_enabled)
 
-                logger.info("ToasterUI build done 2")
+                logger.info("ToasterUI build done, brbe: %s", brbe)
                 continue
 
             if isinstance(event, (bb.command.CommandCompleted,

-- 
To stop receiving notification emails like this one, please contact
the administrator of this repository.