[ 
https://jira.codehaus.org/browse/MSHADE-101?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=272127#comment-272127
 ] 

Benson Margulies edited comment on MSHADE-101 at 7/2/11 6:27 AM:
-----------------------------------------------------------------

Robert, We don't check in files with personal copyrights at Apache. Since you 
have an ICLA on file, you have in general granted permission for your work to 
be incorporated with the standard notice. And you did use a standard notice on 
the java file, just not on the POM. So I'm going to go ahead and fix the notice 
and commit. If this is a problem we'll cope.

      was (Author: bmargulies):
    Robert, We don't check in files with personal copyrights at Apache. Since 
you have an ICLA on file, you have in general granted permission for your work 
to be incorporated with the standard notice. However, in an abundance of 
caution, I'm holding off committing the repaired notices until I hear from you 
that you did indeed intend to submit this under the terms of your icla with 
Apache.

  
> ApacheNoticeResourceTransformer NullPointerException for some licenses when 
> no parameters set.
> ----------------------------------------------------------------------------------------------
>
>                 Key: MSHADE-101
>                 URL: https://jira.codehaus.org/browse/MSHADE-101
>             Project: Maven 2.x Shade Plugin
>          Issue Type: Bug
>    Affects Versions: 1.5
>            Reporter: Robert Burrell Donkin
>            Assignee: Benson Margulies
>         Attachments: 
> ApacheNoticeResourceTransformer-NullPointerException.patch
>
>
> For some dependencies, ApacheNoticeResourceTransformer throws a 
> NullPointerException when no parameters are set.
> Included in the patch are 
> * an integration test case which demonstrates the problem, 
> * a unit test case (including tests around this problem) and
> * my proposed (narrow) fix for ApacheNoticeResourceTransformer.
> ATM no parameter is mandatory. This may indicate an intention to support a 
> basic use case without parameters. So this patch opts for a reasonable 
> default.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to