-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
[GR-51017] Use JDK 17 for building Quarkus #7998
Conversation
.github/workflows/quarkus.yml
Outdated
# Use Java 17 to build Quarkus as that's the lowest supported JDK version currently | ||
- uses: graalvm/setup-graalvm@v1 | ||
with: | ||
version: 'latest' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
version
is deprecated. As much as I like using setup-graalvm
, maybe it's better to use setup-java
instead because here, we only care about compiling with javac?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mostly yes. We decided to use GraalVM just as a very basic test that Quarkus compiles with it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated to using Temurin builds of JDK 17
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! Could we change this to use Oracle JDK? I can make the change unless you'd like to do it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure, please go ahead.
Quarkus no longer supports JDK 11. Furthermore, no actual tests are being run with GraalVM in JVM-mode so using it instead of Temurin builds doesn't seem to have any benefits.
3d087da
to
68aa1b5
Compare
Quarkus no longer supports JDK 11
See https://github.com/oracle/graal/actions/runs/7162366442/job/19499236553#step:12:2328