On Mon, 1 Dec 2025 06:08:00 GMT, Jaikiran Pai <[email protected]> wrote:

> Can I please get a review of this change which proposes to address the issue 
> noted in https://bugs.openjdk.org/browse/JDK-8372787?
> 
> The commit in this PR takes into account the `IllegalStateException` thrown 
> by `JarFile` APIs and wraps them into a `IOException` to conform with the 
> expectations of the `ModuleReader` APIs.
> 
> A new jtreg test has been introduced to reproduce the issue and verify the 
> fix. CI testing is currently in progress with this change.

@jaikiran Would it be possible to update the bug with a summary as to how this 
arise when using --patch-module? The proposed test is using the internal 
ModulePatcher directly and I think the starting point for discussion is an 
outline as to how anything using --patch-module runs into this.

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

PR Comment: https://git.openjdk.org/jdk/pull/28569#issuecomment-3594856116

Reply via email to