• crlishka's avatar
    Update contrib/docker builds with features for ngraph-tensorflow integration builds (#195) · c6ff47d0
    crlishka authored
    * Refactored to run builds as user, not as root.  Added make install.  Added parallel build.  Expanded make clean to remove BUILD directory.  Moved log generation to makefile, so Jenkins job can let make do all of the work.
    
    * Changes/fixes to address code review.  Mainly adding double-quotes around variables and using $(...) expansion where appropriate (and semantically identical to back-quotes).
    
    * Add double-quotes for variable expansion safety.  Write error message in run_as_user.sh to stderr.
    
    * Remove chown_files.sh, made obsolete by running as user inside docker container.  Add set -e and set -u to run_as_user.sh, for further protection.
    c6ff47d0
run_as_user.sh 3.32 KB