search

mediathekview / MediathekView

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

Fehlende JVM Parameter erkannt #719

Closed toby63 closed 10 months ago

toby63 commented 10 months ago

Beim Start von Mediathekview erscheint folgende Fehlermeldung:

Bitte stellen Sie sicher dass folgende Paramter übergeben werden:

- -XX:+UseShenandoahGC
- -XX:ShenandoagGCHeuristics=compact
- -XX:+UseStringDeduplication
- -XX:MaxRAMPercentage=XX.X
- --add-opens=java.desktop/sun.awt.X11=ALL-UNNAMED

-Xmx sollte nicht mehr genutzt werden!

Version: 14.0.0. (kompiliert) OS: Manjaro Linux

dadidadit commented 10 months ago

Bitte stellen Sie sicher dass folgende Paramter übergeben werden: < .....

Und, hast du sichergestellt, dass die entsprechenden Parameter auch vorhanden sind bzw. die Xmx..-Parameter entfernt wurden?

In den fertigen angebotenen Mediathekview-Paketen für Linux tritt dieser Fehler sicher nicht auf, denn darin sind die erforderlichen Parameter und die benötigte Java-Version 20 enthalten.

https://mediathekview.de/download/

toby63 commented 10 months ago

@dadidadit Danke für den Hinweis.

Da bisher garkeine Parameter in den jeweiligen Start-Dateien übergeben wurden, ging ich davon aus, dass das irgenteine Code-Sache oder sogar einfach ein Bug ist. Allerdings habe ich tatsächlich im offiziellen Appimage entsprechende Passagen (https://github.com/mediathekview/MediathekView/blob/master/scripte/appimage.sh#L88) gefunden.

Dann werde ich das wohl anpassen müssen.

derreisende77 commented 10 months ago

Nein, ist kein bug sondern ein Feature, was auch im CHANGELOG.md in 4. Bullet inkl. workaround dokumentiert ist. Die Parameter sind wichtig und sollten auch genutzt werden wenn man es selbst baut. Der Dialog ist nur eine Warnung, kein Fehler.

toby63 commented 10 months ago

@derreisende77 Nunja der 4. Punkt im Changelog sagt nichts über die konkreten Parameter und dessen Gründe usw. Es wäre für Maintainer usw. sicher sinnvoll sowas etwas detailierter in Changelogs einzufügen.

derreisende77 commented 10 months ago

Nun das sehe ich etwas anders. Der normale Nutzer wird nicht durch den Dialog tangiert, die Startparameter sind für jede Plattform in den install4j Dateien hinterlegt. Das CHANGELOG dient auch primär der Information der „normalen“ Nutzer. Letztendlich trifft es nur Wagemutige die die Software selbst bauen wollen/müssen. Da diese i.d.R. Zugriff auf den kompletten Quellcode haben können diese in den .install4j Ordnern auch die aktuell zu nutzenden Parameter je nach Plattform einsehen. Und da wir/ich die Maintainer der Pakete sind werden Aktualisierungen der Parameter ja auch hier im git dokumentiert.