[MERGED] python/osmo-python-tests[master]: Explicitly check syntax in jenkins tests

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

Harald Welte gerrit-no-reply at lists.osmocom.org
Mon Dec 18 13:32:21 UTC 2017


Harald Welte has submitted this change and it was merged.

Change subject: Explicitly check syntax in jenkins tests
......................................................................


Explicitly check syntax in jenkins tests

By default setup.py ignores compilation errorsand proceed with
installation anyway. Make sure that it fails if any errorsis encountered
bymanually call compilation step.

Change-Id: I2235e3fff21b7c09d3fba00ecdccca7f4515e071
Related: OS#2770
---
M contrib/jenkins.sh
1 file changed, 2 insertions(+), 0 deletions(-)

Approvals:
  Harald Welte: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/contrib/jenkins.sh b/contrib/jenkins.sh
index 4e51dd1..a60438e 100755
--- a/contrib/jenkins.sh
+++ b/contrib/jenkins.sh
@@ -5,9 +5,11 @@
 # FIXME: remove once python 2 support is deprecated
 python2 ./setup.py install
 python2 tests/test_py2.py
+python2 -m compileall osmopy
 
 rm -rf ./build
 python3 ./setup.py install
 python3 tests/test_py3.py
+python3 -m compileall osmopy
 
 # TODO: add more tests

-- 
To view, visit https://gerrit.osmocom.org/5481
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I2235e3fff21b7c09d3fba00ecdccca7f4515e071
Gerrit-PatchSet: 3
Gerrit-Project: python/osmo-python-tests
Gerrit-Branch: master
Gerrit-Owner: Max <msuraev at sysmocom.de>
Gerrit-Reviewer: Harald Welte <laforge at gnumonks.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr at sysmocom.de>
Gerrit-Reviewer: Pau Espin Pedrol <pespin at sysmocom.de>
Gerrit-Reviewer: lynxis lazus <lynxis at fe80.eu>



More information about the gerrit-log mailing list