-
-
Notifications
You must be signed in to change notification settings - Fork 25.8k
CI Adds manylinux1 to building wheels #19235
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
Conversation
Thank you @thomasjpfan for your PR! I would prefer to set these environment variables:
To build the wheels for |
I think the advantage of building both is that we can be sure that we are able to build the "newer" version of |
If I am not wrong, Edit:
in the already defined build matrix and use:
By this way, we would not need to duplicate code. Off topic: I am missing the |
Ah you are correct, numpy builds the full matrix
Yea we are out of credits :( |
I edit my original answer with a possible idea to avoid code duplication.
😢 |
Before we merge this, I would want to make sure this actually resolves the original issue. In the original issue, the readthedocs instance is using pip 20.3, which, in principle, should have fetched |
We can upload the file in |
FYI in the original issue, I just think that the build was starting at the moment of the upload and only the |
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.
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.
Thank you @thomasjpfan!
I tagged this PR for 0.24.1 as we can backport this to the 0.24.X branch and trigger a build there to generate manylinux1 artifacts and upload them manually with twine. |
I will manage the backport. |
Co-authored-by: Olivier Grisel <olivier.grisel@ensta.org>
Reference Issues/PRs
Fixes #19233
What does this implement/fix? Explain your changes.
We have been getting errors from users that are downloading the source from pypi. This PR adds
manylinux1
to wheel builder.