On Sat, 9 Nov 2024 02:18:22 GMT, jyxzwd <d...@openjdk.org> wrote:
> Use the built-in file system provider rather than the custom file system
> provider.
> Add "public static FileSystemProvider create" method in
> DefaultFileSystemProvider which is from java8API to be compatible against
> runtime.
src/java.base/share/classes/jdk/internal/jimage/ImageReaderFactory.java line 51:
> 49: private static final String JAVA_HOME =
> System.getProperty("java.home");
> 50: private static final Path BOOT_MODULES_JIMAGE =
> 51:
> sun.nio.fs.DefaultFileSystemProvider.create().getPath(JAVA_HOME, "lib",
> "modules");
This is JDK internal so if the classes in jrt-fs.jar are loaded by a custom
class loader, in for example JDK 17 or 21, then this will fail.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/21997#discussion_r1835320287
