Re: [Xen-devel] [PATCH] tools/configure: correct enable/disable-feature option

[Ian Campbell <Ian.Campbell@xxxxxxxxxx>]

On Wed, 2012-03-14 at 18:13 +0000, Olaf Hering wrote:
> # HG changeset patch
> # User Olaf Hering <olaf@xxxxxxxxx>
> # Date 1331748796 -3600
> # Node ID 36690251f6bfe153f390ef200ff286e64f2582d7
> # Parent  cd473b1fb313bb107cf1c32ce224f265a5de097e
> tools/configure: correct enable/disable-feature option
> 
> If --disable-feature is not specified then feature should default to n.

Maybe I'm misunderstanding what you are saying. I'd have though that it
would depend on the feature whether it was on by default or not and in
other cases it will depend on whether the prerequisites are met.

> But with the current code disable with set feature to y, and enable will
> set feature to n. Reverse the logic in the two .m4 files to use correct
> default values.
> The failure currently is that default := is y while it should be n.
> 
> Signed-off-by: Olaf Hering <olaf@xxxxxxxxx>
> 
> diff -r cd473b1fb313 -r 36690251f6bf tools/m4/disable_feature.m4
> --- a/tools/m4/disable_feature.m4
> +++ b/tools/m4/disable_feature.m4
> @@ -7,7 +7,7 @@ AS_IF([test "x$enable_$1" = "xno"], [
>  ], [test "x$enable_$1" = "xyes"], [
>      ax_cv_$1="y"
>  ], [test -z $ax_cv_$1], [
> -    ax_cv_$1="y"
> +    ax_cv_$1="n"
>  ])
>  $1=$ax_cv_$1
>  AC_SUBST($1)])
> diff -r cd473b1fb313 -r 36690251f6bf tools/m4/enable_feature.m4
> --- a/tools/m4/enable_feature.m4
> +++ b/tools/m4/enable_feature.m4
> @@ -7,7 +7,7 @@ AS_IF([test "x$enable_$1" = "xyes"], [
>  ], [test "x$enable_$1" = "xno"], [
>      ax_cv_$1="n"
>  ], [test -z $ax_cv_$1], [
> -    ax_cv_$1="n"
> +    ax_cv_$1="y"
>  ])
>  $1=$ax_cv_$1
>  AC_SUBST($1)])
> 
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@xxxxxxxxxxxxx
> http://lists.xen.org/xen-devel



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel