docker: Fix return code of build_qemu()
authorFam Zheng <famz@redhat.com>
Tue, 5 Sep 2017 02:56:09 +0000 (10:56 +0800)
committerFam Zheng <famz@redhat.com>
Fri, 22 Sep 2017 02:20:34 +0000 (10:20 +0800)
Without "set -e", the "&&" makes sure that the return code reflects the
result status, and that make only runs if configure succeeds.

Signed-off-by: Fam Zheng <famz@redhat.com>
Message-Id: <20170905025614.579-2-famz@redhat.com>
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
Based-on: 20170905021201.25684-1-famz@redhat.com

tests/docker/common.rc

index 64b36ba2d098d68f8da4d5f4a42e8eada7a98a7c..3b45eb91c6b7182b6cd6d720e783bae1feae5a22 100755 (executable)
@@ -30,6 +30,5 @@ build_qemu()
                  $@"
     echo "Configure options:"
     echo $config_opts
-    $QEMU_SRC/configure $config_opts
-    make $MAKEFLAGS
+    $QEMU_SRC/configure $config_opts && make $MAKEFLAGS
 }