-
Notifications
You must be signed in to change notification settings - Fork 69
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
Add Gradle Toolchain Support #357
Comments
We would like to, but we need the Gradle team's help. See gradle/gradle#18028 |
Note that according to @melix's comment "The PR, in itself, is irrelevant though" now that Gradle 7.6 added support for Java Toolchain downloads from arbitrary repositories, also see this comment of mine. |
In conjunction with https://github.com/gradle/foojay-toolchains this now basically works as outlined e.g. at oss-review-toolkit/ort@main...graalvm-upgrade. |
Thanks @sschuberth but I couldn't get that working. Test project: For me:
Probably I'm using it wrong. Any help would be great! |
@jamesward Edit: Also, just delete all of |
Tried again with 8.0-rc-1 and cleared the jdks and got:
|
Yes, for that issue you need my patch to Gradle. But even that is not enough yet: As Gradle's Copy-task cannot handle symlinks, some build file logic is required to fix that up. I've updated my example branch accordingly. |
Oh cool. Thanks for all your work on this! |
FYI, with my fix in Gradle 8.1, I'm able to use this code to bootstrap GraalVM via the
|
According to this PR oracle/graal#5995 , Native Image is now shipped as part of the GraalVM JDK and thus no longer needs to be installed via |
@melix wjat is the current status of the toolchain support in Gradle? Are there any plans to support "JDK features" such as JavaFX or Native Image that would help automatically pick the right JDK for the user? GraalVM has become easier to set up and use in the last couple of months, not sure this issue is still relevant? |
The issue is still relevant to me. We still have to disable toolchain support by default because of the limitations of the Gradle APIs. There is no progress made on that, as per a conversation I had with @ljacomet at Devoxx. |
Given the inclusion of |
The only missing piece is gradle/gradle#25264 |
I don't think that's the only thing missing. As far as I can tell there's no way for Gradle to make the difference between a regular Oracle JDK and an Oracle GraalVM JDK, (nor can it make the difference between another vendor's GraalVM), because it can only "look" a the |
Exactly. Also see my gradle/gradle#25521 in this context. |
I'm using:
And have a Gradle toolchain configured:
And I get:
It'd be awesome if the Gradle Native plugin worked together with Gradle Toolchains.
The text was updated successfully, but these errors were encountered: