[hbase] branch branch-2.1 updated: HBASE-21999 [DEBUG] Exit if git returns empty revision!

[st...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

stack pushed a commit to branch branch-2.1
in repository https://gitbox.apache.org/repos/asf/hbase.git


The following commit(s) were added to refs/heads/branch-2.1 by this push:
     new c8f9f56  HBASE-21999 [DEBUG] Exit if git returns empty revision!
c8f9f56 is described below

commit c8f9f56e947d27ba2f609215bb6a89f5bc57ce4b
Author: stack <st...@apache.org>
AuthorDate: Wed Mar 6 16:44:50 2019 -0800

    HBASE-21999 [DEBUG] Exit if git returns empty revision!
---
 hbase-common/src/saveVersion.sh | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/hbase-common/src/saveVersion.sh b/hbase-common/src/saveVersion.sh
index 730224f..97fe6b4 100644
--- a/hbase-common/src/saveVersion.sh
+++ b/hbase-common/src/saveVersion.sh
@@ -18,6 +18,8 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
+set -e
+
 unset LANG
 unset LC_CTYPE
 
@@ -45,10 +47,12 @@ else
   revision="Unknown"
   url="file://$cwd"
 fi
-which md5sum > /dev/null
-if [ "$?" != "0" ] ; then
-  which md5 > /dev/null
-  if [ "$?" != "0" ] ; then
+if [ -z $revision ]; then
+  echo "$revision is empty!"
+  exit 1
+fi
+if ! [  -x "$(command -v md5sum)" ]; then
+  if ! [ -x "$(command -v md5)" ]; then
     srcChecksum="Unknown"
   else
     srcChecksum=`find hbase-*/src/main/ | grep -e "\.java" -e "\.proto" | LC_ALL=C sort | xargs md5 | md5 | cut -d ' ' -f 1`