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

BUG: Fixed mingw.lib error #7119

Merged
merged 1 commit into from
Jan 31, 2016
Merged

BUG: Fixed mingw.lib error #7119

merged 1 commit into from
Jan 31, 2016

Conversation

gfyoung
Copy link
Contributor

@gfyoung gfyoung commented Jan 25, 2016

Addresses issue in #647 in which attempts to compile numpy with MinGW and Fortran result in a missing mingw.lib error. Implemented the change proposed in that issue, tested locally on my machine, and all tests pass.

@rgommers
Copy link
Member

@gfyoung thanks for working on this one. With what Mingw32 version did you test this?

I'd like to test this on my Mingw32 install next week (traveling now) before merging.

@rgommers rgommers self-assigned this Jan 26, 2016
@rgommers
Copy link
Member

(if I haven't looked at this by the end of next week, feel free to ping me)

@gfyoung
Copy link
Contributor Author

gfyoung commented Jan 26, 2016

Sure thing. I'm not sure which number is the actual version number you're asking for, but my version of mingw32-base, which is the basic MinGW installation, is version 2013072200.

@rgommers
Copy link
Member

Okay, tested this with my old numpy-vendor setup and verified the directory layout as given in gh-647. All looks good, so merging. Thanks @gfyoung

rgommers added a commit that referenced this pull request Jan 31, 2016
@rgommers rgommers merged commit cacdfd3 into numpy:master Jan 31, 2016
@rgommers rgommers added this to the 1.12.0 release milestone Jan 31, 2016
@gfyoung gfyoung deleted the mingw_patch branch January 31, 2016 20:53
@gfyoung gfyoung mentioned this pull request Jan 31, 2016
charris added a commit that referenced this pull request Jan 31, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants