]> code.ossystems Code Review - openembedded-core.git/commitdiff
kernel-yocto: propagate configuration errors to bbclass
authorBruce Ashfield <bruce.ashfield@windriver.com>
Tue, 13 Jun 2017 07:26:17 +0000 (03:26 -0400)
committerRichard Purdie <richard.purdie@linuxfoundation.org>
Mon, 17 Jul 2017 12:49:01 +0000 (13:49 +0100)
As pointed out by klapperichpaul@johndeere.com, missing configuration
fragments were being picked up twice, once by the tools and once by the
bbclass. Unfortunately, the tools error message was being detected as
configs, and hence no error was reported at all.

Rather than catching the output of the tools, we can instead check the
return code and propagate the error message from the tools directly to
the user.

[YOCTO #11649]

Signed-off-by: Bruce Ashfield <bruce.ashfield@windriver.com>
Signed-off-by: Ross Burton <ross.burton@intel.com>
meta/classes/kernel-yocto.bbclass

index 50226f6d492ee30d71f6a76b46e9fcd2f01bfba7..1ca0756c495958f3764854f8aef369b180cde4dc 100644 (file)
@@ -281,7 +281,8 @@ do_kernel_configme() {
 
        meta_dir=$(kgit --meta)
        configs="$(scc --configs -o ${meta_dir})"
-       if [ -z "${configs}" ]; then
+       if [ $? -ne 0 ]; then
+               bberror "${configs}"
                bbfatal_log "Could not find configuration queue (${meta_dir}/config.queue)"
        fi