Skip to content

Finally update to gradle 9#439

Open
xeruf wants to merge 48 commits intomainfrom
chore/gradle7
Open

Finally update to gradle 9#439
xeruf wants to merge 48 commits intomainfrom
chore/gradle7

Conversation

@xeruf
Copy link
Member

@xeruf xeruf commented Feb 21, 2026

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
Copy link
Member

@soerendomroes soerendomroes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

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

Comment on lines 61 to 75
"a player joined" should {
players[0].joinGame(TestPlugin.TEST_PLUGIN_UUID)
"create a room for it" {
await("Room opened") { lobby.games.size shouldBe 1 }
val room = lobby.games.single()
room.clients shouldHaveSize 1
"return GameResult on step" {
withClue("return GameResult on step") {
val roomListener = observeRoom(room.id)
admin.control(room.id).step(true)
val result = roomListener.waitForMessage(GameResult::class)
playerHandlers[0].gameResult shouldBe result
result.win shouldBe Team.ONE
result.isRegular shouldBe false
result.win?.reason.shouldBeInstanceOf<Violation.LEFT>()
result.win?.winner shouldBe Team.TWO
result.isRegular shouldBe true
admin.closed shouldBe false
}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Der Test scheint sich sehr geändert zu haben. Hat sich hier auch das Verhalten geändert?

log/
replays/ No newline at end of file
replays/
plans/
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Was ist in plans?

@soerendomroes
Copy link
Member

@xeruf Gibt es dazu schon einen GUI branch der damit kompatibel ist?

@soerendomroes
Copy link
Member

Der build failed bei mir noch:

FAILURE: Build failed with an exception.

* 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.
    
    Possible solutions:
      1. Declare task ':server:createStartScripts' as an input of ':server:startScripts'.
      2. Declare an explicit dependency on ':server:createStartScripts' from ':server:startScripts' using Task#dependsOn.
      3. Declare an explicit dependency on ':server:createStartScripts' from ':server:startScripts' using Task#mustRunAfter.
    
    For more information, please refer to https://docs.gradle.org/9.3.0/userguide/validation_problems.html#implicit_dependency in the Gradle documentation.
  - 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:copyConfig' without declaring an explicit or implicit dependency. This can lead to incorrect results being produced, depending on what order the tasks are executed.
    
    Possible solutions:
      1. Declare task ':server:copyConfig' as an input of ':server:startScripts'.
      2. Declare an explicit dependency on ':server:copyConfig' from ':server:startScripts' using Task#dependsOn.
      3. Declare an explicit dependency on ':server:copyConfig' from ':server:startScripts' using Task#mustRunAfter.
    
    For more information, please refer to https://docs.gradle.org/9.3.0/userguide/validation_problems.html#implicit_dependency in the Gradle documentation.

* Try:
> Run with --scan to get full insights from a Build Scan (powered by Develocity).

Muss ich außer zu pullen noch etwas einstellen?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

2020 war nicht Blokus dran. Das ist Hive. Kannst du das bitte beim Verschieben mit umbenennen?

return result
}

/** @suppress */
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Was macht hier suppress?

@xeruf
Copy link
Member Author

xeruf commented Feb 27, 2026

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
Copy link
Member

soerendomroes commented Feb 27, 2026

./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
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
Copy link
Member Author

xeruf commented Mar 1, 2026

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
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
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 on lines +17 to +18
os: [ubuntu-latest, windows-latest, macos-latest]
jdk: [11, 24]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi, kannst du bitte die höchste LTS version nehmen (Java 21)?

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