Created attachment 341442 [details, diff] jfreechart-1.0.14.diff against 1.0.13 Please bump, release was back in 21 Nov 2011. Modified build.xml patch, small EAPI=4 cleanup.
Created attachment 341444 [details, diff] jfreechart-1.0.14-build.xml.patch required build.xml patch, smaller than the 1.0.13 patch.
Depends on >=dev-java/jcommon-1.0.17, also works with jcommon-1.0.18 which contains a bug fix. Tested with icedtea-7.
Created attachment 341522 [details, diff] jfreechart-1.0.14.diff against 1.0.13 (v2) gained EAPI=4 knowledge from the jcommon bump request. :)
Created attachment 341532 [details, diff] jfreechart-1.0.14.diff against 1.0.13 (v3) some more modifications. less dependencies! - introducing 'servlet' IUSE for I have no use for those few classes that require servlet-api:2.3 - itext-2.1.5 seems more appropriate a dependency (same as bundled version) - junit-4 based on README.txt ("The JUnit tests included with JFreeChart have been created using JUnit 4.3.1.") Tested successfully with icedtea-7.
Created attachment 341538 [details, diff] jfreechart-1.0.14.diff against 1.0.13 (v3) correcting that junit oops: not sure what's the correct test dependency, leaving it as-is.
Well, one test fails: [junit] FAILED [junit] expected:<1.199142E12> but was:<1.1991456E12> [junit] junit.framework.AssertionFailedError: expected:<1.199142E12> but was:<1.1991456E12> [junit] at org.jfree.data.time.junit.TimeSeriesCollectionTests.testFindDomainBounds(TimeSeriesCollectionTests.java:360) [junit] which is... 1 hour? Not sure what to do there.
Created attachment 348798 [details, diff] jfreechart-1.0.14-fix-TimeSeriesCollectionTests.patch Patch for test failure
Created attachment 348800 [details, diff] jfreechart-1.0.14.diff against 1.0.13 (v4) ebuild update - now passes all tests successfully and builds, using icedtea-7.2.3.8
<flag name='servlet'>Build a servlet using the servlet API.</flag> I'm not using this package, did I get this USE flag description right? * Possibly unneeded dependencies found * itext Any comment on this? Though it seems they still bring itext along so this feels like a false positive. + 22 May 2013; Tom Wijsman <TomWij@gentoo.org> + +files/jfreechart-1.0.14-build.xml.patch, + +files/jfreechart-1.0.14-fix-TimeSeriesCollectionTests.patch, + +jfreechart-1.0.14.ebuild, metadata.xml: + Version bump to 1.0.14, contributed by Andreas Sturmlechner (genstorm). Fixes + bug #460952.
From the package.html inside the servlet directory: "Classes for providing useful servlet and JSP functionality." When I bumped the itext dependency I just edited what was there from 1.0.13, but in fact there is no reference in the source, only the Readme mentions "the demo application uses itext", which we don't install anyway. So, as long as we don't add an examples flag to install the demo binaries (no sources shipped), I guess we can drop the dep.
Created attachment 348950 [details, diff] jfreechart-1.0.14-r1.ebuild.diff I gave it a try: This installs the demo to /usr/share/jfreechart-1.0/lib/jfreechart-demo.jar when USE="examples" is set.
Created attachment 348952 [details, diff] jfreechart-1.0.14-r1.ebuild.diff actually, make that better. (test-built with icedtea-7)
Closing this one in favor of bug 528402 (1.0.19 version bump).
