Skip to content
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

Fixed a vcvarsall.bat error on win32/Py2.7 when trying to compile with m... #1805

Merged
merged 1 commit into from May 31, 2012

Conversation

dhirschfeld
Copy link

This is the same error as cython pull request 129.

cython/cython#129

This makes %cython also work for me but the cython devs have yet to confirm that this is the correct fix.

@takluyver
Copy link
Member

Thanks, @dhirschfeld .

@fperez
Copy link
Member

fperez commented May 31, 2012

This looks good to me code-wise, and since we don't have a win32/mingw machine to test on, we'll go with your report that it helps until further notice. Thanks a lot for contributing the fix! I'll merge it now.

fperez added a commit that referenced this pull request May 31, 2012
Fixes Cython compilation errors on certain Windows/mingw configurations.
@fperez fperez merged commit a1fcd08 into ipython:master May 31, 2012
mattvonrocketstein pushed a commit to mattvonrocketstein/ipython that referenced this pull request Nov 3, 2014
…s-fix

Fixes Cython compilation errors on certain Windows/mingw configurations.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants