From ad10c32a89b770c08e8c21f65d2ddc961e1e6d48 Mon Sep 17 00:00:00 2001 From: wangxiyuan Date: Fri, 20 Apr 2018 16:43:19 +0800 Subject: [PATCH] Fix the doc CI failure CI job now uses pip >= 10.0.0 to install the package. In pip >= 10.0.0, if the package name is missing, the command "pip install -U" returns error, but in pip < 10.0.0 it just logs warning. So this patch make sure the package name exists first. Change-Id: Id900640a7133f837ece8c6c0e7d2e4c17665a53d --- tools/tox_install.sh | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/tools/tox_install.sh b/tools/tox_install.sh index e61b63a8..f3423d95 100755 --- a/tools/tox_install.sh +++ b/tools/tox_install.sh @@ -26,5 +26,7 @@ pip install -c"$localfile" openstack-requirements # install will be constrained and we need to unconstrain it. edit-constraints "$localfile" -- "$CLIENT_NAME" -pip install -c"$localfile" -U "$@" +if [[ -n "$@" ]]; then + pip install -c"$localfile" -U "$@" +fi exit $?