Reformat all of the plugins SConscripts to be more Pythonic 33/20533/4
authorMats Wichmann <mats@linux.com>
Fri, 2 Jun 2017 12:49:40 +0000 (06:49 -0600)
committerPhil Coval <philippe.coval@osg.samsung.com>
Mon, 12 Jun 2017 14:16:08 +0000 (14:16 +0000)
There are not intended to be any functional changes.

Change-Id: Ifb5cadc8f6de73b55d5f6fece7cb401f34b81674
Signed-off-by: Mats Wichmann <mats@linux.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/20533
Tested-by: jenkins-iotivity <jenkins@iotivity.org>
Reviewed-by: George Nash <george.nash@intel.com>
Reviewed-by: Phil Coval <philippe.coval@osg.samsung.com>
plugins/SConscript
plugins/samples/linux/SConscript
plugins/src/SConscript
plugins/unittests/SConscript
plugins/zigbee_wrapper/src/SConscript
plugins/zigbee_wrapper/telegesis_wrapper/SConscript
plugins/zigbee_wrapper/telegesis_wrapper/src/SConscript

index 2e3f3f0..3decb72 100644 (file)
@@ -29,7 +29,9 @@ target_os = env.get('TARGET_OS')
 build_sample = env.get('BUILD_SAMPLE')
 src_dir = env.get('SRC_DIR')
 
-if target_os not in ['android', 'arduino', 'darwin', 'ios', 'tizen', 'msys_nt', 'windows']:
+if target_os not in [
+        'android', 'arduino', 'darwin', 'ios', 'tizen', 'msys_nt', 'windows'
+]:
 
     SConscript('zigbee_wrapper/SConscript')
 
@@ -40,4 +42,3 @@ if target_os not in ['android', 'arduino', 'darwin', 'ios', 'tizen', 'msys_nt',
     if build_sample == 'ON':
         if target_os in ['linux']:
             SConscript('samples/' + target_os + '/SConscript')
-
index 091c790..9fc158d 100644 (file)
@@ -30,55 +30,49 @@ pi_dir = os.path.join(src_dir, 'plugins')
 ######################################################################
 # Build flags
 ######################################################################
-samples_env.PrependUnique(CPPPATH = [
-               os.path.join(src_dir, 'resource', 'oc_logger', 'include'),
-               os.path.join(src_dir, 'resource', 'csdk', 'logger', 'include'),
-               os.path.join(src_dir, 'resource', 'csdk', 'include'),
-               os.path.join(src_dir, 'resource', 'csdk', 'stack', 'include'),
-                os.path.join(pi_dir, 'include')
-                ])
+samples_env.PrependUnique(CPPPATH=[
+    os.path.join(src_dir, 'resource', 'oc_logger', 'include'),
+    os.path.join(src_dir, 'resource', 'csdk', 'logger', 'include'),
+    os.path.join(src_dir, 'resource', 'csdk', 'include'),
+    os.path.join(src_dir, 'resource', 'csdk', 'stack', 'include'),
+    os.path.join(pi_dir, 'include')
+])
 
-samples_env.AppendUnique(CXXFLAGS = ['-std=c++0x', '-Wall', '-Wextra', '-Werror'])
-samples_env.AppendUnique(RPATH = [env.get('BUILD_DIR')])
-samples_env.AppendUnique(LIBPATH = [env.get('BUILD_DIR')])
+samples_env.AppendUnique(
+    CXXFLAGS=['-std=c++0x', '-Wall', '-Wextra', '-Werror'])
+samples_env.AppendUnique(RPATH=[env.get('BUILD_DIR')])
+samples_env.AppendUnique(LIBPATH=[env.get('BUILD_DIR')])
 
 if target_os in ['darwin', 'ios']:
-       samples_env.PrependUnique(LIBS = ['m',
-                                          'octbstack',
-                                          'ocsrm',
-                                          'connectivity_abstraction',
-                                          'coap' ])
+    samples_env.PrependUnique(
+        LIBS=['m', 'octbstack', 'ocsrm', 'connectivity_abstraction', 'coap'])
 elif target_os not in ['arduino']:
-       samples_env.PrependUnique(LIBS = ['m',
-                                          'octbstack',
-                                          'ocsrm',
-                                          'connectivity_abstraction',
-                                          'coap',
-                                          'plugin_interface'])
-       samples_env.AppendUnique(LIBS = ['rt'])
-       if target_arch in ['x86_64', 'arm64']:
-               samples_env.AppendUnique(CPPFLAGS = ['-Llib64'])
-       else:
-               samples_env.AppendUnique(CPPFLAGS = ['-Llib'])
-       samples_env.AppendUnique(LIBS = ['pthread'])
+    samples_env.PrependUnique(LIBS=[
+        'm', 'octbstack', 'ocsrm', 'connectivity_abstraction', 'coap',
+        'plugin_interface'
+    ])
+    samples_env.AppendUnique(LIBS=['rt'])
+    if target_arch in ['x86_64', 'arm64']:
+        samples_env.AppendUnique(CPPFLAGS=['-Llib64'])
+    else:
+        samples_env.AppendUnique(CPPFLAGS=['-Llib'])
+    samples_env.AppendUnique(LIBS=['pthread'])
 
 if env.get('SECURED') == '1':
-    samples_env.AppendUnique(LIBS = ['mbedtls','mbedx509','mbedcrypto'])
+    samples_env.AppendUnique(LIBS=['mbedtls', 'mbedx509', 'mbedcrypto'])
 
-samples_env.AppendUnique(CPPDEFINES = ['TB_LOG'])
+samples_env.AppendUnique(CPPDEFINES=['TB_LOG'])
 
 ######################################################################
 # Source files and Targets
 ######################################################################
 iotivityandzigbeeserver = samples_env.Program('iotivityandzigbeeserver',
-        ['IotivityandZigbeeServer.c'])
+                                              ['IotivityandZigbeeServer.c'])
 iotivityandzigbeeclient = samples_env.Program('iotivityandzigbeeclient',
-        ['IotivityandZigbeeClient.c'])
+                                              ['IotivityandZigbeeClient.c'])
 
 list_of_samples = [iotivityandzigbeeserver, iotivityandzigbeeclient]
 
 Alias("samples", list_of_samples)
 
 env.AppendTarget('samples')
-
-
index b9945c2..6140ed3 100644 (file)
@@ -29,64 +29,68 @@ target_os = env.get('TARGET_OS')
 src_dir = env.get('SRC_DIR')
 pi_path = os.path.join(src_dir, 'plugins')
 
-print"Reading PI script"
+print "Reading PI script"
 
 ######################################################################
 # Build flags
 ######################################################################
 with_unforked_libcoap = env.get('WITH_UNFORKED_LIBCOAP')
 if with_unforked_libcoap == '1':
-    env.PrependUnique(CPPPATH = ['#extlibs/libcoap/libcoap/include'])
+    env.PrependUnique(CPPPATH=['#extlibs/libcoap/libcoap/include'])
 else:
-    env.PrependUnique(CPPPATH = [os.path.join(src_dir, 'resource', 'csdk', 'connectivity', 'lib', 'libcoap-4.1.1', 'include')])
+    env.PrependUnique(CPPPATH=[
+        os.path.join(src_dir, 'resource', 'csdk', 'connectivity', 'lib',
+                     'libcoap-4.1.1', 'include')
+    ])
 
-env.PrependUnique(CPPPATH = [ os.path.join(src_dir, 'resource', 'c_common', 'oic_malloc', 'include'),
-                              os.path.join(src_dir, 'resource', 'c_common', 'oic_string', 'include'),
-                              os.path.join(src_dir, 'resource', 'c_common', 'oic_time', 'include'),
-                              os.path.join(src_dir, 'resource', 'oc_logger', 'include'),
-                              os.path.join(src_dir, 'resource', 'csdk', 'logger', 'include'),
-                              os.path.join(src_dir, 'resource', 'csdk', 'stack', 'include'),
-                              os.path.join(src_dir, 'resource', 'csdk', 'security', 'include'),
-                              os.path.join(src_dir, 'resource', 'csdk', 'include')
-                            ])
-env.AppendUnique(CPPPATH = [ os.path.join(pi_path, 'include'),
-                             os.path.join(pi_path, 'include', 'internal'),
-                             os.path.join(pi_path, 'zigbee_wrapper', 'include'),
-                             os.path.join(pi_path, 'include', 'internal')
-                           ])
+env.PrependUnique(CPPPATH=[
+    os.path.join(src_dir, 'resource', 'c_common', 'oic_malloc', 'include'),
+    os.path.join(src_dir, 'resource', 'c_common', 'oic_string', 'include'),
+    os.path.join(src_dir, 'resource', 'c_common', 'oic_time', 'include'),
+    os.path.join(src_dir, 'resource', 'oc_logger', 'include'),
+    os.path.join(src_dir, 'resource', 'csdk', 'logger', 'include'),
+    os.path.join(src_dir, 'resource', 'csdk', 'stack', 'include'),
+    os.path.join(src_dir, 'resource', 'csdk', 'security', 'include'),
+    os.path.join(src_dir, 'resource', 'csdk', 'include')
+])
+env.AppendUnique(CPPPATH=[
+    os.path.join(pi_path, 'include'),
+    os.path.join(pi_path, 'include', 'internal'),
+    os.path.join(pi_path, 'zigbee_wrapper', 'include'),
+    os.path.join(pi_path, 'include', 'internal')
+])
 
 if target_os not in ['arduino', 'windows']:
-       env.AppendUnique(CPPDEFINES = ['WITH_POSIX'])
+    env.AppendUnique(CPPDEFINES=['WITH_POSIX'])
 
-if target_os in ['darwin','ios']:
-       env.AppendUnique(CPPDEFINES = ['_DARWIN_C_SOURCE'])
+if target_os in ['darwin', 'ios']:
+    env.AppendUnique(CPPDEFINES=['_DARWIN_C_SOURCE'])
 
-env.AppendUnique(CXXFLAGS = ['-std=c++0x', '-Wall', '-Wextra', '-Werror'])
-env.AppendUnique(RPATH = [env.get('BUILD_DIR')])
-env.AppendUnique(LIBPATH = [env.get('BUILD_DIR')])
+env.AppendUnique(CXXFLAGS=['-std=c++0x', '-Wall', '-Wextra', '-Werror'])
+env.AppendUnique(RPATH=[env.get('BUILD_DIR')])
+env.AppendUnique(LIBPATH=[env.get('BUILD_DIR')])
 
-env.PrependUnique(LIBS = ['zigbee_wrapper'])
+env.PrependUnique(LIBS=['zigbee_wrapper'])
 
 if env.get('LOGGING'):
-       env.AppendUnique(CPPDEFINES = ['TB_LOG'])
+    env.AppendUnique(CPPDEFINES=['TB_LOG'])
 
 #####################################################################
 # Source files and Target(s)
 ######################################################################
 pi_src = [
-         os.path.join(src_dir, 'resource', 'csdk', 'logger', 'src', 'logger.c'),
-         'pluginlist.c',
-         'plugininterface.c',
-            ]
+    os.path.join(src_dir, 'resource', 'csdk', 'logger', 'src', 'logger.c'),
+    'pluginlist.c',
+    'plugininterface.c',
+]
 
-env.AppendUnique(PI_SRC = pi_src)
+env.AppendUnique(PI_SRC=pi_src)
 
 print "Include path is %s" % env.get('CPPPATH')
 print "Files path is %s" % env.get('PI_SRC')
 if target_os in ['android', 'tizen']:
-       calib = env.SharedLibrary('plugin_interface', env.get('PI_SRC'))
+    calib = env.SharedLibrary('plugin_interface', env.get('PI_SRC'))
 else:
-       calib = env.StaticLibrary('plugin_interface', env.get('PI_SRC'))
+    calib = env.StaticLibrary('plugin_interface', env.get('PI_SRC'))
 env.InstallTarget(calib, 'plugin_interface')
 env.UserInstallTargetLib(calib, 'plugin_interface')
-
index c1391df..8e1ade2 100644 (file)
@@ -20,6 +20,7 @@
 
 import os
 import os.path
+from tools.scons.RunTest import run_test
 
 # SConscript file for Local PKI google tests
 gtest_env = SConscript('#extlibs/gtest/SConscript')
@@ -32,22 +33,22 @@ target_os = unittests_env.get('TARGET_OS')
 ######################################################################
 # Build flags
 ######################################################################
-unittests_env.PrependUnique(CPPPATH = [
-                os.path.join(pi_dir, 'include'),
-               ])
+unittests_env.PrependUnique(CPPPATH=[
+    os.path.join(pi_dir, 'include'),
+])
 
-unittests_env.AppendUnique(LIBPATH = [unittests_env.get('BUILD_DIR')])
-unittests_env.PrependUnique(LIBS = [
-               'plugin_interface',
-               'oc',
-               'octbstack',
-               'oc_logger',
-               'connectivity_abstraction',
-               'coap'
-               ])
+unittests_env.AppendUnique(LIBPATH=[unittests_env.get('BUILD_DIR')])
+unittests_env.PrependUnique(LIBS=[
+    'plugin_interface',
+    'oc',
+    'octbstack',
+    'oc_logger',
+    'connectivity_abstraction',
+    'coap',
+])
 
 if unittests_env.get('LOGGING'):
-       unittests_env.AppendUnique(CPPDEFINES = ['TB_LOG'])
+    unittests_env.AppendUnique(CPPDEFINES=['TB_LOG'])
 
 ######################################################################
 # Source files and Targets
@@ -58,10 +59,6 @@ Alias("piunittests", [piunittests])
 
 unittests_env.AppendTarget('piunittests')
 if unittests_env.get('TEST') == '1':
-       if target_os in ['linux']:
-                from tools.scons.RunTest import *
-                run_test(unittests_env,
-                         'plugins_unittests.memcheck',
-                         'plugins/unittests/piunittests',
-                         piunittests)
-
+    if target_os in ['linux']:
+        run_test(unittests_env, 'plugins_unittests.memcheck',
+                 'plugins/unittests/piunittests', piunittests)
index 21b1bf2..108a448 100644 (file)
@@ -30,54 +30,57 @@ target_os = env.get('TARGET_OS')
 src_dir = env.get('SRC_DIR')
 zw_path = os.path.join(src_dir, 'plugins')
 
-print"Reading Zigbee Wrapper (ZW) script"
+print "Reading Zigbee Wrapper (ZW) script"
 
 ######################################################################
 # Build flags
 ######################################################################
 
-env.PrependUnique(CPPPATH = [ os.path.join(src_dir, 'resource', 'c_common', 'oic_malloc', 'include'),
-                              os.path.join(src_dir, 'resource', 'c_common', 'oic_string', 'include'),
-                              os.path.join(src_dir, 'resource', 'oc_logger', 'include'),
-                              os.path.join(src_dir, 'resource', 'csdk', 'logger', 'include'),
-                              os.path.join(src_dir, 'resource', 'csdk', 'include'),
-                              os.path.join(src_dir, 'resource', 'csdk', 'stack', 'include'),
-                              ])
-env.AppendUnique(CPPPATH = [ os.path.join(zw_path, 'include'),
-                             os.path.join(zw_path, 'include', 'internal'),
-                             os.path.join(zw_path, 'zigbee_wrapper', 'include')
-                             ])
+env.PrependUnique(CPPPATH=[
+    os.path.join(src_dir, 'resource', 'c_common', 'oic_malloc', 'include'),
+    os.path.join(src_dir, 'resource', 'c_common', 'oic_string', 'include'),
+    os.path.join(src_dir, 'resource', 'oc_logger', 'include'),
+    os.path.join(src_dir, 'resource', 'csdk', 'logger', 'include'),
+    os.path.join(src_dir, 'resource', 'csdk', 'include'),
+    os.path.join(src_dir, 'resource', 'csdk', 'stack', 'include'),
+])
+env.AppendUnique(CPPPATH=[
+    os.path.join(zw_path, 'include'),
+    os.path.join(zw_path, 'include', 'internal'),
+    os.path.join(zw_path, 'zigbee_wrapper', 'include')
+])
 
 if target_os not in ['arduino', 'windows']:
-       env.AppendUnique(CPPDEFINES = ['WITH_POSIX'])
+    env.AppendUnique(CPPDEFINES=['WITH_POSIX'])
 
-if target_os in ['darwin','ios']:
-       env.AppendUnique(CPPDEFINES = ['_DARWIN_C_SOURCE'])
+if target_os in ['darwin', 'ios']:
+    env.AppendUnique(CPPDEFINES=['_DARWIN_C_SOURCE'])
 
-env.AppendUnique(CXXFLAGS = ['-std=c++0x', '-Wall', '-Wextra', '-Werror'])
-env.AppendUnique(RPATH = [env.get('BUILD_DIR')])
-env.AppendUnique(LIBPATH = [env.get('BUILD_DIR')])
+env.AppendUnique(CXXFLAGS=['-std=c++0x', '-Wall', '-Wextra', '-Werror'])
+env.AppendUnique(RPATH=[env.get('BUILD_DIR')])
+env.AppendUnique(LIBPATH=[env.get('BUILD_DIR')])
 
-env.PrependUnique(LIBS = ['telegesis_wrapper'])
+env.PrependUnique(LIBS=['telegesis_wrapper'])
 
 if env.get('LOGGING'):
-       env.AppendUnique(CPPDEFINES = ['TB_LOG'])
+    env.AppendUnique(CPPDEFINES=['TB_LOG'])
 
 #####################################################################
 # Source files and Target(s)
 ######################################################################
 
 zw_src = [
-         os.path.join(src_dir, 'resource', 'c_common', 'oic_malloc', 'src', 'oic_malloc.c'),
-         os.path.join(src_dir, 'resource', 'csdk', 'logger', 'src', 'logger.c'),
-            'zigbee_wrapper.c',
-         ]
+    os.path.join(src_dir, 'resource', 'c_common', 'oic_malloc', 'src',
+                 'oic_malloc.c'),
+    os.path.join(src_dir, 'resource', 'csdk', 'logger', 'src', 'logger.c'),
+    'zigbee_wrapper.c',
+]
 
-env.AppendUnique(ZW_SRC = zw_src)
+env.AppendUnique(ZW_SRC=zw_src)
 
 if target_os in ['android', 'tizen']:
-       calib = env.SharedLibrary('zigbee_wrapper', env.get('ZW_SRC'))
+    calib = env.SharedLibrary('zigbee_wrapper', env.get('ZW_SRC'))
 else:
-       calib = env.StaticLibrary('zigbee_wrapper', env.get('ZW_SRC'))
+    calib = env.StaticLibrary('zigbee_wrapper', env.get('ZW_SRC'))
 env.InstallTarget(calib, 'libzigbee_wrapper')
 env.UserInstallTargetLib(calib, 'libzigbee_wrapper')
index 893f83c..391b0bb 100644 (file)
@@ -28,4 +28,3 @@ Import('env')
 print "Reading top Telegesis Wrapper."
 
 env.SConscript('src/SConscript')
-
index b2d71b0..284405a 100644 (file)
@@ -33,37 +33,37 @@ tw_path = os.path.join(src_dir, 'plugins')
 # Source files and Target(s)
 ######################################################################
 
-print"Reading Telegesis Wrapper (TW) script"
+print "Reading Telegesis Wrapper (TW) script"
 
-env.AppendUnique(CPPPATH = [ os.path.join(tw_path, 'include'),
-                             os.path.join(tw_path, 'include', 'internal'),
-                             os.path.join(src_dir, 'resource', 'oc_logger', 'include'),
-                             os.path.join(src_dir, 'resource', 'csdk', 'include'),
-                             os.path.join(src_dir, 'resource', 'csdk', 'stack', 'include'),
-                             os.path.join(src_dir, 'resource', 'csdk', 'logger', 'include'),
-                             os.path.join(src_dir, 'resource', 'c_common', 'oic_string', 'include'),
-                             os.path.join(tw_path, 'zigbee_wrapper', 'telegesis_wrapper', 'include')
-                             ])
+env.AppendUnique(CPPPATH=[
+    os.path.join(tw_path, 'include'),
+    os.path.join(tw_path, 'include', 'internal'),
+    os.path.join(src_dir, 'resource', 'oc_logger', 'include'),
+    os.path.join(src_dir, 'resource', 'csdk', 'include'),
+    os.path.join(src_dir, 'resource', 'csdk', 'stack', 'include'),
+    os.path.join(src_dir, 'resource', 'csdk', 'logger', 'include'),
+    os.path.join(src_dir, 'resource', 'c_common', 'oic_string', 'include'),
+    os.path.join(tw_path, 'zigbee_wrapper', 'telegesis_wrapper', 'include')
+])
 
 if target_os not in ['arduino', 'windows']:
-       env.AppendUnique(CPPDEFINES = ['WITH_POSIX'])
+    env.AppendUnique(CPPDEFINES=['WITH_POSIX'])
 
-if target_os in ['darwin','ios']:
-       env.AppendUnique(CPPDEFINES = ['_DARWIN_C_SOURCE'])
+if target_os in ['darwin', 'ios']:
+    env.AppendUnique(CPPDEFINES=['_DARWIN_C_SOURCE'])
 
 tw_src = [
-         os.path.join(src_dir, 'resource', 'c_common', 'oic_string', 'src', 'oic_string.c'),
-         os.path.join(src_dir, 'resource', 'csdk', 'logger', 'src', 'logger.c'),
-         'twsocketlist.c',
-         'telegesis_socket.c',
-         'telegesis_wrapper.c'
-         ]
+    os.path.join(src_dir, 'resource', 'c_common', 'oic_string', 'src',
+                 'oic_string.c'),
+    os.path.join(src_dir, 'resource', 'csdk', 'logger', 'src', 'logger.c'),
+    'twsocketlist.c', 'telegesis_socket.c', 'telegesis_wrapper.c'
+]
 
-env.AppendUnique(TW_SRC = tw_src)
+env.AppendUnique(TW_SRC=tw_src)
 
 if target_os in ['android', 'tizen']:
-       calib = env.SharedLibrary('telegesis_wrapper', env.get('TW_SRC'))
+    calib = env.SharedLibrary('telegesis_wrapper', env.get('TW_SRC'))
 else:
-       calib = env.StaticLibrary('telegesis_wrapper', env.get('TW_SRC'))
+    calib = env.StaticLibrary('telegesis_wrapper', env.get('TW_SRC'))
 env.InstallTarget(calib, 'libtelegesis_wrapper')
 env.UserInstallTargetLib(calib, 'libtelegesis_wrapper')