Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 524528 - dev-java/rhino-1.7.7 tests failing / test suite disabled
Summary: dev-java/rhino-1.7.7 tests failing / test suite disabled
Status: CONFIRMED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Java (show other bugs)
Hardware: All Linux
: Normal enhancement (vote)
Assignee: Java team
URL:
Whiteboard:
Keywords:
Depends on: 575590
Blocks:
  Show dependency tree
 
Reported: 2014-10-05 18:27 UTC by Rafał Mużyło
Modified: 2017-08-24 20:12 UTC (History)
2 users (show)

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Rafał Mużyło 2014-10-05 18:27:06 UTC
Can't really tell much about it, but going by bugzilla link in release notes of 1.7.4, thing of note is dependency on dev-java/xml-xmlbeans:2, instead of 1.

With that correction (in deps and src_prepare), the old ebuild finishes successfully - not sure if it really works, but icedtea did finish build (again, not sure if the build was actually correct).
Comment 1 Rafał Mużyło 2014-10-05 18:36:50 UTC
...on a not quite related note: why doesn't slot 0 (1.4) fulfill webstart dep of >icedtea-7.2.4.3, while older versions (in slot 7) do ?
Comment 2 Rafał Mużyło 2014-10-06 22:33:01 UTC
Now, on a related note: probably I should open a new bug for it, but =dev-java/piccolo-1.04, =dev-java/annogen-0.1.0 and =dev-java/xml-xmlbeans-2.6.0 lack x86 keyword.

That might create a few repoman warnings once rhino 1.7.4 is added to the tree (wrt. icedtea again).
Comment 3 OGINO Masanori 2015-06-10 13:17:29 UTC
Version 1.7.5 and 1.7.6 has been released until now.

SRC_URI for 1.7.6 is https://github.com/mozilla/rhino/releases/download/Rhino1_7_6_RELEASE/rhino1.7.6.zip
SRC_URI for 1.7.5 is https://github.com/mozilla/rhino/releases/download/Rhino1_7R5_RELEASE/rhino1_7R5.zip

Note that upstream has changed the versioning scheme so 1.7.6 is not 1.7R6, but (really) 1.7.6.
Comment 4 Patrice Clement gentoo-dev 2015-06-10 13:21:49 UTC
Thanks for the heads up. Don't hesitate to give it a try and knock together an ebuild.
Comment 5 Rafał Mużyło 2015-06-10 16:20:22 UTC
I'm tempted to insert the obvious Frankenstein quite here...

Anyway, again, old ebuild (with noted changes) still works - it seems to work even better than it did before, as while ebuild wasn't dieing, as in 1.7.4 there was a compilation error on old-ex4-compile target...

However, it seems that a further simplification is possible and a minor change is needed:
going by release notes and build.properties, it seems xml-xmlbeans aren't needed anymore and java dep should now be 1.6.

...if the above is true, this will obsolete the mentioned keyword problem.
Comment 6 Patrice Clement gentoo-dev 2015-08-05 14:57:56 UTC
https://github.com/mozilla/rhino/releases/tag/Rhino1_7_7_RELEASE

(In reply to Rafał Mużyło from comment #5)
> However, it seems that a further simplification is possible and a minor
> change is needed:
> going by release notes and build.properties, it seems xml-xmlbeans aren't
> needed anymore and java dep should now be 1.6.

Yes, indeed. Excerpted from build.gradle:
43 dependencies {
44     testCompile "junit:junit:4.12"
45 }

Yet, rhino devs have written Ant build.xml files and left them scattered a bit everywhere in the project directory, which will make bumping to rhino-1.7.7 a cinch:
monsieurp@epsilon /tmp/rhino-Rhino1_7_7_RELEASE $ find . -type f -name \*.xml
./maven/maven-pom.xml
./src/build.xml
./toolsrc/org/mozilla/javascript/tools/debugger/build.xml
./toolsrc/build.xml
./testsrc/build.xml
./xmlimplsrc/build.xml
./build.xml
Comment 7 Patrice Clement gentoo-dev 2015-08-06 10:14:42 UTC
+*rhino-1.7.7 (06 Aug 2015)
+
+  06 Aug 2015; Patrice Clement <monsieurp@gentoo.org>
+  +files/rhino-1.7.7-testsrc-build.xml.patch, +rhino-1.7.7.ebuild:
+  Version bump. Fixes bug 524528.
+

I have disabled the tests cause the test suite generates a lot of StackOverFlow errors and takes a while to complete. I'm not sure it's worth looking into.
Comment 8 Patrice Clement gentoo-dev 2015-08-06 12:15:18 UTC
btw thanks Sebastian Pipping for helping with the bumping :)
Comment 9 Sebastian Pipping gentoo-dev 2015-08-06 21:56:47 UTC
Re-opening as suggested by Patrice.

Reported upstream at https://github.com/mozilla/rhino/issues/225 .