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 lemburg
Recipients lemburg, paul.moore, steve.dower, tim.golden, zach.ware
Date 2015-04-16.12:58:43
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1429189123.39.0.755762327907.issue23970@psf.upfronthosting.co.za>
In-reply-to
Content
We do some monkey-patching on MSVCCompiler as well, but mostly because distutils doesn't define a good way to subclass or configure a compiler class.

IMO, monkey patching is a really bad idea. distutils should be evolved to provide more proper extension mechanisms like the ones via distclass and cmdclass.

That said, we only use the compilers that Python itself is compiled with for each version, so your change should work for us (modulo some fixes we'd have to do to support it).

One detail you are not considering with your patch is that distutils can well be used to build other tools and those may need older VC versions. Couldn't you at least keep the old compiler support around in a separate file ?
History
Date User Action Args
2015-04-16 12:58:43lemburgsetrecipients: + lemburg, paul.moore, tim.golden, zach.ware, steve.dower
2015-04-16 12:58:43lemburgsetmessageid: <1429189123.39.0.755762327907.issue23970@psf.upfronthosting.co.za>
2015-04-16 12:58:43lemburglinkissue23970 messages
2015-04-16 12:58:43lemburgcreate