On Wed, 17 Dec 2025 16:55:15 GMT, David Beaumont <[email protected]> wrote:

> Removing code guarded by now obsolete preview patching property.
> This should be a no-op functionally.
> Some minor tidying around arguments.cpp since now there's no reason to init 
> the jimage file in the module patching code.

test/jdk/tools/jlink/runtimeImage/PackagedModulesVsRuntimeImageLinkTest.java 
line 58:

> 56:  * @build tests.* jdk.test.lib.process.OutputAnalyzer
> 57:  *        jdk.test.lib.process.ProcessTools
> 58:  * @run main/othervm/timeout=1200 -ea -esa 
> -DDISABLE_PREVIEW_PATCHING=false -Xmx1g PackagedModulesVsRuntimeImageLinkTest

Not 100% sure why this was explicitly leaving patching on, but it passes 
locally.

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

PR Review Comment: 
https://git.openjdk.org/valhalla/pull/1819#discussion_r2627919939

Reply via email to