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.orgHello Harald Welte, Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/5061
to look at the new patch set (#4).
Use python3 for osmo_ctrl.py
It's a standalone script illustrating the use of ctrl protocol from
python. Since it's not used as a library and nothing depends on it, we
can safely switch to python3.
Change-Id: I2461dd9af67771beed5306116e8a1b0ee2285aa8
---
M contrib/jenkins.sh
M scripts/osmo_ctrl.py
M setup.py
3 files changed, 7 insertions(+), 4 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/python/osmo-python-tests refs/changes/61/5061/4
diff --git a/contrib/jenkins.sh b/contrib/jenkins.sh
index 1598836..cd94868 100755
--- a/contrib/jenkins.sh
+++ b/contrib/jenkins.sh
@@ -9,7 +9,6 @@
$PY2 ./setup.py install
$PY2 tests/test_py2.py
$PY2 $COM_FLAGS osmopy
-$PY2 $COM_FLAGS scripts/osmo_ctrl.py
$PY2 $COM_FLAGS scripts/osmodumpdoc.py
$PY2 $COM_FLAGS scripts/osmotestvty.py
$PY2 $COM_FLAGS scripts/osmotestconfig.py
@@ -28,4 +27,7 @@
$PY3 $COM_FLAGS scripts/soap.py
$PY3 $COM_FLAGS scripts/twisted_ipa.py
+cd scripts
+./osmo_ctrl.py --help
+
# TODO: add more tests
diff --git a/scripts/osmo_ctrl.py b/scripts/osmo_ctrl.py
index bec6d1d..8c0608f 100755
--- a/scripts/osmo_ctrl.py
+++ b/scripts/osmo_ctrl.py
@@ -1,8 +1,8 @@
-#!/usr/bin/env python2
+#!/usr/bin/env python3
# -*- mode: python-mode; py-indent-tabs-mode: nil -*-
"""
/*
- * Copyright (C) 2016 sysmocom s.f.m.c. GmbH
+ * Copyright (C) 2016-2017 sysmocom s.f.m.c. GmbH
*
* All Rights Reserved
*
@@ -21,7 +21,7 @@
* 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
*/
"""
-from __future__ import print_function
+
from optparse import OptionParser
from osmopy.osmo_ipa import Ctrl
import socket
diff --git a/setup.py b/setup.py
index 533272e..bab9c38 100755
--- a/setup.py
+++ b/setup.py
@@ -25,6 +25,7 @@
elif sys.version_info.major == 3:
scripts = ["scripts/osmo_interact_vty.py",
"scripts/osmo_interact_ctrl.py",
+ "scripts/osmo_ctrl.py",
"scripts/soap.py",
"scripts/twisted_ipa.py",
"scripts/osmo_verify_transcript_vty.py",
--
To view, visit https://gerrit.osmocom.org/5061
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I2461dd9af67771beed5306116e8a1b0ee2285aa8
Gerrit-PatchSet: 4
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>