jamesfredley commented on issue #15413: URL: https://github.com/apache/grails-core/issues/15413#issuecomment-3928200298
@gsartori for generated apps and for grails-core, we did have `toolchains` (since Gradle recommends them by default) this way for ~6 months during Grails 7 development, but then switched to release flags `compileJava.options.release = 25` These are the reasons toolchains were not done by default - Confusing to end users that are not aware of toolchains and expect the app to build and bootrun using the JDK they have specificized for the environment - Some JDK vendors and versions require adding https://plugins.gradle.org/plugin/org.gradle.toolchains.foojay-resolver-convention - toolchains override JDK matrix in GitHub workflow This was just for default, not a hard 1 vs the other or do not us recommendation. Until Gradle 8.14.4 you had to use toolchains to build Grails apps with 25 and I used it extensively. When toolchains are more widely understood we could possibly revisit, but I think the current state is likely the right default for generated apps for the reasons above. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
