Skip to content

Update to Gradle 8 with higher JDK Support#439

Merged
soerendomroes merged 55 commits into
mainfrom
chore/gradle7
Jun 25, 2026
Merged

Update to Gradle 8 with higher JDK Support#439
soerendomroes merged 55 commits into
mainfrom
chore/gradle7

Conversation

@xeruf

@xeruf xeruf commented Feb 21, 2026

Copy link
Copy Markdown
Member

supports higher JDK versions, fancy dokka documentation and a lot more improvements including folder structure

@xeruf xeruf mentioned this pull request Feb 21, 2026
@xeruf xeruf force-pushed the chore/gradle7 branch 2 times, most recently from 3d70429 to c3503ba Compare February 21, 2026 23:16
@xeruf xeruf requested a review from soerendomroes February 21, 2026 23:32
xeruf and others added 26 commits February 22, 2026 00:35
Migrate maven to maven-publish
@xeruf

xeruf commented Feb 27, 2026

Copy link
Copy Markdown
Member Author

Ich verstehe nicht ganz, warum jetzt alle plugins verschoben werden.

damit es für die zukunft leichter zu maintainen bleibt wenn wir rotieren

Hattest du getestet, ob man mit dem SimpleClient für die Schüler auch mit einer niedrigeren Java Version auskommt?

JDK 17 ist das minimum mit Gradle 9, dann müssten die Schüler die gradle version downgraden oder das ohne gradle machen

Der build failed bei mir noch:

vorher mal ./gradlew clean ausgeführt?

@soerendomroes

soerendomroes commented Feb 27, 2026

Copy link
Copy Markdown
Member

./gradlew clean hat geholfen.

Leider läuft der Spieler nicht und es scheint auch noch keinen gui-branch geben, um alles zu testen.

Das liegt wohl daran, dass der player:run die gui auch baut?

@soerendomroes

Copy link
Copy Markdown
Member

Wenn wir mit für die Schulen noch Java 8 und 11 kompatibel sein wollen, dann müsste man den Spieler wahrscheinlich in ein eigenes Repo mit eigenem build bauen.

Wie siehst du das?

@xeruf

xeruf commented Mar 1, 2026

Copy link
Copy Markdown
Member Author

ne man könnte das auch händisch mit bundlen - aber wollen wir diese versionen wirklich weiter maintainen? gabs mal ne umfrage bei den tutoren ob die schulen die die versionen noch benutzen wirklich nix anderes haben?

Leider läuft der Spieler nicht und es scheint auch noch keinen gui-branch geben, um alles zu testen.

versteh ich nicht, der wird wie üblich mit bundle gebaut

Das liegt wohl daran, dass der player:run die gui auch baut?

ne hat nix mit der gui zu tun

@soerendomroes

Copy link
Copy Markdown
Member

ne man könnte das auch händisch mit bundlen - aber wollen wir diese versionen wirklich weiter maintainen? gabs mal ne umfrage bei den tutoren ob die schulen die die versionen noch benutzen wirklich nix anderes haben?

Die Umfrage dazu gibt es erst zum Ende der Saison. Die Tutoren, mit denen ich sprechen konnte, meinten, dass es noch ein paar Schulen gibt die das brauchen. Und die Versionen sind ja mindestens bis nächstes Jahr noch LTS.

Leider läuft der Spieler nicht und es scheint auch noch keinen gui-branch geben, um alles zu testen.

versteh ich nicht, der wird wie üblich mit bundle gebaut

Ok, anscheinend ist mein Developer setup ein bisschen anders als sonst und die Dinge die ich testen möchte.
Gut, dass wir heute mal schnacken.

@soerendomroes

Copy link
Copy Markdown
Member

@xeruf

Leider baut das bei mir (auch nach clean) nicht:

* What went wrong:
Some problems were found with the configuration of task ':server:startScripts' (type 'CreateStartScripts').
  - Gradle detected a problem with the following location: '/home/sdo/Documents/socha/Development/gui/backend/server/build/runnable/server.jar'.
    
    Reason: Task ':server:startScripts' uses this output of task ':server:createStartScripts' without declaring an explicit or implicit dependency. This can lead to incorrect results being produced, depending on what order the tasks are executed.

Klappt das bei dir einfach?

Comment thread .github/workflows/gradle.yml Outdated
@xeruf

xeruf commented Mar 18, 2026

Copy link
Copy Markdown
Member Author

weil das nur mündlich besprochen wurde und hier noch nicht steht - wir gehen runter auf gradle 8, was jdk 11 bis 24 supported und damit alles wichtige abdeckt

@soerendomroes

Copy link
Copy Markdown
Member

https://en.wikipedia.org/wiki/Java_version_history

Java 24 ist nicht das neueste Java mit einer kleineren Nummer als 25. Die LTS Versionen werden weiter geupdated, Java 24 ist nicht mehr supported. Es lohnt sich entsprechend beim Test sowas wie 8, 11, 17 und 21 zu testen.

Und wir hatten uns auf Gradle 8.14.X geeinigt, da dies auch noch Java 8 supported.
https://docs.gradle.org/current/userguide/compatibility.html

@xeruf

xeruf commented Mar 19, 2026

Copy link
Copy Markdown
Member Author

wir nutzen shadow 9.1 weswegen das backend bauen mindestens jdk 11 braucht, aber teilnehmer können trotzdem noch jdk 8 nutzen weil ich shadow da auf 8.3 gesetzt hab - das ist kein problem
https://github.com/GradleUp/shadow?tab=readme-ov-file#compatibility-matrix

@xeruf

xeruf commented Mar 19, 2026

Copy link
Copy Markdown
Member Author

hab chore/gradle7 gui branch nochmal aktualisiert

@xeruf xeruf changed the title Finally update to gradle 9 Update to Gradle 8 with higher JDK Support Mar 31, 2026
@soerendomroes

Copy link
Copy Markdown
Member

@xeruf what is the status here? Should we meet and merge this?

Have you already tested whether the gui works with this?

Comment thread settings.gradle.kts
@soerendomroes soerendomroes dismissed their stale review June 25, 2026 10:38

I am him.

@soerendomroes soerendomroes merged commit 0a8fb7d into main Jun 25, 2026
19 of 20 checks passed
@soerendomroes soerendomroes deleted the chore/gradle7 branch June 25, 2026 10:47
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