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

Mandy Chung mandy.chung at
Fri Nov 20 03:00:06 UTC 2015

I reviewed the top repo change.

modules.xml looks fine.

jdk.dynalink  should be in MAIN_MODULES since it has exported APIs.  jdk.scripting.nashorn should be moved too.  They are not sole service providers.  Since you are on this file, can you move jdk.scripting.nashorn to MAIN_MODULES as well?


> On Nov 19, 2015, at 3:15 PM, Attila Szegedi <szegedia at> wrote:
> Please review JDK-8141338 "Move jdk.internal.dynalink package to jdk.dynalink" for <>. This is basically the implementation step for integrating JEP 276. This changeset will introduce a new public API that has CCC approval (request 8075866), and is also the implementation step of JEP 276 which is now targeted for 9 and thus can be integrated.
> The changes in this changeset fall into several categories:
> - renaming of jdk.internal.dynalink.* package to jdk.dynalink.* package, with ripple effects in Nashorn classes that import from these packages
> - changes to modules.xml and some build files to accommodate a new public module and a dependency of Nashorn on it
> - new tests
> I’m sending this webrev to several lists with the following rationales:
> - nashorn-dev as the primary users and expected reviewers (also, the Dynalink module code lives in jdk9/nashorn/src/jdk.dynalink). A lot of newly added test code was contributed by Sundar.
> - jigsaw-dev because of modules.xml changes
> - jdk9-dev for build changes (build file changes were graciously contributed by Erik Joelsson and Sundar)
> - core-libs-dev since that’s the designated JEP 276 discussion list.
> Nashorn changes: <> 
> top-level jdk9 changes: <>
> Thanks,
>  Attila.

More information about the jigsaw-dev mailing list