Skip to content

Conversation

@rwtolbert
Copy link
Contributor

Small update to add Windows on Arm64 support. This does require changing the (os/arch) value from 'aarch64' to 'arm64' just for Windows on Arm.

Also requires the latest version 3.14 of the WiX toolset.

@bakpakin
Copy link
Member

Looks good to me, but is there any real need to change :aarch64 to :arm64 for windows only? afaik they are the same thing, with aarch64 being the more "official" name for the ISA.

Summary: Small update to add Windows on Arm64 support. Also requires the latest version 3.14 of the WiX toolset.
@rwtolbert
Copy link
Contributor Author

rwtolbert commented Dec 31, 2024

just pushed an update that removes the change in (os/arch).

WiX uses "arm64" and Python uses "arm64". Clang and Rust use "aarch64" so there is no clear answer.

@bakpakin bakpakin merged commit f252933 into janet-lang:master Jan 1, 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

2 participants