Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Only build aarch64 wheels for cpython manylinux #14259

Merged
merged 3 commits into from
Oct 21, 2022

Conversation

erikjohnston
Copy link
Member

As they are too slow to build for everything else

@erikjohnston erikjohnston added the X-Release-Blocker Must be resolved before making a release label Oct 21, 2022
@erikjohnston erikjohnston marked this pull request as ready for review October 21, 2022 13:22
@erikjohnston erikjohnston requested a review from a team as a code owner October 21, 2022 13:22
pyproject.toml Outdated
@@ -318,7 +318,7 @@ build-backend = "poetry.core.masonry.api"

[tool.cibuildwheel]
# Skip unsupported platforms (by us or by Rust).
skip = "cp36* *-musllinux_i686"
skip = "cp36* *-musllinux_i686 pp*aarch64 musllinux_aarch64"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this be *-musllinux_aarch64?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

D'oh!

Copy link
Contributor

@DMRobertson DMRobertson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this complements #14253 --- we should hopefully only build abi3 wheels for arm64 too?

@erikjohnston
Copy link
Member Author

I think this complements #14253 --- we should hopefully only build abi3 wheels for arm64 too?

I think that should happen automatically

Copy link
Contributor

@DMRobertson DMRobertson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As Patrick says elsewhere, I think this was meant to be targeted at 1.70? Otherwise LGTM (but we should check which wheels get built).

And I think:

Resolves #14234.
Resolves #14233.

@erikjohnston erikjohnston changed the base branch from develop to release-v1.70 October 21, 2022 13:55
@erikjohnston erikjohnston enabled auto-merge (squash) October 21, 2022 13:56
@erikjohnston erikjohnston merged commit 1c64215 into release-v1.70 Oct 21, 2022
@erikjohnston erikjohnston deleted the erikj/aarch64_wheel branch October 21, 2022 14:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
X-Release-Blocker Must be resolved before making a release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Decide which arm wheels we want to publish pp37-manylinux_aarch64 wheel fails tests after build
2 participants