Mailing-List: contact dev-help@groovy.incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@groovy.incubator.apache.org
MIME-Version: 1.0
In-Reply-To: 
 <CAByu6UXQinEYkMtn83TpE9GcOkVWzBXXjzEVZY08vtnEzOCX3Q@mail.gmail.com>
References: <55E3E83C.1010409@gmx.net>
	<CAByu6UXQinEYkMtn83TpE9GcOkVWzBXXjzEVZY08vtnEzOCX3Q@mail.gmail.com>
Date: Tue, 1 Sep 2015 12:36:25 +0200
Message-ID: 
 <CAByu6UU47RdvAb3URqNfrYf4ksMiHgEckfR-vXdy4WdgRgmVQw@mail.gmail.com>
Subject: Re: Failing Groovysh Test with JDK 9 SNAPSHOT on Buildserver
From: Thibault Kruse <tibokruse@googlemail.com>
To: dev@groovy.incubator.apache.org
Content-Type: text/plain; charset=UTF-8

See https://github.com/apache/incubator-groovy/pull/107 for a possible fix

On Tue, Sep 1, 2015 at 11:13 AM, Thibault Kruse
<tibokruse@googlemail.com> 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