-
-
Notifications
You must be signed in to change notification settings - Fork 36
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
also enable recent patches on windows #234
Conversation
Hi! This is the friendly automated conda-forge-linting service. I just wanted to let you know that I linted all conda-recipes in your PR ( |
2bb09f1
to
73b8c72
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks @h-vetinari. Feel free to merge now, or wait for another PR if that's better (potential merge conflict maybe, there's a lot going on).
REM This builds a Fortran file which calls a C function, but numpy.distutils | ||
REM creates an isolated DLL for these fortran functions and therefore it doesn't | ||
REM see these C functions. workaround this by compiling it ourselves and | ||
REM sneaking it with the blas libraries | ||
REM TODO: rewrite wrap_g77_abi.f with iso_c_binding when the compiler supports it | ||
cl.exe scipy\_build_utils\src\wrap_g77_abi_c.c -c /MD | ||
cl.exe /I%NP_INC% /I%PY_INC% scipy\_build_utils\src\wrap_g77_abi_c.c -c /MD |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This seems fine to me - whatever works really, since this line is a complete hack anyway that is going to disappear soon.
Excluded from #233 to make forward progress, I realised we should ensure these patches work on windows, so that 1.11 remains buildable with distutils.