Providing build option to disable BLE server 41/199041/3 accepted/tizen/unified/20190208.061532 submit/tizen/20190201.062348 submit/tizen/20190206.231730
authorAbhishek Sansanwal <abhishek.s94@samsung.com>
Fri, 1 Feb 2019 03:42:36 +0000 (09:12 +0530)
committerAbhishek Sansanwal <abhishek.s94@samsung.com>
Fri, 1 Feb 2019 04:47:06 +0000 (10:17 +0530)
Build option DISABLE_BLE_SERVER is added so that BLE server
can be disabled when running in CLIENT_SERVER mode.

Just provide DISABLE_BLE_SERVER = 1 option to disable it.

https://github.sec.samsung.net/RS7-IOTIVITY/IoTivity/pull/394
(cherry picked from ea611133c372df685cb58842dd6d7c06276d0aeb)

Change-Id: I7552f378406e92b6c847cd73debfd31336d074a5
Signed-off-by: agrkush <kush.agrawal@samsung.com>
Signed-off-by: Kush Agrawal <kush.agrawal@samsung.com>
Signed-off-by: Abhishek Sansanwal <abhishek.s94@samsung.com>
14 files changed:
build_common/SConscript
packaging/iotivity.spec
resource/csdk/connectivity/SConscript
resource/csdk/connectivity/build/SConscript
resource/csdk/connectivity/build/tizen/scons/SConscript
resource/csdk/connectivity/src/bt_le_adapter/caleadapter.c
resource/csdk/stack/samples/tizen/SimpleClientServer/SConscript
resource/csdk/stack/samples/tizen/build/SConscript
resource/csdk/stack/samples/tizen/build/gbsbuild.sh
resource/csdk/stack/samples/tizen/build/packaging/com.oic.ri.spec
resource/csdk/stack/samples/tizen/build/scons/SConscript
tools/tizen/iotivity-vd-tv-es-tizen30.spec
tools/tizen/iotivity-vd-tv-tizen30.spec
tools/tizen/iotivity.spec

index 4a9a350244a8f763d839e2cc9d2cd64a2b509e82..8612220baeff2c8385b3576fd177e7e4c296ecdf 100644 (file)
@@ -102,6 +102,7 @@ help_vars.Add(ListVariable('WITH_MQ', 'Build with MQ publisher/broker', 'OFF', [
 help_vars.Add(BoolVariable('WITH_CLOUD', 'Build including AccountManager class and Cloud Client sample', False))
 help_vars.Add(ListVariable('RD_MODE', 'Resource Directory build mode', 'CLIENT', ['CLIENT', 'SERVER']))
 help_vars.Add(BoolVariable('DISABLE_PRESENCE', 'Disable Presence Feature', False))
+help_vars.Add(BoolVariable('DISABLE_BLE_SERVER', 'Disable BLE server', False))
 
 help_vars.Add(BoolVariable('SIMULATOR', 'Build with simulator module', False))
 
@@ -362,6 +363,9 @@ if env.get('DISABLE_TCP_SERVER'):
 if env.get('DISABLE_PRESENCE') == False:
     env.AppendUnique(CPPDEFINES = ['WITH_PRESENCE'])
 
+if env.get('DISABLE_BLE_SERVER'):
+    defines.append('-DDISABLE_BLE_SERVER=1')
+
 libs = []
 if env.get('SECURED') == '1':
     defines.append('-D__WITH_DTLS__=1')
index f5b7f60ea44808ea1dd3f6f53c56242240e439dd..764c9630484551e345b033cb6bc34029d4930846 100755 (executable)
@@ -86,6 +86,7 @@ Source1002: %{name}-test.manifest
 %{!?BLE_CUSTOM_ADV: %define BLE_CUSTOM_ADV False}
 %{!?BLE_DIVISION: %define BLE_DIVISION VD}
 %{!?BLE_TIZEN_30: %define BLE_TIZEN_30 True}
+%{!?DISABLE_BLE_SERVER: %define DISABLE_BLE_SERVER 0}
 %{!?MULTIPLE_OWNER: %define MULTIPLE_OWNER 1}
 
 BuildRequires:  expat-devel
@@ -183,6 +184,7 @@ scons %{JOB} --prefix=%{_prefix} \
     BLE_CUSTOM_ADV=%{BLE_CUSTOM_ADV} \
     BLE_DIVISION=%{BLE_DIVISION} \
     BLE_TIZEN_30=%{BLE_TIZEN_30} \
+    DISABLE_BLE_SERVER=%{DISABLE_BLE_SERVER} \
     MULTIPLE_OWNER=%{MULTIPLE_OWNER} \
     #eol
 
index a38cdbce2da07391ee26651bd46bc89cad490d4e..d5ebfe2ff771f86c8c5089238b2929037b45c6ca 100644 (file)
@@ -10,6 +10,7 @@ build_sample = env.get('BUILD_SAMPLE')
 with_ra = env.get('WITH_RA')
 with_tcp = env.get('WITH_TCP')
 disable_tcp_server = env.get('DISABLE_TCP_SERVER')
+disable_ble_server = env.get('DISABLE_BLE_SERVER')
 with_mq = env.get('WITH_MQ')
 ble_custom_adv = env.get('BLE_CUSTOM_ADV')
 
@@ -33,6 +34,9 @@ if 'ALL' in transport:
                        env.AppendUnique(CPPDEFINES = ['TCP_ADAPTER'])
        if disable_tcp_server == True:
                        env.AppendUnique(CPPDEFINES = ['DISABLE_TCP_SERVER'])
+       if disable_ble_server == True:
+               if target_os in ['tizen']:
+                       env.AppendUnique(CPPDEFINES = ['DISABLE_BLE_SERVER'])
        if target_os in ['linux']:
                env.AppendUnique(CPPDEFINES = ['IP_ADAPTER','NO_EDR_ADAPTER','LE_ADAPTER'])
        elif target_os == 'tizen':
@@ -65,6 +69,10 @@ else:
                        print "BLE Custom advertisement supported"
                else:
                        print "BLE Custom advertisement not supported"
+               if disable_ble_server == True:
+                       if target_os in ['tizen']:
+                               env.AppendUnique(CPPDEFINES = ['DISABLE_BLE_SERVER'])
+                               print "BLE Server Disabled"
        else:
                env.AppendUnique(CPPDEFINES = ['NO_LE_ADAPTER'])
 
index 6aaa3c0edb5f92086d8340a9f6cc9a293ce28e10..05be0219d5ab5f7fc9c1d25c50fd09d244eaa91b 100644 (file)
@@ -67,6 +67,7 @@ help_vars.Add(EnumVariable('ROUTING', 'Enable routing', 'EP', allowed_values=('G
 help_vars.Add(EnumVariable('BUILD_SAMPLE', 'Build with sample', 'ON', allowed_values=('ON', 'OFF')))
 help_vars.Add(BoolVariable('WITH_TCP', 'Enable TCP', False))
 help_vars.Add(BoolVariable('DISABLE_TCP_SERVER', 'Disable TCP server', False))
+help_vars.Add(BoolVariable('DISABLE_BLE_SERVER', 'Disable BLE server', False))
 help_vars.Add(ListVariable('WITH_MQ', 'Build with MQ publisher/subscriber/broker', 'OFF', ['OFF', 'SUB', 'PUB', 'BROKER']))
 
 help_vars.AddVariables(('DEVICE_NAME', 'Network display name for device', 'OIC-DEVICE', None, None),)
index 6b6d55b22cf0b065aa80c1920c17bc6f8ef211ad..1042e31b0c5f657520b5de7f5b8f122351e1f19b 100644 (file)
@@ -10,6 +10,7 @@ target_os = env.get('TARGET_OS')
 transport = env.get('TARGET_TRANSPORT')
 with_tcp = env.get('WITH_TCP')
 disable_tcp_server = env.get('DISABLE_TCP_SERVER')
+disable_ble_server = env.get('DISABLE_BLE_SERVER')
 
 print "Given Transport is %s" % transport
 
@@ -57,6 +58,9 @@ else:
                env.AppendUnique(CPPDEFINES = ['DISABLE_TCP_SERVER'])
                print "TCP server is disabled"
 
+       if disable_ble_server == True:
+               env.AppendUnique(CPPDEFINES = ['DISABLE_BLE_SERVER'])
+               print "BLE server is disabled"
 
 env.SConscript(['../con/lib/libcoap-4.1.1/SConscript'])
 env.SConscript(['../con/SConscript'])
index f9d000b6a7094ee9d98d04231f65a37ee02fae5c..78e4c12ea38845cfb8b9b8e461934841e581341b 100644 (file)
@@ -2379,6 +2379,7 @@ static CAResult_t CALEAdapterClientSendData(const CAEndpoint_t *remoteEndpoint,
 static CAResult_t CALEAdapterGattServerStart()
 {
     OIC_LOG_V(DEBUG, CALEADAPTER_TAG, "%s", __func__);
+    CAResult_t result = CA_STATUS_FAILED;
 
     if (caglobals.bleFlags & CA_LE_SERVER_DISABLE)
     {
@@ -2387,7 +2388,10 @@ static CAResult_t CALEAdapterGattServerStart()
         return CA_STATUS_OK;
     }
 
-    CAResult_t result = CAStartLEGattServer();
+#ifndef DISABLE_BLE_SERVER
+    OIC_LOG_V(INFO, CALEADAPTER_TAG, "Starting LE GATT Server");
+    result = CAStartLEGattServer();
+#endif
 
 #ifndef SINGLE_THREAD
     /*
index 704e44f204d1ed4631794efffb71f019e99350b7..a3f9fa75d6f6a15b1f931fb9eb72e3e42683b444 100644 (file)
@@ -60,6 +60,7 @@ help_vars.Add(BoolVariable('WITH_PROXY', 'CoAP-HTTP Proxy', False)) # set to 'no
 help_vars.Add(ListVariable('WITH_MQ', 'Build with MQ publisher/subscriber/broker', 'OFF', ['OFF', 'SUB', 'PUB', 'BROKER']))
 help_vars.Add(BoolVariable('WITH_TCP', 'Build with TCP', False))
 help_vars.Add(BoolVariable('DISABLE_TCP_SERVER', 'Disable TCP server', False))
+help_vars.Add(BoolVariable('DISABLE_BLE_SERVER', 'Disable BLE server', True))
 
 ######################################################################
 # Platform(build target) specific options: SDK/NDK & toolchain
index a2f27fa3f57ea92367a6d273e0dcc0b0c537e6d8..225d6178a863cca97b94da62e37543abb5d3adb6 100644 (file)
@@ -16,6 +16,7 @@ routing = env.get('ROUTING')
 with_proxy = env.get('WITH_PROXY')
 with_tcp = env.get('WITH_TCP')
 with_mq = env.get('WITH_MQ')
+disable_ble_server = env.get('DISABLE_BLE_SERVER')
 
 with_upstream_libcoap = env.get('WITH_UPSTREAM_LIBCOAP')
 if with_upstream_libcoap == '1':
@@ -52,7 +53,7 @@ if (('BLE' in transport) or ('ALL' in transport)):
        gbs_command_prefix += "gbsprofile=tizen_4_0_unified_armv7l "
 
 if target_os == 'tizen':
-       command = gbs_command_prefix + "./resource/csdk/stack/samples/tizen/build/gbsbuild.sh %s %s %s %s %s %s %s %s %s" % (transport, secured, buildsample, release_mode, logging, routing, with_tcp, with_proxy, with_mq)
+       command = gbs_command_prefix + "./resource/csdk/stack/samples/tizen/build/gbsbuild.sh %s %s %s %s %s %s %s %s %s %s" % (transport, secured, buildsample, release_mode, logging, routing, with_tcp, with_proxy, with_mq, disable_ble_server)
        print "Created Command is %s" % command
        gbs_script = env.Command('gbs_build', None, command)
        AlwaysBuild ('gbs_script')
index 8d761f0ec840194ca51fedd12d8419557232b954..141b8d86c46356c70d8331ccf665cd1f39bcfb95 100755 (executable)
@@ -36,6 +36,9 @@ export WITH_PROXY=$8
 echo $9
 export WITH_MQ=$9
 
+echo $10
+export DISABLE_BLE_SERVER=$10
+
 echo $TARGET_TRANSPORT
 echo $BUILD_SAMPLE
 echo $WITH_MQ
@@ -110,7 +113,7 @@ nproc=$(expr 1 + $(expr `nproc --ignore=1` / 2 ) )
 gbscommand_prefix=${gbscommand_prefix}" --define '_smp_mflags -j${nproc}'"
 
 echo "Calling core gbs build command"
-gbscommand=${gbscommand_prefix}" -B ~/GBS-ROOT-RI-OIC --include-all --repository ./ --define 'TARGET_TRANSPORT $1' --define 'SECURED $2' --define 'RELEASE $4' --define 'LOGGING $5' --define 'ROUTING $6' --define 'WITH_TCP $7' --define 'WITH_PROXY $8' --define 'WITH_MQ $9'"
+gbscommand=${gbscommand_prefix}" -B ~/GBS-ROOT-RI-OIC --include-all --repository ./ --define 'TARGET_TRANSPORT $1' --define 'SECURED $2' --define 'RELEASE $4' --define 'LOGGING $5' --define 'ROUTING $6' --define 'WITH_TCP $7' --define 'WITH_PROXY $8' --define 'WITH_MQ $9' --define 'DISABLE_BLE_SERVER $10'"
 echo $gbscommand
 if eval $gbscommand; then
    echo "Core build is successful"
index 9f1cec93a0932173ff04f7e61a9bf91d85482f71..dd16d6a2ab1ccba883e65a29bd5c66dd1bc7824d 100644 (file)
@@ -31,6 +31,7 @@ Source0: http://mirrors.kernel.org/%{name}/%{version}/%{name}-%{version}.tar.gz
 %{!?WITH_MQ: %define WITH_MQ OFF}
 %{!?WITH_PROXY: %define WITH_PROXY 0}
 %{!?WITH_TCP: %define WITH_TCP 0}
+%{!?DISABLE_BLE_SERVER: %define DISABLE_BLE_SERVER 0}
 
 BuildRequires: pkgconfig(dlog)
 BuildRequires: pkgconfig(ttrace)
@@ -62,6 +63,7 @@ scons %{JOB} --prefix=%{_prefix} \
     VERBOSE=%{VERBOSE} \
     WITH_MQ=%{WITH_MQ} \
     WITH_PROXY=%{WITH_PROXY} \
+    DISABLE_BLE_SERVER=%{DISABLE_BLE_SERVER} \
     WITH_TCP=%{WITH_TCP} \
     #eol
 
index 46e24beafef53f3e47a92dae2e4ffa0ed9ba5bc2..1390ace675bde4064e7eb53c79b36fdb6ddd1058 100644 (file)
@@ -9,6 +9,7 @@ Import('env')
 target_os = env.get('TARGET_OS')
 transport = env.get('TARGET_TRANSPORT')
 with_mq = env.get('WITH_MQ')
+disable_ble_server = env.get('DISABLE_BLE_SERVER')
 
 print "Given Transport is %s" % transport
 
@@ -27,6 +28,9 @@ if env.get('OIC_SUPPORT_TIZEN_TRACE') == 'True':
 if 'ALL' in transport:
                env.AppendUnique(CPPDEFINES = ['WIFI_ADAPTER', 'NO_ETHERNET_ADAPTER','EDR_ADAPTER','LE_ADAPTER'])
                print "CA Transport is ALL"
+               if env.get('DISABLE_BLE_SERVER'):
+                       env.AppendUnique(CPPDEFINES = ['DISABLE_BLE_SERVER'])
+                       print "BLE Server Disabled"
 else:
        if 'BT' in transport:
                env.AppendUnique(CPPDEFINES = ['EDR_ADAPTER'])
@@ -37,6 +41,9 @@ else:
        if 'BLE' in transport:
                env.AppendUnique(CPPDEFINES = ['LE_ADAPTER'])
                print "CA Transport is BLE"
+               if env.get('DISABLE_BLE_SERVER'):
+                       env.AppendUnique(CPPDEFINES = ['DISABLE_BLE_SERVER'])
+                       print "BLE Server Disabled"
        else:
                env.AppendUnique(CPPDEFINES = ['NO_LE_ADAPTER'])
 
index 7b4ce2ffb426aeced05946f299cdedc17dc63494..97da3c74b899b18c907caa16842ce13015a37b62 100644 (file)
@@ -84,6 +84,7 @@ Source1002: %{name}-test.manifest
 %{!?BLE_CUSTOM_ADV: %define BLE_CUSTOM_ADV False}
 %{!?BLE_DIVISION: %define BLE_DIVISION VD}
 %{!?BLE_TIZEN_30: %define BLE_TIZEN_30 True}
+%{!?DISABLE_BLE_SERVER: %define DISABLE_BLE_SERVER 0}
 %{!?MULTIPLE_OWNER: %define MULTIPLE_OWNER 0}
 
 BuildRequires:  expat-devel
@@ -181,6 +182,7 @@ scons %{JOB} --prefix=%{_prefix} \
     BLE_CUSTOM_ADV=%{BLE_CUSTOM_ADV} \
     BLE_DIVISION=%{BLE_DIVISION} \
     BLE_TIZEN_30=%{BLE_TIZEN_30} \
+    DISABLE_BLE_SERVER=%{DISABLE_BLE_SERVER} \
     MULTIPLE_OWNER=%{MULTIPLE_OWNER} \
     #eol
 
index 557c55deda79cf3d8a0789fb44880791e3cef832..8c6bd8099f1258a9f3fafcbe52f6809e51a8c983 100644 (file)
@@ -84,6 +84,7 @@ Source1002: %{name}-test.manifest
 %{!?BLE_CUSTOM_ADV: %define BLE_CUSTOM_ADV False}
 %{!?BLE_DIVISION: %define BLE_DIVISION VD}
 %{!?BLE_TIZEN_30: %define BLE_TIZEN_30 True}
+%{!?DISABLE_BLE_SERVER: %define DISABLE_BLE_SERVER 0}
 %{!?MULTIPLE_OWNER: %define MULTIPLE_OWNER 0}
 
 BuildRequires:  expat-devel
@@ -181,6 +182,7 @@ scons %{JOB} --prefix=%{_prefix} \
     BLE_CUSTOM_ADV=%{BLE_CUSTOM_ADV} \
     BLE_DIVISION=%{BLE_DIVISION} \
     BLE_TIZEN_30=%{BLE_TIZEN_30} \
+    DISABLE_BLE_SERVER=%{DISABLE_BLE_SERVER} \
     MULTIPLE_OWNER=%{MULTIPLE_OWNER} \
     #eol
 
index 903a32ba49158b356ed55cd2598ea7f55e58d3ca..e571f229990d978b719cbde5eb63e895745e5c54 100644 (file)
@@ -83,6 +83,7 @@ Source1002: %{name}-test.manifest
 %{!?BLE_CUSTOM_ADV: %define BLE_CUSTOM_ADV False}
 %{!?BLE_DIVISION: %define BLE_DIVISION VD}
 %{!?BLE_TIZEN_30: %define BLE_TIZEN_30 True}
+%{!?DISABLE_BLE_SERVER: %define DISABLE_BLE_SERVER 0}
 %{!?MULTIPLE_OWNER: %define MULTIPLE_OWNER 0}
 
 BuildRequires:  expat-devel
@@ -179,6 +180,7 @@ scons %{JOB} --prefix=%{_prefix} \
     BLE_CUSTOM_ADV=%{BLE_CUSTOM_ADV} \
     BLE_DIVISION=%{BLE_DIVISION} \
     BLE_TIZEN_30=%{BLE_TIZEN_30} \
+    DISABLE_BLE_SERVER=%{DISABLE_BLE_SERVER} \
     MULTIPLE_OWNER=%{MULTIPLE_OWNER} \
     #eol