From d418fa90f6daf75bb9336182f79c0b25aa6feecd Mon Sep 17 00:00:00 2001 From: Tomohiro Kusumi Date: Mon, 1 May 2017 21:48:19 +0300 Subject: [PATCH] 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 --- configure | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) 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) -- 2.25.1