Skip to content
This repository was archived by the owner on Jul 5, 2023. It is now read-only.

Add support to build aarch64 wheels#182

Merged
hauntsaninja merged 3 commits intopython:masterfrom
tbbharaj:tbbharaj/aarch64-wheels
Jan 24, 2022
Merged

Add support to build aarch64 wheels#182
hauntsaninja merged 3 commits intopython:masterfrom
tbbharaj:tbbharaj/aarch64-wheels

Conversation

@tbbharaj
Copy link
Copy Markdown
Contributor

@tbbharaj tbbharaj commented Dec 7, 2021

In reference to issue #180, created this PR to add support for aarch64 wheels.
Happy to address any feedbacks!
Successful builds: https://github.com/tbbharaj/typed_ast/actions/runs/1546050914

Thank you

@the-knights-who-say-ni
Copy link
Copy Markdown

Hello, and thanks for your contribution!

I'm a bot set up to make sure that the project can legally accept this contribution by verifying everyone involved has signed the PSF contributor agreement (CLA).

Recognized GitHub username

We couldn't find a bugs.python.org (b.p.o) account corresponding to the following GitHub usernames:

@tbbharaj

This might be simply due to a missing "GitHub Name" entry in one's b.p.o account settings. This is necessary for legal reasons before we can look at this contribution. Please follow the steps outlined in the CPython devguide to rectify this issue.

You can check yourself to see if the CLA has been received.

Thanks again for the contribution, we look forward to reviewing it!

env:
CIBW_BUILD: "cp${{ matrix.python-version }}-*"
CIBW_SKIP: "*-manylinux_i686 *-win32"
CIBW_SKIP: "*-manylinux_i686 *-win32 *-macosx_arm64"
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Is this intentional? We still want arm64 wheels for macOS right?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yes, added it back to be keep it same as before

Copy link
Copy Markdown
Collaborator

@hauntsaninja hauntsaninja left a comment

Choose a reason for hiding this comment

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

Thank you! Sorry, between waiting for the CLA business and the holidays this got a little bit lost. Just had one question :-)

@hauntsaninja
Copy link
Copy Markdown
Collaborator

Hmm, looks like aarch64 build hung on Python 3.10. Let me try kicking it, in case it was a fluke.

@tbbharaj
Copy link
Copy Markdown
Contributor Author

Hmm, looks like aarch64 build hung on Python 3.10. Let me try kicking it, in case it was a fluke.

@hauntsaninja On my branch I was able to get aarch64 build on python 3.10 successfully. https://github.com/tbbharaj/typed_ast/actions/runs/1713796535

@tbbharaj
Copy link
Copy Markdown
Contributor Author

@hauntsaninja Does this look good to merge?

@hauntsaninja
Copy link
Copy Markdown
Collaborator

Yup, thanks for your patience. It's a little unfortunate that the build seems flakey, but we'll see how bad of a problem that is going forward. I'll release a 1.5.2 with aarch64 wheels.

@hauntsaninja hauntsaninja merged commit bbf3eb1 into python:master Jan 24, 2022
@hauntsaninja
Copy link
Copy Markdown
Collaborator

Thanks, 1.5.2 is out!

@tbbharaj
Copy link
Copy Markdown
Contributor Author

tbbharaj commented Jan 24, 2022

Thanks, 1.5.2 is out!

Thank you!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants