You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tez.apache.org by ss...@apache.org on 2015/05/29 23:05:37 UTC

tez git commit: TEZ-2503 addendum. Move findbugs_version determination to after running findbugs. (sseth)

Repository: tez
Updated Branches:
  refs/heads/master 7bc350505 -> 87f26a2ed


TEZ-2503 addendum. Move findbugs_version determination to after running
findbugs. (sseth)


Project: http://git-wip-us.apache.org/repos/asf/tez/repo
Commit: http://git-wip-us.apache.org/repos/asf/tez/commit/87f26a2e
Tree: http://git-wip-us.apache.org/repos/asf/tez/tree/87f26a2e
Diff: http://git-wip-us.apache.org/repos/asf/tez/diff/87f26a2e

Branch: refs/heads/master
Commit: 87f26a2ed10a097d20f69c5ab0b6e33a0877125e
Parents: 7bc3505
Author: Siddharth Seth <ss...@apache.org>
Authored: Fri May 29 14:05:16 2015 -0700
Committer: Siddharth Seth <ss...@apache.org>
Committed: Fri May 29 14:05:16 2015 -0700

----------------------------------------------------------------------
 build-tools/test-patch.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/tez/blob/87f26a2e/build-tools/test-patch.sh
----------------------------------------------------------------------
diff --git a/build-tools/test-patch.sh b/build-tools/test-patch.sh
index d41c54b..d1d5a24 100755
--- a/build-tools/test-patch.sh
+++ b/build-tools/test-patch.sh
@@ -525,7 +525,6 @@ buildAndInstall () {
 ###############################################################################
 ### Check there are no changes in the number of Findbugs warnings
 checkFindbugsWarnings () {
-  findbugs_version=$(${AWK} 'match($0, /findbugs-maven-plugin:[^:]*:findbugs/) { print substr($0, RSTART + 22, RLENGTH - 31); exit }' "${PATCH_DIR}/patchFindBugsOutput.txt")
   echo ""
   echo ""
   echo "======================================================================"
@@ -541,6 +540,7 @@ checkFindbugsWarnings () {
   echo "$MVN clean test findbugs:findbugs -DskipTests < /dev/null > $PATCH_DIR/patchFindBugsOutput.txt 2>&1"
   $MVN clean test findbugs:findbugs -DskipTests < /dev/null > $PATCH_DIR/patchFindBugsOutput.txt 2>&1
   rc=$?
+  findbugs_version=$(${AWK} 'match($0, /findbugs-maven-plugin:[^:]*:findbugs/) { print substr($0, RSTART + 22, RLENGTH - 31); exit }' "${PATCH_DIR}/patchFindBugsOutput.txt")
 
   if [ $rc != 0 ] ; then
     JIRA_COMMENT="$JIRA_COMMENT