[Mesa-dev] [PATCH 2/2] automake: Remove OPT_FLAGS.

[Brian Paul]

On 07/26/2012 10:52 AM, Eric Anholt wrote:
> If you want to change your compiler arguments, just set CFLAGS/CXXFLAGS.
> Having Mesa have this separate variable is a great way to have your arguments
> not thoroughly propagated to all compiler invocations.
> ---
>
> I don't feel strongly about removing the variable from configure, just
> as long as we don't have to propagate it into Makefiles.  I use shared
> configure args including CFLAGS and CXXFLAGS across many projects, so
> I've never seen the use for this one, but if people really care to
> keep it, we should just append it to CFLAGS/CXXFLAGS in configure.ac.

This sounds good but someone else will have to review.

But while you're in this area, it would be nice if passing 
--enable-debug would set the -g flag and omit the -O2 flag.  It 
doesn't seem right that one has to explicitly set CFLAGS and CXXFLAGS 
to -g when --enable-debug is used.

-Brian