search

flathub / com.makemkv.MakeMKV

https://flathub.org/apps/details/com.makemkv.MakeMKV
9 stars 20 forks source link

This disc requires Java runtime (JRE), but none was found. #45

Open SuNNjek opened 1 year ago

SuNNjek commented 1 year ago

When trying to archive a BD+ disc, I got the following warning in the output log:

This disc requires Java runtime (JRE), but none was found. Certain functions will fail, please install Java. See http://www.makemkv.com/bdjava/ for details.

Also, the resulting MKV file seemed to be slightly broken, or at least some segments were saved in the wrong order, causing skips in the movie. When running MakeMKV without flatpak (via the AUR package) so it would use the JRE I have installed on my system, the MKV file was saved correctly.

Would it be possible to include a Java runtime in this flatpak? You can look at how PrismLauncher (a Minecraft launcher, which also requires a JRE) does it here: https://github.com/flathub/org.prismlauncher.PrismLauncher/blob/master/org.prismlauncher.PrismLauncher.yml

SuNNjek commented 1 year ago

Ok, it seems the flatpak does provide a JRE but doesn't seem to use it by default. I can manually set the Java path in the settings to /app/jre/bin/java. Would it be possible to have this set automatically by default or, even better, have the flatpak automatically detect it?

wberrier commented 1 year ago

I have thought something similar. Although, currently we share settings with non-flatpak makemkv settings. So, they may conflict... :thinking:

Perkolator commented 7 months ago

I have a note saved for setting the custom java location for Flatpak MakeMKV:

Protection -> Custom Java executable location:

  1. Find where Java is installed (in terminal): readlink -f $(which java)

    E.g.: /usr/lib/jvm/java-11-openjdk-amd64/bin/java

  2. Click the button beside the location box and navigate to the Java folder and select the executable.

    Because Flatpak doesn't have all read rights to the system, the location box is populated with something like: /run/user/1000/doc/6f392a36/java but it should work just fine.

SchoolGuy commented 7 months ago

I ran into this problem too today. The workaround by @Perkolator works like a charm.