[ http://jira.codehaus.org/browse/MJAR-32?page=comments#action_59321 ] 

Pawel Pastula commented on MJAR-32:
-----------------------------------

After applying your patch, some tests fail.
To be precise it's: testVerifyInPlaceSignedJar
when you invoke:
mojo.setSignedJar( null );
you will get NPE in JarSignMojo.java:221:
addArgIfNotEmpty( arguments, "-signedjar", this.signedjar.getPath() );
Here is the solution to this problem:
addArgIfNotEmpty( arguments, "-signedjar", this.signedjar != null ? 
this.signedjar.getPath() : null );

I didn't attach a patch since I don't know to which revision this patch should 
be applied.

> [JarSignMojo] Verification of a signed jar fails
> ------------------------------------------------
>
>          Key: MJAR-32
>          URL: http://jira.codehaus.org/browse/MJAR-32
>      Project: Maven 2.x Jar Plugin
>         Type: Bug

>  Environment: Linux Slackware 10.1, JDK 1.4.2_02
>     Reporter: Pawel Pastula
>     Priority: Blocker
>  Attachments: MJAR-32.diff
>
>
> When verification is executed there is no way the verification can succeed 
> since if uses unsigned jar file instead of signedjar file:
> verify.setJarPath( getJarFile() ); in JarSignMojo.java:182.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to