summaryrefslogtreecommitdiffstats
path: root/configure
diff options
context:
space:
mode:
authorPeter Maydell2022-08-25 17:07:02 +0200
committerPeter Maydell2022-09-22 17:38:29 +0200
commit563661c05677ad7c4b3f956507f3529df42afbf7 (patch)
tree22f19079cc081e6caae456aff7ad41dceab98b2d /configure
parentconfigure: Remove use of backtick `...` syntax (diff)
downloadqemu-563661c05677ad7c4b3f956507f3529df42afbf7.tar.gz
qemu-563661c05677ad7c4b3f956507f3529df42afbf7.tar.xz
qemu-563661c05677ad7c4b3f956507f3529df42afbf7.zip
configure: Check mkdir result directly, not via $?
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
Diffstat (limited to 'configure')
-rwxr-xr-xconfigure3
1 files changed, 1 insertions, 2 deletions
diff --git a/configure b/configure
index 8b48e72a13..b39561b2ed 100755
--- 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