yetus-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bus...@apache.org
Subject yetus git commit: YETUS-467 Update hbase personality after YETUS-322 "flip verify_ result codes"
Date Sat, 29 Oct 2016 02:03:51 GMT
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 <busbey@apache.org>


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 <stack@apache.org>
Authored: Fri Oct 28 16:00:56 2016 -0700
Committer: Sean Busbey <busbey@apache.org>
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
 


Mime
View raw message