Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 558310 - app-admin/collectd - Use the Java eclasses
Summary: app-admin/collectd - Use the Java eclasses
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Johan Bergström
URL:
Whiteboard:
Keywords: PATCH
Depends on:
Blocks: 549508
  Show dependency tree
 
Reported: 2015-08-20 21:00 UTC by James Le Cuirot
Modified: 2015-09-03 09:01 UTC (History)
2 users (show)

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


Attachments
Patch against collectd-5.4.1-r1.ebuild (collectd-java.patch,2.63 KB, patch)
2015-08-20 21:00 UTC, James Le Cuirot
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description James Le Cuirot gentoo-dev 2015-08-20 21:00:57 UTC
Created attachment 409702 [details, diff]
Patch against collectd-5.4.1-r1.ebuild

Java team needs to remove java-config-wrapper, which this erroneously depends on. The existing ebuild does not make use of the Java eclasses, which can lead to problems. I have:

 * Made use of java-pkg-opt-2.
 * Added a DEPEND on virtual/jdk because javac, jar and JNI headers are required.
 * Used sed against the Java Makefile.am to set proper -source and -target flags.
 * Used the eclasses to properly set JAVA_HOME instead of calling java-config.
 * Registered the installed jars so that Gentoo metadata is created, allowing other packages to locate them and use the correct Java version against them.

If no one complains within 2 weeks, I'll commit these changes myself.
Comment 1 James Le Cuirot gentoo-dev 2015-08-20 22:01:48 UTC
Forgot to mention that the older ebuilds should probably be removed in the process as they are unused.
Comment 2 James Le Cuirot gentoo-dev 2015-09-03 09:01:19 UTC
Time up! Committed.