Message101680
MaL> The patch you checked in still unconditionally overrides the
MaL> CFLAGS setting applied by AC_PROG_CC in case no CFLAGS variable
MaL> is set.
MaL>
MaL> The issue now is: AC_PROG_CC no longer initializes CFLAGS
MaL> if not set.
Sorry, but I don't understand. Why should it be initialized?
I don't like the default value because it enables optimization when --with-pydebug is used and I consider that as a bug. If no configure option is used, Python adds -O3 as before. About -g: Python always add it, so the -g from AC_PROG_CC was redundant.
In Makefile.pre.in, prefixes and suffixes are added to the CFLAGS: "CFLAGS= $(BASECFLAGS) @CFLAGS@ $(OPT) $(EXTRA_CFLAGS)". But I consider this as a separate issue. |
|
Date |
User |
Action |
Args |
2010-03-25 09:14:36 | vstinner | set | recipients:
+ vstinner, lemburg, loewis, Arfrever |
2010-03-25 09:14:36 | vstinner | set | messageid: <1269508476.61.0.76435997971.issue8211@psf.upfronthosting.co.za> |
2010-03-25 09:14:35 | vstinner | link | issue8211 messages |
2010-03-25 09:14:34 | vstinner | create | |
|