diff options
author | Masahiro Yamada <yamada.m@jp.panasonic.com> | 2015-02-03 19:14:44 +0900 |
---|---|---|
committer | Sascha Hauer <s.hauer@pengutronix.de> | 2015-02-04 12:51:22 +0100 |
commit | 5e1f14d960c58a70f16a129ac65da579a393e234 (patch) | |
tree | f65066881d1b6e1b78974c40869937be28c0507d /lib | |
parent | ade5b109ac88d0903f25e5ac175cbf1d7eafbdfe (diff) | |
download | barebox-5e1f14d960c58a70f16a129ac65da579a393e234.tar.gz barebox-5e1f14d960c58a70f16a129ac65da579a393e234.tar.xz |
MAKEALL: check the return code of "make" rather than "printf"
Currently, MAKEALL always reports "Configure: OK" and "Compile: OK"
regardless of the result of the configuration and compile.
$ LANG=C CROSS_COMPILE=arm-linux-gnueabi- ./MAKEALL -a arm foo_defconfig
Building arm foo_defconfig
make[2]: *** [foo_defconfig] Error 1
make[1]: *** [foo_defconfig] Error 2
make: *** [foo_defconfig] Error 2
Configure: OK
***
*** Configuration file ".config" not found!
***
*** Please run some configurator (e.g. "make oldconfig" or
*** "make menuconfig" or "make xconfig").
***
make[3]: *** [silentoldconfig] Error 1
make[2]: *** [silentoldconfig] Error 2
make[1]: *** No rule to make target `include/config/auto.conf',
needed by `include/config/kernel.release'. Stop.
make[1]: *** Waiting for unfinished jobs....
make: *** [_all] Error 2
Compile: OK
arm-linux-gnueabi-size: 'makeall_builddir/barebox': No such file
Compiled in 2s
The check_pipe_status() function must be called right after "make"
command, not "printf" command. ("printf" probably succeeds all
the time.)
Signed-off-by: Masahiro Yamada <yamada.m@jp.panasonic.com>
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
Diffstat (limited to 'lib')
0 files changed, 0 insertions, 0 deletions