-
-
Notifications
You must be signed in to change notification settings - Fork 10.9k
TST: Windows f2py
tests hang indefinitely with msys gcc-fortran on PATH
#9342
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
Comments
@matthew-brett, is this issue still valid? |
@matthew-brett, is this issue still topical? |
I successfully tested this with Windows 10 and
and the UCRT64 GCC toolchain from msys2:
However, a lot of tests are skipped right now due to #9673, so I leave this open for now. |
With un-skipped test (#9673) I got a lot of errors, but not hang or segfault with |
@carlkl - I get the same as you, after laboriously editing out the decorators for skipping the tests:
|
Errors appear to be of form:
|
If I manually add However, there are 39 errors left, also with i.e.
It seems, that `.\_test_ext_module_5403.cp39-win_amd64.pyd was successfully build, but the build directory has been removed before running the test? |
@carlkl or @matthew-brett, it'd be really great to see if these errors are still present since the transition to |
f2py
tests hang indefinitely with msys gcc-fortran on PATH
I can certainly investigate this, but it will take some time ... |
Short test with Windows 10 64-bit
so must of the the tests are skipped, is this intended? |
Ah, sorry, jumped the gun, testing these should be after #25134.. |
With Appveyor file:
Without adding msys64 to the path, install + f2py tests pass in 30 seconds or so:
https://ci.appveyor.com/project/matthew-brett/windows-wheel-builder/build/1.0.53#L29
Adding msys64 to the path - hangs until build timeout (I canceled this one after 90 seconds):
https://ci.appveyor.com/project/matthew-brett/windows-wheel-builder/build/1.0.52#L16
Adding msys64 to the path, but uninstalling gcc-fortran, tests pass again:
https://ci.appveyor.com/project/matthew-brett/windows-wheel-builder/build/1.0.51#L40
The text was updated successfully, but these errors were encountered: