Skip to content

Update Gradle (Wrapper) to 8.14#680

Merged
berry120 merged 3 commits into
quelea-projection:masterfrom
ebbez:update-gradle
May 2, 2025
Merged

Update Gradle (Wrapper) to 8.14#680
berry120 merged 3 commits into
quelea-projection:masterfrom
ebbez:update-gradle

Conversation

@ebbez
Copy link
Copy Markdown
Contributor

@ebbez ebbez commented Apr 24, 2025

Updates Gradle to 8.14 (and removes the wrapper { block from build.gradle since it went un-updated).

(The Gradle setup action should verify the integrity of the Gradle Wrapper jar file, but if you want to verify manually you can find the checksums on https://gradle.org/release-checksums/ )

image

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.

I have a vague feeling the reason we didn't upgrade before was because the snapcraft image failed with a newer gradle version for some reason (seems like it blocked head requests that seemed to be used on newer versions) - but this was a while back, so I might merge in a bit just to see if it functions, but if not would have to back it out.

@ebbez ebbez changed the title Update Gradle (Wrapper) to 8.13 Update Gradle (Wrapper) to 8.14 Apr 27, 2025
@ebbez
Copy link
Copy Markdown
Contributor Author

ebbez commented Apr 27, 2025

Gradle 8.14 was released 2 days ago so I bumped the version to 8.14.

If it breaks Snaps then it's no good so definitely feel free to revert it then! I am trying to get for some needed dependencies/build tools up to date, mainly because DK/JFX 24 are probably needed for Linux due to the shrinking maximized window issue, and Gradle 8.14 supports JDK 24 officially, but of course the pre-existing installers take priority.

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.

Let's merge and see what happens to the snaps!

@berry120 berry120 merged commit 9f3952d into quelea-projection:master May 2, 2025
1 check passed
@ebbez ebbez deleted the update-gradle branch May 2, 2025 18:18
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