From 1e96d54d9f928c4ea4bf0564ef9900f6bd03acd5 Mon Sep 17 00:00:00 2001 From: gunan Date: Tue, 12 Sep 2017 00:47:55 -0700 Subject: [PATCH] Also accept non-k8 CPU types in build pip package. (#12975) * Also accept non-k8 CPU types in build pip package. Fixes #12735 * Make the script work with `set -e`. --- .../tools/pip_package/build_pip_package.sh | 26 +++++++++---------- 1 file changed, 13 insertions(+), 13 deletions(-) diff --git a/tensorflow/tools/pip_package/build_pip_package.sh b/tensorflow/tools/pip_package/build_pip_package.sh index f48fdcc9ec9..cbf06a97d02 100755 --- a/tensorflow/tools/pip_package/build_pip_package.sh +++ b/tensorflow/tools/pip_package/build_pip_package.sh @@ -98,6 +98,7 @@ function main() { "${TMPDIR}/external" RUNFILES=bazel-bin/tensorflow/tools/pip_package/simple_console_for_window_unzip/runfiles/org_tensorflow else + RUNFILES=bazel-bin/tensorflow/tools/pip_package/build_pip_package.runfiles/org_tensorflow if [ -d bazel-bin/tensorflow/tools/pip_package/build_pip_package.runfiles/org_tensorflow/external ]; then # Old-style runfiles structure (--legacy_external_runfiles). cp -R \ @@ -108,12 +109,12 @@ function main() { bazel-bin/tensorflow/tools/pip_package/build_pip_package.runfiles/org_tensorflow/external \ "${TMPDIR}/external" # Copy MKL libs over so they can be loaded at runtime - so_lib_dir="bazel-bin/tensorflow/tools/pip_package/build_pip_package.runfiles/org_tensorflow/_solib_k8" - if [ -d ${so_lib_dir} ]; then - mkl_so_dir=$(ls ${so_lib_dir} | grep mkl) - if [ $? -eq 0 ]; then - mkdir "${TMPDIR}/_solib_k8" - cp -R ${so_lib_dir}/${mkl_so_dir} "${TMPDIR}/_solib_k8" + so_lib_dir=$(ls $RUNFILES | grep solib) || true + if [ -n "${so_lib_dir}" ]; then + mkl_so_dir=$(ls ${RUNFILES}/${so_lib_dir} | grep mkl) || true + if [ -n "${mkl_so_dir}" ]; then + mkdir "${TMPDIR}/${so_lib_dir}" + cp -R ${RUNFILES}/${so_lib_dir}/${mkl_so_dir} "${TMPDIR}/${so_lib_dir}" fi fi else @@ -127,16 +128,15 @@ function main() { bazel-bin/tensorflow/tools/pip_package/build_pip_package.runfiles \ "${TMPDIR}/external" # Copy MKL libs over so they can be loaded at runtime - so_lib_dir="bazel-bin/tensorflow/tools/pip_package/build_pip_package.runfiles/org_tensorflow/_solib_k8" - if [ -d ${so_lib_dir} ]; then - mkl_so_dir=$(ls ${so_lib_dir} | grep mkl) - if [ $? -eq 0 ]; then - mkdir "${TMPDIR}/_solib_k8" - cp -R ${so_lib_dir}/${mkl_so_dir} "${TMPDIR}/_solib_k8" + so_lib_dir=$(ls $RUNFILES | grep solib) || true + if [ -n "${so_lib_dir}" ]; then + mkl_so_dir=$(ls ${RUNFILES}/${so_lib_dir} | grep mkl) || true + if [ -n "${mkl_so_dir}" ]; then + mkdir "${TMPDIR}/${so_lib_dir}" + cp -R ${RUNFILES}/${so_lib_dir}/${mkl_so_dir} "${TMPDIR}/${so_lib_dir}" fi fi fi - RUNFILES=bazel-bin/tensorflow/tools/pip_package/build_pip_package.runfiles/org_tensorflow fi # protobuf pip package doesn't ship with header files. Copy the headers