Skip to content

Conversation

@Johni0702
Copy link
Contributor

This PR fixes various deprecated things in an effort to make the repo build with Gradle 9.
See individual commits for details.

This is required because Minecraft 26.1 requires Java 25 which requires Gradle 9, and Essential includeBuilds this repo, and therefore will require it to be compatible with Gradle 9.

This PR does not yet upgrade the Gradle used by this repository, because we still want it to remain compatible with Gradle 8 (at least until Essential upgrades to Gradle 9).

These changes do not affect the produced jars (they are bit-for-bit identical to the jars produced prior to these changes), with one exception: The container-modlauncher{8,9}.jar files (both produced by the Shadow plugin) are not bit-for-bit identical because it seems the newer Shadow plugin now uses the 2.0 ZIP spec, while the old one used the 1.0 spec. Their content appears identical though.

To the latest version which still works on Gradle 8.8.
The space-to-assign (`url ""` being equal to `url = ""`) Groovy syntax
is being deprecated, so while we have to touch all these lines, we may
as well centralize all repos into the `settings.gradle.kts` file.
Projects for which the folder does not exist are deprecated.
Relying on automatic test framework implementation dependencies is
deprecated.
Access to `project` during task execution is incompatible with the
Configuration Cache feature which is enabled by default with Gradle 9.
Because `Task`, which is referenced from `TaskOutputs`, isn't
serializable as far as the Configuration Cache is concerned.
@github-actions
Copy link

github-actions bot commented Jan 8, 2026

Test Results

17 files  ±0  17 suites  ±0   11m 15s ⏱️ -8s
90 tests ±0  90 ✅ ±0  0 💤 ±0  0 ❌ ±0 
94 runs  ±0  94 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit d7b9b04. ± Comparison against base commit fd7e1a4.

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.

2 participants