It fails for both a console and a groovysh test. There is some output for these tests earlier in the report saying: [:groovy-groovysh:test] *** Error in `/opt/jdk9-build/j2sdk-image/bin/java': double free or corruption (fasttop): 0x00007fe038b0d300 *** It might be good to bisect this to the change causing this if possible.
On Sun, Sep 6, 2015 at 9:16 AM, Pascal Schumacher <[email protected]> wrote: > GroovyShell Test all work now :), but now gradle exits with an non-zero exit > code when running groovy-console tests with indy: > http://ci.groovy-lang.org/viewLog.html?buildId=26577&buildTypeId=Groovy_Jdk9Build&tab=buildLog#_focus=85490&state=85490 > > Anybody has an idea why? > > -Pascal > > > Am 01.09.2015 um 19:19 schrieb Pascal Schumacher: >> >> Merged. Thanks a lot. :) >> >> - Pascal >> >> Am 01.09.2015 um 12:36 schrieb Thibault Kruse: >>> >>> See https://github.com/apache/incubator-groovy/pull/107 for a possible >>> fix >>> >>> On Tue, Sep 1, 2015 at 11:13 AM, Thibault Kruse >>> <[email protected]> wrote: >>>> >>>> The tests makes sure that completing java.util. includes 'Set', as in >>>> java.util.Set. I'll assume java9 does not move the Set class. I guess >>>> there is a change affecting method >>>> PackageHelperImpl.getClassnames(), which was originally copied from >>>> jline1. >>>> >>>> I see it has adaptations for jigsaw made by Cedric: >>>> https://github.com/apache/incubator-groovy/commit/0e384ec3 >>>> (not pointing fingers, just analyzing the code) >>>> >>>> And the breaking test follows that new codepath. >>>> >>>> Debugging a bit, I notice that the files contained in 'jrt:/' does not >>>> contains only "/modules/java.base/java/util/Set.class". At a glance, >>>> it seems the assumptions in >>>> PackageHelperImpl.getPackagesAndClassesFromJigsaw() do not hold with >>>> the current Java9 implementation, since it produces a Class >>>> java.base.java.util.Set, but not java.util.Set. >>>> >>>> A quick fix is to change: >>>> -if (elems) { >>>> - elems = elems[3..<elems.length] >>>> +if (elems && elems.length > 2) { >>>> + elems = elems[3..<elems.length] >>>> >>>> in *two* places in that method. >>>> >>>> I have no idea whether there is a standard for the folder layout >>>> FileSystems.newFileSystem(URI.create("jrt:/")) should return, or >>>> whether and why this has changed since Cedric made his additions. >>>> >>>> However this might be related: >>>> >>>> http://mail.openjdk.java.net/pipermail/jigsaw-dev/2014-November/004044.html >> >> >
