[ARVADOS-DEV] updated: 7512b31d3c7e3ae890a37b2222f113c9b351d86a

[git at public.curoverse.com]

Summary of changes:
 jenkins/package-testing/deb-common-test-packages.sh        |  8 ++++----
 jenkins/package-testing/test-package-arvados-api-server.sh |  2 +-
 jenkins/package-testing/test-package-arvados-sso-server.sh |  2 +-
 jenkins/package-testing/test-package-arvados-workbench.sh  |  2 +-
 jenkins/package-testing/test-packages-centos6.sh           | 12 ++++++------
 5 files changed, 13 insertions(+), 13 deletions(-)

       via  7512b31d3c7e3ae890a37b2222f113c9b351d86a (commit)
      from  846a9311a328b9cbadc8b70d34134b72b0b6b332 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit 7512b31d3c7e3ae890a37b2222f113c9b351d86a
Author: Brett Smith <brett at curoverse.com>
Date:   Wed Jan 20 18:52:36 2016 -0500

    8008: Quote more variables in package test scripts.

diff --git a/jenkins/package-testing/deb-common-test-packages.sh b/jenkins/package-testing/deb-common-test-packages.sh
index 0d15436..3928ba2 100755
--- a/jenkins/package-testing/deb-common-test-packages.sh
+++ b/jenkins/package-testing/deb-common-test-packages.sh
@@ -6,12 +6,12 @@ set -eu
 # from the invocation path.
 target=$(echo $0 | sed 's/.*test-packages-\([^.]*\)\.sh.*/\1/')
 
-export ARV_PACKAGES_DIR=/arvados/packages/$target
+export ARV_PACKAGES_DIR="/arvados/packages/$target"
 
 dpkg-query --show > "$ARV_PACKAGES_DIR/$1.before"
 
 apt-get -qq update
-apt-get --assume-yes --force-yes install $1
+apt-get --assume-yes --force-yes install "$1"
 
 dpkg-query --show > "$ARV_PACKAGES_DIR/$1.after"
 
@@ -22,7 +22,7 @@ set -e
 mkdir -p /tmp/opts
 cd /tmp/opts
 
-export ARV_PACKAGES_DIR=/arvados/packages/$target
+export ARV_PACKAGES_DIR="/arvados/packages/$target"
 
 dpkg-deb -x $ARV_PACKAGES_DIR/"$1"_*.deb .
 
@@ -34,4 +34,4 @@ done <<EOF
 $(find -name '*.so')
 EOF
 
-exec /jenkins/package-testing/common-test-packages.sh $1
+exec /jenkins/package-testing/common-test-packages.sh "$1"
diff --git a/jenkins/package-testing/test-package-arvados-api-server.sh b/jenkins/package-testing/test-package-arvados-api-server.sh
index 64494aa..e975448 100755
--- a/jenkins/package-testing/test-package-arvados-api-server.sh
+++ b/jenkins/package-testing/test-package-arvados-api-server.sh
@@ -17,4 +17,4 @@ case "$TARGET" in
         ;;
 esac
 
-/usr/local/rvm/bin/rvm-exec default bundle list >$ARV_PACKAGES_DIR/arados-api-server.gems
+/usr/local/rvm/bin/rvm-exec default bundle list >"$ARV_PACKAGES_DIR/arvados-api-server.gems"
diff --git a/jenkins/package-testing/test-package-arvados-sso-server.sh b/jenkins/package-testing/test-package-arvados-sso-server.sh
index 28c5618..c1a377e 100755
--- a/jenkins/package-testing/test-package-arvados-sso-server.sh
+++ b/jenkins/package-testing/test-package-arvados-sso-server.sh
@@ -104,7 +104,7 @@ if [[ "$FORMAT" == "deb" ]]; then
   dpkg-reconfigure arvados-sso-server || EXITCODE=3
 
   cd /var/www/arvados-sso/current/
-  /usr/local/rvm/bin/rvm-exec default bundle list >$ARV_PACKAGES_DIR/arvados-sso-server.gems
+  /usr/local/rvm/bin/rvm-exec default bundle list >"$ARV_PACKAGES_DIR/arvados-sso-server.gems"
 
   # Test 3: the package should remove cleanly
   apt-get remove arvados-sso-server --yes || EXITCODE=3
diff --git a/jenkins/package-testing/test-package-arvados-workbench.sh b/jenkins/package-testing/test-package-arvados-workbench.sh
index e255786..1be4dea 100755
--- a/jenkins/package-testing/test-package-arvados-workbench.sh
+++ b/jenkins/package-testing/test-package-arvados-workbench.sh
@@ -17,4 +17,4 @@ case "$TARGET" in
         ;;
 esac
 
-/usr/local/rvm/bin/rvm-exec default bundle list >$ARV_PACKAGES_DIR/arvados-workbench.gems
+/usr/local/rvm/bin/rvm-exec default bundle list >"$ARV_PACKAGES_DIR/arvados-workbench.gems"
diff --git a/jenkins/package-testing/test-packages-centos6.sh b/jenkins/package-testing/test-packages-centos6.sh
index ad80564..1960944 100755
--- a/jenkins/package-testing/test-packages-centos6.sh
+++ b/jenkins/package-testing/test-packages-centos6.sh
@@ -7,14 +7,14 @@ touch /var/lib/rpm/*
 
 export ARV_PACKAGES_DIR=/arvados/packages/centos6
 
-rpm -qa | sort > $ARV_PACKAGES_DIR/$1.before
+rpm -qa | sort > "$ARV_PACKAGES_DIR/$1.before"
 
 yum install --assumeyes $1
 
-rpm -qa | sort > $ARV_PACKAGES_DIR/$1.after
+rpm -qa | sort > "$ARV_PACKAGES_DIR/$1.after"
 
 set +e
-diff $ARV_PACKAGES_DIR/$1.before $ARV_PACKAGES_DIR/$1.after > $ARV_PACKAGES_DIR/$1.diff
+diff "$ARV_PACKAGES_DIR/$1.before" "$ARV_PACKAGES_DIR/$1.after" >"$ARV_PACKAGES_DIR/$1.diff"
 set -e
 
 SCL=""
@@ -25,7 +25,7 @@ fi
 mkdir -p /tmp/opts
 cd /tmp/opts
 
-rpm2cpio $ARV_PACKAGES_DIR/$1-*.rpm | cpio -idm 2>/dev/null
+rpm2cpio "$ARV_PACKAGES_DIR/$1"-*.rpm | cpio -idm 2>/dev/null
 
 shared=$(find -name '*.so')
 if test -n "$shared" ; then
@@ -38,7 +38,7 @@ if test -n "$shared" ; then
 fi
 
 if test -n "$SCL" ; then
-    exec $SCL "/jenkins/package-testing/common-test-packages.sh $1"
+    exec $SCL "/jenkins/package-testing/common-test-packages.sh '$1'"
 else
-    exec /jenkins/package-testing/common-test-packages.sh $1
+    exec /jenkins/package-testing/common-test-packages.sh "$1"
 fi

-----------------------------------------------------------------------


hooks/post-receive
--