Browse Source

Avoid tox_install.sh for constraints support

We do not need tox_install.sh, pip can handle constraints itself
and install the project correctly. Thus update tox.ini and remove
the now obsolete tools/tox_install.sh file.

This follows https://review.openstack.org/#/c/508061 to remove
tools/tox_install.sh.

Change-Id: I2310cc11d5d0b53b42d2deb34111dfaaac34e911
Andreas Jaeger 1 year ago
parent
commit
dba909fd35
2 changed files with 5 additions and 36 deletions
  1. 0
    30
      tools/tox_install.sh
  2. 5
    6
      tox.ini

+ 0
- 30
tools/tox_install.sh View File

@@ -1,30 +0,0 @@
1
-#!/usr/bin/env bash
2
-
3
-# Client constraint file contains this client version pin that is in conflict
4
-# with installing the client from source. We should remove the version pin in
5
-# the constraints file before applying it for from-source installation.
6
-
7
-CONSTRAINTS_FILE="$1"
8
-shift 1
9
-
10
-set -e
11
-
12
-# NOTE(tonyb): Place this in the tox enviroment's log dir so it will get
13
-# published to logs.openstack.org for easy debugging.
14
-localfile="$VIRTUAL_ENV/log/upper-constraints.txt"
15
-
16
-if [[ "$CONSTRAINTS_FILE" != http* ]]; then
17
-    CONSTRAINTS_FILE="file://$CONSTRAINTS_FILE"
18
-fi
19
-# NOTE(tonyb): need to add curl to bindep.txt if the project supports bindep
20
-curl "$CONSTRAINTS_FILE" --insecure --progress-bar --output "$localfile"
21
-
22
-pip install -c"$localfile" openstack-requirements
23
-
24
-# This is the main purpose of the script: Allow local installation of
25
-# the current repo. It is listed in constraints file and thus any
26
-# install will be constrained and we need to unconstrain it.
27
-edit-constraints "$localfile" -- "$CLIENT_NAME"
28
-
29
-pip install -c"$localfile" -U "$@"
30
-exit $?

+ 5
- 6
tox.ini View File

@@ -3,12 +3,11 @@ minversion = 2.0
3 3
 envlist = py35,py27,pypy,pep8
4 4
 
5 5
 [testenv]
6
-setenv =
7
-    VIRTUAL_ENV={envdir}
8
-    BRANCH_NAME=master
9
-    CLIENT_NAME=oslo.context
10
-install_command = {toxinidir}/tools/tox_install.sh {env:UPPER_CONSTRAINTS_FILE:https://git.openstack.org/cgit/openstack/requirements/plain/upper-constraints.txt} {opts} {packages}
11
-deps = -r{toxinidir}/test-requirements.txt
6
+install_command = pip install -U {opts} {packages}
7
+deps =
8
+  -c{env:UPPER_CONSTRAINTS_FILE:https://git.openstack.org/cgit/openstack/requirements/plain/upper-constraints.txt}
9
+ -r{toxinidir}/test-requirements.txt
10
+ -r{toxinidir}/requirements.txt
12 11
 commands = python setup.py testr --slowest --testr-args='{posargs}'
13 12
 
14 13
 [testenv:pep8]

Loading…
Cancel
Save