openstack/monasca-persister
Code Issues Proposed changes

Sync tools/tox_install.sh

Browse Source 
Sync tools/tox_install.sh with python-openstackclient. This brings in
the following improvements:

* Do not leave temporary directory around, instead delete temporary
  directory directly after usage (change
  I939eae82dba3287fd4e4086128ebf4609a0e0770).
* Do not set ZUUL_BRANCH explicitely and remove unused if condition
  (change I0077c986a17d6bb92791474e03d1e77776e9382f).

Change-Id: I89b86e20d063f53338c87e76d9a39d600d35f3cd
This commit is contained in:
Andreas Jaeger 2016-08-30 20:17:06 +02:00
parent 0529f0ca3a
commit a9b647d743
1 changed files with 34 additions and 34 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
tools/tox_install.sh
View File

@ -17,9 +17,8 @@ CONSTRAINTS_FILE=$1
shift
install_cmd="pip install"
if [[ $CONSTRAINTS_FILE != "unconstrained" ]]; then
mydir=$(mktemp -dt "$PACKAGE_NAME-tox_install-XXXXXXX")
trap "rm -rf $mydir" EXIT
localfile=$mydir/upper-constraints.txt
if [[ $CONSTRAINTS_FILE != http* ]]; then
CONSTRAINTS_FILE=file://$CONSTRAINTS_FILE
@ -31,7 +30,6 @@ if [[ $CONSTRAINTS_FILE != "unconstrained" ]]; then
echo "ALREADY INSTALLED" > /tmp/tox_install.txt
echo "Requirements already installed; using existing package"
elif [ -x "$ZUUL_CLONER" ]; then
export ZUUL_BRANCH=${ZUUL_BRANCH-$BRANCH}
echo "ZUUL CLONER" > /tmp/tox_install.txt
pushd $mydir
$ZUUL_CLONER --cache-dir \
@ -50,8 +48,10 @@ if [[ $CONSTRAINTS_FILE != "unconstrained" ]]; then
$install_cmd -U -e ${REQUIREMENTS_PIP_LOCATION}
fi
# This is the main purpose of the script: Allow local installation of
# the current repo. It is listed in constraints file and thus any
# install will be constrained and we need to unconstrain it.
edit-constraints $localfile -- $PACKAGE_NAME "-e file://$PWD#egg=$PACKAGE_NAME"
fi
$install_cmd -U $*
exit $?