[ARVADOS-DEV] updated: 2fe4a6a068e720e72b1695de133b4363e9a69a40

[git at public.curoverse.com]

Summary of changes:
 jenkins/package-test-dockerfiles/centos6/Dockerfile |  4 ++++
 jenkins/run-build-packages-one-target.sh            |  4 +++-
 jenkins/run-library.sh                              | 16 ++++++++++------
 3 files changed, 17 insertions(+), 7 deletions(-)

       via  2fe4a6a068e720e72b1695de133b4363e9a69a40 (commit)
      from  ebdbb0bb23ceb1d1373472e6405fa2293f51616b (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 2fe4a6a068e720e72b1695de133b4363e9a69a40
Author: Peter Amstutz <peter.amstutz at curoverse.com>
Date:   Wed Jan 13 16:41:39 2016 -0500

    Fixing our fpm rails package building to handle fpm-info.sh.

diff --git a/jenkins/package-test-dockerfiles/centos6/Dockerfile b/jenkins/package-test-dockerfiles/centos6/Dockerfile
index 0020d5f..6870ec3 100644
--- a/jenkins/package-test-dockerfiles/centos6/Dockerfile
+++ b/jenkins/package-test-dockerfiles/centos6/Dockerfile
@@ -12,4 +12,8 @@ RUN touch /var/lib/rpm/* && \
     /usr/local/rvm/bin/rvm alias create default ruby-2.1 && \
     /usr/local/rvm/bin/rvm-exec default gem install bundle fpm
 
+RUN cd / root && \
+    curl -OL 'http://pkgs.repoforge.org/rpmforge-release/rpmforge-release-0.5.3-1.el6.rf.x86_64.rpm' && \
+    rpm -ivh rpmforge-release-0.5.3-1.el6.rf.x86_64.rpm
+
 COPY localrepo.repo /etc/yum.repos.d/localrepo.repo
\ No newline at end of file
diff --git a/jenkins/run-build-packages-one-target.sh b/jenkins/run-build-packages-one-target.sh
index ad46452..af6e078 100755
--- a/jenkins/run-build-packages-one-target.sh
+++ b/jenkins/run-build-packages-one-target.sh
@@ -21,6 +21,8 @@ WORKSPACE=path         Path to the Arvados source tree to build packages from
 
 EOF
 
+set -e
+
 if ! [[ -n "$WORKSPACE" ]]; then
   echo >&2 "$helpmessage"
   echo >&2
@@ -131,7 +133,7 @@ if test -z "$packages" ; then
         keep-web
         libarvados-perl"
 
-    if test $target == centos6 ; then
+    if test "$target" = centos6 ; then
         packages="$packages python27-python-arvados-fuse
                   python27-python-arvados-python-client"
     else
diff --git a/jenkins/run-library.sh b/jenkins/run-library.sh
index 623e844..0579814 100755
--- a/jenkins/run-library.sh
+++ b/jenkins/run-library.sh
@@ -235,18 +235,17 @@ fpm_build () {
   # that will take precedence, as desired.
   COMMAND_ARR+=(--iteration "$(default_iteration "$PACKAGE" "$VERSION")")
 
-  # Append remaining function arguments directly to fpm's command line.
-  for i; do
-    COMMAND_ARR+=("$i")
-  done
+  # 'dir' type packages are provided in the form /path/to/source=/path/to/dest
+  # so strip off the 2nd part to check for fpm-info below.
+  PACKAGE_DIR=$(echo $PACKAGE | sed 's/\/=.*//')
 
   # Append --depends X and other arguments specified by fpm-info.sh in
   # the package source dir. These are added last so they can override
   # the arguments added by this script.
   declare -a fpm_args=()
   declare -a fpm_depends=()
-  if [[ -d "$PACKAGE" ]]; then
-      FPM_INFO="$PACKAGE/fpm-info.sh"
+  if [[ -d "$PACKAGE_DIR" ]]; then
+      FPM_INFO="$PACKAGE_DIR/fpm-info.sh"
   else
       FPM_INFO="${WORKSPACE}/backports/${PACKAGE_TYPE}-${PACKAGE}/fpm-info.sh"
   fi
@@ -259,6 +258,11 @@ fpm_build () {
   done
   COMMAND_ARR+=("${fpm_args[@]}")
 
+  # Append remaining function arguments directly to fpm's command line.
+  for i; do
+    COMMAND_ARR+=("$i")
+  done
+
   COMMAND_ARR+=("$PACKAGE")
 
   debug_echo -e "\n${COMMAND_ARR[@]}\n"

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


hooks/post-receive
--