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 steve.dower
Recipients christopher.hogan, dstufft, eric.araujo, paul.moore, r.david.murray, rohitjamuar, steve.dower, tim.golden, zach.ware
Date 2017-02-04.23:54:49
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1486252490.15.0.155700662938.issue26876@psf.upfronthosting.co.za>
In-reply-to
Content
Taking another look at the patch, I'm not real keen on the C/LDFLAGS section. I think if we want to support setting these, we should fully override the default settings (otherwise you can't specify certain options that are in the defaults), and avoid splitting them (which probably means changing the spawn() call to not quote arguments - right now the .split() logic is just wrong).

I'm sorry this is tough to get together - the distutils compilers are really fragile, which is why we would really rather migrate people off them onto some other build framework. Right now though, that framework doesn't exist. (One potential starting point is my msbuildcompiler at https://github.com/zooba/pyfindvs/tree/master/pyfindvs/msbuildcompiler will generate a .vcxproj file and build it, which allows much more reliable behaviour for options and detecting installs.)
History
Date User Action Args
2017-02-04 23:54:50steve.dowersetrecipients: + steve.dower, paul.moore, tim.golden, eric.araujo, r.david.murray, zach.ware, dstufft, christopher.hogan, rohitjamuar
2017-02-04 23:54:50steve.dowersetmessageid: <1486252490.15.0.155700662938.issue26876@psf.upfronthosting.co.za>
2017-02-04 23:54:50steve.dowerlinkissue26876 messages
2017-02-04 23:54:49steve.dowercreate