RFR: 8294972: Convert jdk.jlink internal plugins to use the Classfile API [v8]

Mandy Chung mchung at openjdk.org
Wed Mar 15 22:33:30 UTC 2023


On Wed, 15 Mar 2023 16:42:53 GMT, Adam Sotona <asotona at openjdk.org> wrote:

>> jdk.jlink internal plugins are heavily using ASM
>> 
>> This patch converts ASM calls to Classfile API.
>> 
>> Please review.
>> Thanks,
>> Adam
>
> Adam Sotona has updated the pull request incrementally with one additional commit since the last revision:
> 
>   fixed SystemModulesPlugin

These convenient methods really simplify it and improve the readability.    This patch seems to fix `BasicTest.java` test failure I mentioned to you offline.   What was the issue with the previous revision?


$ JTwork/scratch/mysmallimage/bin/java -Xlog:init=debug -m test/jdk.test.Test                                 
Error occurred during initialization of boot layer
java.lang.InternalError: java.lang.ClassNotFoundException: jdk/internal/module/SystemModules$all
	at java.base/jdk.internal.module.SystemModuleFinders.systemModules(SystemModuleFinders.java:132)
	at java.base/jdk.internal.module.ModuleBootstrap.boot2(ModuleBootstrap.java:228)
	at java.base/jdk.internal.module.ModuleBootstrap.boot(ModuleBootstrap.java:174)
	at java.base/java.lang.System.initPhase2(System.java:2222)
Caused by: java.lang.ClassNotFoundException: jdk/internal/module/SystemModules$all
	at java.base/java.lang.Class.forName0(Native Method)
	at java.base/java.lang.Class.forName(Class.java:391)
	at java.base/java.lang.Class.forName(Class.java:382)
	at java.base/jdk.internal.module.SystemModuleFinders.systemModules(SystemModuleFinders.java:129)
	... 3 more

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

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


More information about the core-libs-dev mailing list