-
Notifications
You must be signed in to change notification settings - Fork 662
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
Use manylinux: auto for aarch64 builds #3444
Conversation
Not sure this will work. Let's try. |
with: | ||
target: ${{ matrix.platform.target }} | ||
# On `aarch64`, use `manylinux: 2_28`; otherwise, use `manylinux: auto`. | ||
manylinux: ${{ matrix.platform.arch == 'aarch64' && '2_28' || 'auto' }} | ||
manylinux: auto |
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.
Try this workaround:
- uses: PyO3/maturin-aciton@v1
env:
# Workaround ring 0.17 build issue
CFLAGS_aarch64_unknown_linux_gnu: "-D__ARM_ARCH=8"
with:
manylinux: auto
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.
Wow I think that worked? Thanks! Are there any limitations here? Will the wheel work as expected?
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.
It should work as expected, I think the obvious limitation is that it might break in the future if ring/boringssl actually requires a newer GCC version.
2f2361b
to
92900b7
Compare
See: #3439