[lworld] RFR: 8377162: [lworld] getResourceAsStream() doesn't work in preview mode for exploded images [v2]
Alan Bateman
alanb at openjdk.org
Wed Feb 18 18:09:20 UTC 2026
On Tue, 17 Feb 2026 11:19:57 GMT, David Beaumont <duke at openjdk.org> wrote:
>> Allow exploded module readers created for exploded system images to support preview mode.
>
> David Beaumont has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains five additional commits since the last revision:
>
> - Merge branch 'lworld' into jdk_8377162_exploded_modules/squashed
> - revert unnecessary filter in ModulePatcher
> - Latest version
> - Limit awareness of preview resources to system modules only.
>
> * limit preview handling
> * Patch ModuleReaderTest from mainline
> - Preview mode for exploded system module readers
>
> * with better flag plumbing
> * Simpler approach with linked hash set
> * Revert weird import change
> * First cut at a preview aware module reader for exploded builds
> * Test for partiy of JRT file-system and class resources in preview mode
src/java.base/share/classes/jdk/internal/module/ModuleReferences.java line 453:
> 451: files.map(f -> Resources.toResourceName(dir, f))
> 452: .filter(s -> !s.isEmpty())
> 453: .filter(s -> previewDir == null || !isPreviewEntry(s))
Okay, so if preview features are not enabled then the META-INF/preview/** resources will appear in the list, just doubling checking.
-------------
PR Review Comment: https://git.openjdk.org/valhalla/pull/2032#discussion_r2823733005
More information about the valhalla-dev
mailing list