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

Packaging: Add java7/8 java-package paths to init script #8815

Conversation

@spinscale
Copy link
Member

commented Dec 8, 2014

If you use the java-package tool to create java packages, those
paths also should be added to the debian init script.

Also updated the docs, that it is ok to install java8.

Closes #7383

@rjernst

This comment has been minimized.

Copy link
Member

commented Dec 8, 2014

This looks fine, but should java 8 be added at the very end of the list?

@spinscale

This comment has been minimized.

Copy link
Member Author

commented Dec 8, 2014

thats actually a good question. Do we have a precedence on java7 vs. java8?

/cc @clintongormley

@clintongormley

This comment has been minimized.

Copy link
Member

commented Dec 11, 2014

I'd say that we prefer Java 8 over Java 7, but perhaps @rmuir can give more informed advice?

@dakrone

This comment has been minimized.

Copy link
Member

commented Dec 11, 2014

Since we switched to the invoke dynamic version of the Groovy jar, it looks like scripting is faster on JDK 8 (just commenting)

@rmuir

This comment has been minimized.

Copy link
Contributor

commented Dec 11, 2014

+1 for java8 first. it has less probability of being broken.

Packaging: Add java7/8 java-package paths to debian init script
If you use the java-package tool to create java packages, those
paths also should be added to the debian init script.

Also updated the docs, that it is ok to install java8.

Closes #7383

@spinscale spinscale force-pushed the spinscale:fix/packaging-debian-add-oracle-java7-java8-path branch to 544ef8c Dec 11, 2014

@spinscale spinscale merged commit 544ef8c into elastic:master Dec 11, 2014

@clintongormley clintongormley added >enhancement and removed review labels Mar 19, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants
You can’t perform that action at this time.