-
Notifications
You must be signed in to change notification settings - Fork 253
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
auditwheel: error: cannot repair "...cp37m-linux_x86_64.whl" to "manylinux2010_x86_64" #417
Comments
Related to issue #247 |
Sorry for this issue. The problem was a C file leftover created by cffi when testing my package on the travis machine, then going to manylinux container without cleaning that up first. |
Huh, OK, I though we had a check to print a better error message in such a case, somewhere. Thanks for reporting the solution, btw! :-) |
Hi @YannickJadoul . Thanks a lot for the looking into it. That is what happened. Steps to the errorMy package has C extension built using CFFI python package: https://github.com/limix/bgen-reader-py/blob/master/build_ext.py Before the usual
I do some tests on my package, which involves in build_ext'ng it. By doing it, CFFI produces a C file inside my package gluing all the things together, which is based on the current the environment (i.e., outside of the manylinux images). When then entering into the cibuildwheel pipeline, build_ext is ran again but this time CFFI believes that the gluing-C file is up-to-date, instead of generating a new one based on the current manylinux environment. This must have caused the error:
SolutionMy solution was simply to issue an I'm not sure if there is something that cibuildwheel could do about it. |
Hi, @horta. Thanks for the extra info! @joerick Does the (reopening until this last part is figured out) |
I see what you mean. Here is the verbose=3 travis output: https://travis-ci.com/github/limix/bgen-reader-py/jobs/370550804 Have a look at the line https://travis-ci.com/github/limix/bgen-reader-py/jobs/370550804#L3649 onward:
What the above tells me is that CFFI checked that (inside manylinux) Do you think this is more an user problem, CFFI or cibuildwheel? It feels to me that is more an user problem but I have to admit that I spent around 4 hours trying to figure it out. |
Thanks for checking that! :-) So it is a combination of effects, indeed, but there is a
CFFI is definitely not to blame, I believe. You're intuition is right: it's a combination of the user and cibuildwheel. The thing is, we had some functionality where cibuildwheel would warn you, after #139 (i.e., PR #263). But now that you've confirmed that there was a |
I think that the problem comes from using same job for test and wheel build. Best option is to define separate job for this. Then building wheel starts on clean project. |
That actually makes a lot of sense, @Czaki . It will prevent further problems in the future, thanks! |
Thanks for the prod @YannickJadoul - |
Thanks for sticking along and helping debugging, @horta! :-) |
The relevant bit:
Link: https://travis-ci.com/github/limix/bgen-reader-py/jobs/370485778
It seems to have happened after I added
pyproject.toml
to my project.The text was updated successfully, but these errors were encountered: