You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by sy...@apache.org on 2015/12/07 17:56:23 UTC

[46/50] [abbrv] hbase git commit: HBASE-14930 check_compatibility.sh needs smarter exit codes

HBASE-14930 check_compatibility.sh needs smarter exit codes

Signed-off-by: Andrew Purtell <ap...@apache.org>


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

Branch: refs/heads/hbase-12439
Commit: 4f36632d7b7f987e91327f9ce1673a842f4eb864
Parents: c9a0148
Author: Dima Spivak <ds...@cloudera.com>
Authored: Fri Dec 4 11:36:16 2015 -0800
Committer: Andrew Purtell <ap...@apache.org>
Committed: Fri Dec 4 18:55:52 2015 -0800

----------------------------------------------------------------------
 dev-support/check_compatibility.sh | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hbase/blob/4f36632d/dev-support/check_compatibility.sh
----------------------------------------------------------------------
diff --git a/dev-support/check_compatibility.sh b/dev-support/check_compatibility.sh
index b9a7238..b6c11e0 100755
--- a/dev-support/check_compatibility.sh
+++ b/dev-support/check_compatibility.sh
@@ -93,7 +93,7 @@ if ! ARG_LIST=$(${GETOPT} -q -o abfhno:qr:s \
     -l all,binary-only,force-download,help,no-checkout,options:,quick,repo:,source-only \
     -- "${@}"); then
   usage >&2
-  exit 1
+  exit 2
 fi
 eval set -- "${ARG_LIST[@]}"
 
@@ -145,7 +145,7 @@ while ((${#})); do
       # message.
       else
         usage >&2
-        exit 1
+        exit 2
       fi
       ;;
   esac
@@ -166,7 +166,7 @@ for ref in 1 2; do
       echo "Cloning ${REPO_URL} into ${SCRIPT_DIRECTORY}/target/compatibility/${ref}..."
       if ! git clone ${REPO_URL} ${SCRIPT_DIRECTORY}/target/compatibility/${ref}; then
 	echo "Error while cloning ${REPO_URL}. Exiting..." >&2
-	exit 1
+	exit 2
       fi
     elif [ "${ref}" = "2" ]; then
       # Avoid cloning from Git twice by copying first repo into different folder.
@@ -181,12 +181,12 @@ for ref in 1 2; do
     echo "Checking out ${COMMIT[${ref}]} into ${ref}/..."
     if ! git checkout -f ${COMMIT[${ref}]}; then
       echo "Error while checking out ${COMMIT[${ref}]}. Exiting..." >&2
-      exit 1
+      exit 2
     fi
     echo "Building ${COMMIT[${ref}]}..."
     if ! mvn clean package -DskipTests; then
       echo "Maven could not successfully package ${COMMIT[${ref}]}. Exiting..." >&2
-      exit 1
+      exit 2
     fi
     popd > /dev/null
   fi
@@ -205,7 +205,7 @@ for ref in 1 2; do
             "Attempting to build ${COMMIT[${ref}]}..."
         if ! mvn clean package -DskipTests; then
           echo "Maven could not successfully package ${COMMIT[${ref}]}. Exiting..." >&2
-          exit 1
+          exit 2
         fi
         popd > /dev/null
       done
@@ -213,7 +213,7 @@ for ref in 1 2; do
       JARS=$(find ${SCRIPT_DIRECTORY}/target/compatibility/${ref} "${JAR_FIND_EXPRESSION[@]}")
       if [ ${#JARS[@]} -eq 0 ]; then
 	echo "Unable to find any JARs matching the find expression. Exiting..." >&2
-        exit 1
+        exit 2
       fi
 
     # If no JARs were found and --no-checkout was not specified, fail immediately.
@@ -242,7 +242,7 @@ if [ ! -d ${SCRIPT_DIRECTORY}/target/compatibility/javaACC ] || [ -n "${FORCE_DO
   if ! git clone https://github.com/lvc/japi-compliance-checker.git \
       ${SCRIPT_DIRECTORY}/target/compatibility/javaACC; then
     echo "Failed to download Java API Compliance Checker. Exiting..." >&2
-    exit 1
+    exit 2
   fi
 fi