[Mesa-dev] [PATCH] configure: Don't override user -g or -O options for debug builds

[Marek Olšák]

You can also just add -DDEBUG by yourself since you override the flags
anyway (me too).

Marek

On Mon, Jul 14, 2014 at 8:49 PM, Ian Romanick <idr at freedesktop.org> wrote:
> From: Ian Romanick <ian.d.romanick at intel.com>
>
> I already pass '-ggdb3 -O1' or '-ggdb3 -Og' for CFLAGS, and I don't want
> configure to change them for me.
>
> Signed-off-by: Ian Romanick <ian.d.romanick at intel.com>
> ---
>  configure.ac | 14 ++++++++++++--
>  1 file changed, 12 insertions(+), 2 deletions(-)
>
> diff --git a/configure.ac b/configure.ac
> index 4646212..04abcc0 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -308,10 +308,20 @@ AC_ARG_ENABLE([debug],
>  if test "x$enable_debug" = xyes; then
>      DEFINES="$DEFINES -DDEBUG"
>      if test "x$GCC" = xyes; then
> -        CFLAGS="$CFLAGS -g -O0"
> +        if ! echo "$CFLAGS" | grep -q -e '-g'; then
> +            CFLAGS="$CFLAGS -g"
> +        fi
> +        if ! echo "$CFLAGS" | grep -q -e '-O'; then
> +            CFLAGS="$CFLAGS -O0"
> +        fi
>      fi
>      if test "x$GXX" = xyes; then
> -        CXXFLAGS="$CXXFLAGS -g -O0"
> +        if ! echo "$CXXFLAGS" | grep -q -e '-g'; then
> +            CXXFLAGS="$CXXFLAGS -g"
> +        fi
> +        if ! echo "$CXXFLAGS" | grep -q -e '-O'; then
> +            CXXFLAGS="$CXXFLAGS -O0"
> +        fi
>      fi
>  fi
>
> --
> 1.8.1.4
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev