Author: bobby
Date: Wed May 9 14:08:28 2012
New Revision: 1336213
URL: http://svn.apache.org/viewvc?rev=1336213&view=rev
Log:
HADOOP-8354. test-patch findbugs may fail if a dependent module is changed
Contributed by Tom White and Robert Evans.
Modified:
hadoop/common/trunk/dev-support/test-patch.sh
Modified: hadoop/common/trunk/dev-support/test-patch.sh
URL:
http://svn.apache.org/viewvc/hadoop/common/trunk/dev-support/test-patch.sh?rev=1336213&r1=1336212&r2=1336213&view=diff
==============================================================================
--- hadoop/common/trunk/dev-support/test-patch.sh (original)
+++ hadoop/common/trunk/dev-support/test-patch.sh Wed May 9 14:08:28 2012
@@ -529,6 +529,24 @@ $JIRA_COMMENT_FOOTER"
}
###############################################################################
+### Install the new jars so tests and findbugs can find all of the updated
jars
+buildAndInstall () {
+ echo ""
+ echo ""
+ echo "======================================================================"
+ echo "======================================================================"
+ echo " Installing all of the jars"
+ echo "======================================================================"
+ echo "======================================================================"
+ echo ""
+ echo ""
+ echo "$MVN install -Dmaven.javadoc.skip=true -DskipTests
-D${PROJECT_NAME}PatchProcess"
+ $MVN install -Dmaven.javadoc.skip=true -DskipTests
-D${PROJECT_NAME}PatchProcess
+ return $?
+}
+
+
+###############################################################################
### Check there are no changes in the number of Findbugs warnings
checkFindbugsWarnings () {
findbugs_version=`${FINDBUGS_HOME}/bin/findbugs -version`
@@ -891,6 +909,7 @@ checkEclipseGeneration
### Checkstyle not implemented yet
#checkStyle
#(( RESULT = RESULT + $? ))
+buildAndInstall
checkFindbugsWarnings
(( RESULT = RESULT + $? ))
checkReleaseAuditWarnings
