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