[Mesa-dev] [PATCH 1/2] configure: allow C{,XX}FLAGS override

[Brian Paul]

On 07/13/2011 01:26 PM, Marcin Baczyński wrote:
> ---
>
>   Without this patch CFLAGS=-fstrict-aliasing ./autogen.sh ... has no effect,
>   as the flag is followed by -fno-strict-aliasing added by configure.
>
>   configure.ac |    8 ++++++++
>   1 files changed, 8 insertions(+), 0 deletions(-)
>
> diff --git a/configure.ac b/configure.ac
> index eace790..ef9f4b2 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -17,6 +17,10 @@ AC_INIT([Mesa],[mesa_version],
>   AC_CONFIG_AUX_DIR([bin])
>   AC_CANONICAL_HOST
>
> +dnl Save user CFLAGS and CXXFLAGS so one can override the default ones
> +USER_CFLAGS="$CFLAGS"
> +USER_CXXFLAGS="$CXXFLAGS"
> +
>   dnl Versions for external dependencies
>   LIBDRM_REQUIRED=2.4.24
>   LIBDRM_RADEON_REQUIRED=2.4.24
> @@ -1906,6 +1910,10 @@ dnl Restore LDFLAGS and CPPFLAGS
>   LDFLAGS="$_SAVE_LDFLAGS"
>   CPPFLAGS="$_SAVE_CPPFLAGS"
>
> +dnl Add user CFLAGS and CXXFLAGS
> +CFLAGS="$CFLAGS $USER_CFLAGS"
> +CXXFLAGS="$CXXFLAGS $USER_CXXFLAGS"
> +
>   dnl Substitute the config
>   AC_CONFIG_FILES([configs/autoconf])
>

Looks OK to me, but an autoconf expert should probably verify that 
this is the right fix.  Anyone???

-Brian