diff mbox

configure: fix Linux AIO detection

Message ID 20090831131812.57f12e52@doriath
State Superseded
Headers show

Commit Message

Luiz Capitulino Aug. 31, 2009, 4:18 p.m. UTC 
We should set $linux_aio to 'no' if detection failed, otherwise
its contents will be empty, which is a bug as we test for 'yes'
or 'no'.
    
Signed-off-by: Luiz Capitulino <lcapitulino@redhat.com>

Comments

Juan Quintela Aug. 31, 2009, 4:28 p.m. UTC | #1 
Luiz Capitulino <lcapitulino@redhat.com> wrote:
> We should set $linux_aio to 'no' if detection failed, otherwise
> its contents will be empty, which is a bug as we test for 'yes'
> or 'no'.
>     
> Signed-off-by: Luiz Capitulino <lcapitulino@redhat.com>

acked-by: Juan Quintela <quintela@redhat.com>
Good catch.

> diff --git a/configure b/configure
> index b805f10..4c60c4d 100755
> --- a/configure
> +++ b/configure
> @@ -1327,6 +1327,7 @@ EOF
>      if test "$linux_aio" = "yes" ; then
>        feature_not_found "linux AIO"
>      fi
> +    linux_aio=no
>    fi
>  fi
>
diff mbox

Patch

diff --git a/configure b/configure
index b805f10..4c60c4d 100755
--- a/configure
+++ b/configure
@@ -1327,6 +1327,7 @@  EOF
     if test "$linux_aio" = "yes" ; then
       feature_not_found "linux AIO"
     fi
+    linux_aio=no
   fi
 fi