[PATCH] python/osmo-python-tests[master]: Use python3 for osmo_ctrl.py

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
Thu Dec 21 13:23:01 UTC 2017


Hello 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>



More information about the gerrit-log mailing list