[arrow] branch master updated: ARROW-2960: [Packaging] Fix verify-release-candidate for binary packages and fix release cutting script for lib64 cmake issue

[cp...@apache.org]

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

cpcloud pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/arrow.git


The following commit(s) were added to refs/heads/master by this push:
     new 567e33b  ARROW-2960: [Packaging] Fix verify-release-candidate for binary packages and fix release cutting script for lib64 cmake issue
567e33b is described below

commit 567e33b5dd23dd236be46bbd90b15d85b8847775
Author: Phillip Cloud <cp...@gmail.com>
AuthorDate: Wed Aug 1 12:53:50 2018 -0400

    ARROW-2960: [Packaging] Fix verify-release-candidate for binary packages and fix release cutting script for lib64 cmake issue
    
    Author: Phillip Cloud <cp...@gmail.com>
    
    Closes #2356 from cpcloud/ARROW-2960 and squashes the following commits:
    
    5f92c9f6 <Phillip Cloud> ARROW-2960:  Fix verify-release-candidate for binary packages and fix release cutting script for lib64 cmake issue
---
 dev/release/02-source.sh                |  1 +
 dev/release/verify-release-candidate.sh | 27 +++++++++++++++++++++------
 2 files changed, 22 insertions(+), 6 deletions(-)

diff --git a/dev/release/02-source.sh b/dev/release/02-source.sh
index 1712427..e72bb5d 100755
--- a/dev/release/02-source.sh
+++ b/dev/release/02-source.sh
@@ -79,6 +79,7 @@ cpp_install_dir=${PWD}/${extract_dir}/cpp/install
 cd ${extract_dir}/cpp/build
 cmake .. \
   -DCMAKE_INSTALL_PREFIX=${cpp_install_dir} \
+  -DCMAKE_INSTALL_LIBDIR=${cpp_install_dir}/lib \
   -DARROW_BUILD_TESTS=no
 make -j8
 make install
diff --git a/dev/release/verify-release-candidate.sh b/dev/release/verify-release-candidate.sh
index 9a18bce..74ec61c 100755
--- a/dev/release/verify-release-candidate.sh
+++ b/dev/release/verify-release-candidate.sh
@@ -40,13 +40,19 @@ case $# in
 esac
 
 set -ex
+set -o pipefail
 
 HERE=$(cd `dirname "${BASH_SOURCE[0]:-$0}"` && pwd)
 
 ARROW_DIST_URL='https://dist.apache.org/repos/dist/dev/arrow'
 
 download_dist_file() {
-  curl -f -O $ARROW_DIST_URL/$1
+  curl \
+    --silent \
+    --show-error \
+    --fail \
+    --location \
+    --remote-name $ARROW_DIST_URL/$1
 }
 
 download_rc_file() {
@@ -71,18 +77,27 @@ fetch_archive() {
 
 verify_binary_artifacts() {
   # download the binaries folder for the current RC
-  download_rc_file binaries
+  rcname=apache-arrow-${VERSION}-rc${RC_NUMBER}
+  wget -P "$rcname" \
+    --quiet \
+    --no-host-directories \
+    --cut-dirs=5 \
+    --show-progress \
+    --no-parent \
+    --reject 'index.html*' \
+    --recursive "$ARROW_DIST_URL/$rcname/binaries/"
 
   # verify the signature and the checksums of each artifact
-  find binaries -name '*.asc' | while read sigfile; do
+  find $rcname/binaries -name '*.asc' | while read sigfile; do
     artifact=${sigfile/.asc/}
-    gpg --verify $sigfile $artifact
+    gpg --verify $sigfile $artifact || exit 1
 
     # go into the directory because the checksum files contain only the
     # basename of the artifact
     pushd $(dirname $artifact)
-    shasum -a 1 -c $artifact.sha1
-    shasum -a 256 -c $artifact.sha256
+    base_artifact=$(basename $artifact)
+    shasum -a 1 -c $base_artifact.sha1 || exit 1
+    shasum -a 256 -c $base_artifact.sha256 || exit 1
     popd
   done
 }