* [for-denzil] bitbake: command: add error to return of runCommand
@ 2012-12-18 17:18 Matthew McClintock
2012-12-18 21:28 ` Richard Purdie
0 siblings, 1 reply; 5+ messages in thread
From: Matthew McClintock @ 2012-12-18 17:18 UTC (permalink / raw)
To: openembedded-core
From: Christopher Larson <chris_larson@mentor.com>
Currently, command.py can return an error message from runCommand, due to
being unable to run the command, yet few of our UIs (just hob) can handle it
today. This can result in seeing a TypeError with traceback in certain rare
circumstances.
To resolve this, we need a clean way to get errors back from runCommand,
without having to isinstance() the return value. This implements such a thing
by making runCommand also return an error (or None if no error occurred).
As runCommand now has a method of returning errors, we can also alter the
getCmdLineAction bits such that the returned value is just the action, not an
additional message. If a sync command wants to return an error, it raises
CommandError(message), and the message will be passed to the caller
appropriately.
Example Usage:
result, error = server.runCommand(...)
if error:
log.error('Unable to run command: %s' % error)
return 1
(Bitbake rev: 717831b8315cb3904d9b590e633000bc897e8fb6)
Signed-off-by: Christopher Larson <chris_larson@mentor.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
---
bitbake/lib/bb/command.py | 43 +++++++++++++++----------
bitbake/lib/bb/server/process.py | 2 +-
bitbake/lib/bb/ui/crumbs/hobeventhandler.py | 5 ++-
bitbake/lib/bb/ui/depexp.py | 38 ++++++++++++++--------
bitbake/lib/bb/ui/goggle.py | 17 +++++-----
bitbake/lib/bb/ui/knotty.py | 45 ++++++++++++++++++---------
bitbake/lib/bb/ui/ncurses.py | 21 ++++++++-----
7 files changed, 112 insertions(+), 59 deletions(-)
diff --git a/bitbake/lib/bb/command.py b/bitbake/lib/bb/command.py
index fd8912a..00b854e 100644
--- a/bitbake/lib/bb/command.py
+++ b/bitbake/lib/bb/command.py
@@ -44,6 +44,9 @@ class CommandFailed(CommandExit):
self.error = message
CommandExit.__init__(self, 1)
+class CommandError(Exception):
+ pass
+
class Command:
"""
A queue of asynchronous commands for bitbake
@@ -57,21 +60,25 @@ class Command:
self.currentAsyncCommand = None
def runCommand(self, commandline):
- try:
- command = commandline.pop(0)
- if command in CommandsSync.__dict__:
- # Can run synchronous commands straight away
- return getattr(CommandsSync, command)(self.cmds_sync, self, commandline)
- if self.currentAsyncCommand is not None:
- return "Busy (%s in progress)" % self.currentAsyncCommand[0]
- if command not in CommandsAsync.__dict__:
- return "No such command"
- self.currentAsyncCommand = (command, commandline)
- self.cooker.server_registration_cb(self.cooker.runCommands, self.cooker)
- return True
- except:
- import traceback
- return traceback.format_exc()
+ command = commandline.pop(0)
+ if hasattr(CommandsSync, command):
+ # Can run synchronous commands straight away
+ command_method = getattr(self.cmds_sync, command)
+ try:
+ result = command_method(self, commandline)
+ except CommandError as exc:
+ return None, exc.args[0]
+ except Exception:
+ return None, traceback.format_exc()
+ else:
+ return result, None
+ if self.currentAsyncCommand is not None:
+ return None, "Busy (%s in progress)" % self.currentAsyncCommand[0]
+ if command not in CommandsAsync.__dict__:
+ return None, "No such command"
+ self.currentAsyncCommand = (command, commandline)
+ self.cooker.server_registration_cb(self.cooker.runCommands, self.cooker)
+ return True, None
def runAsyncCommand(self):
try:
@@ -139,7 +146,11 @@ class CommandsSync:
"""
Get any command parsed from the commandline
"""
- return command.cooker.commandlineAction
+ cmd_action = command.cooker.commandlineAction
+ if cmd_action['msg']:
+ raise CommandError(msg)
+ else:
+ return cmd_action['action']
def getVariable(self, command, params):
"""
diff --git a/bitbake/lib/bb/server/process.py b/bitbake/lib/bb/server/process.py
index ba91336..3ae9ede 100644
--- a/bitbake/lib/bb/server/process.py
+++ b/bitbake/lib/bb/server/process.py
@@ -48,7 +48,7 @@ class ServerCommunicator():
if self.connection.poll(.5):
return self.connection.recv()
else:
- return None
+ return None, "Timeout while attempting to communicate with bitbake server"
except KeyboardInterrupt:
pass
diff --git a/bitbake/lib/bb/ui/crumbs/hobeventhandler.py b/bitbake/lib/bb/ui/crumbs/hobeventhandler.py
index 52acec1..21bea23 100644
--- a/bitbake/lib/bb/ui/crumbs/hobeventhandler.py
+++ b/bitbake/lib/bb/ui/crumbs/hobeventhandler.py
@@ -102,7 +102,10 @@ class HobHandler(gobject.GObject):
def runCommand(self, commandline):
try:
- return self.server.runCommand(commandline)
+ result, error = self.server.runCommand(commandline)
+ if error:
+ raise Exception("Error running command '%s': %s" % (commandline, error))
+ return result
except Exception as e:
self.commands_async = []
self.clear_busy()
diff --git a/bitbake/lib/bb/ui/depexp.py b/bitbake/lib/bb/ui/depexp.py
index b62adbf..1254128 100644
--- a/bitbake/lib/bb/ui/depexp.py
+++ b/bitbake/lib/bb/ui/depexp.py
@@ -198,17 +198,23 @@ class gtkthread(threading.Thread):
def main(server, eventHandler):
try:
- cmdline = server.runCommand(["getCmdLineAction"])
- if cmdline and not cmdline['action']:
- print(cmdline['msg'])
- return
- elif not cmdline or (cmdline['action'] and cmdline['action'][0] != "generateDotGraph"):
+ cmdline, error = server.runCommand(["getCmdLineAction"])
+ if error:
+ print("Error getting bitbake commandline: %s" % error)
+ return 1
+ elif not cmdline:
+ print("Nothing to do. Use 'bitbake world' to build everything, or run 'bitbake --help' for usage information.")
+ return 1
+ elif not cmdline or cmdline[0] != "generateDotGraph":
print("This UI is only compatible with the -g option")
- return
- ret = server.runCommand(["generateDepTreeEvent", cmdline['action'][1], cmdline['action'][2]])
- if ret != True:
- print("Couldn't run command! %s" % ret)
- return
+ return 1
+ ret, error = server.runCommand(["generateDepTreeEvent", cmdline[1], cmdline[2]])
+ if error:
+ print("Error running command '%s': %s" % (cmdline, error))
+ return 1
+ elif ret != True:
+ print("Error running command '%s': returned %s" % (cmdline, ret))
+ return 1
except xmlrpclib.Fault as x:
print("XMLRPC Fault getting commandline:\n %s" % x)
return
@@ -229,7 +235,9 @@ def main(server, eventHandler):
try:
event = eventHandler.waitEvent(0.25)
if gtkthread.quit.isSet():
- server.runCommand(["stateStop"])
+ _, error = server.runCommand(["stateStop"])
+ if error:
+ print('Unable to cleanly stop: %s' % error)
break
if event is None:
@@ -302,9 +310,13 @@ def main(server, eventHandler):
break
if shutdown == 1:
print("\nSecond Keyboard Interrupt, stopping...\n")
- server.runCommand(["stateStop"])
+ _, error = server.runCommand(["stateStop"])
+ if error:
+ print('Unable to cleanly stop: %s' % error)
if shutdown == 0:
print("\nKeyboard Interrupt, closing down...\n")
- server.runCommand(["stateShutdown"])
+ _, error = server.runCommand(["stateShutdown"])
+ if error:
+ print('Unable to cleanly shutdown: %s' % error)
shutdown = shutdown + 1
pass
diff --git a/bitbake/lib/bb/ui/goggle.py b/bitbake/lib/bb/ui/goggle.py
index b2fd274..c0785b7 100644
--- a/bitbake/lib/bb/ui/goggle.py
+++ b/bitbake/lib/bb/ui/goggle.py
@@ -80,16 +80,19 @@ def main (server, eventHandler):
running_build.connect ("build-failed", running_build_failed_cb)
try:
- cmdline = server.runCommand(["getCmdLineAction"])
- if not cmdline:
+ cmdline, error = server.runCommand(["getCmdLineAction"])
+ if err:
+ print("Error getting bitbake commandline: %s" % error)
+ return 1
+ elif not cmdline:
print("Nothing to do. Use 'bitbake world' to build everything, or run 'bitbake --help' for usage information.")
return 1
- elif not cmdline['action']:
- print(cmdline['msg'])
+ ret, error = server.runCommand(cmdline)
+ if error:
+ print("Error running command '%s': %s" % (cmdline, error))
return 1
- ret = server.runCommand(cmdline['action'])
- if ret != True:
- print("Couldn't get default commandline! %s" % ret)
+ elif ret != True:
+ print("Error running command '%s': returned %s" % (cmdline, ret))
return 1
except xmlrpclib.Fault as x:
print("XMLRPC Fault getting commandline:\n %s" % x)
diff --git a/bitbake/lib/bb/ui/knotty.py b/bitbake/lib/bb/ui/knotty.py
index 0167416..439a792 100644
--- a/bitbake/lib/bb/ui/knotty.py
+++ b/bitbake/lib/bb/ui/knotty.py
@@ -107,9 +107,18 @@ class TerminalFilter(object):
def main(server, eventHandler, tf = TerminalFilter):
# Get values of variables which control our output
- includelogs = server.runCommand(["getVariable", "BBINCLUDELOGS"])
- loglines = server.runCommand(["getVariable", "BBINCLUDELOGS_LINES"])
- consolelogfile = server.runCommand(["getVariable", "BB_CONSOLELOG"])
+ includelogs, error = server.runCommand(["getVariable", "BBINCLUDELOGS"])
+ if error:
+ logger.error("Unable to get the value of BBINCLUDELOGS variable: %s" % error)
+ return 1
+ loglines, error = server.runCommand(["getVariable", "BBINCLUDELOGS_LINES"])
+ if error:
+ logger.error("Unable to get the value of BBINCLUDELOGS_LINES variable: %s" % error)
+ return 1
+ consolelogfile, error = server.runCommand(["getVariable", "BB_CONSOLELOG"])
+ if error:
+ logger.error("Unable to get the value of BB_CONSOLELOG variable: %s" % error)
+ return 1
helper = uihelper.BBUIHelper()
@@ -125,19 +134,22 @@ def main(server, eventHandler, tf = TerminalFilter):
logger.addHandler(consolelog)
try:
- cmdline = server.runCommand(["getCmdLineAction"])
- if not cmdline:
+ cmdline, error = server.runCommand(["getCmdLineAction"])
+ if error:
+ logger.error("Unable to get bitbake commandline arguments: %s" % error)
+ return 1
+ elif not cmdline:
print("Nothing to do. Use 'bitbake world' to build everything, or run 'bitbake --help' for usage information.")
return 1
- elif not cmdline['action']:
- print(cmdline['msg'])
+ ret, error = server.runCommand(cmdline)
+ if error:
+ logger.error("Command '%s' failed: %s" % (cmdline, error))
return 1
- ret = server.runCommand(cmdline['action'])
- if ret != True:
- print("Couldn't get default commandline! %s" % ret)
+ elif ret != True:
+ logger.error("Command '%s' failed: returned %s" % (cmdline, ret))
return 1
except xmlrpclib.Fault as x:
- print("XMLRPC Fault getting commandline:\n %s" % x)
+ logger.error("XMLRPC Fault getting commandline:\n %s" % x)
return 1
parseprogress = None
@@ -318,14 +330,19 @@ def main(server, eventHandler, tf = TerminalFilter):
if ioerror.args[0] == 4:
pass
except KeyboardInterrupt:
+ import time
termfilter.clearFooter()
if main.shutdown == 1:
print("\nSecond Keyboard Interrupt, stopping...\n")
- server.runCommand(["stateStop"])
+ _, error = server.runCommand(["stateStop"])
+ if error:
+ logger.error("Unable to cleanly stop: %s" % error)
if main.shutdown == 0:
- interrupted = True
print("\nKeyboard Interrupt, closing down...\n")
- server.runCommand(["stateShutdown"])
+ interrupted = True
+ _, error = server.runCommand(["stateShutdown"])
+ if error:
+ logger.error("Unable to cleanly shutdown: %s" % error)
main.shutdown = main.shutdown + 1
pass
diff --git a/bitbake/lib/bb/ui/ncurses.py b/bitbake/lib/bb/ui/ncurses.py
index 8524446..896ed01 100644
--- a/bitbake/lib/bb/ui/ncurses.py
+++ b/bitbake/lib/bb/ui/ncurses.py
@@ -230,15 +230,18 @@ class NCursesUI:
shutdown = 0
try:
- cmdline = server.runCommand(["getCmdLineAction"])
+ cmdline, error = server.runCommand(["getCmdLineAction"])
if not cmdline:
print("Nothing to do. Use 'bitbake world' to build everything, or run 'bitbake --help' for usage information.")
return
- elif not cmdline['action']:
- print(cmdline['msg'])
+ elif error:
+ print("Error getting bitbake commandline: %s" % error)
return
- ret = server.runCommand(cmdline['action'])
- if ret != True:
+ ret, error = server.runCommand(cmdline)
+ if error:
+ print("Error running command '%s': %s" % (cmdline, error))
+ return
+ elif ret != True:
print("Couldn't get default commandlind! %s" % ret)
return
except xmlrpclib.Fault as x:
@@ -337,10 +340,14 @@ class NCursesUI:
exitflag = True
if shutdown == 1:
mw.appendText("Second Keyboard Interrupt, stopping...\n")
- server.runCommand(["stateStop"])
+ _, error = server.runCommand(["stateStop"])
+ if error:
+ print("Unable to cleanly stop: %s" % error)
if shutdown == 0:
mw.appendText("Keyboard Interrupt, closing down...\n")
- server.runCommand(["stateShutdown"])
+ _, error = server.runCommand(["stateShutdown"])
+ if error:
+ print("Unable to cleanly shutdown: %s" % error)
shutdown = shutdown + 1
pass
--
1.7.9.7
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [for-denzil] bitbake: command: add error to return of runCommand
2012-12-18 17:18 [for-denzil] bitbake: command: add error to return of runCommand Matthew McClintock
@ 2012-12-18 21:28 ` Richard Purdie
2012-12-19 19:01 ` McClintock Matthew-B29882
0 siblings, 1 reply; 5+ messages in thread
From: Richard Purdie @ 2012-12-18 21:28 UTC (permalink / raw)
To: Matthew McClintock; +Cc: openembedded-core
On Tue, 2012-12-18 at 11:18 -0600, Matthew McClintock wrote:
> From: Christopher Larson <chris_larson@mentor.com>
>
> Currently, command.py can return an error message from runCommand, due to
> being unable to run the command, yet few of our UIs (just hob) can handle it
> today. This can result in seeing a TypeError with traceback in certain rare
> circumstances.
>
> To resolve this, we need a clean way to get errors back from runCommand,
> without having to isinstance() the return value. This implements such a thing
> by making runCommand also return an error (or None if no error occurred).
>
> As runCommand now has a method of returning errors, we can also alter the
> getCmdLineAction bits such that the returned value is just the action, not an
> additional message. If a sync command wants to return an error, it raises
> CommandError(message), and the message will be passed to the caller
> appropriately.
>
> Example Usage:
>
> result, error = server.runCommand(...)
> if error:
> log.error('Unable to run command: %s' % error)
> return 1
>
> (Bitbake rev: 717831b8315cb3904d9b590e633000bc897e8fb6)
This patch has bugs in it. See recent fixes in master.
> Signed-off-by: Christopher Larson <chris_larson@mentor.com>
> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> ---
> bitbake/lib/bb/command.py | 43 +++++++++++++++----------
> bitbake/lib/bb/server/process.py | 2 +-
> bitbake/lib/bb/ui/crumbs/hobeventhandler.py | 5 ++-
> bitbake/lib/bb/ui/depexp.py | 38 ++++++++++++++--------
> bitbake/lib/bb/ui/goggle.py | 17 +++++-----
> bitbake/lib/bb/ui/knotty.py | 45 ++++++++++++++++++---------
> bitbake/lib/bb/ui/ncurses.py | 21 ++++++++-----
> 7 files changed, 112 insertions(+), 59 deletions(-)
>
> diff --git a/bitbake/lib/bb/command.py b/bitbake/lib/bb/command.py
> index fd8912a..00b854e 100644
> --- a/bitbake/lib/bb/command.py
> +++ b/bitbake/lib/bb/command.py
> @@ -44,6 +44,9 @@ class CommandFailed(CommandExit):
> self.error = message
> CommandExit.__init__(self, 1)
>
> +class CommandError(Exception):
> + pass
> +
> class Command:
> """
> A queue of asynchronous commands for bitbake
> @@ -57,21 +60,25 @@ class Command:
> self.currentAsyncCommand = None
>
> def runCommand(self, commandline):
> - try:
> - command = commandline.pop(0)
> - if command in CommandsSync.__dict__:
> - # Can run synchronous commands straight away
> - return getattr(CommandsSync, command)(self.cmds_sync, self, commandline)
> - if self.currentAsyncCommand is not None:
> - return "Busy (%s in progress)" % self.currentAsyncCommand[0]
> - if command not in CommandsAsync.__dict__:
> - return "No such command"
> - self.currentAsyncCommand = (command, commandline)
> - self.cooker.server_registration_cb(self.cooker.runCommands, self.cooker)
> - return True
> - except:
> - import traceback
> - return traceback.format_exc()
> + command = commandline.pop(0)
> + if hasattr(CommandsSync, command):
> + # Can run synchronous commands straight away
> + command_method = getattr(self.cmds_sync, command)
> + try:
> + result = command_method(self, commandline)
> + except CommandError as exc:
> + return None, exc.args[0]
> + except Exception:
> + return None, traceback.format_exc()
Missing import traceback.
> + else:
> + return result, None
> + if self.currentAsyncCommand is not None:
> + return None, "Busy (%s in progress)" % self.currentAsyncCommand[0]
> + if command not in CommandsAsync.__dict__:
> + return None, "No such command"
> + self.currentAsyncCommand = (command, commandline)
> + self.cooker.server_registration_cb(self.cooker.runCommands, self.cooker)
> + return True, None
>
> def runAsyncCommand(self):
> try:
> @@ -139,7 +146,11 @@ class CommandsSync:
> """
> Get any command parsed from the commandline
> """
> - return command.cooker.commandlineAction
> + cmd_action = command.cooker.commandlineAction
> + if cmd_action['msg']:
> + raise CommandError(msg)
Error, "msg" not defined.
Cheers,
Richard
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [for-denzil] bitbake: command: add error to return of runCommand
2012-12-18 21:28 ` Richard Purdie
@ 2012-12-19 19:01 ` McClintock Matthew-B29882
0 siblings, 0 replies; 5+ messages in thread
From: McClintock Matthew-B29882 @ 2012-12-19 19:01 UTC (permalink / raw)
To: Richard Purdie; +Cc: openembedded-core
On Tue, Dec 18, 2012 at 3:28 PM, Richard Purdie
<richard.purdie@linuxfoundation.org> wrote:
> On Tue, 2012-12-18 at 11:18 -0600, Matthew McClintock wrote:
>> From: Christopher Larson <chris_larson@mentor.com>
>>
>> Currently, command.py can return an error message from runCommand, due to
>> being unable to run the command, yet few of our UIs (just hob) can handle it
>> today. This can result in seeing a TypeError with traceback in certain rare
>> circumstances.
>>
>> To resolve this, we need a clean way to get errors back from runCommand,
>> without having to isinstance() the return value. This implements such a thing
>> by making runCommand also return an error (or None if no error occurred).
>>
>> As runCommand now has a method of returning errors, we can also alter the
>> getCmdLineAction bits such that the returned value is just the action, not an
>> additional message. If a sync command wants to return an error, it raises
>> CommandError(message), and the message will be passed to the caller
>> appropriately.
>>
>> Example Usage:
>>
>> result, error = server.runCommand(...)
>> if error:
>> log.error('Unable to run command: %s' % error)
>> return 1
>>
>> (Bitbake rev: 717831b8315cb3904d9b590e633000bc897e8fb6)
>
> This patch has bugs in it. See recent fixes in master.
>
>> Signed-off-by: Christopher Larson <chris_larson@mentor.com>
>> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
>> ---
>> bitbake/lib/bb/command.py | 43 +++++++++++++++----------
>> bitbake/lib/bb/server/process.py | 2 +-
>> bitbake/lib/bb/ui/crumbs/hobeventhandler.py | 5 ++-
>> bitbake/lib/bb/ui/depexp.py | 38 ++++++++++++++--------
>> bitbake/lib/bb/ui/goggle.py | 17 +++++-----
>> bitbake/lib/bb/ui/knotty.py | 45 ++++++++++++++++++---------
>> bitbake/lib/bb/ui/ncurses.py | 21 ++++++++-----
>> 7 files changed, 112 insertions(+), 59 deletions(-)
>>
>> diff --git a/bitbake/lib/bb/command.py b/bitbake/lib/bb/command.py
>> index fd8912a..00b854e 100644
>> --- a/bitbake/lib/bb/command.py
>> +++ b/bitbake/lib/bb/command.py
>> @@ -44,6 +44,9 @@ class CommandFailed(CommandExit):
>> self.error = message
>> CommandExit.__init__(self, 1)
>>
>> +class CommandError(Exception):
>> + pass
>> +
>> class Command:
>> """
>> A queue of asynchronous commands for bitbake
>> @@ -57,21 +60,25 @@ class Command:
>> self.currentAsyncCommand = None
>>
>> def runCommand(self, commandline):
>> - try:
>> - command = commandline.pop(0)
>> - if command in CommandsSync.__dict__:
>> - # Can run synchronous commands straight away
>> - return getattr(CommandsSync, command)(self.cmds_sync, self, commandline)
>> - if self.currentAsyncCommand is not None:
>> - return "Busy (%s in progress)" % self.currentAsyncCommand[0]
>> - if command not in CommandsAsync.__dict__:
>> - return "No such command"
>> - self.currentAsyncCommand = (command, commandline)
>> - self.cooker.server_registration_cb(self.cooker.runCommands, self.cooker)
>> - return True
>> - except:
>> - import traceback
>> - return traceback.format_exc()
>> + command = commandline.pop(0)
>> + if hasattr(CommandsSync, command):
>> + # Can run synchronous commands straight away
>> + command_method = getattr(self.cmds_sync, command)
>> + try:
>> + result = command_method(self, commandline)
>> + except CommandError as exc:
>> + return None, exc.args[0]
>> + except Exception:
>> + return None, traceback.format_exc()
>
> Missing import traceback.
>
>> + else:
>> + return result, None
>> + if self.currentAsyncCommand is not None:
>> + return None, "Busy (%s in progress)" % self.currentAsyncCommand[0]
>> + if command not in CommandsAsync.__dict__:
>> + return None, "No such command"
>> + self.currentAsyncCommand = (command, commandline)
>> + self.cooker.server_registration_cb(self.cooker.runCommands, self.cooker)
>> + return True, None
>>
>> def runAsyncCommand(self):
>> try:
>> @@ -139,7 +146,11 @@ class CommandsSync:
>> """
>> Get any command parsed from the commandline
>> """
>> - return command.cooker.commandlineAction
>> + cmd_action = command.cooker.commandlineAction
>> + if cmd_action['msg']:
>> + raise CommandError(msg)
>
> Error, "msg" not defined.
Thanks, will look for these and add them / send to list as well. Look
for another patch when I get to it.
-M
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [for-denzil] bitbake: command: add error to return of runCommand
2012-11-01 18:16 Matthew McClintock
@ 2012-11-30 2:56 ` Scott Garman
0 siblings, 0 replies; 5+ messages in thread
From: Scott Garman @ 2012-11-30 2:56 UTC (permalink / raw)
To: poky
On 11/01/2012 11:16 AM, Matthew McClintock wrote:
> From: Christopher Larson <chris_larson@mentor.com>
>
> Currently, command.py can return an error message from runCommand, due to
> being unable to run the command, yet few of our UIs (just hob) can handle it
> today. This can result in seeing a TypeError with traceback in certain rare
> circumstances.
>
> To resolve this, we need a clean way to get errors back from runCommand,
> without having to isinstance() the return value. This implements such a thing
> by making runCommand also return an error (or None if no error occurred).
>
> As runCommand now has a method of returning errors, we can also alter the
> getCmdLineAction bits such that the returned value is just the action, not an
> additional message. If a sync command wants to return an error, it raises
> CommandError(message), and the message will be passed to the caller
> appropriately.
>
> Example Usage:
>
> result, error = server.runCommand(...)
> if error:
> log.error('Unable to run command: %s' % error)
> return 1
>
> (Bitbake rev: 717831b8315cb3904d9b590e633000bc897e8fb6)
>
> Signed-off-by: Christopher Larson <chris_larson@mentor.com>
> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> ---
> bitbake/lib/bb/command.py | 43 +++++++++++++++----------
> bitbake/lib/bb/server/process.py | 2 +-
> bitbake/lib/bb/ui/crumbs/hobeventhandler.py | 5 ++-
> bitbake/lib/bb/ui/depexp.py | 38 ++++++++++++++--------
> bitbake/lib/bb/ui/goggle.py | 17 +++++-----
> bitbake/lib/bb/ui/knotty.py | 45 ++++++++++++++++++---------
> bitbake/lib/bb/ui/ncurses.py | 21 ++++++++-----
> 7 files changed, 112 insertions(+), 59 deletions(-)
>
> diff --git a/bitbake/lib/bb/command.py b/bitbake/lib/bb/command.py
> index fd8912a..00b854e 100644
> --- a/bitbake/lib/bb/command.py
> +++ b/bitbake/lib/bb/command.py
> @@ -44,6 +44,9 @@ class CommandFailed(CommandExit):
> self.error = message
> CommandExit.__init__(self, 1)
>
> +class CommandError(Exception):
> + pass
> +
> class Command:
> """
> A queue of asynchronous commands for bitbake
> @@ -57,21 +60,25 @@ class Command:
> self.currentAsyncCommand = None
>
> def runCommand(self, commandline):
> - try:
> - command = commandline.pop(0)
> - if command in CommandsSync.__dict__:
> - # Can run synchronous commands straight away
> - return getattr(CommandsSync, command)(self.cmds_sync, self, commandline)
> - if self.currentAsyncCommand is not None:
> - return "Busy (%s in progress)" % self.currentAsyncCommand[0]
> - if command not in CommandsAsync.__dict__:
> - return "No such command"
> - self.currentAsyncCommand = (command, commandline)
> - self.cooker.server_registration_cb(self.cooker.runCommands, self.cooker)
> - return True
> - except:
> - import traceback
> - return traceback.format_exc()
> + command = commandline.pop(0)
> + if hasattr(CommandsSync, command):
> + # Can run synchronous commands straight away
> + command_method = getattr(self.cmds_sync, command)
> + try:
> + result = command_method(self, commandline)
> + except CommandError as exc:
> + return None, exc.args[0]
> + except Exception:
> + return None, traceback.format_exc()
> + else:
> + return result, None
> + if self.currentAsyncCommand is not None:
> + return None, "Busy (%s in progress)" % self.currentAsyncCommand[0]
> + if command not in CommandsAsync.__dict__:
> + return None, "No such command"
> + self.currentAsyncCommand = (command, commandline)
> + self.cooker.server_registration_cb(self.cooker.runCommands, self.cooker)
> + return True, None
>
> def runAsyncCommand(self):
> try:
> @@ -139,7 +146,11 @@ class CommandsSync:
> """
> Get any command parsed from the commandline
> """
> - return command.cooker.commandlineAction
> + cmd_action = command.cooker.commandlineAction
> + if cmd_action['msg']:
> + raise CommandError(msg)
> + else:
> + return cmd_action['action']
>
> def getVariable(self, command, params):
> """
> diff --git a/bitbake/lib/bb/server/process.py b/bitbake/lib/bb/server/process.py
> index ba91336..3ae9ede 100644
> --- a/bitbake/lib/bb/server/process.py
> +++ b/bitbake/lib/bb/server/process.py
> @@ -48,7 +48,7 @@ class ServerCommunicator():
> if self.connection.poll(.5):
> return self.connection.recv()
> else:
> - return None
> + return None, "Timeout while attempting to communicate with bitbake server"
> except KeyboardInterrupt:
> pass
>
> diff --git a/bitbake/lib/bb/ui/crumbs/hobeventhandler.py b/bitbake/lib/bb/ui/crumbs/hobeventhandler.py
> index 52acec1..21bea23 100644
> --- a/bitbake/lib/bb/ui/crumbs/hobeventhandler.py
> +++ b/bitbake/lib/bb/ui/crumbs/hobeventhandler.py
> @@ -102,7 +102,10 @@ class HobHandler(gobject.GObject):
>
> def runCommand(self, commandline):
> try:
> - return self.server.runCommand(commandline)
> + result, error = self.server.runCommand(commandline)
> + if error:
> + raise Exception("Error running command '%s': %s" % (commandline, error))
> + return result
> except Exception as e:
> self.commands_async = []
> self.clear_busy()
> diff --git a/bitbake/lib/bb/ui/depexp.py b/bitbake/lib/bb/ui/depexp.py
> index b62adbf..1254128 100644
> --- a/bitbake/lib/bb/ui/depexp.py
> +++ b/bitbake/lib/bb/ui/depexp.py
> @@ -198,17 +198,23 @@ class gtkthread(threading.Thread):
>
> def main(server, eventHandler):
> try:
> - cmdline = server.runCommand(["getCmdLineAction"])
> - if cmdline and not cmdline['action']:
> - print(cmdline['msg'])
> - return
> - elif not cmdline or (cmdline['action'] and cmdline['action'][0] != "generateDotGraph"):
> + cmdline, error = server.runCommand(["getCmdLineAction"])
> + if error:
> + print("Error getting bitbake commandline: %s" % error)
> + return 1
> + elif not cmdline:
> + print("Nothing to do. Use 'bitbake world' to build everything, or run 'bitbake --help' for usage information.")
> + return 1
> + elif not cmdline or cmdline[0] != "generateDotGraph":
> print("This UI is only compatible with the -g option")
> - return
> - ret = server.runCommand(["generateDepTreeEvent", cmdline['action'][1], cmdline['action'][2]])
> - if ret != True:
> - print("Couldn't run command! %s" % ret)
> - return
> + return 1
> + ret, error = server.runCommand(["generateDepTreeEvent", cmdline[1], cmdline[2]])
> + if error:
> + print("Error running command '%s': %s" % (cmdline, error))
> + return 1
> + elif ret != True:
> + print("Error running command '%s': returned %s" % (cmdline, ret))
> + return 1
> except xmlrpclib.Fault as x:
> print("XMLRPC Fault getting commandline:\n %s" % x)
> return
> @@ -229,7 +235,9 @@ def main(server, eventHandler):
> try:
> event = eventHandler.waitEvent(0.25)
> if gtkthread.quit.isSet():
> - server.runCommand(["stateStop"])
> + _, error = server.runCommand(["stateStop"])
> + if error:
> + print('Unable to cleanly stop: %s' % error)
> break
>
> if event is None:
> @@ -302,9 +310,13 @@ def main(server, eventHandler):
> break
> if shutdown == 1:
> print("\nSecond Keyboard Interrupt, stopping...\n")
> - server.runCommand(["stateStop"])
> + _, error = server.runCommand(["stateStop"])
> + if error:
> + print('Unable to cleanly stop: %s' % error)
> if shutdown == 0:
> print("\nKeyboard Interrupt, closing down...\n")
> - server.runCommand(["stateShutdown"])
> + _, error = server.runCommand(["stateShutdown"])
> + if error:
> + print('Unable to cleanly shutdown: %s' % error)
> shutdown = shutdown + 1
> pass
> diff --git a/bitbake/lib/bb/ui/goggle.py b/bitbake/lib/bb/ui/goggle.py
> index b2fd274..c0785b7 100644
> --- a/bitbake/lib/bb/ui/goggle.py
> +++ b/bitbake/lib/bb/ui/goggle.py
> @@ -80,16 +80,19 @@ def main (server, eventHandler):
> running_build.connect ("build-failed", running_build_failed_cb)
>
> try:
> - cmdline = server.runCommand(["getCmdLineAction"])
> - if not cmdline:
> + cmdline, error = server.runCommand(["getCmdLineAction"])
> + if err:
> + print("Error getting bitbake commandline: %s" % error)
> + return 1
> + elif not cmdline:
> print("Nothing to do. Use 'bitbake world' to build everything, or run 'bitbake --help' for usage information.")
> return 1
> - elif not cmdline['action']:
> - print(cmdline['msg'])
> + ret, error = server.runCommand(cmdline)
> + if error:
> + print("Error running command '%s': %s" % (cmdline, error))
> return 1
> - ret = server.runCommand(cmdline['action'])
> - if ret != True:
> - print("Couldn't get default commandline! %s" % ret)
> + elif ret != True:
> + print("Error running command '%s': returned %s" % (cmdline, ret))
> return 1
> except xmlrpclib.Fault as x:
> print("XMLRPC Fault getting commandline:\n %s" % x)
> diff --git a/bitbake/lib/bb/ui/knotty.py b/bitbake/lib/bb/ui/knotty.py
> index 0167416..439a792 100644
> --- a/bitbake/lib/bb/ui/knotty.py
> +++ b/bitbake/lib/bb/ui/knotty.py
> @@ -107,9 +107,18 @@ class TerminalFilter(object):
> def main(server, eventHandler, tf = TerminalFilter):
>
> # Get values of variables which control our output
> - includelogs = server.runCommand(["getVariable", "BBINCLUDELOGS"])
> - loglines = server.runCommand(["getVariable", "BBINCLUDELOGS_LINES"])
> - consolelogfile = server.runCommand(["getVariable", "BB_CONSOLELOG"])
> + includelogs, error = server.runCommand(["getVariable", "BBINCLUDELOGS"])
> + if error:
> + logger.error("Unable to get the value of BBINCLUDELOGS variable: %s" % error)
> + return 1
> + loglines, error = server.runCommand(["getVariable", "BBINCLUDELOGS_LINES"])
> + if error:
> + logger.error("Unable to get the value of BBINCLUDELOGS_LINES variable: %s" % error)
> + return 1
> + consolelogfile, error = server.runCommand(["getVariable", "BB_CONSOLELOG"])
> + if error:
> + logger.error("Unable to get the value of BB_CONSOLELOG variable: %s" % error)
> + return 1
>
> helper = uihelper.BBUIHelper()
>
> @@ -125,19 +134,22 @@ def main(server, eventHandler, tf = TerminalFilter):
> logger.addHandler(consolelog)
>
> try:
> - cmdline = server.runCommand(["getCmdLineAction"])
> - if not cmdline:
> + cmdline, error = server.runCommand(["getCmdLineAction"])
> + if error:
> + logger.error("Unable to get bitbake commandline arguments: %s" % error)
> + return 1
> + elif not cmdline:
> print("Nothing to do. Use 'bitbake world' to build everything, or run 'bitbake --help' for usage information.")
> return 1
> - elif not cmdline['action']:
> - print(cmdline['msg'])
> + ret, error = server.runCommand(cmdline)
> + if error:
> + logger.error("Command '%s' failed: %s" % (cmdline, error))
> return 1
> - ret = server.runCommand(cmdline['action'])
> - if ret != True:
> - print("Couldn't get default commandline! %s" % ret)
> + elif ret != True:
> + logger.error("Command '%s' failed: returned %s" % (cmdline, ret))
> return 1
> except xmlrpclib.Fault as x:
> - print("XMLRPC Fault getting commandline:\n %s" % x)
> + logger.error("XMLRPC Fault getting commandline:\n %s" % x)
> return 1
>
> parseprogress = None
> @@ -318,14 +330,19 @@ def main(server, eventHandler, tf = TerminalFilter):
> if ioerror.args[0] == 4:
> pass
> except KeyboardInterrupt:
> + import time
> termfilter.clearFooter()
> if main.shutdown == 1:
> print("\nSecond Keyboard Interrupt, stopping...\n")
> - server.runCommand(["stateStop"])
> + _, error = server.runCommand(["stateStop"])
> + if error:
> + logger.error("Unable to cleanly stop: %s" % error)
> if main.shutdown == 0:
> - interrupted = True
> print("\nKeyboard Interrupt, closing down...\n")
> - server.runCommand(["stateShutdown"])
> + interrupted = True
> + _, error = server.runCommand(["stateShutdown"])
> + if error:
> + logger.error("Unable to cleanly shutdown: %s" % error)
> main.shutdown = main.shutdown + 1
> pass
>
> diff --git a/bitbake/lib/bb/ui/ncurses.py b/bitbake/lib/bb/ui/ncurses.py
> index 8524446..896ed01 100644
> --- a/bitbake/lib/bb/ui/ncurses.py
> +++ b/bitbake/lib/bb/ui/ncurses.py
> @@ -230,15 +230,18 @@ class NCursesUI:
> shutdown = 0
>
> try:
> - cmdline = server.runCommand(["getCmdLineAction"])
> + cmdline, error = server.runCommand(["getCmdLineAction"])
> if not cmdline:
> print("Nothing to do. Use 'bitbake world' to build everything, or run 'bitbake --help' for usage information.")
> return
> - elif not cmdline['action']:
> - print(cmdline['msg'])
> + elif error:
> + print("Error getting bitbake commandline: %s" % error)
> return
> - ret = server.runCommand(cmdline['action'])
> - if ret != True:
> + ret, error = server.runCommand(cmdline)
> + if error:
> + print("Error running command '%s': %s" % (cmdline, error))
> + return
> + elif ret != True:
> print("Couldn't get default commandlind! %s" % ret)
> return
> except xmlrpclib.Fault as x:
> @@ -337,10 +340,14 @@ class NCursesUI:
> exitflag = True
> if shutdown == 1:
> mw.appendText("Second Keyboard Interrupt, stopping...\n")
> - server.runCommand(["stateStop"])
> + _, error = server.runCommand(["stateStop"])
> + if error:
> + print("Unable to cleanly stop: %s" % error)
> if shutdown == 0:
> mw.appendText("Keyboard Interrupt, closing down...\n")
> - server.runCommand(["stateShutdown"])
> + _, error = server.runCommand(["stateShutdown"])
> + if error:
> + print("Unable to cleanly shutdown: %s" % error)
> shutdown = shutdown + 1
> pass
>
>
I have pulled this into sgarman/denzil-next on poky-contrib.
Scott
--
Scott Garman
Embedded Linux Engineer - Yocto Project
Intel Open Source Technology Center
^ permalink raw reply [flat|nested] 5+ messages in thread
* [for-denzil] bitbake: command: add error to return of runCommand
@ 2012-11-01 18:16 Matthew McClintock
2012-11-30 2:56 ` Scott Garman
0 siblings, 1 reply; 5+ messages in thread
From: Matthew McClintock @ 2012-11-01 18:16 UTC (permalink / raw)
To: poky
From: Christopher Larson <chris_larson@mentor.com>
Currently, command.py can return an error message from runCommand, due to
being unable to run the command, yet few of our UIs (just hob) can handle it
today. This can result in seeing a TypeError with traceback in certain rare
circumstances.
To resolve this, we need a clean way to get errors back from runCommand,
without having to isinstance() the return value. This implements such a thing
by making runCommand also return an error (or None if no error occurred).
As runCommand now has a method of returning errors, we can also alter the
getCmdLineAction bits such that the returned value is just the action, not an
additional message. If a sync command wants to return an error, it raises
CommandError(message), and the message will be passed to the caller
appropriately.
Example Usage:
result, error = server.runCommand(...)
if error:
log.error('Unable to run command: %s' % error)
return 1
(Bitbake rev: 717831b8315cb3904d9b590e633000bc897e8fb6)
Signed-off-by: Christopher Larson <chris_larson@mentor.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
---
bitbake/lib/bb/command.py | 43 +++++++++++++++----------
bitbake/lib/bb/server/process.py | 2 +-
bitbake/lib/bb/ui/crumbs/hobeventhandler.py | 5 ++-
bitbake/lib/bb/ui/depexp.py | 38 ++++++++++++++--------
bitbake/lib/bb/ui/goggle.py | 17 +++++-----
bitbake/lib/bb/ui/knotty.py | 45 ++++++++++++++++++---------
bitbake/lib/bb/ui/ncurses.py | 21 ++++++++-----
7 files changed, 112 insertions(+), 59 deletions(-)
diff --git a/bitbake/lib/bb/command.py b/bitbake/lib/bb/command.py
index fd8912a..00b854e 100644
--- a/bitbake/lib/bb/command.py
+++ b/bitbake/lib/bb/command.py
@@ -44,6 +44,9 @@ class CommandFailed(CommandExit):
self.error = message
CommandExit.__init__(self, 1)
+class CommandError(Exception):
+ pass
+
class Command:
"""
A queue of asynchronous commands for bitbake
@@ -57,21 +60,25 @@ class Command:
self.currentAsyncCommand = None
def runCommand(self, commandline):
- try:
- command = commandline.pop(0)
- if command in CommandsSync.__dict__:
- # Can run synchronous commands straight away
- return getattr(CommandsSync, command)(self.cmds_sync, self, commandline)
- if self.currentAsyncCommand is not None:
- return "Busy (%s in progress)" % self.currentAsyncCommand[0]
- if command not in CommandsAsync.__dict__:
- return "No such command"
- self.currentAsyncCommand = (command, commandline)
- self.cooker.server_registration_cb(self.cooker.runCommands, self.cooker)
- return True
- except:
- import traceback
- return traceback.format_exc()
+ command = commandline.pop(0)
+ if hasattr(CommandsSync, command):
+ # Can run synchronous commands straight away
+ command_method = getattr(self.cmds_sync, command)
+ try:
+ result = command_method(self, commandline)
+ except CommandError as exc:
+ return None, exc.args[0]
+ except Exception:
+ return None, traceback.format_exc()
+ else:
+ return result, None
+ if self.currentAsyncCommand is not None:
+ return None, "Busy (%s in progress)" % self.currentAsyncCommand[0]
+ if command not in CommandsAsync.__dict__:
+ return None, "No such command"
+ self.currentAsyncCommand = (command, commandline)
+ self.cooker.server_registration_cb(self.cooker.runCommands, self.cooker)
+ return True, None
def runAsyncCommand(self):
try:
@@ -139,7 +146,11 @@ class CommandsSync:
"""
Get any command parsed from the commandline
"""
- return command.cooker.commandlineAction
+ cmd_action = command.cooker.commandlineAction
+ if cmd_action['msg']:
+ raise CommandError(msg)
+ else:
+ return cmd_action['action']
def getVariable(self, command, params):
"""
diff --git a/bitbake/lib/bb/server/process.py b/bitbake/lib/bb/server/process.py
index ba91336..3ae9ede 100644
--- a/bitbake/lib/bb/server/process.py
+++ b/bitbake/lib/bb/server/process.py
@@ -48,7 +48,7 @@ class ServerCommunicator():
if self.connection.poll(.5):
return self.connection.recv()
else:
- return None
+ return None, "Timeout while attempting to communicate with bitbake server"
except KeyboardInterrupt:
pass
diff --git a/bitbake/lib/bb/ui/crumbs/hobeventhandler.py b/bitbake/lib/bb/ui/crumbs/hobeventhandler.py
index 52acec1..21bea23 100644
--- a/bitbake/lib/bb/ui/crumbs/hobeventhandler.py
+++ b/bitbake/lib/bb/ui/crumbs/hobeventhandler.py
@@ -102,7 +102,10 @@ class HobHandler(gobject.GObject):
def runCommand(self, commandline):
try:
- return self.server.runCommand(commandline)
+ result, error = self.server.runCommand(commandline)
+ if error:
+ raise Exception("Error running command '%s': %s" % (commandline, error))
+ return result
except Exception as e:
self.commands_async = []
self.clear_busy()
diff --git a/bitbake/lib/bb/ui/depexp.py b/bitbake/lib/bb/ui/depexp.py
index b62adbf..1254128 100644
--- a/bitbake/lib/bb/ui/depexp.py
+++ b/bitbake/lib/bb/ui/depexp.py
@@ -198,17 +198,23 @@ class gtkthread(threading.Thread):
def main(server, eventHandler):
try:
- cmdline = server.runCommand(["getCmdLineAction"])
- if cmdline and not cmdline['action']:
- print(cmdline['msg'])
- return
- elif not cmdline or (cmdline['action'] and cmdline['action'][0] != "generateDotGraph"):
+ cmdline, error = server.runCommand(["getCmdLineAction"])
+ if error:
+ print("Error getting bitbake commandline: %s" % error)
+ return 1
+ elif not cmdline:
+ print("Nothing to do. Use 'bitbake world' to build everything, or run 'bitbake --help' for usage information.")
+ return 1
+ elif not cmdline or cmdline[0] != "generateDotGraph":
print("This UI is only compatible with the -g option")
- return
- ret = server.runCommand(["generateDepTreeEvent", cmdline['action'][1], cmdline['action'][2]])
- if ret != True:
- print("Couldn't run command! %s" % ret)
- return
+ return 1
+ ret, error = server.runCommand(["generateDepTreeEvent", cmdline[1], cmdline[2]])
+ if error:
+ print("Error running command '%s': %s" % (cmdline, error))
+ return 1
+ elif ret != True:
+ print("Error running command '%s': returned %s" % (cmdline, ret))
+ return 1
except xmlrpclib.Fault as x:
print("XMLRPC Fault getting commandline:\n %s" % x)
return
@@ -229,7 +235,9 @@ def main(server, eventHandler):
try:
event = eventHandler.waitEvent(0.25)
if gtkthread.quit.isSet():
- server.runCommand(["stateStop"])
+ _, error = server.runCommand(["stateStop"])
+ if error:
+ print('Unable to cleanly stop: %s' % error)
break
if event is None:
@@ -302,9 +310,13 @@ def main(server, eventHandler):
break
if shutdown == 1:
print("\nSecond Keyboard Interrupt, stopping...\n")
- server.runCommand(["stateStop"])
+ _, error = server.runCommand(["stateStop"])
+ if error:
+ print('Unable to cleanly stop: %s' % error)
if shutdown == 0:
print("\nKeyboard Interrupt, closing down...\n")
- server.runCommand(["stateShutdown"])
+ _, error = server.runCommand(["stateShutdown"])
+ if error:
+ print('Unable to cleanly shutdown: %s' % error)
shutdown = shutdown + 1
pass
diff --git a/bitbake/lib/bb/ui/goggle.py b/bitbake/lib/bb/ui/goggle.py
index b2fd274..c0785b7 100644
--- a/bitbake/lib/bb/ui/goggle.py
+++ b/bitbake/lib/bb/ui/goggle.py
@@ -80,16 +80,19 @@ def main (server, eventHandler):
running_build.connect ("build-failed", running_build_failed_cb)
try:
- cmdline = server.runCommand(["getCmdLineAction"])
- if not cmdline:
+ cmdline, error = server.runCommand(["getCmdLineAction"])
+ if err:
+ print("Error getting bitbake commandline: %s" % error)
+ return 1
+ elif not cmdline:
print("Nothing to do. Use 'bitbake world' to build everything, or run 'bitbake --help' for usage information.")
return 1
- elif not cmdline['action']:
- print(cmdline['msg'])
+ ret, error = server.runCommand(cmdline)
+ if error:
+ print("Error running command '%s': %s" % (cmdline, error))
return 1
- ret = server.runCommand(cmdline['action'])
- if ret != True:
- print("Couldn't get default commandline! %s" % ret)
+ elif ret != True:
+ print("Error running command '%s': returned %s" % (cmdline, ret))
return 1
except xmlrpclib.Fault as x:
print("XMLRPC Fault getting commandline:\n %s" % x)
diff --git a/bitbake/lib/bb/ui/knotty.py b/bitbake/lib/bb/ui/knotty.py
index 0167416..439a792 100644
--- a/bitbake/lib/bb/ui/knotty.py
+++ b/bitbake/lib/bb/ui/knotty.py
@@ -107,9 +107,18 @@ class TerminalFilter(object):
def main(server, eventHandler, tf = TerminalFilter):
# Get values of variables which control our output
- includelogs = server.runCommand(["getVariable", "BBINCLUDELOGS"])
- loglines = server.runCommand(["getVariable", "BBINCLUDELOGS_LINES"])
- consolelogfile = server.runCommand(["getVariable", "BB_CONSOLELOG"])
+ includelogs, error = server.runCommand(["getVariable", "BBINCLUDELOGS"])
+ if error:
+ logger.error("Unable to get the value of BBINCLUDELOGS variable: %s" % error)
+ return 1
+ loglines, error = server.runCommand(["getVariable", "BBINCLUDELOGS_LINES"])
+ if error:
+ logger.error("Unable to get the value of BBINCLUDELOGS_LINES variable: %s" % error)
+ return 1
+ consolelogfile, error = server.runCommand(["getVariable", "BB_CONSOLELOG"])
+ if error:
+ logger.error("Unable to get the value of BB_CONSOLELOG variable: %s" % error)
+ return 1
helper = uihelper.BBUIHelper()
@@ -125,19 +134,22 @@ def main(server, eventHandler, tf = TerminalFilter):
logger.addHandler(consolelog)
try:
- cmdline = server.runCommand(["getCmdLineAction"])
- if not cmdline:
+ cmdline, error = server.runCommand(["getCmdLineAction"])
+ if error:
+ logger.error("Unable to get bitbake commandline arguments: %s" % error)
+ return 1
+ elif not cmdline:
print("Nothing to do. Use 'bitbake world' to build everything, or run 'bitbake --help' for usage information.")
return 1
- elif not cmdline['action']:
- print(cmdline['msg'])
+ ret, error = server.runCommand(cmdline)
+ if error:
+ logger.error("Command '%s' failed: %s" % (cmdline, error))
return 1
- ret = server.runCommand(cmdline['action'])
- if ret != True:
- print("Couldn't get default commandline! %s" % ret)
+ elif ret != True:
+ logger.error("Command '%s' failed: returned %s" % (cmdline, ret))
return 1
except xmlrpclib.Fault as x:
- print("XMLRPC Fault getting commandline:\n %s" % x)
+ logger.error("XMLRPC Fault getting commandline:\n %s" % x)
return 1
parseprogress = None
@@ -318,14 +330,19 @@ def main(server, eventHandler, tf = TerminalFilter):
if ioerror.args[0] == 4:
pass
except KeyboardInterrupt:
+ import time
termfilter.clearFooter()
if main.shutdown == 1:
print("\nSecond Keyboard Interrupt, stopping...\n")
- server.runCommand(["stateStop"])
+ _, error = server.runCommand(["stateStop"])
+ if error:
+ logger.error("Unable to cleanly stop: %s" % error)
if main.shutdown == 0:
- interrupted = True
print("\nKeyboard Interrupt, closing down...\n")
- server.runCommand(["stateShutdown"])
+ interrupted = True
+ _, error = server.runCommand(["stateShutdown"])
+ if error:
+ logger.error("Unable to cleanly shutdown: %s" % error)
main.shutdown = main.shutdown + 1
pass
diff --git a/bitbake/lib/bb/ui/ncurses.py b/bitbake/lib/bb/ui/ncurses.py
index 8524446..896ed01 100644
--- a/bitbake/lib/bb/ui/ncurses.py
+++ b/bitbake/lib/bb/ui/ncurses.py
@@ -230,15 +230,18 @@ class NCursesUI:
shutdown = 0
try:
- cmdline = server.runCommand(["getCmdLineAction"])
+ cmdline, error = server.runCommand(["getCmdLineAction"])
if not cmdline:
print("Nothing to do. Use 'bitbake world' to build everything, or run 'bitbake --help' for usage information.")
return
- elif not cmdline['action']:
- print(cmdline['msg'])
+ elif error:
+ print("Error getting bitbake commandline: %s" % error)
return
- ret = server.runCommand(cmdline['action'])
- if ret != True:
+ ret, error = server.runCommand(cmdline)
+ if error:
+ print("Error running command '%s': %s" % (cmdline, error))
+ return
+ elif ret != True:
print("Couldn't get default commandlind! %s" % ret)
return
except xmlrpclib.Fault as x:
@@ -337,10 +340,14 @@ class NCursesUI:
exitflag = True
if shutdown == 1:
mw.appendText("Second Keyboard Interrupt, stopping...\n")
- server.runCommand(["stateStop"])
+ _, error = server.runCommand(["stateStop"])
+ if error:
+ print("Unable to cleanly stop: %s" % error)
if shutdown == 0:
mw.appendText("Keyboard Interrupt, closing down...\n")
- server.runCommand(["stateShutdown"])
+ _, error = server.runCommand(["stateShutdown"])
+ if error:
+ print("Unable to cleanly shutdown: %s" % error)
shutdown = shutdown + 1
pass
--
1.7.9.7
^ permalink raw reply related [flat|nested] 5+ messages in thread
end of thread, other threads:[~2012-12-19 19:31 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-12-18 17:18 [for-denzil] bitbake: command: add error to return of runCommand Matthew McClintock
2012-12-18 21:28 ` Richard Purdie
2012-12-19 19:01 ` McClintock Matthew-B29882
-- strict thread matches above, loose matches on Subject: below --
2012-11-01 18:16 Matthew McClintock
2012-11-30 2:56 ` Scott Garman
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.