IllegalAccessError with --patch-module

[Alan Bateman]

On 06/01/2021 23:49, Johannes Kuhn wrote:
> Indeed a bug.
>
> Happens under the following conditions:
> * Patched module is an automatic module
> * Patch adds one or more additional packages to the module.
>
> :
>
> Proposed fix:
> * Change requires to not check if it is an automatic module if the 
> builder is not strict.
> * Move the copying of the requires out of the if.

Thanks for the mails, it is bug. The additional information that the 
patch adds packages to the automatic module was important to understand 
the mails. One other thing is that the bug does not surface when the set 
of root modules includes an explicit module or an unpatched automatic 
module. So very obscure scenario that probably explains why there hasn't 
been reported before now.

The patcher should be using strict mode when re-building the module 
descriptor for a patched automatic modules. The non-strict mode is for 
explicit modules as the identifiers (that may not be valid in the Java 
Language) are read from the module-info.class. I'll create a bug for this.

-Alan.