From 686e04177674de657c5ee8aca6580c65a5a58166 Mon Sep 17 00:00:00 2001 From: Peter Somogyi Date: Mon, 13 Jul 2020 14:32:47 +0200 Subject: [PATCH 1/2] HBASE-24710 Incorrect checksum calculation in saveVersion.sh --- hbase-common/src/saveVersion.sh | 22 +++++++++++----------- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/hbase-common/src/saveVersion.sh b/hbase-common/src/saveVersion.sh index 1e61eec932ca..6ff5cc75d762 100644 --- a/hbase-common/src/saveVersion.sh +++ b/hbase-common/src/saveVersion.sh @@ -30,19 +30,19 @@ nativeOutputDirectory="$2/native/utils/" pushd . cd .. -user=`whoami | sed -n -e 's/\\\/\\\\\\\\/p'` +user=$(whoami | sed -n -e "s/\\\/\\\\\\\\/p") if [ "$user" == "" ] then - user=`whoami` + user=$(whoami) fi -date=`date` -cwd=`pwd` +date=$(date) +cwd=$(pwd) if [ -d .svn ]; then - revision=`(svn info | sed -n -e 's/Last Changed Rev: \(.*\)/\1/p') || true` - url=`(svn info | sed -n -e 's/^URL: \(.*\)/\1/p') || true` + revision=$( (svn info | sed -n -e "s/Last Changed Rev: \(.*\)/\1/p") || true) + url=$( (svn info | sed -n -e 's/^URL: \(.*\)/\1/p') || true) elif [ -d .git ]; then - revision=`git log -1 --no-show-signature --pretty=format:"%H" || true` - hostname=`hostname` + revision=$(git log -1 --no-show-signature --pretty=format:"%H" || true) + hostname=$(hostname) url="git://${hostname}${cwd}" fi if [ -z "${revision}" ]; then @@ -50,14 +50,14 @@ if [ -z "${revision}" ]; then revision="Unknown" url="file://$cwd" fi -if ! [ -x "$(command -v openssl)" ]; then +if ! [ -x "$(command -v openssl)" ]; then if ! [ -x "$(command -v gpg)" ]; then srcChecksum="Unknown" else - srcChecksum=`find hbase-*/src/main/ | grep -e "\.java" -e "\.proto" | LC_ALL=C sort | xargs gpg --print-md sha512 | gpg --print-md sha512 | cut -d ' ' -f 1` + srcChecksum=$(find hbase-*/src/main/ | grep -e "\.java" -e "\.proto" | LC_ALL=C sort xargs gpg --print-md sha512 | gpg --print-md sha512 | tr '\n' ' ' | sed 's/[[:space:]]*//g') fi else - srcChecksum=`find hbase-*/src/main/ | grep -e "\.java" -e "\.proto" | LC_ALL=C sort | xargs openssl dgst -sha512 | openssl dgst -sha512 | cut -d ' ' -f 1` + srcChecksum=$(find hbase-*/src/main/ | grep -e "\.java" -e "\.proto" | LC_ALL=C sort xargs openssl dgst -sha512 | openssl dgst -sha512 | sed 's/^.* //') fi popd From 73f809afd0569d6b696d54ea2724f8568c98ce67 Mon Sep 17 00:00:00 2001 From: Peter Somogyi Date: Wed, 15 Jul 2020 19:01:52 +0200 Subject: [PATCH 2/2] Add missing pipe --- hbase-common/src/saveVersion.sh | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/hbase-common/src/saveVersion.sh b/hbase-common/src/saveVersion.sh index 6ff5cc75d762..8d93b41218e8 100644 --- a/hbase-common/src/saveVersion.sh +++ b/hbase-common/src/saveVersion.sh @@ -54,10 +54,10 @@ if ! [ -x "$(command -v openssl)" ]; then if ! [ -x "$(command -v gpg)" ]; then srcChecksum="Unknown" else - srcChecksum=$(find hbase-*/src/main/ | grep -e "\.java" -e "\.proto" | LC_ALL=C sort xargs gpg --print-md sha512 | gpg --print-md sha512 | tr '\n' ' ' | sed 's/[[:space:]]*//g') + srcChecksum=$(find hbase-*/src/main/ | grep -e "\.java" -e "\.proto" | LC_ALL=C sort | xargs gpg --print-md sha512 | gpg --print-md sha512 | tr '\n' ' ' | sed 's/[[:space:]]*//g') fi else - srcChecksum=$(find hbase-*/src/main/ | grep -e "\.java" -e "\.proto" | LC_ALL=C sort xargs openssl dgst -sha512 | openssl dgst -sha512 | sed 's/^.* //') + srcChecksum=$(find hbase-*/src/main/ | grep -e "\.java" -e "\.proto" | LC_ALL=C sort | xargs openssl dgst -sha512 | openssl dgst -sha512 | sed 's/^.* //') fi popd