Review request for JDK-8141338: Move jdk.internal.dynalink package to jdk.dynalink

Alan Bateman Alan.Bateman at
Mon Nov 23 15:40:10 UTC 2015

On 23/11/2015 15:27, Attila Szegedi wrote:
> Folks,
> I integrated the changes Mandy suggested; please review these (build related) changes:
> jdk9 top level: <>
> jdk9/jdk: <>
> For the sake of completeness, the jdk/nashorn changes are here: <> but they have already been reviewed by Hannes and Sundar; only the above two (jdk9 and jdk9/jdk) have been modified compared to the original review request.
> Sundar was kind enough to verify that JDK9 builds fine with these changes.
The jdk repo looks okay (just had to change your link to find it :-)

In the top-level repo then you've moved jdk.scripting.nashorn from 
PROVIDER_MODULES to MAIN_MODULES. The reason that we've had it in 
PROVIDER_MODULES is because we treat it as a service provider module (it 
provides an implementation of javax.script.ScriptEngineFactory).


More information about the jigsaw-dev mailing list