[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
Mon Dec 18 14:38: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 (#2).

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/2

diff --git a/contrib/jenkins.sh b/contrib/jenkins.sh
index 39e502c..e04d7e8 100755
--- a/contrib/jenkins.sh
+++ b/contrib/jenkins.sh
@@ -6,7 +6,6 @@
 python2 ./setup.py install
 python2 tests/test_py2.py
 python2 -m compileall osmopy
-python2 -m compileall scripts/osmo_ctrl.py
 python2 -m compileall scripts/osmodumpdoc.py
 python2 -m compileall scripts/osmotestvty.py
 python2 -m compileall scripts/osmotestconfig.py
@@ -27,4 +26,7 @@
 python3 -m compileall scripts/soap.py
 python3 -m compileall 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 455650a..bbb4b93 100755
--- a/setup.py
+++ b/setup.py
@@ -28,6 +28,7 @@
 	           "scripts/osmotestvty.py",
                    "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: 2
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