Done as you suggested.
---------------------------------------------------------------------------
Tom Lane wrote:
> Bruce Momjian <pgman@candle.pha.pa.us> writes:
> > OK, patch attached and applied. It centralizes the optimization
> > defaults into configure.in, rather than having CFLAGS= in the template
> > files.
>
> I think there's a problem here:
>
> > + # configure sets CFLAGS to -O2 for gcc, so this is only for non-gcc
> > + if test x"$CFLAGS" = x""; then
> > + CFLAGS="-O"
> > + fi
> > if test "$enable_debug" = yes && test "$ac_cv_prog_cc_g" = yes; then
> > CFLAGS="$CFLAGS -g"
> > fi
>
> since this will cause "configure --enable-debug" to default to selecting
> CFLAGS="-O -g" for non-gcc compilers. On a lot of compilers that
> combination does not work, and will generate tons of useless warnings.
> I think it might be better to do
>
> if test "$enable_debug" = yes && test "$ac_cv_prog_cc_g" = yes; then
> CFLAGS="$CFLAGS -g"
> + else
> + # configure sets CFLAGS to -O2 for gcc, so this is only for non-gcc
> + if test x"$CFLAGS" = x""; then
> + CFLAGS="-O"
> + fi
> fi
>
> regards, tom lane
>
--
Bruce Momjian | http://candle.pha.pa.us
pgman@candle.pha.pa.us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup. | Newtown Square, Pennsylvania 19073