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

Update moduleplugin to v1.8.10 #120

Merged
merged 3 commits into from
Feb 3, 2022

Conversation

FlorianKirmaier
Copy link
Contributor

updating module plugin, to fix issues with newer Gradle versions and Java17.
The issue is also described here:
https://discuss.gradle.org/t/unable-to-build-project-with-gradle-7-3-1-on-java-17/41601
and here
java9-modularity/gradle-modules-plugin#199

updating module plugin, to fix issues with newer gradle versions
@FlorianKirmaier FlorianKirmaier changed the title Update build.gradle Updating moduleplugin to 1.8.8 to avoid issues with Java17 and newer Gradle versions Jan 18, 2022
@palexdev
Copy link

Can someone take a look at this? This is kind of an important fix

@abhinayagarwal abhinayagarwal changed the title Updating moduleplugin to 1.8.8 to avoid issues with Java17 and newer Gradle versions Update moduleplugin to v1.8.10 Feb 3, 2022
Copy link
Collaborator

@abhinayagarwal abhinayagarwal left a comment

Choose a reason for hiding this comment

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

Tested with Gradle 7.3 and Java 17. Seems to fix the issue while still keeping compatibility with Java 11 👍

@abhinayagarwal abhinayagarwal merged commit 02d36b8 into openjfx:master Feb 3, 2022
@FlorianKirmaier
Copy link
Contributor Author

Thanks for merging.
So, can we get a new build, please. 😊

@abhinayagarwal
Copy link
Collaborator

Now available with v0.0.12 🎉

usta added a commit to usta/openjfx-docs that referenced this pull request Mar 15, 2022
* Need to increase javafxplugin version because of a bug exist for build with java11+ in moduleplugin which used in older versions of javafxplugin. It is fixed by openjfx/javafx-gradle-plugin#120 Which in merged with 0.0.12

* Need to increase jlink because of a bug exist in jlink for building java16 versions  ( beryx/badass-jlink-plugin#176 )
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

3 participants