[ https://issues.apache.org/jira/browse/MCOMPILER-321?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16503872#comment-16503872 ]
Hudson commented on MCOMPILER-321: ---------------------------------- Build succeeded in Jenkins: Maven TLP » maven-compiler-plugin » MCOMPILER-321 #4 See https://builds.apache.org/job/maven-box/job/maven-compiler-plugin/job/MCOMPILER-321/4/ > Problematic Java 9 modules are silently ignored > ----------------------------------------------- > > Key: MCOMPILER-321 > URL: https://issues.apache.org/jira/browse/MCOMPILER-321 > Project: Maven Compiler Plugin > Issue Type: Bug > Reporter: Marcel Kolsteren > Priority: Major > Fix For: 3.7.1 > > Attachments: maven-compiler-plugin-bug.zip > > > Before the compilation starts, the compiler plugin prepares the module path > and the class path. Unfortunately, this process is very lenient. If the > plugin cannot find a module that is specified in the module-info file, it > doesn't report this problem, and it just carries on. In the end, the compiler > will give a "module not found", giving no clue about the root cause. > As an example, I attached a simple maven project with a module info file that > refers to an automatic module named {{jython-standalone}}. The dependencies > contain a jar file {{jython-standalone-2.7.0.jar}}. That jar file contains no > module information, and it would normally be resolvable as an automatic > module. However, when building the project, the Java compiler fails with this > error: > {{module not found: jython.standalone}} > The root cause of the error is that the jar file contains a class at top > level, which is illegal for a module. So, the error message "module not > found" is misleading. > I think this is a bug, because the maven-compiler-plugin fails to perform its > task of building a correct module path, but doesn't tell, and just carries > on. Instead, it should stop, and give an informative error message. > Relevant source code: > * [https://github.com/apache/maven-compiler-plugin], class {{CompilerMojo}} > * [https://github.com/codehaus-plexus/plexus-languages], class > {{LocationManager}} and {{MainClassModuleNameExtractor}} > * > [https://docs.oracle.com/javase/9/docs/api/java/lang/module/ModuleFinder.html] > The {{MainClassModuleNameExtractor}} uses the {{ModuleFinder}} class of the > JDK to find modules. I think that these two error situations should lead to > an error and a clear description of possible causes: > * {{ModuleFinder#findAll}} returns an empty set. > * {{ModuleFinder#findAll}} results in a {{FindException}}. When this happens > is well documented in the {{ModuleFinder#of}} method. This exception is > currently swallowed. -- This message was sent by Atlassian JIRA (v7.6.3#76005)