[issue5572] distutils ignores the LIBS configure env var

Collin Winter added the comment: I have no interest in forward-porting the patch, closing accordingly. At least there will be a record of this patch for anyone interested in getting a free 10% performance boost for their Python 2.x or 3.[012] systems. -- assignee: tarek -> eric.araujo

[issue5572] distutils ignores the LIBS configure env var

Éric Araujo added the comment: I’m afraid performance improvements are not bug fixes, which are the only possible changes in distutils now. This change is however appreciated for packaging. -- ___ Python tracker

[issue5572] distutils ignores the LIBS configure env var

Collin Winter added the comment: I have provided justification in the original patch submission. Without this patch, we were unable to cleanly apply gcc's feedback-directed optimization system to Python. FDO yields significant performance improvements. --

[issue5572] distutils ignores the LIBS configure env var

Éric Araujo added the comment: > This wasn't so much a feature request as a request for review. >From a tracker process viewpoint, any report is a bug, a feature request or a >doc bug. A bug can be defined as a discrepancy between the doc and the code. >Given that I found no mention of LIBS

[issue5572] distutils ignores the LIBS configure env var

Collin Winter added the comment: This wasn't so much a feature request as a request for review. I had already implemented the necessary changes and provided a patch. We found this change necessary when working on Unladen Swallow. -- ___ Python trac

[issue5572] distutils ignores the LIBS configure env var

Éric Araujo added the comment: Reclassifying as a feature request for the packaging module. The distutils docs don’t say using LIBS is supported and there aren’t many people asking for it on this bug report. -- components: +Distutils2 -Distutils nosy: +alexis, eric.araujo versions: +

[issue5572] distutils ignores the LIBS configure env var

Changes by Mark Lawrence : -- versions: +Python 3.2 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.

[issue5572] distutils ignores the LIBS configure env var

Collin Winter added the comment: 2009/5/4 <"Tarek Ziadé "@psf.upfronthosting.co.za>: > About the patch: > > is there a particular reason why you have changed the call to > 'set_library' into calls to 'add_library' in build_ext in your patch ? Yes. It seems like the proper thing to do is to mer

[issue5572] distutils ignores the LIBS configure env var

Tarek Ziadé added the comment: > May be I miss something but I could not found why distutils > has to use LIBS to link a module. It doesn't right now, but depending on the extension you want to build, you might want to configure it and make sure it is passed to the compiler. About the patch:

[issue5572] distutils ignores the LIBS configure env var

Roumen Petrov added the comment: The LIBS contain all module dependent libraries, libpython is linked with LIBS, modules are linked with libpython. May be I miss something but I could not found why distutils has to use LIBS to link a module. -- ___ P

[issue5572] distutils ignores the LIBS configure env var

Collin Winter added the comment: The patch attached to issue 5060 seems very tightly focused on a problem with gcc FDO. This is a more general patch that solves the problem of distutils ignoring LIBS. -- ___ Python tracker

[issue5572] distutils ignores the LIBS configure env var

Roumen Petrov added the comment: issue 5060 address this -- nosy: +rpetrov ___ Python tracker ___ ___ Python-bugs-list mailing list Un

[issue5572] distutils ignores the LIBS configure env var

New submission from Collin Winter : If you pass LIBS to ./configure (as in "LIBS=-lgcov ./configure"), distutils ignores this when building extension modules, which breaks when using certain gcc options which require certain libraries (I'm thinking of -fprofile-generate). The attached patch remed