On Thu, 21 Dec 2023 19:10:59 GMT, Calvin Cheung <cche...@openjdk.org> wrote:

> Please review this change for enabling full module graph even when 
> -Xbootclasspath/a is specified. The validation of -Xbootclasspath/a is 
> already being done in `FileMapInfo::validate_boot_class_paths()`. The full 
> module graph will be disabled if there's a mismatch in -Xbootclasspath/a 
> between dump time and runtime.
> 
> The changes in ClassLoaders.java is for setting up the append class path for 
> the boot loader retrieved from the CDS archive. It is required because some 
> existing tests are using the `getResourceAsStream()` api. Those tests would 
> fail without the change.
> 
> Passed tiers 1 - 4 testing.

This pull request has now been integrated.

Changeset: 3b1e56a4
Author:    Calvin Cheung <cche...@openjdk.org>
URL:       
https://git.openjdk.org/jdk/commit/3b1e56a4275addeadcefe180b5ce60d9d74cca7b
Stats:     64 lines in 4 files changed: 50 ins; 11 del; 3 mod

8322322: Support archived full module graph when -Xbootclasspath/a is used

Reviewed-by: alanb, mchung

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

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

Reply via email to