[jdk21u-dev] RFR: 8322809: SystemModulesMap::classNames and moduleNames arrays do not match the order [v2]

Goetz Lindenmaier goetz at openjdk.org
Mon Dec 2 08:10:40 UTC 2024


On Fri, 29 Nov 2024 10:25:25 GMT, Severin Gehwolf <sgehwolf at openjdk.org> wrote:

>> Goetz Lindenmaier has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Remove jdk.jfr from net.foo module-info
>
> test/jdk/tools/jlink/plugins/SystemModuleDescriptors/src/net.foo/module-info.java line 25:
> 
>> 23: 
>> 24: module net.foo {
>> 25:     requires jdk.jfr;
> 
> Not needed if there is no main class in that module.

Hi @jerboaa, thanks for looking at this change. 
I updated the module info.  The test still passes.

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

PR Review Comment: https://git.openjdk.org/jdk21u-dev/pull/1189#discussion_r1865396701


More information about the jdk-updates-dev mailing list