-
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
Update PyPy to 7.3.5 and use PyPA's manylinux images with PyPy #671
Merged
Merged
Changes from all commits
Commits
Show all changes
15 commits
Select commit
Hold shift + click to select a range
511606c
Try out PyPy 7.3.4rc1 (dropping PyPy 3.6)
YannickJadoul 1ebad94
PyPy now has 64-bit Windows binaries
YannickJadoul 921a626
PyPy 7.3.4rc2
YannickJadoul 6789fb5
Fix Windows tests
YannickJadoul ac40e31
Change overlooked architecture entry of PyPy in build-platforms.toml
YannickJadoul 1a6b9d2
PyPy 7.3.4 released
YannickJadoul acbfce0
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 3ec1bfa
Try out PyPy 7.3.5rc2
YannickJadoul 0b0a37a
Update to PyPy 7.3.5rc3
YannickJadoul d608cc5
Update PyPy to 7.3.5 on macOS/Windows
mayeut 976d8a0
Use quay.io/pypa/manylinux2010_x86_64 for PyPy
mayeut 9abf199
Fix tests linked to manylinux2010_x86_64 usage for PyPy
mayeut 7126c10
Add manylinux2014 & manylinux_2_24 support for PyPy
mayeut c29a701
Add aarch64 support for PyPy
mayeut d846654
Add i686 support for PyPy
mayeut File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,27 +1,38 @@ | ||
[x86_64] | ||
manylinux1 = quay.io/pypa/manylinux1_x86_64:2021-05-16-740a2ec | ||
manylinux2010 = quay.io/pypa/manylinux2010_x86_64:2021-05-16-606ba6c | ||
manylinux2014 = quay.io/pypa/manylinux2014_x86_64:2021-05-16-606ba6c | ||
manylinux_2_24 = quay.io/pypa/manylinux_2_24_x86_64:2021-05-16-606ba6c | ||
manylinux1 = quay.io/pypa/manylinux1_x86_64:2021-05-24-df68af6 | ||
manylinux2010 = quay.io/pypa/manylinux2010_x86_64:2021-05-24-84874a2 | ||
manylinux2014 = quay.io/pypa/manylinux2014_x86_64:2021-05-24-84874a2 | ||
manylinux_2_24 = quay.io/pypa/manylinux_2_24_x86_64:2021-05-24-84874a2 | ||
|
||
[i686] | ||
manylinux1 = quay.io/pypa/manylinux1_i686:2021-05-16-740a2ec | ||
manylinux2010 = quay.io/pypa/manylinux2010_i686:2021-05-16-606ba6c | ||
manylinux2014 = quay.io/pypa/manylinux2014_i686:2021-05-16-606ba6c | ||
manylinux_2_24 = quay.io/pypa/manylinux_2_24_i686:2021-05-16-606ba6c | ||
manylinux1 = quay.io/pypa/manylinux1_i686:2021-05-24-df68af6 | ||
manylinux2010 = quay.io/pypa/manylinux2010_i686:2021-05-24-84874a2 | ||
manylinux2014 = quay.io/pypa/manylinux2014_i686:2021-05-24-84874a2 | ||
manylinux_2_24 = quay.io/pypa/manylinux_2_24_i686:2021-05-24-84874a2 | ||
|
||
[pypy_x86_64] | ||
manylinux2010 = pypywheels/manylinux2010-pypy_x86_64:2021-05-08-8650a6d | ||
manylinux2010 = quay.io/pypa/manylinux2010_x86_64:2021-05-24-84874a2 | ||
manylinux2014 = quay.io/pypa/manylinux2014_x86_64:2021-05-24-84874a2 | ||
manylinux_2_24 = quay.io/pypa/manylinux_2_24_x86_64:2021-05-24-84874a2 | ||
|
||
[pypy_i686] | ||
manylinux2010 = quay.io/pypa/manylinux2010_i686:2021-05-24-84874a2 | ||
manylinux2014 = quay.io/pypa/manylinux2014_i686:2021-05-24-84874a2 | ||
manylinux_2_24 = quay.io/pypa/manylinux_2_24_i686:2021-05-24-84874a2 | ||
|
||
[aarch64] | ||
manylinux2014 = quay.io/pypa/manylinux2014_aarch64:2021-05-16-606ba6c | ||
manylinux_2_24 = quay.io/pypa/manylinux_2_24_aarch64:2021-05-16-606ba6c | ||
manylinux2014 = quay.io/pypa/manylinux2014_aarch64:2021-05-24-84874a2 | ||
manylinux_2_24 = quay.io/pypa/manylinux_2_24_aarch64:2021-05-24-84874a2 | ||
|
||
[ppc64le] | ||
manylinux2014 = quay.io/pypa/manylinux2014_ppc64le:2021-05-16-606ba6c | ||
manylinux_2_24 = quay.io/pypa/manylinux_2_24_ppc64le:2021-05-16-606ba6c | ||
manylinux2014 = quay.io/pypa/manylinux2014_ppc64le:2021-05-24-84874a2 | ||
manylinux_2_24 = quay.io/pypa/manylinux_2_24_ppc64le:2021-05-24-84874a2 | ||
|
||
[s390x] | ||
manylinux2014 = quay.io/pypa/manylinux2014_s390x:2021-05-16-606ba6c | ||
manylinux_2_24 = quay.io/pypa/manylinux_2_24_s390x:2021-05-16-606ba6c | ||
manylinux2014 = quay.io/pypa/manylinux2014_s390x:2021-05-24-84874a2 | ||
manylinux_2_24 = quay.io/pypa/manylinux_2_24_s390x:2021-05-24-84874a2 | ||
|
||
[pypy_aarch64] | ||
manylinux2014 = quay.io/pypa/manylinux2014_aarch64:2021-05-24-84874a2 | ||
manylinux_2_24 = quay.io/pypa/manylinux_2_24_aarch64:2021-05-24-84874a2 | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
I will do a single comment here that might apply throughout the remaining of the changes.
Should we remove the
CIBW_MANYLINUX_PYPY_*
environment variables (adding a deprecation warning forCIBW_MANYLINUX_PYPY_X86_64_IMAGE
) ?I went with "we should keep it" and thus "them" for consistency's sake.
The only thing remaining would thus be how to best order those options in docs (either group by CPython/PyPy or by arch ? The current layout is by age).
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.
I was thinking something similar, but seems you beat me :-) If manylinux now "compresses" the two into one, then shouldn't we do so as well?
The more I think about it, the more I think we should do this, if we can deprecate
CIBW_MANYLINUX_PYPY_X86_64_IMAGE
. Extra argument: the PyPy manylinux images also include everything from manylinux itself, as a base. So if someone needs to use these old ones, it should still be possible?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.
About deprecating
CIBW_MANYLINUX_PYPY_X86_64_IMAGE
: this is slightly more tricky. Before, when we did so, we basically added it to the new setting. But now, if bothCIBW_MANYLINUX_X86_64_IMAGE
andCIBW_MANYLINUX_PYPY_X86_64_IMAGE
are set, it's not clear what the new value ofCIBW_MANYLINUX_X86_64_IMAGE
should be?Given that I'm not expecting a lot of users to actually set
CIBW_MANYLINUX_PYPY_X86_64_IMAGE
, would it be possible to just raise an error and break the config?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.
That's the other thing I noticed, yes. If we would keep them, I think maybe just lsting all PyPy things at the end in the same order as the default ones would be best, as we also e.g. do this for the Python versions.
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.
You're right, I didn't fully thought this through.
One benefit of keeping a separate key for PyPy is that it allows the following config to be valid:
Seeing this https://foss.heptapod.net/pypy/pypy/-/issues/3425 might also prove having a separate key for PyPy to be worth it.
While I do agree it should probably be dropped sooner rather than later, the "pip situation" does not agree with that...