Skip to content
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

Java 11 #450

Closed
mstormi opened this issue Oct 3, 2018 · 5 comments
Closed

Java 11 #450

mstormi opened this issue Oct 3, 2018 · 5 comments
Labels
base system Related to the base system help wanted Extra attention is needed

Comments

@mstormi
Copy link
Contributor

mstormi commented Oct 3, 2018

openHAB is to move to Java 11 soon, see this for current status.

We need to offer this to at least be an install option.

@EliasGabrielsson EliasGabrielsson added the base system Related to the base system label Oct 8, 2018
@EliasGabrielsson EliasGabrielsson added this to the Image v1.6 milestone Oct 8, 2018
@EliasGabrielsson
Copy link
Contributor

Notice license change from oracle when doing this.

@EliasGabrielsson EliasGabrielsson added the help wanted Extra attention is needed label Oct 8, 2018
@kaikreuzer
Copy link
Member

openHAB is to move to Java 11 soon

Not at all "soon" - we are just evaluating and trying to fix issues, so that IF someone uses Java 11 by accident can still use it. The recommendation will be Java 8 for the foreseeable future.

For openHABian, we use Zulu Embedded JVM, so Oracle licensing does not impact us.
There are no Zulu Embedded builds of Java 11 available, so this is another reason to stick to Java 8.

I'd suggest to only open an issue here once there is a decision to upgrade from 8 to anything newer.

@mstormi
Copy link
Contributor Author

mstormi commented Oct 31, 2018

I disagree a little here and I think you shouldn't close it as fast as you did.
First, the issue is meant to provide Java 11 just as an install option (not default) and I think it makes sense to have that available in openHABian before OH moves to Java 11 so people can start testing early.

Also remind you openHABian runs (and is supposed to run) on non-ARM hardware as well, and there's Java 11 Zulu on x86 already available.
Also, until now Oracle is a supported alternative on both, ARM and non-ARM, so either we decide to discontinue support for Oracle or we mustn't ignore licensing issues.

@kaikreuzer
Copy link
Member

This issue made the impression as if there can/should be something done in openHABian. I don't agree that we should add features where we know that it is NOT working, just for the sake of testing.

Also remind you openHABian runs (and is supposed to run) on non-ARM hardware as well

Ok, good point - I was so far only aware that we package Zulu embedded in the images (and I cannot see anything in the features that mentions x86 JVMs as an install option).

so either we decide to discontinue support for Oracle

Not sure who and why this has ever been added, but I'd be all for removing it indeed.

@ThomDietrich
Copy link
Member

Hey guys!

so either we decide to discontinue support for Oracle

Not sure who and why this has ever been added, but I'd be all for removing it indeed.

The routine was working when we decided to switch over to Zulu. At this point I decided to offer the Oracle solution as an alternative, in case Zulu created issues for some. Never removed it since.
@mstormi feel free to remove the menu entry.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
base system Related to the base system help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

4 participants