[incubator-nuttx-testing] branch master updated: Call testbuild.sh with -x option to set it fail fast for check build error

[xi...@apache.org]

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 384af2d  Call testbuild.sh with -x option to set it fail fast for check build error
384af2d is described below

commit 384af2da1aad5bba0b18b073693e1ae56941c16d
Author: liuhaitao <li...@xiaomi.com>
AuthorDate: Fri Feb 7 09:47:39 2020 +0800

    Call testbuild.sh with -x option to set it fail fast for check build error
    
    For PR check build, call testbuild.sh with -x option to set it fail fast if
    build error occured. Note that for full build, it keeps on going build anyway.
    
    Change-Id: I46b77cc4632dd2c8e1749ab81a275fa1ff5a505c
    Signed-off-by: liuhaitao <li...@xiaomi.com>
---
 cibuild.sh | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/cibuild.sh b/cibuild.sh
index 953854b..a939744 100755
--- a/cibuild.sh
+++ b/cibuild.sh
@@ -172,12 +172,13 @@ function install_tools {
 
 function run_builds {
   local ncpus=`grep -c ^processor /proc/cpuinfo`
+  local options="-si -j $ncpus"
 
-  $nuttx/tools/testbuild.sh -si -j $ncpus $WD/testlist/${build}list.dat
-  if [ $? != 0 ]; then
-    echo "ERROR: $BUILD build failed (error $?)"
-    exit 1
+  if [ "X$build" = "Xcheck" ]; then
+    options="$options -x"
   fi
+
+  $nuttx/tools/testbuild.sh $options $WD/testlist/${build}list.dat
 }
 
 if [ -z "$1" ]; then