search

mediathekview / MediathekView

Das Programm MediathekView durchsucht die Online-Mediatheken verschiedener Sender
https://mediathekview.de
GNU General Public License v3.0
855 stars 95 forks source link

Switch from AdoptOpenJDK to Adoptium (Eclipse Temurin runtime) #619

Closed reneleonhardt closed 2 years ago

reneleonhardt commented 2 years ago

install4j 9.0.5 introduced support for Adoptium (Eclipse Temurin runtime) and defaults to it.

Dependencies and relevant pipelines upgraded.

reneleonhardt commented 2 years ago

Ist Sonar inzwischen richtig konfiguriert? Kotlin Dateien schlagen weiterhin fehl. [ERROR] Cannot read : You're not authorized to run analysis. Please contact the project administrator.

alex1702 commented 2 years ago

Öhm erklär mir noch schnell wieso der Wechsel sinnvoll ist?

alex1702 commented 2 years ago

Sehe ich das richtig, dass es der Nachfolger von AdoptOpenJDK ist? Wieso müssen die spacken eigentlich immer alles umbenennen, wenn die in anderen Organisationen aufgehen.

reneleonhardt commented 2 years ago

Öhm erklär mir noch schnell wieso der Wechsel sinnvoll ist?

AdoptOpenJDK liefert keine neuen Versionen mehr, das Projekt wurde beendet, die letzte bleibt bei 16.0.1 stehen. Adoptium liefert 16.0.2 und in zwei Wochen dann Java 17.

reneleonhardt commented 2 years ago

Sehe ich das richtig, dass es der Nachfolger von AdoptOpenJDK ist? Wieso müssen die spacken eigentlich immer alles umbenennen, wenn die in anderen Organisationen aufgehen.

Tja, ich vermute mal, dass sie vielleicht aus rechtlichen Gründen nichts mehr vom alten Namen irgendwo referenzieren dürfen. Wahrscheinlich wollen sie mit Temurin auch sicherstellen, dass es nun eine Eclipse Runtime ist, und nicht die Organisation gemeint ist... nun müssen alle Stellen in allen Projekten angepasst werden, hier war es ja noch einfach ;)

reneleonhardt commented 2 years ago

Sieht für mich soweit okay aus. Zukünftig wäre es jedoch cool, library upgrades die sich nicht auf den PR beziehen separat zu machen. Es gibt ein zwei libs die vor einem upgrade umfänglich getestet werden müssen da das Programm zwar compiled, aber danach nicht mehr richtig funktioniert.

Ich kann die Library Upgrades auch rauslösen, hab nicht als Kommentar gesehen, dass die gesondert getestet werden müssen. Kann man dafür automatische Tests schreiben, was funktionierte in der Vergangenheit nicht?

derreisende77 commented 2 years ago

Nee, das kann schon drin bleiben, war nur ein Hinweis für die Zukunft. Ich merge das heute abend rein wenn ich einen rechner habe. Die TilesFX Lib zum Beispiel verhält sich teilweise nach updates nicht mehr richtig. Das sieht man nur wenn das Programm läuft weil z.B. Bandbreitenanzeige nicht mehr richtig aktualisiert wird. Testen an sich automatisiert geht da nicht wirklich.

reneleonhardt commented 2 years ago

Die TilesFX Lib zum Beispiel verhält sich teilweise nach updates nicht mehr richtig.

Achso, das wäre dann aber durch die Library selbst abzutesten, wenn Benutzung/Aufrufe sich nicht ändern, aber die Ausgabe verschwindet, die könnten ja z.B. Screenshots diffen, oder noch besser ihre internen Buffer.