Rat checks should exclude target directories created by the maven build (similar to other projects like hbase)
Add target directory to rat excludes.
Just a one liner change.. adding */target/* to Rat exclusion list in pom.xml
Wondering if we need to check the docs using rat. I think the docs currently pass the test and I believe they end up in target/, but we do shenanigans to move the files around so I don't know where they live during the rat check. However I'm not sure they are required to pass the test because they are generated.
Mike.. could you take a look and commit it if there are no issues.
Mike .. could you ?
This issue affects developers wishing to build locally. Using the project from Eclipse produces extra files which Rat fails on. The embedded elasticsearch used in the unit tests also produces files which should be ignored.
I have attached a patch, FLUME-1686-2.patch, which extends the above patch to cover these situations.
+1. But I am leaving out the last line (data/**) since that is not checked into the Flume tree
Pushed to trunk & flume-1.4 branches. Thanks Roshan & Edward!
Integrated in flume-trunk #377 (See https://builds.apache.org/job/flume-trunk/377/)
FLUME-1686. Exclude target directories & Eclipse files from rat checks. (Revision 3b29a3c07cbd6e5ac70cc55e506ffaf7ceecfbe2)
Result = FAILURE
mpercy : http://git-wip-us.apache.org/repos/asf/flume/repo?p=flume.git&a=commit&h=3b29a3c07cbd6e5ac70cc55e506ffaf7ceecfbe2