From 6e3f13ffb47102b5df2da91fbc3f5da3179245b2 Mon Sep 17 00:00:00 2001 From: Ed Bartosh Date: Wed, 2 Mar 2016 22:47:41 -0800 Subject: [PATCH] toasterui: check if setEventMask succeeded Currently toasterui ignores return value of setEventMask command, which created confusing difference between set of events set by this command and the real set used in the code. Checked if setEventMask succeeded. Print error message and exit if it's not. Signed-off-by: Ed Bartosh Signed-off-by: brian avery Signed-off-by: Richard Purdie --- lib/bb/ui/toasterui.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/lib/bb/ui/toasterui.py b/lib/bb/ui/toasterui.py index 4bb33f1008f..fec69621f1b 100644 --- a/lib/bb/ui/toasterui.py +++ b/lib/bb/ui/toasterui.py @@ -154,8 +154,10 @@ def main(server, eventHandler, params): logger.addHandler(console) logger.setLevel(logging.INFO) llevel, debug_domains = bb.msg.constructLogOptions() - server.runCommand(["setEventMask", server.getEventHandle(), llevel, debug_domains, _evt_list]) - + result, error = server.runCommand(["setEventMask", server.getEventHandle(), llevel, debug_domains, _evt_list]) + if not result or error: + logger.error("can't set event mask: %s", error) + return 1 # verify and warn build_history_enabled = True -- 2.47.3