jbonofre commented on PR #166:
URL: https://github.com/apache/creadur-rat/pull/166#issuecomment-1826606268

   > Thanks for the contribution @jbonofre
   > 
   > I'm still curious to get to know more details about why you want to scan 
in hidden directories? Can you provide more info about that.
   
   @ottlinger at ASF, we should have license header in all files included in 
project release source distributions (which is the minimum a project has to 
provide for each release). Some projects include hidden directories/iles in 
their source distributions (for instance, Apache Iceberg includes hidden 
directories like .palantir, .baseline, etc). Those directories/files can't be 
excluded else it means that it would not be possible to build the project from 
the source distribution.
   To be "clean", these directories/files have to include ASF/license headers 
and so they should be considered by RAT.
   
   As workaround, RAT gradle|maven plugins use a "custom" walker to scan hidden 
directories/files. So I think it makes sense to improve our default walker to 
support this behavior and so having the same support in CLI.
   
   That's basically the background :)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@creadur.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to