Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP
Browse files

BOOKKEEPER-483: precommit tests only check toplevel rat file, not the…

… one for submodules. (ivank via fpj)

git-svn-id: https://svn.apache.org/repos/asf/zookeeper/bookkeeper/trunk@1421346 13f79535-47bb-0310-9956-ffa450edef68
  • Loading branch information...
commit d932c9b2181cc9f9b6bb3b43abd5af04cda206b8 1 parent 496f442
@fpj fpj authored
Showing with 12 additions and 3 deletions.
  1. +2 −0  CHANGES.txt
  2. +10 −3 bin/test-patch-08-rat
View
2  CHANGES.txt
@@ -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)
View
13 bin/test-patch-08-rat
@@ -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

0 comments on commit d932c9b

Please sign in to comment.
Something went wrong with that request. Please try again.