Skip to content

Change GH Action JDK+FX Zulu to just the Temurin JDK#681

Merged
berry120 merged 1 commit into
quelea-projection:masterfrom
ebbez:build-temurin
Apr 26, 2025
Merged

Change GH Action JDK+FX Zulu to just the Temurin JDK#681
berry120 merged 1 commit into
quelea-projection:masterfrom
ebbez:build-temurin

Conversation

@ebbez
Copy link
Copy Markdown
Contributor

@ebbez ebbez commented Apr 24, 2025

The Temurin JDK comes preinstalled on the Github-hosted Runners so it saves some time from installing. The FX libraries aren't used for building, but are rather only needed for the runtime, which is still taken care of because the runtimes are installed by the the standalone installers that still come with the bundled JRE+FX (Zulu) from Azul.

(The libraries required during compilation/building with Gradle are provided by the Gradle, by the JavaFX plugin and the dependencies from build.gradle and don't need to come bundled with the JDK; only for buildtime/compile, only runtime needs and still gets that JDK+FX bundle)

The Temurin JDK comes preinstalled on the Github-hosted Runners so it saves some time from installing. The FX libraries aren't used for building, but are rather only needed for the runtime, which is taken care of by bundling the standalone installers with the JRE+FX (Zulu) from Azul. (Not needed for build as the JFX libraries are provided by the JavaFX plugin and the dependencies from build.gradle
Copy link
Copy Markdown
Member

@berry120 berry120 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome, yup might as well use this one!

@berry120 berry120 merged commit aafd00e into quelea-projection:master Apr 26, 2025
1 check passed
@ebbez ebbez deleted the build-temurin branch April 28, 2025 13:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants