[PATCH] Fix toplevel configure --enable-multilib handling (PR bootstrap/43328)

Ralf Wildenhues Ralf.Wildenhues@gmx.de
Tue Mar 30 19:02:00 GMT 2010


Ping <http://gcc.gnu.org/ml/gcc-patches/2010-03/msg00509.html>

* Ralf Wildenhues wrote on Sat, Mar 13, 2010 at 10:31:58AM CET:
> Tested by running configure with --disable-multilib, --enable-multilib,
> or neither, and comparing the generated config.status file, ensuring the
> latter two only differ in TOPLEVEL_CONFIGURE_ARGUMENTS and recheck.
> 
> OK for trunk?  I'll sync to src then.
> 
> The bug was that an explicitly passed --enable-multilib would also
> propagate to build_configargs and host_configargs.

> Fix toplevel configure --enable-multilib handling.
> 
> ChangeLog:
> 2010-03-13  Ralf Wildenhues  <Ralf.Wildenhues@gmx.de>
> 
> 	PR bootstrap/43328
> 	* configure.ac: Do not pass --enable-multilib nor
> 	--disable-multilib in baseargs.  Accept explicitly passed
> 	--enable_multilib.
> 	* configure: Regenerate.
> 
> diff --git a/configure.ac b/configure.ac
> index 3464166..7305e98 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -2879,6 +2879,9 @@ do
>        skip_next=$separate_arg
>        continue
>        ;;
> +    --enable-multilib | --disable-multilib)
> +      continue
> +      ;;
>      -*)
>        # An option.  Add it.
>        case $ac_arg in
> @@ -2947,10 +2950,12 @@ if test x${is_cross_compiler} = xyes ; then
>    target_configargs="--with-cross-host=${host_noncanonical} ${target_configargs}"
>  fi
>  
> -# Default to --enable-multilib.
> -if test x${enable_multilib} = x ; then
> +# Pass --enable-multilib to target dirs; default to --enable-multilib.
> +case $enable_multilib in
> +'' | yes)
>    target_configargs="--enable-multilib ${target_configargs}"
> -fi
> +  ;;
> +esac
>  
>  # Pass --with-newlib if appropriate.  Note that target_configdirs has
>  # changed from the earlier setting of with_newlib.



More information about the Gcc-patches mailing list