Java Platform Module System (JPMS) compatibility #70

New Issue

open-autonomous-connection · 2020-06-06T00:40:12Z

micheljung commented

2020-06-06 00:40:12 +00:00

(Migrated from github.com)

JPMS requires that all classes are within a package. This is not the case for:

lua.java
luac.java
luajc.java

Which prevents luaj from being used as a module:

 Caused by: java.lang.module.InvalidModuleDescriptorException: luajc.class found in top-level directory (unnamed package not allowed in module)
	at java.base/jdk.internal.module.ModulePath.toPackageName(ModulePath.java:718)
	at java.base/java.util.stream.ReferencePipeline$3$1.accept(ReferencePipeline.java:195)
	at java.base/java.util.HashMap$KeySpliterator.forEachRemaining(HashMap.java:1603)
	at java.base/java.util.stream.AbstractPipeline.copyInto(AbstractPipeline.java:484)
	at java.base/java.util.stream.AbstractPipeline.wrapAndCopyInto(AbstractPipeline.java:474)
	at java.base/java.util.stream.ReduceOps$ReduceOp.evaluateSequential(ReduceOps.java:913)
	at java.base/java.util.stream.AbstractPipeline.evaluate(AbstractPipeline.java:234)
	at java.base/java.util.stream.ReferencePipeline.collect(ReferencePipeline.java:578)
	at java.base/jdk.internal.module.ModulePath.deriveModuleDescriptor(ModulePath.java:529)
	at java.base/jdk.internal.module.ModulePath.readJar(ModulePath.java:642)
	... 11 more

Tested with AdoptOpenJDK 11.0.7 and luaj 3.0.1

JPMS requires that all classes are within a package. This is not the case for: - lua.java - luac.java - luajc.java Which prevents luaj from being used as a module: ``` Caused by: java.lang.module.InvalidModuleDescriptorException: luajc.class found in top-level directory (unnamed package not allowed in module) at java.base/jdk.internal.module.ModulePath.toPackageName(ModulePath.java:718) at java.base/java.util.stream.ReferencePipeline$3$1.accept(ReferencePipeline.java:195) at java.base/java.util.HashMap$KeySpliterator.forEachRemaining(HashMap.java:1603) at java.base/java.util.stream.AbstractPipeline.copyInto(AbstractPipeline.java:484) at java.base/java.util.stream.AbstractPipeline.wrapAndCopyInto(AbstractPipeline.java:474) at java.base/java.util.stream.ReduceOps$ReduceOp.evaluateSequential(ReduceOps.java:913) at java.base/java.util.stream.AbstractPipeline.evaluate(AbstractPipeline.java:234) at java.base/java.util.stream.ReferencePipeline.collect(ReferencePipeline.java:578) at java.base/jdk.internal.module.ModulePath.deriveModuleDescriptor(ModulePath.java:529) at java.base/jdk.internal.module.ModulePath.readJar(ModulePath.java:642) ... 11 more ``` Tested with AdoptOpenJDK 11.0.7 and luaj 3.0.1

👍 1

open-autonomous-connection commented

2026-03-02 13:27:06 +00:00

Fixed

open-autonomous-connection closed this issue

2026-03-02 13:27:06 +00:00

open-autonomous-connection referenced this issue from a commit

2026-03-02 13:27:24 +00:00

Fixed issue: #70

Sign in to join this conversation.