Skip to content

Update Gradle (Wrapper JAR) to 9.2.1#700

Merged
berry120 merged 1 commit intoquelea-projection:masterfrom
ebbez:update-gradle-wrapper
Jan 6, 2026
Merged

Update Gradle (Wrapper JAR) to 9.2.1#700
berry120 merged 1 commit intoquelea-projection:masterfrom
ebbez:update-gradle-wrapper

Conversation

@ebbez
Copy link
Contributor

@ebbez ebbez commented Jan 6, 2026

Turned out the Wrapper JAR was still Gradle 8.6. This PR updates Gradle + the Gradle Wrapper JAR & scripts to 9.2.1.

@berry120
Copy link
Member

berry120 commented Jan 6, 2026

Looks like there' a few other things changed here too, do these need changing or can we just update the jar path?

@berry120 berry120 mentioned this pull request Jan 6, 2026
@ebbez
Copy link
Contributor Author

ebbez commented Jan 6, 2026

I don't think the .gradlew(.bat) scripts necessarily need to change, but the script modifications were the result of running ./gradlew.bat wrapper --gradle-version 9.2.1 (the command to update the wrapper), so it might be nice to have those up-to-date as well.

It seems primarily that they added the SPDX license-identifier, changed something about resolving a path physically to the cd command (from previously the pwd command), and changed from classpath to executing the jar directly with --jar instead of --classpath. Doesn't seem to be doing too much changes.

But if it's preferred for the scripts to stay the same then I'll remove those modifications :)

@berry120
Copy link
Member

berry120 commented Jan 6, 2026

Happy if that was the auto-update script :)

@berry120 berry120 merged commit 4de536a into quelea-projection:master Jan 6, 2026
1 check passed
@ebbez ebbez deleted the update-gradle-wrapper branch January 6, 2026 23:00
JessyJP pushed a commit to JessyJP/Quelea that referenced this pull request Jan 16, 2026
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