From 60ca31e09597ec71f71b29dc91db8d3c95783fe1 Mon Sep 17 00:00:00 2001 From: Paul Eggleton Date: Mon, 7 Aug 2017 11:40:42 +0200 Subject: Revert "tinfoil: fix duplication of log messages" In combination with the recent server reworking, this change actually prevents messages sent from tasks from being logged properly. This will of course give us the duplicated messages back, and I really hate to do that effectively a second time, but that's better than seeing no error at all in the case of a failure - we'll have to find the proper way of avoiding the duplication that doesn't result in some messages going missing. This reverts commit 8a5bae76f91f2411187c638a42fa3c762052cf11. Signed-off-by: Paul Eggleton --- lib/bb/main.py | 4 ++-- lib/bb/tinfoil.py | 3 +-- 2 files changed, 3 insertions(+), 4 deletions(-) diff --git a/lib/bb/main.py b/lib/bb/main.py index 431f6f479..c51c6f2e4 100755 --- a/lib/bb/main.py +++ b/lib/bb/main.py @@ -394,10 +394,10 @@ def bitbake_main(configParams, configuration): return 1 -def setup_bitbake(configParams, configuration, extrafeatures=None, setup_logging=True): +def setup_bitbake(configParams, configuration, extrafeatures=None): # Ensure logging messages get sent to the UI as events handler = bb.event.LogHandler() - if setup_logging and not configParams.status_only: + if not configParams.status_only: # In status only mode there are no logs and no UI logger.addHandler(handler) diff --git a/lib/bb/tinfoil.py b/lib/bb/tinfoil.py index e246b3d2a..cd0587e27 100644 --- a/lib/bb/tinfoil.py +++ b/lib/bb/tinfoil.py @@ -383,8 +383,7 @@ class Tinfoil: self.server_connection, ui_module = setup_bitbake(config_params, cookerconfig, - extrafeatures, - setup_logging=False) + extrafeatures) self.ui_module = ui_module -- cgit 1.2.3-korg