You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-commits@hadoop.apache.org by aw...@apache.org on 2015/09/04 23:47:27 UTC

hadoop git commit: HADOOP-12375. Incomplete checking for findbugs executable (Jagadesh Kiran N via aw)

Repository: hadoop
Updated Branches:
  refs/heads/HADOOP-12111 a6b6c44b7 -> 8bb85842d


HADOOP-12375. Incomplete checking for findbugs executable (Jagadesh Kiran N via aw)


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

Branch: refs/heads/HADOOP-12111
Commit: 8bb85842df5dc37fac25767215b2eaeb396bd607
Parents: a6b6c44
Author: Allen Wittenauer <aw...@apache.org>
Authored: Fri Sep 4 14:47:18 2015 -0700
Committer: Allen Wittenauer <aw...@apache.org>
Committed: Fri Sep 4 14:47:18 2015 -0700

----------------------------------------------------------------------
 dev-support/test-patch.d/findbugs.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hadoop/blob/8bb85842/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 16ef8d7..df479af 100755
--- a/dev-support/test-patch.d/findbugs.sh
+++ b/dev-support/test-patch.d/findbugs.sh
@@ -64,7 +64,7 @@ function findbugs_parse_args
 ## @return       1 findbugs is missing some component
 function findbugs_is_installed
 {
-  if [[ ! -e "${FINDBUGS_HOME}/bin/findbugs" ]]; then
+  if [[ ! -x "${FINDBUGS_HOME}/bin/findbugs" ]]; then
     printf "\n\n%s is not executable.\n\n" "${FINDBUGS_HOME}/bin/findbugs"
     add_vote_table -1 findbugs "Findbugs is not installed."
     return 1