You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@kudu.apache.org by "Jean-Daniel Cryans (Code Review)" <ge...@cloudera.org> on 2016/03/02 05:44:37 UTC

[kudu-CR](branch-0.7.x) Add RAT excludes file and parsing script for releases

Hello Kudu Jenkins,

I'd like you to do a code review.  Please visit

    http://gerrit.cloudera.org:8080/2396

to review the following change.

Change subject: Add RAT excludes file and parsing script for releases
......................................................................

Add RAT excludes file and parsing script for releases

The RAT tool itself doesn't have very robust support for exclude files
(only filenames, not paths, are supported), so I've written a little
Python script to parse the output of the RAT XML report and apply rules
from a simple path-globbing exclude file. Instructions in the README.

Ultimately, it would be nice to fully automate downloading and running
RAT, but this is better than nothing.

Change-Id: Id6850e71665c0846b5f57d0527b930a629f0ab4a
Reviewed-on: http://gerrit.cloudera.org:8080/2180
Tested-by: Kudu Jenkins
Reviewed-by: Jean-Daniel Cryans
---
A build-support/release/README.adoc
A build-support/release/check-rat-report.py
A build-support/release/rat_exclude_files.txt
3 files changed, 265 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/96/2396/1
-- 
To view, visit http://gerrit.cloudera.org:8080/2396
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Id6850e71665c0846b5f57d0527b930a629f0ab4a
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: branch-0.7.x
Gerrit-Owner: Jean-Daniel Cryans
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy <mp...@apache.org>