Update to Gradle 8 with higher JDK Support#439
Conversation
3d70429 to
c3503ba
Compare
Migrate maven to maven-publish
|
@xeruf Gibt es dazu schon einen GUI branch der damit kompatibel ist? |
|
Der build failed bei mir noch: Muss ich außer zu pullen noch etwas einstellen? |
damit es für die zukunft leichter zu maintainen bleibt wenn wir rotieren
JDK 17 ist das minimum mit Gradle 9, dann müssten die Schüler die gradle version downgraden oder das ohne gradle machen
vorher mal |
|
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? |
|
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? |
|
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?
versteh ich nicht, der wird wie üblich mit
ne hat nix mit der gui zu tun |
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.
Ok, anscheinend ist mein Developer setup ein bisschen anders als sonst und die Dinge die ich testen möchte. |
|
Leider baut das bei mir (auch nach clean) nicht: Klappt das bei dir einfach? |
|
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 |
|
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. |
|
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 |
backend itself needs at least jdk 11 to build with shadow 9.1 but this enables participants to still use jdk 8 https://github.com/GradleUp/shadow?tab=readme-ov-file#compatibility-matrix
|
hab chore/gradle7 gui branch nochmal aktualisiert |
|
@xeruf what is the status here? Should we meet and merge this? Have you already tested whether the gui works with this? |
| includeBuild("gradle/custom-tasks") | ||
|
|
||
| include("sdk", "server", "plugin", "plugin2025", "plugin2026", "player", "test-client") | ||
| // TODO rename to name of game rather than plugin from 2027 |
There was a problem hiding this comment.
What is the plan here?
This might be a good place to document how a new plugin should be added.
I guess that the name plugin2023 and so on must be used in the gui. In the gui build.gradle should hence be some documentation on how to choose a specific plugin.
As I understand this, plugin202X is only called like this, since it is the name used in the gui. Moreover, I remember that the gui just uses the highest number, e.g. plugin2099 would be used if it exists. Is this correct?
Line 12 and onwards maps these plugins to actual projects.
Hence, to add a new project, I create a folder with everything inside call it , add plugin20XX and everything is ready?
supports higher JDK versions, fancy dokka documentation and a lot more improvements including folder structure