Skip to content
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

Add windows arm64 build #3457

Merged
merged 2 commits into from
Feb 21, 2025
Merged

Add windows arm64 build #3457

merged 2 commits into from
Feb 21, 2025

Conversation

finnagin
Copy link
Contributor

Simple change to cross-compile a windows arm64 wheel. (#3456)

While this cannot be tested in the CI until GitHub starts supporting Windows arm64 machines for open source projects, I did test the wheel this produced locally on an arm64 Windows machine and it passed all tests.

@finnagin
Copy link
Contributor Author

It looks like the python 3.14 tests are failing due to a depreciation warning. (DeprecationWarning: 'asyncio.set_event_loop' is deprecated and slated for removal in Python 3.16) I tried rerunning the tests on a different branch without my arm64 build changes just to be sure I didn't cause it and it still failed.

bdarnell added a commit to bdarnell/tornado that referenced this pull request Feb 20, 2025
from tornadoweb#3457 and if this works i'll merge that one,
i just need it on my fork to test the ci build
@bdarnell
Copy link
Member

Thanks! I've run a test of the build workflow with this change to make sure it doesn't slow things down too much and it looks fine: it adds about 75 seconds to the windows build which previously took about 10 minutes. This is currently irrelevant to the total build time because the linux builds take 22 minutes (although this will change when I get a linux arm build runner so we don't have to build and test arm in emulation).

@bdarnell bdarnell merged commit 0ab655e into tornadoweb:master Feb 21, 2025
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants