[Mesa-dev] [PATCH] configure: Don't use $CLANG since it will collide with the static analyzer.

[Ian Romanick]

On 12/09/2011 06:12 AM, nobled wrote:
> From: Jeremy Huddleston<jeremyhu at apple.com>
> Date: Thu, 5 May 2011 14:08:57 -0700
>
> We just prefix the $CLANG environment variable in configure.ac with acv_mesa_
>
> Found by: tinderbox
> Signed-off-by: Jeremy Huddleston<jeremyhu at apple.com>

Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>

Jeremy should have commit access to push this at his leisure.  I suppose 
this should also go on the 7.11 branch as well, right?

> ---
>
> Originally posted here:
> http://lists.freedesktop.org/archives/mesa-dev/2011-May/007411.html
>
>   configure.ac |    8 ++++----
>   1 files changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/configure.ac b/configure.ac
> index a4943e1..162945b 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -75,13 +75,13 @@ AC_COMPILE_IFELSE(
>          not clang
>   #endif
>   ]])],
> -[CLANG=yes], [CLANG=no])
> +[acv_mesa_CLANG=yes], [acv_mesa_CLANG=no])
>
> -AC_MSG_RESULT([$CLANG])
> +AC_MSG_RESULT([$acv_mesa_CLANG])
>
>   dnl If we're using GCC, make sure that it is at least version 3.3.0.  Older
>   dnl versions are explictly not supported.
> -if test "x$GCC" = xyes -a "x$CLANG" = xno; then
> +if test "x$GCC" = xyes -a "x$acv_mesa_CLANG" = xno; then
>       AC_MSG_CHECKING([whether gcc version is sufficient])
>       major=0
>       minor=0
> @@ -155,7 +155,7 @@ esac
>   dnl Add flags for gcc and g++
>   if test "x$GCC" = xyes; then
>       CFLAGS="$CFLAGS -Wall -Wmissing-prototypes -std=c99"
> -    if test "x$CLANG" = "xno"; then
> +    if test "x$acv_mesa_CLANG" = "xno"; then
>          CFLAGS="$CFLAGS -ffast-math"
>       fi
>
>
>
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev