On Wed, 16 Dec 2020 14:40:14 GMT, Alan Bateman <al...@openjdk.org> wrote:

> javadoc-modules.conf is probably okay although someone finding this in the 
> repo might initially think it's the configuration for the javadoc modules. 
> That plus it sets DOCS_MODULES, so maybe it should be apidocs-modules.conf.

I'm okay with apidocs-modules.conf or docs-modules.conf.   It's good to match 
the make target name which makes it easier to understand what this 
configuration is for.

> module-loader-map.conf works as the configuration file that defines 
> BOOT_MODULES and PLATFORM_MODULES. I think AGGREGATOR_MODULES should be 
> dropped and "java.se" added to PLATFORM_MODULES. If I remember correctly, 
> this was separated out in JDK 9 and 10 because of the java.se.ee aggregator 
> module (that one was removed in Java 11 by JEP 320).

module-loader-map.conf works for me too.   We can drop AGGREGATOR_MODULES  
since java.se is the sole aggregator module in JDK now.

-------------

PR: https://git.openjdk.java.net/jdk/pull/1781

Reply via email to