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

Remove java.toolchain from build, fix Win arm64 #75

Merged
merged 1 commit into from
Dec 24, 2023

Conversation

mbland
Copy link
Owner

@mbland mbland commented Dec 24, 2023

After #73, running .\gradlew.bat build on arm64 Windows would fail with "WindowsRegistry is not supported on this operating system." Removing the java.toolchain block from strcalc/build.gradle.kts allows it to succeed, as Gradle no longer tries to query the Windows registry to discover the Java toolchain. It will just pick up the toolchain from the environment.

Running .\gradlew.bat -q javaToolchains will still produce the failure, however.

See:

After #73, running `.\gradlew.bat build` on arm64 Windows would fail
with "WindowsRegistry is not supported on this operating system."
Removing the `java.toolchain` block from strcalc/build.gradle.kts allows
it to succeed, as Gradle no longer tries to query the Windows registry
to discover the Java toolchain. It will just pick up the toolchain from
the environment.

Running `.\gradlew.bat -q javaToolchains` will still produce the
failure, however.

See:
- https://docs.gradle.org/current/userguide/toolchains.html
- gradle/native-platform: WindowsRegistry is not supported on this
  operating system. #274
  gradle/native-platform#274 (comment)
- gradle/gradle: Support ARM64 Windows #21703
  gradle/gradle#21703
@mbland mbland self-assigned this Dec 24, 2023
@mbland mbland merged commit 9d14a6e into main Dec 24, 2023
3 checks passed
@mbland mbland deleted the remove-java-toolchain-fix-win-arm64 branch December 24, 2023 00:41
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.

None yet

1 participant