fixeria has submitted this change. ( 
https://gerrit.osmocom.org/c/python/osmo-python-tests/+/37540?usp=email )

Change subject: contrib/jenkins.sh: use virtualenv and pip
......................................................................

contrib/jenkins.sh: use virtualenv and pip

setup.py install is deprecated:

https://packaging.python.org/en/latest/discussions/setup-py-deprecated/

Change-Id: I42ba71e2fd0ceb8466758ea46e9ae33e3d4fcb91
---
M contrib/jenkins.sh
1 file changed, 18 insertions(+), 9 deletions(-)

Approvals:
  pespin: Looks good to me, but someone else must approve
  osmith: Looks good to me, approved
  Jenkins Builder: Verified




diff --git a/contrib/jenkins.sh b/contrib/jenkins.sh
index 810cb7e..916a537 100755
--- a/contrib/jenkins.sh
+++ b/contrib/jenkins.sh
@@ -2,16 +2,12 @@

 set -ex

-COM_FLAGS='-m compileall'
-
-rm -rf ./build
 PY3=python3
-PY3_LIST="osmopy scripts/osmo_ctrl.py scripts/osmo_rate_ctr2csv.py 
scripts/osmodumpdoc.py scripts/osmotestvty.py scripts/osmotestconfig.py 
scripts/osmo_interact_ctrl.py scripts/osmo_interact_vty.py 
scripts/osmo_verify_transcript_ctrl.py scripts/osmo_verify_transcript_vty.py 
scripts/soap.py scripts/twisted_ipa.py"
-$PY3 ./setup.py install --user
-for f in $PY3_LIST
-do
-    $PY3 $COM_FLAGS $f
-done
+
+rm -rf ./env
+virtualenv --system-site-packages env
+. ./env/bin/activate
+pip install .

 # Run async server which tests scripts/osmo_ctrl.py interaction
 $PY3 tests/test_py3.py

--
To view, visit 
https://gerrit.osmocom.org/c/python/osmo-python-tests/+/37540?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: python/osmo-python-tests
Gerrit-Branch: master
Gerrit-Change-Id: I42ba71e2fd0ceb8466758ea46e9ae33e3d4fcb91
Gerrit-Change-Number: 37540
Gerrit-PatchSet: 2
Gerrit-Owner: fixeria <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <[email protected]>
Gerrit-Reviewer: osmith <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>
Gerrit-MessageType: merged

Reply via email to