]> git.proxmox.com Git - mirror_qemu.git/commitdiff
configure: Check mkdir result directly, not via $?
authorPeter Maydell <peter.maydell@linaro.org>
Thu, 25 Aug 2022 15:07:02 +0000 (16:07 +0100)
committerPeter Maydell <peter.maydell@linaro.org>
Thu, 22 Sep 2022 15:38:29 +0000 (16:38 +0100)
Shellcheck warns that we have one place where we run a command and
then check if it failed using $?; this is better written to simply
check the command in the 'if' statement directly.

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Message-id: 20220825150703.4074125-7-peter.maydell@linaro.org

configure

index 8b48e72a135a6ccb8032951d1982c5f96e1aeff8..b39561b2ed7edc16e796877b2a388ec8b78fab65 100755 (executable)
--- a/configure
+++ b/configure
@@ -67,8 +67,7 @@ fi
 # it when configure exits.)
 TMPDIR1="config-temp"
 rm -rf "${TMPDIR1}"
-mkdir -p "${TMPDIR1}"
-if [ $? -ne 0 ]; then
+if ! mkdir -p "${TMPDIR1}"; then
     echo "ERROR: failed to create temporary directory"
     exit 1
 fi