-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Re-enable ARM Windows builds in release pipeline #1141
Comments
Does this mean we want support aarch64 windows? Then we need e.g. launcher binaries and some testing that puffin actually works. |
I'll turn this into a more general issue. This was just about getting CI building but that makes sense. I assume I should similarly disable the i686 Windows then? |
I don't understand which of the errors was the fatal one. On linux with
|
@konstin This is because FYI, |
Thank you, release mode indeed works! |
Any plans here? Did it just need to be built in release mode? Windows ARM is another platform cibuildwheel supports. I also have a Windows ARM dev box that I'd like to be able to use uv on. :) |
If you guys would be helped at all by someone testing or compiling on a Windows ARM machine, I have one and am happy to help! |
So with the advent of the Snapdragon X chips, we're seeing Windows ARM machines become more commonplace. Older ARM chips were pretty lame, but they're dramatically improving, and dev ecosystem support is too. I have one team member who uses Windows ARM as their personal dev machine. |
If someone wants to put some effort into getting an ARM build working I'm all for it. I think we could start with something low impact like the #8269 build then move to distributing releases as a secondary goal. |
@zanieb can you give me some guidance on how to go about it? I have a snapdragon laptop and heavily use UV, would love to contribute. |
I added a runner called Lines 529 to 560 in 9d417da
|
So @zanieb basically, I need to clone the uv repo, run the steps to build it on my local machine, fix any bugs, and then write: Lines 529 to 560 in 9d417da
Is this the right approach? |
That sounds correct to me, yep! |
Ok @zanieb I was able to get It worked without any bugs/hiccups. Do you think:
will work? Will |
Feel free to just post the PR and we can see if CI works.
You need to change the
Presumably it will work — if not we can just drop the dev-drive. |
@zanieb Can you tell me the label for windows aarch64 runner? According to this https://github.blog/news-insights/product-news/arm64-on-github-actions-powering-faster-more-efficient-build-systems/#get-started-using-arm-hosted-runners-today and https://www.youtube.com/watch?v=cgI6SBP8pEM it is only available to enterprise github accounts, does |
I set one up, see #1141 (comment) |
Based on discussion in #10306, this adds building aarch64 exe using x86 runner. See: #10402 (comment) Addresses: #1141 --------- Co-authored-by: Zanie Blue <contact@zanie.dev>
See: https://github.com/astral-sh/puffin/actions/runs/7673412766 (failure to build
libgit2
).See: https://github.com/astral-sh/puffin/actions/runs/7673603363 (failure to build
libgit2
, even after enabling vendoring).The text was updated successfully, but these errors were encountered: