I see Niall has been working on an ant build file to produce RAT reports for
license headers of the subversion tree [1].

It would be helpful if the patch was altered to provide an exclude file,
rather than list the excludes directly into the build.xml file itself.
This will allow the ASF Buildbot RAT reports to use the exclude file when
producing its own reports.

So, for instance, instead of:

+            <fileset dir="${project.dir}">
+
+                <!-- RAT Files (if build-rat.xml is dropped in project
directory) -->
+                <exclude name="apache-rat-*.jar"/>
+                <exclude name="apache-rat-*.jar"/>
+                <exclude name="build-rat.*"/>
+                <exclude name="rat-report*.txt"/>
+
+                <!-- the following are not distributed -->
+                <exclude name="contrib/"/>
+                <exclude name="www/"/>
+
+                <!-- Text files -->
+                <exclude name="**/*.txt"/>
+                ....
+            </fileset>

We could instead have:

+            <fileset dir="${project.dir}">
+
+                <excludesfile name="rat-avoid.txt"/>

and then just place the excludes list in rat-avoid.txt.

See how Apache Forrest does it at

http://svn.apache.org/repos/asf/forrest/trunk/etc/rat-avoid.txt

This way, the output at

http://ci.apache.org/projects/subversion/rat-output.txt

can be made more accurate, as seen by Forrests output:


http://ci.apache.org/projects/forrest/rat-output.txt

I hope this makes sense,

Thanks

Gav...


[1] - http://subversion.tigris.org/issues/show_bug.cgi?id=3529


Reply via email to