[Cppunit-devel] another configure.in change

[Steve M. R. <ste...@vi...>]

I just noticed another oddity in configure.in that violates
the principle of "least surprise".  I propose we do away
with the following:

AC_ARG_ENABLE(debug-mode,   
[  --enable-debug-mode     enable debugging mode],
  [
    CFLAGS="-D_DEBUG -g3 -Wall"
    CXXFLAGS="-D_DEBUG -g3 -Wall"
  ],[
    CFLAGS="-O2 "
    CXXFLAGS="-O2"
  ]) 

Rationale: besides surprising me that "-g" is omitted by default, it
is clearly wrong that --enable-debug-mode sets GCC-specific flags like
"-Wall".  The builder always has the option to set the flags themself,
so this difference is completely gratuitous.

-Steve


-- 
by Rocket to the Moon,
by Airplane to the Rocket,
by Taxi to the Airport,
by Frontdoor to the Taxi,
by throwing back the blanket and laying down the legs ...
- They Might Be Giants