[ 
http://issues.apache.org/jira/browse/MYFACES-1488?page=comments#action_12448526 
] 
            
Paul Spencer commented on MYFACES-1488:
---------------------------------------

Grant,
I was using the Checkstyle plugin to verify the file header matched the Apache 
license text when creating the patch submitted for this issue.  I suspect this 
is a clean approach since it is integrated into the Maven build process.  As an 
additional benefit, Checkstyle can also be used to detect other formatting 
issues, like unused imports...

I had started looking into the "Multimodule Configuration" [1] as a way of 
doing the configuration in one place and using it in many modules.  But I was 
waiting on feedback on the patch and mailing list post related to this issue.

As to "what next".
1) A new JIRA Issue should be create for implementing  Checkstyle, RAT,...
2) If you want to take the lead on this, it is OK with me.  If you want me to 
take the lead, I will but I will be dependent on a MyFaces committer since I am 
not one.

Paul Spencer

[1] 
http://maven.apache.org/plugins/maven-checkstyle-plugin/examples/multi-module-config.html

> ASF Source Header and Copyright Notice Policy
> ---------------------------------------------
>
>                 Key: MYFACES-1488
>                 URL: http://issues.apache.org/jira/browse/MYFACES-1488
>             Project: MyFaces Core
>          Issue Type: Improvement
>            Reporter: Wendy Smoak
>         Assigned To: Grant Smith
>            Priority: Blocker
>             Fix For: 1.1.5-SNAPSHOT
>
>         Attachments: shared-core.patch
>
>
> All the ASF License headers in the source files need to be updated to remove 
> the copyright notice as per the new policy:
>     http://www.apache.org/legal/src-headers.html
> We could also run Robert Burrell Donkin's "RAT" (release audit tool) to see 
> if anything else is missing
>    http://code.google.com/p/arat/

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

        

Reply via email to