RFR: 8353185: Introduce the concept of upgradeable files in context of JEP 493 [v4]

Severin Gehwolf sgehwolf at openjdk.org
Tue Apr 8 13:34:22 UTC 2025


On Mon, 7 Apr 2025 17:30:53 GMT, Alan Bateman <alanb at openjdk.org> wrote:

>> Severin Gehwolf has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Review v2
>
> src/jdk.jlink/share/classes/jdk/tools/jlink/internal/LinkableRuntimeImage.java line 71:
> 
>> 69:     private static InputStream getDiffInputStream(String module) throws IOException {
>> 70:         String resourceName = String.format(DIFF_PATTERN, module);
>> 71:         return JDK_JLINK_MOD.getResourceAsStream(resourceName);
> 
> FYI you can use LinkableRuntimeImage.class.getResourceAsStream here as the resource is in the current module.

It's very odd, but when I attempt this then the resource is not found. It seems to fail on the module name verification. For example: `jlink --help | tail -n2` shows as `disabled` for an enabled linkable runtime image.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/24388#discussion_r2033202346


More information about the build-dev mailing list