You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@zookeeper.apache.org by fp...@apache.org on 2012/12/13 16:44:20 UTC
svn commit: r1421346 - in /zookeeper/bookkeeper/trunk: CHANGES.txt
bin/test-patch-08-rat
Author: fpj
Date: Thu Dec 13 15:44:19 2012
New Revision: 1421346
URL: http://svn.apache.org/viewvc?rev=1421346&view=rev
Log:
BOOKKEEPER-483: precommit tests only check toplevel rat file, not the one for submodules. (ivank via fpj)
Modified:
zookeeper/bookkeeper/trunk/CHANGES.txt
zookeeper/bookkeeper/trunk/bin/test-patch-08-rat
Modified: zookeeper/bookkeeper/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/zookeeper/bookkeeper/trunk/CHANGES.txt?rev=1421346&r1=1421345&r2=1421346&view=diff
==============================================================================
--- zookeeper/bookkeeper/trunk/CHANGES.txt (original)
+++ zookeeper/bookkeeper/trunk/CHANGES.txt Thu Dec 13 15:44:19 2012
@@ -34,6 +34,8 @@ Trunk (unreleased changes)
BOOKKEEPER-484: Misc fixes for test scripts (ivank via fpj)
+ BOOKKEEPER-483: precommit tests only check toplevel rat file, not the one for submodules. (ivank via fpj)
+
bookkeeper-server:
BOOKKEEPER-183: Provide tools to read/check data files in bookie server (sijie via ivank)
Modified: zookeeper/bookkeeper/trunk/bin/test-patch-08-rat
URL: http://svn.apache.org/viewvc/zookeeper/bookkeeper/trunk/bin/test-patch-08-rat?rev=1421346&r1=1421345&r2=1421346&view=diff
==============================================================================
--- zookeeper/bookkeeper/trunk/bin/test-patch-08-rat (original)
+++ zookeeper/bookkeeper/trunk/bin/test-patch-08-rat Thu Dec 13 15:44:19 2012
@@ -96,18 +96,25 @@ checkForWarnings() {
fi
}
###############################################################################
-
+copyRatFiles() {
+ TAG=$1
+ rm -f ${REPORTDIR}/${TASKNAME}-$TAG.txt
+ for f in $(find . -name rat.txt); do
+ cat $f >> ${REPORTDIR}/${TASKNAME}-$TAG.txt
+ done
+}
+###############################################################################
parseArgs "$@"
case $OP in
pre)
mvn apache-rat:check ${MVNPASSTHRU} > $STDOUT
- cp target/rat.txt ${REPORTDIR}/${TASKNAME}-clean.txt
+ copyRatFiles clean
;;
post)
mvn apache-rat:check ${MVNPASSTHRU} > $STDOUT
- cp target/rat.txt ${REPORTDIR}/${TASKNAME}-patch.txt
+ copyRatFiles patch
;;
report)
checkForWarnings