Updates to /tools for Python 3 compatibility

Author: madchutney

tools/export/exporters.py

@@ -28,6 +28,7 @@
 from tools.targets import TARGET_MAP
 from tools.utils import mkdir
 from tools.resources import FileType, FileRef
+from future.utils import with_metaclass
 
 """Just a template for subclassing"""
 
@@ -57,14 +58,13 @@ def __init__(*args, **kwargs):
     CLS.NAME = "%s (DEPRECATED)" % old_name
     return CLS
 
-class Exporter(object):
+class Exporter(with_metaclass(ABCMeta, object)):
     """Exporter base class
 
     This class is meant to be extended by individual exporters, and provides a
     few helper methods for implementing an exporter with either jinja2 or
     progen.
     """
-    __metaclass__ = ABCMeta
     TEMPLATE_DIR = dirname(__file__)
     DOT_IN_RELATIVE_PATH = False
     NAME = None

tools/flash_algo/__init__.py

@@ -134,7 +134,7 @@ def format_algo_data(self, spaces, group_size, fmt):
             blob = self.algo_data[:]
             pad_size = 0 if len(blob) % 4 == 0 else 4 - len(blob) % 4
             blob = blob + "\x00" * pad_size
-            integer_list = struct.unpack("<" + "L" * (len(blob) / 4), blob)
+            integer_list = struct.unpack("<" + "L" * (len(blob) // 4), blob)
             line_list = []
             for pos in range(0, len(integer_list), group_size):
                 group = ["0x%08x" % value for value in

tools/host_tests/echo_flow_control.py

@@ -14,7 +14,7 @@
 See the License for the specific language governing permissions and
 limitations under the License.
 """
-from host_test import Test
+from .host_test import Test
 
 
 class EchoTest(Test):

tools/host_tests/hello_auto.py

@@ -16,6 +16,7 @@
 """
 
 class HelloTest():
+class HelloTest(object):
     HELLO_WORLD = "Hello World"
 
     def test(self, selftest):

tools/host_tests/host_registry.py

@@ -15,7 +15,7 @@
 limitations under the License.
 """
 
-class HostRegistry:
+class HostRegistry(object):
     """ Class stores registry with host tests and objects representing them
     """ 
     HOST_TESTS = {} # host_test_name -> host_test_ojbect

tools/host_tests/host_test.py

@@ -18,8 +18,8 @@
 # Check if 'serial' module is installed
 try:
     from serial import Serial
-except ImportError, e:
-    print "Error: Can't import 'serial' module: %s"% e
+except ImportError as e:
+    print("Error: Can't import 'serial' module: %s"% e)
     exit(-1)
 
 import os
@@ -29,7 +29,7 @@
 from time import sleep, time
 from optparse import OptionParser
 
-import host_tests_plugins
+from . import host_tests_plugins
 
 # This is a little tricky. We need to add upper directory to path so
 # we can find packages we want from the same level as other files do
@@ -39,7 +39,7 @@
 from tools.test_api import get_module_avail
 
 
-class Mbed:
+class Mbed(object):
     """ Base class for a host driven test
     """
     def __init__(self):
@@ -117,7 +117,7 @@ def __init__(self):
         self.serial_timeout = 1
 
         self.timeout = self.DEFAULT_TOUT if self.options.timeout is None else self.options.timeout
-        print 'MBED: Instrumentation: "%s" and disk: "%s"' % (self.port, self.disk)
+        print('MBED: Instrumentation: "%s" and disk: "%s"' % (self.port, self.disk))
 
     def init_serial_params(self, serial_baud=9600, serial_timeout=1):
         """ Initialize port parameters.
@@ -183,11 +183,11 @@ def pool_for_serial_init(self, serial_baud, serial_timeout, pooling_loops=40, in
                 stdout.write('.')
                 stdout.flush()
             else:
-                print "...port ready!"
+                print("...port ready!")
                 result = True
                 break
         if not result and last_error:
-            print last_error
+            print(last_error)
         return result
 
     def set_serial_timeout(self, timeout):
@@ -221,7 +221,7 @@ def serial_readline(self, timeout=5):
                     c = self.serial.read(1)
                     result += c
                 except Exception as e:
-                    print "MBED: %s"% str(e)
+                    print("MBED: %s"% str(e))
                     result = None
                     break
                 if c == '\n':
@@ -298,7 +298,7 @@ def flush(self):
         return result
 
 
-class HostTestResults:
+class HostTestResults(object):
     """ Test results set by host tests
     """
     def __init__(self):
@@ -389,8 +389,8 @@ def run(self):
                 self.print_result(result)
             else:
                 self.notify("HOST: Passive mode...")
-        except Exception, e:
-            print str(e)
+        except Exception as e:
+            print(str(e))
             self.print_result(self.RESULT_ERROR)
 
     def setup(self):
@@ -406,7 +406,7 @@ def setup(self):
     def notify(self, message):
         """ On screen notification function
         """
-        print message
+        print(message)
         stdout.flush()
 
     def print_result(self, result):

tools/host_tests/mbedrpc.py

@@ -27,10 +27,12 @@
 # >myled.write(1)
 # >
 
-import serial, urllib2, time
+from future import standard_library
+standard_library.install_aliases()
+import serial, urllib.request, urllib.error, urllib.parse, time
 
 # mbed super class
-class mbed:
+class mbed(object):
     def __init__(self):
         print("This will work as a demo but no transport mechanism has been selected")
 
@@ -48,7 +50,7 @@ def rpc(self, name, method, args):
         # creates the command to be sent serially - /name/method arg1 arg2 arg3 ... argN
         str = "/" + name + "/" + method + " " + " ".join(args) + "\n"
         # prints the command being executed
-        print str
+        print(str)
         # writes the command to serial
         self.ser.write(str)
         # strips trailing characters from the line just written
@@ -61,12 +63,12 @@ def __init__(self, ip):
         self.host = "http://" + ip
 
     def rpc(self, name, method, args):
-        response = urllib2.urlopen(self.host + "/rpc/" + name + "/" + method + "%20" + "%20".join(args))
+        response = urllib.request.urlopen(self.host + "/rpc/" + name + "/" + method + "%20" + "%20".join(args))
         return response.read().strip()
 
 
 # generic mbed interface super class
-class mbed_interface():
+class mbed_interface(object):
     # initialize an mbed interface with a transport mechanism and pin name
     def __init__(self, this_mbed, mpin):
         self.mbed = this_mbed
@@ -198,7 +200,7 @@ def read_us(self):
         return float(re.search('\d+\.*\d*', r).group(0))
 
 # Serial
-class Serial():
+class Serial(object):
     def __init__(self, this_mbed, tx, rx=""):
         self.mbed = this_mbed
         if isinstance(tx, str):

tools/host_tests/net_test.py

@@ -14,7 +14,7 @@
 See the License for the specific language governing permissions and
 limitations under the License.
 """
-from host_test import Test, Simple
+from .host_test import Test, Simple
 from sys import stdout
 
 class NETTest(Simple):

tools/host_tests/rpc.py

@@ -14,8 +14,8 @@
 See the License for the specific language governing permissions and
 limitations under the License.
 """
-from host_test import Test
-from mbedrpc import SerialRPC, DigitalOut, DigitalIn, pin
+from .host_test import Test
+from .mbedrpc import SerialRPC, DigitalOut, DigitalIn, pin
 
 
 class RpcTest(Test):
@@ -30,7 +30,7 @@ def test(self):
 
         if hasattr(self.mbed.options, 'micro'):
             if self.mbed.options.micro == 'M0+':
-                print "Freedom Board: PTA12 <-> PTC4"
+                print("Freedom Board: PTA12 <-> PTC4")
                 p_out = pin("PTA12")
                 p_in  = pin("PTC4")
 

tools/host_tests/serial_complete_auto.py

@@ -21,7 +21,7 @@
 import string
 from sys import stdout
 
-class SerialCompleteTest():
+class SerialCompleteTest(object):
 
     def test(self, selftest):
         strip_chars = string.whitespace + "\0"