You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@pagespeed.apache.org by GitBox <gi...@apache.org> on 2018/01/15 21:14:52 UTC

[GitHub] oschaaf opened a new pull request #1717: Update license bannners in net/ and pagespeed/

oschaaf opened a new pull request #1717: Update license bannners in net/ and pagespeed/
URL: https://github.com/apache/incubator-pagespeed-mod/pull/1717
 
 
   This updates license headers in net/ and pagespeed/ for .cc and .h files. The process
   used to do this was: 
   
   1. Strip the original banners with a tiny script
   2. Run Apache RAT [1] to (re)insert banners into sources that do not have them
   3. Perform some fixups, mostly caused by the script in 1.
   
   Note authors are also stripped from the comments in some cases. This is on purpose,
   I believe ASF policy is that these should not be there, and there are arguments for not having
   them. Unfortunately I have not been able to find an authoritative source on the subject yet.
   
   While doing so was not the purpose of this change, a future change may strip the 
   remaining author comments out.
   
   [1] e.g. `java -jar ~/rat/apache-rat-0.13-SNAPSHOT.jar -e .rat-excludes -A --force -d net/`
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services