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 pitrou
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:58:12
SpamBayes Score 3.0053737e-13
Marked as misclassified No
Message-id <1297007890.3765.5.camel@localhost.localdomain>
In-reply-to <1297006369.19.0.00697787596914.issue969718@psf.upfronthosting.co.za>
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. 

I don't understand how you can cross-compile using the host Python
Makefile. Could you explain?

> 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

EXTRA_CFLAGS is not defined in the Makefile so it would probably be
taken from the environment anyway (if my understand of Makefiles is
right). That seems to explain the name "EXTRA_CFLAGS", but I'm no
specialist of the jungle that have building systems become under Unix.
History
Date User Action Args
2011-02-06 15:58:13pitrousetrecipients: + pitrou, lemburg, loewis, brett.cannon, georg.brandl, ronaldoussoren, mark.dickinson, vaxhacker, nyogtha, marienz, jyasskin, tarek, jwilk, eric.araujo, rpetrov, Arfrever, lambacck, skrah, lukasz.langa
2011-02-06 15:58:13pitroulinkissue969718 messages
2011-02-06 15:58:12pitroucreate