HBASE-14930 check_compatibility.sh needs smarter exit codes Signed-off-by: Andrew Purtell <apurt...@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 <dspi...@cloudera.com> Authored: Fri Dec 4 11:36:16 2015 -0800 Committer: Andrew Purtell <apurt...@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