Skip to content

Conversation

@hmartinez82
Copy link
Contributor

@hmartinez82 hmartinez82 commented Dec 20, 2025

GitHub Actions now has a native Windows on ARM runner. The MSY2 CLANGARM64 environment can be to build ARM64 binaries for windows.

This also introduces a new Inno Setup recipe externally set variable to change the name of the output installer file.
The values for CMAKE_SYSTEM_PROCESSOR are AMD64 and ARM64

@hmartinez82 hmartinez82 force-pushed the arm64 branch 2 times, most recently from 3eca8b2 to 4aa392f Compare December 20, 2025 07:17
@hmartinez82
Copy link
Contributor Author

@Jarod42 wait, it's not building yet 😅. Failing towards the very end, looking for clang-format, only failing on ARM64, I'm still trying to find out why: https://github.com/hmartinez82/codelite/actions/runs/20391021465/job/58600199488

@Jarod42
Copy link
Collaborator

Jarod42 commented Dec 20, 2025

I didn't plan to merge :), just give feed back (ok for me, I also activate CI).
For this kind of change, I would let Eranif merges it anyway.

@hmartinez82 hmartinez82 force-pushed the arm64 branch 4 times, most recently from 7e61272 to 62d6384 Compare December 20, 2025 10:06
@hmartinez82
Copy link
Contributor Author

@Jarod42 Getting there
image
But I had to manually copy zlib1.dll, libssl-3.dll, libcrypto-3.dll, libwinpthread-1 to the installed bin folder for it to work. These files don't even have the same same across CLANG64 and CLANGARM64 🫤

@hmartinez82
Copy link
Contributor Author

Ugh, that's because cygcheck doesn't understand ARM64 binaries (it's part of the MSYS environment, not CLANGARM64)

@hmartinez82 hmartinez82 marked this pull request as ready for review December 20, 2025 14:39
@hmartinez82 hmartinez82 requested a review from eranif December 20, 2025 14:39
@hmartinez82
Copy link
Contributor Author

@Jarod42 finally
image

Can you re-run the checks?

@eranif eranif merged commit 80c1482 into eranif:master Dec 29, 2025
7 checks passed
@hmartinez82 hmartinez82 deleted the arm64 branch December 29, 2025 17:34
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.

3 participants