fix travis CI error report when build fails

The code eval $build_cmd will return the status of make in ${?}. That needs to be saved and preserved immediately following eval $build_cmd and then returned before gcc_check_happy completes at }. Otherwise, each of the cp and echo lines will overwrite ${?}. Since echo will always succeed, gcc_check_happy will always succeed, and Travis CI will always believe such matrix targets have succeeded, even when they do not.

Author: yunhanw-google

.travis/script.sh

@@ -51,9 +51,11 @@ gcc_check_happy()
 
     mkdir -p $TRAVIS_BUILD_DIR/happy-test-logs/$1/from-tmp
     eval $build_cmd
+    make_status=${?}
     cp $TRAVIS_BUILD_DIR/build/$build_folder/src/test-apps/happy $TRAVIS_BUILD_DIR/happy-test-logs/$1 -rf
     cp /tmp/happy* $TRAVIS_BUILD_DIR/happy-test-logs/$1/from-tmp
     echo "please check happy-test-log/<UTC time> under link: https://storage.cloud.google.com/openweave"
+    return ${make_status}
 }
 
 case "${BUILD_TARGET}" in
@@ -113,3 +115,4 @@ case "${BUILD_TARGET}" in
         ;;
 
 esac
+