Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 551880 - app-misc/elasticsearch - add jre:1.8 to RDEPEND
Summary: app-misc/elasticsearch - add jre:1.8 to RDEPEND
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Unspecified (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Ferenc Erki
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-06-12 11:16 UTC by Tomáš Mózes
Modified: 2015-06-30 04:04 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
elasticsearch-1.6.0.ebuild (elasticsearch-1.6.0.ebuild,1.69 KB, text/plain)
2015-06-14 08:46 UTC, Ferenc Erki
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Tomáš Mózes 2015-06-12 11:16:30 UTC
The documentation is a bit inconsistent on this:
https://www.elastic.co/guide/en/elasticsearch/reference/current/_installation.html

"Elasticsearch requires Java 7. Specifically as of this writing, it is recommended that you use the Oracle JDK version 1.8.0_25."

I've tested with JRE 8 and it works fine.
Comment 1 Ferenc Erki 2015-06-14 08:46:29 UTC
Created attachment 405122 [details]
elasticsearch-1.6.0.ebuild

Accept virtual/jre:1.8 as RDEPEND. Let me know if this also requires a revision bump.
Comment 2 Ferenc Erki 2015-06-14 08:47:31 UTC
It looks like upstream uses slightly different wording in various places, e.g. this section of the setup docs has a more detailed description of the Java requirements: https://www.elastic.co/guide/en/elasticsearch/reference/current/setup.html#jvm-version

So I also opened a GitHub pull request to help upstream clarifying the docs: https://github.com/elastic/elasticsearch/pull/11649
Comment 3 Guillaume Ceccarelli 2015-06-28 19:01:28 UTC
Any update on pushing this change into the main portage tree?

I'd like to add that Oracle JDK7 has been EOLed, so it doesn't seem to make much sense to constrain ourselves to a 1.7 JRE. Upstream has to (and does) support Java8.
Comment 4 Tony Vroon (RETIRED) gentoo-dev 2015-06-29 14:29:31 UTC
(In reply to Ferenc Erki from comment #1)
> Let me know if this also requires a revision bump.

I would only bump the revision if what is installed on disk changes. For this RDEPEND variable change, it will not. Could you please submit a unified diff in future as it makes it easier for me to see what has changed?

+  29 Jun 2015; Tony Vroon <chainsaw@gentoo.org> elasticsearch-1.6.0.ebuild:
+  Also accept version 8 Java Runtime Environment, as per Ferenc Erki in bug
+  #551880.
Comment 5 Ferenc Erki 2015-06-29 14:51:13 UTC
(In reply to Tony Vroon from comment #4)
> Could you please submit a unified diff in future as it makes it easier for me to see what has changed?

Sure, no problem. Thanks for the feedback!
Comment 6 Tomáš Mózes 2015-06-30 04:04:01 UTC
Thanks. Maybe later we can prioritize jre8 over jre7.