Fix two more problems building with Python3 09/24609/3
authorMats Wichmann <mats@linux.com>
Sat, 7 Apr 2018 23:46:27 +0000 (17:46 -0600)
committerMats Wichmann <mats@linux.com>
Tue, 10 Apr 2018 13:06:38 +0000 (13:06 +0000)
The boost script used string.replace() which is no longer
supported; problem did not turn up until trying a build on Windows

The security - provisioning - unittest script had a Windows
stanza for killing running server processes before launching a
new one. It printed the process id it was going to kill in a way
that caused an AttributError exception - this was a conversion
error from when print statement was changed to print function,
again the code path had not been hit. At the same time a second
exception ocurred because when the first exception took place the
script attempted to us the message attribute of the exception for
printing, but not all exceptions have such an attribute. Change
to just print the string representation of the exception, which
is more reliable.

Additionally, in the libcoap script, the term 'string' is used
which while not broken since the script never imports the string
module, is unnecessarily ambiguous. Use the Python recommended
disambiguation of adding a trailing underscore on an identifier
which might match one used by Python.

Change-Id: I635796a6442caacb6988e21dbe46938f860e333f
Signed-off-by: Mats Wichmann <mats@linux.com>
extlibs/boost/SConscript
extlibs/libcoap/SConscript
resource/csdk/security/provisioning/unittest/SConscript

index b06af6b..cec2d18 100644 (file)
@@ -3,7 +3,7 @@
 # get and install
 #
 
-import os, string, sys, subprocess
+import os, sys, subprocess
 import SCons.Errors
 
 Import('env')
@@ -22,7 +22,7 @@ boost_version = '1.60.0'
 if target_os in ['android']:
     boost_version = '1.58.0'
 
-boost_base_name = 'boost_' + string.replace(boost_version, '.', '_')
+boost_base_name = 'boost_' + boost_version.replace('.', '_')
 boost_arch_name = boost_base_name + '.zip'
 boost_b2_name = boost_base_name + os.sep + 'b2'
 boost_url = 'http://downloads.sourceforge.net/project/boost/boost/' + boost_version + '/' + boost_arch_name + '?r=&ts=1421801329&use_mirror=iweb'
index be4b6d9..03845cb 100644 (file)
@@ -251,8 +251,8 @@ if with_upstream_libcoap == '1':
                 'flsll',
             ]
 
-            def get_define_from_string(string):
-                string_converted = string.replace("/", "_").replace(".", "_").upper()
+            def get_define_from_string(string_):
+                string_converted = string_.replace("/", "_").replace(".", "_").upper()
                 return "HAVE_" + string_converted
 
             for header_file_name in cxx_headers:
index 18b1ff3..eb550bb 100644 (file)
@@ -104,13 +104,13 @@ def kill_all():
                 if not cmd_list[1].isdigit():
                     continue
                 try:
+                    print("killing:" + cmd_list[1])
                     pid = int(cmd_list[1])
-                    print("killing: " + pid)
                     kill(pid, SIGTERM)
                 except Exception as ex:
                     print(line)
                     print("Cannot kill: " + cmd_list[1])
-                    print("exception: " + ex.message)
+                    print("exception: " + str(ex))
     else:
         try:
             from psutil import process_iter