[incubator-nuttx] branch master updated: tools/testbuild.sh: update fail to return the error value instead

[gn...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

gnutt pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git


The following commit(s) were added to refs/heads/master by this push:
     new 57099fe  tools/testbuild.sh: update fail to return the error value instead
57099fe is described below

commit 57099fe4415a7c33279e0e60b6f9026c1649ccde
Author: liuhaitao <li...@xiaomi.com>
AuthorDate: Sun Feb 2 10:29:28 2020 +0800

    tools/testbuild.sh: update fail to return the error value instead
---
 tools/testbuild.sh | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/tools/testbuild.sh b/tools/testbuild.sh
index 59b0218..79561fa 100755
--- a/tools/testbuild.sh
+++ b/tools/testbuild.sh
@@ -154,7 +154,7 @@ cd $nuttx || { echo "ERROR: failed to CD to $nuttx"; exit 1; }
 function distclean {
   if [ -f .config ]; then
     echo "  Cleaning..."
-    ${MAKE} ${JOPTION} ${MAKE_FLAGS} distclean 1>/dev/null || fail=1
+    ${MAKE} ${JOPTION} ${MAKE_FLAGS} distclean 1>/dev/null || fail=$?
   fi
 }
 
@@ -183,7 +183,7 @@ function configure {
     echo "$toolchain=y" >> $nuttx/.config
 
     echo "  Refreshing..."
-    ${MAKE} ${MAKE_FLAGS} olddefconfig 1>/dev/null || fail=1
+    ${MAKE} ${MAKE_FLAGS} olddefconfig 1>/dev/null || fail=$?
   fi
 }
 
@@ -192,7 +192,7 @@ function configure {
 function build {
   echo "  Building NuttX..."
   echo "------------------------------------------------------------------------------------"
-  ${MAKE} ${JOPTION} ${MAKE_FLAGS} 1>/dev/null || fail=1
+  ${MAKE} ${JOPTION} ${MAKE_FLAGS} 1>/dev/null || fail=$?
 }
 
 # Coordinate the steps for the next build test