> Please review this changeset for adding support for `--module` (-m) option 
> for CDS.
> Changes in the `ModuleBootstrap.java` are needed so that the 
> `ArchivedModuleGraph.archive` and `ArchivedBootLayer.archive` are called if 
> the main module is specified. The module name will be stored in the ro region 
> of the CDS archive. During runtime, the archived module name will be compared 
> with the runtime module name. If comparison fails, the archived full module 
> graph won't be used.
> 
> Note: this RFE is a subtask of 
> [JDK-8266329](https://bugs.openjdk.org/browse/JDK-8266329). More subtask(s) 
> will be created to support other options such as `--add-modules`.
> 
> Passed tiers 1 - 4 testing.

Calvin Cheung has updated the pull request incrementally with one additional 
commit since the last revision:

  skip archiving full module graph is there is an incubator module

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/16016/files
  - new: https://git.openjdk.org/jdk/pull/16016/files/6c82fde0..f6b2c3d5

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=16016&range=04
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=16016&range=03-04

  Stats: 81 lines in 5 files changed: 59 ins; 1 del; 21 mod
  Patch: https://git.openjdk.org/jdk/pull/16016.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/16016/head:pull/16016

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

Reply via email to