Add support for windows arm64 architecture for prunsrv.exe#311
Add support for windows arm64 architecture for prunsrv.exe#311wglas85 wants to merge 2 commits intoapache:masterfrom
Conversation
|
You need to install |
garydgregory
left a comment
There was a problem hiding this comment.
You need to install
MSCV Build Tools for ARM64/ARM64ECin order to be able to compile for arm64 using a cmdline set up viavcvarsall.bat arm64.
Hi @wglas85
Thank you for your PR.
I think you'll want to update https://github.com/apache/commons-daemon/blob/master/src/native/windows/README.txt
|
I've added build notes, adding arm64 to the github workflows is left to the maintainers. |
I think this already happens with |
|
Hi @markt-asf Are you OK with this PR? |
|
Thanks for your review, I think the new architecture needs to be added in |
|
To generate the arm64 artifact, the Windows job needs to be updated. The best way to me seems to run it on an arm runner (this needs to be extended with I can create a PR with the necessary changes once this gets merged, if you want. |
|
@jajik thanks for your offer, adding the necessary bits to the workflow is the best option for getting a signed windows binary for extensive testing. I think, it would be best to provide all binaries for all three architectures, I know, Windows can always emulate |
Broadly, yes. I am in favour of adding arm64 support. Some addition docs changes are also required (change log, HOWTO-RELEASE.txt) but those are all minor issues. I would like to see some confirmation that the resulting binary works on arm64 before we start including it in a release. |
|
The resulting binaries work in my Test environment: Win11 under VMWare Fusion on MacOS with openjdk-21.0.10.7-aarch64 |
This adds ARM64 support for Windows binaries as discussed in https://issues.apache.org/jira/browse/DAEMON-462
Based on this PR I kindly request to include an ARM64 binary of
prunsrv.exein futurecommons-daemonreleases.