[PATCH] 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/.

Max gerrit-no-reply at lists.osmocom.org
Mon Dec 18 12:13:02 UTC 2017


Hello Harald Welte, Jenkins Builder,

I'd like you to reexamine a change.  Please visit

    https://gerrit.osmocom.org/5481

to look at the new patch set (#3).

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(-)


  git pull ssh://gerrit.osmocom.org:29418/python/osmo-python-tests refs/changes/81/5481/3

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: newpatchset
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



More information about the gerrit-log mailing list