[ARVADOS] updated: aca461985963f60c185a02c53377d75eebf26266

Git user git at public.curoverse.com
Wed Jun 15 12:59:15 EDT 2016


Summary of changes:
 build/run-build-packages.sh | 1 +
 1 file changed, 1 insertion(+)

       via  aca461985963f60c185a02c53377d75eebf26266 (commit)
      from  e47a01ee19ee44b8004bab7ed7eec5bbf72f175e (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 aca461985963f60c185a02c53377d75eebf26266
Author: Brett Smith <brett at curoverse.com>
Date:   Wed Jun 15 12:58:31 2016 -0400

    Package build directories get same owner as $WORKSPACE.
    
    Since this script runs as root inside the Docker container, the
    directory created on the previous line is normally owned by root too.
    This causes trouble when the package tests run later and try to create
    repository information inside the directory.  See, e.g.,
    <https://ci.curoverse.com/job/build-packages-sso/23/consoleFull> for a
    build that failed because of this.
    
    No issue #.

diff --git a/build/run-build-packages.sh b/build/run-build-packages.sh
index f07b942..0277094 100755
--- a/build/run-build-packages.sh
+++ b/build/run-build-packages.sh
@@ -238,6 +238,7 @@ debug_echo "umask is" `umask`
 
 if [[ ! -d "$WORKSPACE/packages/$TARGET" ]]; then
   mkdir -p $WORKSPACE/packages/$TARGET
+  chown --reference="$WORKSPACE" "$WORKSPACE/packages/$TARGET"
 fi
 
 # Perl packages

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


hooks/post-receive
-- 




More information about the arvados-commits mailing list