New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Detect and use Microsoft Visual C++ Compilers for Python 2.7 package #258

Closed
bb-migration opened this Issue Sep 27, 2014 · 1 comment

Comments

Projects
None yet
1 participant
@bb-migration

bb-migration commented Sep 27, 2014

Originally reported by: stevedower (Bitbucket: stevedower, GitHub: Unknown)


Microsoft just released a compiler package for Python 2.7 containing VC9. It has both the 32-bit and 64-bit compilers and the Windows SDK headers and libs that should be necessary to build C extensions for Windows. The package is at http://aka.ms/vcpython27

It would be great if setuptools could patch distutils.msvc9compiler.find_vcvarsall() to be able to find this compiler when it's installed. The install directory is registered at (HKCU|HKLM)\Software\Microsoft\DevDiv\VCForPython\9.0@InstallDir and is the directory that contains the required vcvarsall.bat file.

This change could justifiably be added to Python 2.7.9, but that won't help people using Python 2.6 or 3.2 or people who aren't able to upgrade their Python 2.7 installation. The compiler package is aimed at package developers (to produce wheels) rather than end users, but it would be nicer to simply say "update setuptools and it'll work".

I'd also like to improve the error message when vcvarsall.bat cannot be found and include a link to the compiler. This can help all Python versions and will reduce the confusion faced by users.


@bb-migration

This comment has been minimized.

bb-migration commented Sep 27, 2014

Original comment by jaraco (Bitbucket: jaraco, GitHub: jaraco):


Update changelog to reference documented issue. Fixes #258.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment