From: Tomohiro Kusumi Date: Mon, 1 May 2017 18:48:19 +0000 (+0300) Subject: configure: Add missing $val != "yes" test to override compile_prog() result X-Git-Tag: fio-2.20~12 X-Git-Url: https://git.kernel.dk/?p=fio.git;a=commitdiff_plain;h=d418fa90f6daf75bb9336182f79c0b25aa6feecd configure: Add missing $val != "yes" test to override compile_prog() result Since it's possible for some platforms to use statically configured configurations (e.g. see around L350 for Cygwin), it should test if the variable hasn't been set to "yes" before initializing the variable with "no", so that compile_prog() result can be safely ignored when the platform knows it's supported. ca205a75 (configure: Make Cygwin take regular configure path) started this due to a major change in the configuration path, which changed Cygwin to take the normal path taken by all other platforms, and these two were added after that change. Signed-off-by: Tomohiro Kusumi Signed-off-by: Jens Axboe --- diff --git a/configure b/configure index 4ca7d52b..21bcaf46 100755 --- a/configure +++ b/configure @@ -612,7 +612,9 @@ echo "POSIX AIO fsync $posix_aio_fsync" ########################################## # POSIX pshared attribute probe -posix_pshared="no" +if test "$posix_pshared" != "yes" ; then + posix_pshared="no" +fi cat > $TMPC < int main(void) @@ -1997,7 +1999,9 @@ echo "march_armv8_a_crc_crypto $march_armv8_a_crc_crypto" ########################################## # cuda probe -cuda="no" +if test "$cuda" != "yes" ; then + cuda="no" +fi cat > $TMPC << EOF #include int main(int argc, char **argv)