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 <[email protected]> Gerrit-Reviewer: Harald Welte <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: Neels Hofmeyr <[email protected]> Gerrit-Reviewer: Pau Espin Pedrol <[email protected]> Gerrit-Reviewer: lynxis lazus <[email protected]>
