-
Notifications
You must be signed in to change notification settings - Fork 270
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
BLD: test wheels and sdist #4906
BLD: test wheels and sdist #4906
Conversation
f94b8a2
to
1da3351
Compare
I don't know exactly what I'm doing wrong but
I'm running out of time for today, but I'll try to come back soon. Any help is welcome. |
I was able to reproduce the linux segfaults locally (inside the cibuildwheel containers), and the backtraces point to static initialization of They initially showed up for Full backtraces:
|
Thanks @yut23. I also dropped the unused includes you referred to here just in case, but I actually didn't realized there was a segfault at al... Maybe it's something that should get reported to cibuildwheel ? In any case, I think we need to figure out how to properly run pytest from outside the source repo first, so we have at least some evidence that the segfault is specifically a problem with the ubuntu container. Currently, sdist and macOS wheels are blocked by the same problem, so, progress ! |
7f543c0
to
5c829a3
Compare
Made some progress regarding configuration, but I'm still seeing collection errors where files named |
It looks like |
Alright, I've figured out why: pytest makes the Relevant pytest code: https://github.com/pytest-dev/pytest/blob/cbf6bd9dd28d8464f7eebe32e51b398e2a5e3039/src/_pytest/main.py#L407-L416 |
Thank you so much for investigating this. Indeed, I never thought it could be a problem with pytest itself. Do you think this should be fixed upstream or is pytest working as intended ? |
superseded by #4913 |
PR Summary
In #4905 it became apparent that our wheels are not tested, and while they're not broken now (as far as I know), it seems worth it to make building safer (albeit probably much longer). Let's see how long it even takes.
PR Checklist