RFR: 8258411: Move module set configuration from Modules.gmk to conf dir

Alan Bateman alanb at openjdk.java.net
Tue Dec 15 16:56:57 UTC 2020


On Tue, 15 Dec 2020 16:11:45 GMT, Magnus Ihse Bursie <ihse at openjdk.org> wrote:

> The hard-coded list of modules in `make/common/Modules.gmk` has always been a wart in the build system. We pride ourself on using discovery instead of hard-coded list. In this case, it is not possible to do do auto-discovery, since the different module sets are configured, not determined.
> 
> Thus the actual lists of module sets should move to the `make/conf` directory.
> 
> This is the first patch in a series where I will move configuration values spread over the build system into the designated `make/conf` directory.

I really dislike patch as it mixes up several things in module-sets.conf. If you really need to move configuration out of Modules.gmk (and I see no reason to do this) then please look at separating out the static mapping of modules to class loaders, the modules used for the interim builds, and the modules used to create API docs.

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

Changes requested by alanb (Reviewer).

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



More information about the build-dev mailing list