diff options
author | Tom Zanussi <tom.zanussi@linux.intel.com> | 2014-02-19 14:53:37 -0600 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2014-02-20 14:28:13 +0000 |
commit | bb0c26960d3b05070346cdfdfd05d6fd4ad0a7b1 (patch) | |
tree | 187d506936d4470b85e4e14db90d8e97e5fff1ab /scripts/lib/mic/utils/oe | |
parent | 8e894d111de3cfe86bd6c6178fb619d94b1be115 (diff) | |
download | ast2050-yocto-poky-bb0c26960d3b05070346cdfdfd05d6fd4ad0a7b1.zip ast2050-yocto-poky-bb0c26960d3b05070346cdfdfd05d6fd4ad0a7b1.tar.gz |
wic: Make exec_native_command() fail if a command isn't found
Because exec_cmd() return values can in certain cases be non-zero yet
non-fatal, we don't want to automatically make them fatal (though
there should at least be a warning in such cases, which this patch
also does); non-zero return values are definitely fatal however if
they mean that a native command wasn't found, so have
exec_native_cmd() check the return value of exec_cmd() for that case,
and bail out if so.
[YOCTO #5835]
(From OE-Core rev: 43ac6e3216c5d985d6f90a28e727e397df616267)
Signed-off-by: Tom Zanussi <tom.zanussi@linux.intel.com>
Signed-off-by: Saul Wold <sgw@linux.intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'scripts/lib/mic/utils/oe')
-rw-r--r-- | scripts/lib/mic/utils/oe/misc.py | 11 |
1 files changed, 9 insertions, 2 deletions
diff --git a/scripts/lib/mic/utils/oe/misc.py b/scripts/lib/mic/utils/oe/misc.py index d65b77a..7ad3aa9 100644 --- a/scripts/lib/mic/utils/oe/misc.py +++ b/scripts/lib/mic/utils/oe/misc.py @@ -51,7 +51,7 @@ def exec_cmd(cmd_and_args, as_shell = False, catch = 3): # parted always fails to reload part table with loop devices. This # prevents us from distinguishing real errors based on return # code. - msger.debug("WARNING: %s returned '%s' instead of 0" % (args[0], rc)) + msger.warning("WARNING: %s returned '%s' instead of 0" % (cmd_and_args, rc)) return (rc, out) @@ -82,7 +82,14 @@ def exec_native_cmd(cmd_and_args, native_sysroot, catch = 3): args = cmd_and_args.split() msger.debug(args) - return exec_cmd(native_cmd_and_args, True, catch) + rc, out = exec_cmd(native_cmd_and_args, True, catch) + + if rc == 127: # shell command-not-found + msger.error("A native (host) program required to build the image " + "was not found (see details above). Please make sure " + "it's installed and try again.") + + return (rc, out) def exec_native_cmd_quiet(cmd_and_args, native_sysroot): |