[jira] [Comment Edited] (MCOMPILER-359) plexus-java 0.9.10 causes a NullPointer in compiler-plugin 3.8.0

["Marvin (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/MCOMPILER-359?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16599830#comment-16599830 ] 

Marvin edited comment on MCOMPILER-359 at 9/1/18 11:01 PM:
-----------------------------------------------------------

See attached, The build succeeds if you remove module-info.java. The opinion from plexus dev is that they are not responsible for handling anything other than jar files and it is the responsibility of the compiler plugin to send only appropriate files or to manage the exceptions, I responded that they should not be throwing a zip-related error if the file is not a zip.


was (Author: wolfthal):
See attached, The build succeeds if you remove module-info.java. The opinion from plexus dev is that they are not responsible for handling anything other than jar files and it the the responsibility of the compiler plugin to send only appropriate files or to manage the exceptions, I responded that they should not be throwing a zip-related error if the file is not a zip.

> plexus-java 0.9.10 causes a NullPointer in compiler-plugin 3.8.0
> ----------------------------------------------------------------
>
>                 Key: MCOMPILER-359
>                 URL: https://issues.apache.org/jira/browse/MCOMPILER-359
>             Project: Maven Compiler Plugin
>          Issue Type: Bug
>    Affects Versions: 3.8.0
>         Environment: Windows 7
> JDK 10.0.2
> Maven 3.5.4
>            Reporter: Marvin
>            Priority: Major
>         Attachments: plugintest.zip
>
>
> If {{<dependencies>}} contains a {{<type>pom</type>}} or {{<type>war</type>}} this call:
>      {{moduleDescriptor = binaryParser.getModuleDescriptor( path );}}
>  throws an {{IOException}}:
>      {{zip END header not found}}
>  because the file is being treated as a jar file.
>  When the call returns to {{CompilerMojo.preparePaths()}} this call returns {{null}}:
>      {{Throwable cause = pathException.getValue().getCause();}}
>  and the next line throws a {{NullPointerException}}:
>      {{while ( cause.getCause() != null )}}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)