Skip to content

Commit 30628d6

Browse files
author
Vijay Vasudevan
committed
Merge pull request tensorflow#1177 from vrv/master.
2 parents e4ebc45 + cd35bb1 commit 30628d6

File tree

1 file changed

+6
-3
lines changed
  • tensorflow/tools/ci_build/builds

1 file changed

+6
-3
lines changed

tensorflow/tools/ci_build/builds/pip.sh

+6-3
Original file line numberDiff line numberDiff line change
@@ -125,7 +125,8 @@ echo "Python binary path to be used in PIP install-test: ${PYTHON_BIN_PATH} "\
125125
PIP_WHL_DIR="pip_test/whl"
126126
PIP_WHL_DIR=$(abs_path ${PIP_WHL_DIR}) # Get absolute path
127127
rm -rf ${PIP_WHL_DIR} && mkdir -p ${PIP_WHL_DIR}
128-
bazel-bin/tensorflow/tools/pip_package/build_pip_package ${PIP_WHL_DIR} &&
128+
bazel-bin/tensorflow/tools/pip_package/build_pip_package ${PIP_WHL_DIR} || \
129+
die "build_pip_package FAILED"
129130

130131
# Perform installation
131132
WHL_PATH=$(ls ${PIP_WHL_DIR}/tensorflow*.whl)
@@ -141,8 +142,10 @@ echo "Installing pip whl file: ${WHL_PATH}"
141142

142143
# Call pip install twice, first time with --upgrade and second time without it
143144
# This addresses the sporadic test failures related to protobuf version
144-
${PYTHON_BIN_PATH} -m pip install -v --user --upgrade ${WHL_PATH} numpy==1.8.2 &&
145-
${PYTHON_BIN_PATH} -m pip install -v --user ${WHL_PATH} &&
145+
${PYTHON_BIN_PATH} -m pip install -v --user --upgrade ${WHL_PATH} numpy==1.8.2 \
146+
|| die "pip install (1st step, with --upgrade) FAILED"
147+
${PYTHON_BIN_PATH} -m pip install -v --user ${WHL_PATH} \
148+
|| die "pip install (2nd step, without --upgrade) FAILED"
146149

147150
# If NO_TEST_ON_INSTALL is set to any non-empty value, skip all Python
148151
# tests-on-install and exit right away

0 commit comments

Comments
 (0)