HBASE-20467 Precommit personality should only run checkstyle once if we're going to run it at the root
Signed-off-by: Sean Busbey <[email protected]> Project: http://git-wip-us.apache.org/repos/asf/hbase/repo Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/d4768114 Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/d4768114 Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/d4768114 Branch: refs/heads/HBASE-19064 Commit: d4768114e860eb03418e6b08e5de20950824327e Parents: 2ab7318 Author: Nihal Jain <[email protected]> Authored: Sat Apr 21 03:41:13 2018 +0530 Committer: Sean Busbey <[email protected]> Committed: Thu Apr 26 07:54:25 2018 -0500 ---------------------------------------------------------------------- dev-support/hbase-personality.sh | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/hbase/blob/d4768114/dev-support/hbase-personality.sh ---------------------------------------------------------------------- diff --git a/dev-support/hbase-personality.sh b/dev-support/hbase-personality.sh index b010503..2b1e2c3 100755 --- a/dev-support/hbase-personality.sh +++ b/dev-support/hbase-personality.sh @@ -125,7 +125,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
