Repository: hbase Updated Branches: refs/heads/branch-2 2aa10cb65 -> 5ed2f899e
HBASE-18834 fix shellcheck warning on hbase personality Signed-off-by: Dima Spivak <dimaspi...@apache.org> Project: http://git-wip-us.apache.org/repos/asf/hbase/repo Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/5ed2f899 Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/5ed2f899 Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/5ed2f899 Branch: refs/heads/branch-2 Commit: 5ed2f899e17d838ad8c790b3f99269d76e01cf9b Parents: 2aa10cb Author: Sean Busbey <bus...@apache.org> Authored: Sun Sep 17 23:54:37 2017 -0500 Committer: Sean Busbey <bus...@apache.org> Committed: Mon Sep 18 08:55:30 2017 -0500 ---------------------------------------------------------------------- dev-support/hbase-personality.sh | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/hbase/blob/5ed2f899/dev-support/hbase-personality.sh ---------------------------------------------------------------------- diff --git a/dev-support/hbase-personality.sh b/dev-support/hbase-personality.sh index 37a4144..6745595 100755 --- a/dev-support/hbase-personality.sh +++ b/dev-support/hbase-personality.sh @@ -117,8 +117,7 @@ function personality_modules if [[ ${testtype} = unit ]]; then extra="${extra} -PrunAllTests" if [[ -n "$EXCLUDE_TESTS_URL" ]]; then - wget "$EXCLUDE_TESTS_URL" -O "excludes" - if [[ $? -eq 0 ]]; then + if wget "$EXCLUDE_TESTS_URL" -O "excludes"; then excludes=$(cat excludes) if [[ -n "${excludes}" ]]; then extra="${extra} -Dtest.exclude.pattern=${excludes}" @@ -129,8 +128,7 @@ function personality_modules "${EXCLUDE_TESTS_URL}. Ignoring and proceeding." fi elif [[ -n "$INCLUDE_TESTS_URL" ]]; then - wget "$INCLUDE_TESTS_URL" -O "includes" - if [[ $? -eq 0 ]]; then + if wget "$INCLUDE_TESTS_URL" -O "includes"; then includes=$(cat includes) if [[ -n "${includes}" ]]; then extra="${extra} -Dtest=${includes}"