This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author lambacck
Recipients Arfrever, brett.cannon, eric.araujo, georg.brandl, jwilk, jyasskin, lambacck, lemburg, loewis, lukasz.langa, marienz, mark.dickinson, nyogtha, pitrou, ronaldoussoren, rpetrov, skrah, tarek, vaxhacker
Date 2011-02-06.15:32:48
SpamBayes Score 9.331929e-08
Marked as misclassified No
Message-id <1297006369.19.0.00697787596914.issue969718@psf.upfronthosting.co.za>
In-reply-to
Content
I am not convinced that the minimal patch would work for my original issue. I wanted to be able to override the -march option which shows up in OPT on Fedora. I was cross-compiling to a target architecture that does not support the -march option so I would not be able to provide a different one as override. 

The proposed minimal patch would leave the OPT value and make it unchangeable because CFLAGS would pull out a value for OPT from the Makefile which shows as in my current Ubuntu system:

OPT=        -DNDEBUG -g -fwrapv -O2 -Wall -Wstrict-prototypes
BASECFLAGS=  -fno-strict-aliasing
CFLAGS=     $(BASECFLAGS) $(OPT) $(EXTRA_CFLAGS)

The debian-sysconfig-flags looks the most correct because it allows override at any point by emulating the CFLAGS = $(BASECFLAGS) $(OPT) $(EXTRA_CFLAGS) logic.

It still looks like it needs an override for EXTRA_CFLAGS and the logic for LD_SHARED looks wrong. In the Makefile LDSHARED is just:
LDSHARED=   $(CC) -shared -Wl,-O1 -Wl,-Bsymbolic-functions

Why does LDSHARED need CFLAGS and CPPFLAGS? When provided as an override but not in the Makefile? Do we need to allow for this override in the case of overriding OPT, BASEFLAGS or EXTRA_CFLAGS?
History
Date User Action Args
2011-02-06 15:32:49lambaccksetrecipients: + lambacck, lemburg, loewis, brett.cannon, georg.brandl, ronaldoussoren, mark.dickinson, vaxhacker, nyogtha, marienz, pitrou, jyasskin, tarek, jwilk, eric.araujo, rpetrov, Arfrever, skrah, lukasz.langa
2011-02-06 15:32:49lambaccksetmessageid: <1297006369.19.0.00697787596914.issue969718@psf.upfronthosting.co.za>
2011-02-06 15:32:48lambaccklinkissue969718 messages
2011-02-06 15:32:48lambacckcreate