[hbase] 01/02: HBASE-20467 Precommit personality should only run checkstyle once if we're going to run it at the root

[nd...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

ndimiduk pushed a commit to branch branch-2
in repository https://gitbox.apache.org/repos/asf/hbase.git

commit 6f675e269cfb0f1d234e28a8e4e7eafc2a2c11a7
Author: Nihal Jain <ni...@gmail.com>
AuthorDate: Sat Apr 21 03:41:13 2018 +0530

    HBASE-20467 Precommit personality should only run checkstyle once if we're going to run it at the root
    
    Signed-off-by: Sean Busbey <bu...@apache.org>
---
 dev-support/hbase-personality.sh | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/dev-support/hbase-personality.sh b/dev-support/hbase-personality.sh
index 72f0b05..3fe9fd9 100755
--- a/dev-support/hbase-personality.sh
+++ b/dev-support/hbase-personality.sh
@@ -153,7 +153,8 @@ function personality_modules
   # If BUILDMODE is 'patch', for unit and compile testtypes, there is no need to run individual
   # modules if root is included. HBASE-18505
   if [[ "${BUILDMODE}" == "full" ]] || \
-     [[ ( "${testtype}" == unit || "${testtype}" == compile ) && "${MODULES[*]}" =~ \. ]]; then
+     ( ( [[ "${testtype}" == unit ]] || [[ "${testtype}" == compile ]] || [[ "${testtype}" == checkstyle ]] ) && \
+     [[ "${MODULES[*]}" =~ \. ]] ); then
     MODULES=(.)
   fi