yetus-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bus...@apache.org
Subject [39/50] [abbrv] yetus git commit: HADOOP-12312. Findbugs HTML report link shows 0 warnings despite errors (Kengo Seki via busbey)
Date Wed, 23 Sep 2015 02:24:09 GMT
HADOOP-12312. Findbugs HTML report link shows 0 warnings despite errors (Kengo Seki via busbey)


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

Branch: refs/heads/master
Commit: b8a06f99262b6dee0abbb765e30bc2bbaa021a0f
Parents: 30d8e14
Author: Sean Busbey <busbey@apache.org>
Authored: Fri Sep 4 09:16:42 2015 -0500
Committer: Sean Busbey <busbey@cloudera.com>
Committed: Fri Sep 4 09:17:09 2015 -0500

----------------------------------------------------------------------
 dev-support/test-patch.d/findbugs.sh | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/yetus/blob/b8a06f99/dev-support/test-patch.d/findbugs.sh
----------------------------------------------------------------------
diff --git a/dev-support/test-patch.d/findbugs.sh b/dev-support/test-patch.d/findbugs.sh
index 4fa5428..16ef8d7 100755
--- a/dev-support/test-patch.d/findbugs.sh
+++ b/dev-support/test-patch.d/findbugs.sh
@@ -252,7 +252,9 @@ function findbugs_postinstall
   local branchxml
   local patchxml
   local newbugsbase
+  local fixedbugsbase
   local new_findbugs_warnings
+  local fixed_findbugs_warnings
   local line
   local firstpart
   local secondpart
@@ -297,6 +299,7 @@ function findbugs_postinstall
     fi
 
     newbugsbase="${PATCH_DIR}/new-findbugs-${fn}"
+    fixedbugsbase="${PATCH_DIR}/fixed-findbugs-${fn}"
 
     "${FINDBUGS_HOME}/bin/computeBugHistory" -useAnalysisTimes -withMessages \
             -output "${combined_xml}" \
@@ -326,8 +329,8 @@ function findbugs_postinstall
     fi
 
     #shellcheck disable=SC2016
-    new_findbugs_fixed_warnings=$("${FINDBUGS_HOME}/bin/filterBugs" -fixed patch \
-        "${combined_xml}" "${newbugsbase}.xml" | ${AWK} '{print $1}')
+    fixed_findbugs_warnings=$("${FINDBUGS_HOME}/bin/filterBugs" -fixed patch \
+        "${combined_xml}" "${fixedbugsbase}.xml" | ${AWK} '{print $1}')
     if [[ $? != 0 ]]; then
       popd >/dev/null
       module_status ${i} -1 "" "${module} cannot run filterBugs (#2) from findbugs"
@@ -338,9 +341,7 @@ function findbugs_postinstall
       continue
     fi
 
-    echo "Found ${new_findbugs_warnings} new Findbugs warnings and ${new_findbugs_fixed_warnings}
newly fixed warnings."
-    findbugs_warnings=$((findbugs_warnings+new_findbugs_warnings))
-    findbugs_fixed_warnings=$((findbugs_fixed_warnings+new_findbugs_fixed_warnings))
+    echo "Found ${new_findbugs_warnings} new Findbugs warnings and ${fixed_findbugs_warnings}
newly fixed warnings."
 
     "${FINDBUGS_HOME}/bin/convertXmlToText" -html "${newbugsbase}.xml" \
         "${newbugsbase}.html"


Mime
View raw message