RFR: 8343839: Detect patched modules and abort run-time image link early

[Severin Gehwolf]

On Tue, 12 Nov 2024 11:24:07 GMT, Alan Bateman <alanb at openjdk.org> wrote:

> Instead, I think we should look at adding a property such as jdk.patched

You mean a new standard property - observable with `System.getProperty("jdk.patched")` as well - which would also show up with `java -XshowSettings:properties`? Is that the direction you are envisioning?

> src/jdk.jlink/share/classes/jdk/tools/jlink/resources/jlink.properties line 128:
> 
>> 126: \ --keep-packaged-modules is not supported
>> 127: err.runtime.link.modified.file={0} has been modified
>> 128: err.runtime.link.patched.module=The current runtime includes module patches.\
> 
> Not sure about "module patches", its more that the current runtime has been patched with --patch-module.

OK.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/22037#discussion_r1838137581
PR Review Comment: https://git.openjdk.org/jdk/pull/22037#discussion_r1838139017