Integrated: 8338747: hasIncubatorModules needs to be generated when module resolution required at startup

Alan Bateman alanb at openjdk.org
Thu Sep 12 08:51:09 UTC 2024


On Mon, 2 Sep 2024 12:27:21 GMT, Alan Bateman <alanb at openjdk.org> wrote:

> This PR proposes changes to the ModuleBootstrap code that is used with CDS (Ahead-of-Time Class Loading & Linking in the future). At things stand, the module graph and boot layer can be archived at dump time (-Xshare:dump) when the initial module is the class path or the initial module is in the run-time image. Future work will extend this to deployments with an application module path or where additional root modules are specified with --add-modules. To get there we need the code that determines whether to archive is in one place, and it needs to know if the module graph contains incubator modules or split packages. 
> 
> Testing: tier1-tier6. There are no new tests, the changes don't expand or change what can be archived. Calvin has done some some testing with the change as it is needed for [JDK-8328313](https://bugs.openjdk.org/browse/JDK-8328313).

This pull request has now been integrated.

Changeset: 1b17e0b1
Author:    Alan Bateman <alanb at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/1b17e0b133cab44029333c832bd046b338ede581
Stats:     72 lines in 1 file changed: 31 ins; 21 del; 20 mod

8338747: hasIncubatorModules needs to be generated when module resolution required at startup

Reviewed-by: iklam, ccheung

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

PR: https://git.openjdk.org/jdk/pull/20818


More information about the core-libs-dev mailing list