-
Notifications
You must be signed in to change notification settings - Fork 14
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Bundling multiple JREs/JDKs in a single flatpak #9
Comments
IMO the Right™ solution is to contribute to the application to make it work on modern Java so you don't need to ship two JREs. It's honestly pretty easy to port to new versions of Java -- I don't want to make it easier to do the wrong thing. |
I mostly agree with your points @mbooth101, however, I don't think flatpaks have enough marketshare (users vs traditional pkg formats) to seriously 'dictate' the 'correct way', and as such, not supporting this workflow might cause more issues than it solves. In other words, I'd say not supporting this would create more attrition with developers than necessary, and workarounds will not always be possible (consider the strong encapsulation '--illegal-access' switch being removed in the future, for example), so maybe in order for flatpak to succeed this perceived 'step back' might be necessary. |
I'm not suggesting we dictate anything, I just suggesting we be good citizens and participate in the community by contributing code upstream to fix the issue -- this is how open source works.
It is not "not supported" you can do it quite easily with the addition of one
I don't see this as "hacky" at all. In fact having a special feature in the OpenJDK extension is worse because then it has to be there forever or I risk breaking applications that start using it. Maybe I'm misunderstanding why this is a big problem for applications. |
This is a good solution, at least for me and it just works (its what i did basically in my case). |
Yes, it's been long on my todo list to write more blog articles about using the OpenJDK extension :-) I'll leave this bug open as a reminder to document this use-case. |
BTW @AndroidC512L @KhalilSantana if you like you can CC me on upstream bugs where this problem exists, I have a lot of experience of getting old Java projects working on modern Java -- maybe I can offer advice. |
I didn't mean dictate in a coersive way (hence the quotes), I meant in a 'support presure' kind of way, like deprecating a function or API.
I was unaware that the workaround was this simple like this, so maybe it's not a problem at all then. |
Impossible when you work with launchers like Minecraft launchers which need to be able to use older gam versions for modding. |
The SDK extensions' scripts in all versions (8, 11 and latest) point in the building process to the same place (/app/jre and /app/jdk). This is a problem for any flatpak package that needs to bundle two different versions of the JRE/JDK (e.g. minecraft launchers due to latest game version requiring Java 16+). There are some ways to workaround it but are a little hacky. I think there might be a solution from the SDK side rather than future packages having to circumvent this limitation.
The text was updated successfully, but these errors were encountered: