[23/47] hbase git commit: HBASE-18834 fix shellcheck warning on hbase personality

2017-09-23 Thread busbey
HBASE-18834 fix shellcheck warning on hbase personality

Signed-off-by: Dima Spivak 


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

Branch: refs/heads/HBASE-18467
Commit: efb95a17945cc91aa639396f6f6c528b52c71fcb
Parents: 2a8fb7a
Author: Sean Busbey 
Authored: Sun Sep 17 23:54:37 2017 -0500
Committer: Sean Busbey 
Committed: Mon Sep 18 08:54:37 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/efb95a17/dev-support/hbase-personality.sh
--
diff --git a/dev-support/hbase-personality.sh b/dev-support/hbase-personality.sh
index cf7c6b9..b7a2f97 100755
--- a/dev-support/hbase-personality.sh
+++ b/dev-support/hbase-personality.sh
@@ -119,8 +119,7 @@ function personality_modules
 yetus_debug "EXCLUDE_TESTS_URL = ${EXCLUDE_TESTS_URL}"
 yetus_debug "INCLUDE_TESTS_URL = ${INCLUDE_TESTS_URL}"
 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)
   yetus_debug "excludes=${excludes}"
   if [[ -n "${excludes}" ]]; then
@@ -132,8 +131,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)
   yetus_debug "includes=${includes}"
   if [[ -n "${includes}" ]]; then



hbase git commit: HBASE-18834 fix shellcheck warning on hbase personality

2017-09-18 Thread busbey
Repository: hbase
Updated Branches:
  refs/heads/branch-2 2aa10cb65 -> 5ed2f899e


HBASE-18834 fix shellcheck warning on hbase personality

Signed-off-by: Dima Spivak 


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 
Authored: Sun Sep 17 23:54:37 2017 -0500
Committer: Sean Busbey 
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}"



hbase git commit: HBASE-18834 fix shellcheck warning on hbase personality

2017-09-18 Thread busbey
Repository: hbase
Updated Branches:
  refs/heads/master 2a8fb7a9a -> efb95a179


HBASE-18834 fix shellcheck warning on hbase personality

Signed-off-by: Dima Spivak 


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

Branch: refs/heads/master
Commit: efb95a17945cc91aa639396f6f6c528b52c71fcb
Parents: 2a8fb7a
Author: Sean Busbey 
Authored: Sun Sep 17 23:54:37 2017 -0500
Committer: Sean Busbey 
Committed: Mon Sep 18 08:54:37 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/efb95a17/dev-support/hbase-personality.sh
--
diff --git a/dev-support/hbase-personality.sh b/dev-support/hbase-personality.sh
index cf7c6b9..b7a2f97 100755
--- a/dev-support/hbase-personality.sh
+++ b/dev-support/hbase-personality.sh
@@ -119,8 +119,7 @@ function personality_modules
 yetus_debug "EXCLUDE_TESTS_URL = ${EXCLUDE_TESTS_URL}"
 yetus_debug "INCLUDE_TESTS_URL = ${INCLUDE_TESTS_URL}"
 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)
   yetus_debug "excludes=${excludes}"
   if [[ -n "${excludes}" ]]; then
@@ -132,8 +131,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)
   yetus_debug "includes=${includes}"
   if [[ -n "${includes}" ]]; then