]> code.ossystems Code Review - openembedded-core.git/commitdiff
build-perf-test-wrapper.sh: correctly check test script exit status
authorMarkus Lehtonen <markus.lehtonen@linux.intel.com>
Mon, 3 Apr 2017 15:58:37 +0000 (18:58 +0300)
committerRichard Purdie <richard.purdie@linuxfoundation.org>
Wed, 5 Apr 2017 11:36:45 +0000 (12:36 +0100)
Test of the exit code was accidentally moved to wrong place when
oe-git-archive was taken into use.

Signed-off-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>
Signed-off-by: Ross Burton <ross.burton@intel.com>
scripts/contrib/build-perf-test-wrapper.sh

index 7b7e4dddc642903a37bdc9b29e9789ecf3fabec1..e804ac24cc20898a2efd911a69d3c9b64ae7c5ae 100755 (executable)
@@ -155,6 +155,14 @@ oe-build-perf-test --out-dir "$results_dir" \
                    "${oe_build_perf_test_extra_opts[@]}" \
                    --lock-file "$base_dir/oe-build-perf.lock"
 
+case $? in
+    1)  echo "ERROR: oe-build-perf-test script failed!"
+        exit 1
+        ;;
+    2)  echo "NOTE: some tests failed!"
+        ;;
+esac
+
 # Commit results to git
 if [ -n "$results_repo" ]; then
     echo -e "\nArchiving results in $results_repo"
@@ -167,14 +175,6 @@ if [ -n "$results_repo" ]; then
         "$results_dir"
 fi
 
-case $? in
-    1)  echo "ERROR: oe-build-perf-test script failed!"
-        exit 1
-        ;;
-    2)  echo "NOTE: some tests failed!"
-        ;;
-esac
-
 echo -ne "\n\n-----------------\n"
 echo "Global results file:"
 echo -ne "\n"