Repository: yetus Updated Branches: refs/heads/master 99540d47f -> 938504b64
YETUS-467 Update hbase personality after YETUS-322 "flip verify_ result codes" YETUS-322 "flip verify_" changed meaning of what is returned by verify_* functions. The hbase personality makes use of verify_needed_test but wasn't updated to use the new idiom testing the return code (all other personalities in the personality dir seem to do the right thing around verify_) Signed-off-by: Sean Busbey <[email protected]> Project: http://git-wip-us.apache.org/repos/asf/yetus/repo Commit: http://git-wip-us.apache.org/repos/asf/yetus/commit/938504b6 Tree: http://git-wip-us.apache.org/repos/asf/yetus/tree/938504b6 Diff: http://git-wip-us.apache.org/repos/asf/yetus/diff/938504b6 Branch: refs/heads/master Commit: 938504b64958650d047f1cd6ace37dd12bd20f62 Parents: 99540d4 Author: Michael Stack <[email protected]> Authored: Fri Oct 28 16:00:56 2016 -0700 Committer: Sean Busbey <[email protected]> Committed: Fri Oct 28 21:02:11 2016 -0500 ---------------------------------------------------------------------- precommit/personality/hbase.sh | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/yetus/blob/938504b6/precommit/personality/hbase.sh ---------------------------------------------------------------------- diff --git a/precommit/personality/hbase.sh b/precommit/personality/hbase.sh index ff2b516..b48691e 100755 --- a/precommit/personality/hbase.sh +++ b/precommit/personality/hbase.sh @@ -197,8 +197,7 @@ function hbaseprotoc_rebuild return 0 fi - verify_needed_test hbaseprotoc - if [[ $? == 0 ]]; then + if ! verify_needed_test hbaseprotoc; then return 0 fi @@ -268,8 +267,7 @@ function hbaseanti_patchfile return 0 fi - verify_needed_test hbaseanti - if [[ $? == 0 ]]; then + if ! verify_needed_test hbaseanti; then return 0 fi
