Modify Tizen BLE 3.0 / 4.0 build option 48/206148/1
authorDoHyun Pyun <dh79.pyun@samsung.com>
Thu, 9 May 2019 04:35:39 +0000 (13:35 +0900)
committerDoHyun Pyun <dh79.pyun@samsung.com>
Tue, 14 May 2019 23:57:04 +0000 (08:57 +0900)
tizen4 flag was defined in the code, but it was not used.
So tizen BLE CA layer is using internal APIs. This patchset
modifies build options and disable internal API usage.

Change-Id: I283085e4525207c4df2cd40b683e6f67b3aa6662
Signed-off-by: DoHyun Pyun <dh79.pyun@samsung.com>
build_common/SConscript
packaging/iotivity.spec [changed mode: 0755->0644]
resource/csdk/connectivity/src/bt_le_adapter/tizen/SConscript
resource/csdk/connectivity/src/bt_le_adapter/tizen/caleserver_mcd.c
resource/csdk/connectivity/src/bt_le_adapter/tizen/caleserver_vd.c
tools/tizen/iotivity.spec

index 577cbff93eb902cec2e0e3856a83a03fcc7efa65..f0322ee161e110dfea87343e3f5caf9c9d5eb099 100755 (executable)
@@ -133,7 +133,7 @@ help_vars.Add(EnumVariable('WITH_UPSTREAM_LIBCOAP', 'Use latest stable version o
 help_vars.Add(EnumVariable('BLE_CUSTOM_ADV', 'Build with CUSTOM Advertisement', 'False', allowed_values=('True', 'False')))
 help_vars.Add(EnumVariable('BLE_DIVISION', 'BLE Build with division code', 'OFF', allowed_values=('OFF', 'MCD', 'VD')))
 help_vars.Add(EnumVariable('BLE_TIZEN_30', 'Build with tizen 3.0 BLE api', 'False', allowed_values=('True', 'False')))
-help_vars.Add(EnumVariable('TIZEN_4', 'Build with tizen 4.0 api', 'False', allowed_values=('True', 'False')))
+help_vars.Add(EnumVariable('BLE_TIZEN_40', 'Build with tizen 4.0 BLE api', 'False', allowed_values=('True', 'False')))
 help_vars.Add(PathVariable('TIZENRT_OS_DIR', 'Absolute Path to TizenRT OS directory', None, PathVariable.PathAccept))
 help_vars.Add(EnumVariable('PLATFORM_TLS', 'Use platform tls instead of local mbedtls', '0', allowed_values=('0', '1')))
 help_vars.Add(EnumVariable('OIC_SUPPORT_TIZEN_TRACE', 'Tizen Trace(T-trace) api availability', 'False', allowed_values=('True', 'False')))
old mode 100755 (executable)
new mode 100644 (file)
index d09744a..46e8332
@@ -86,9 +86,9 @@ Source1002: %{name}-test.manifest
 %{!?RD_MODE: %define RD_MODE CLIENT}
 %{!?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}
+%{!?BLE_TIZEN_40: %define BLE_TIZEN_40 True}
 
 BuildRequires:  expat-devel
 BuildRequires:  python, libcurl-devel
@@ -184,7 +184,7 @@ scons %{JOB} --prefix=%{_prefix} \
     RD_MODE=%{RD_MODE} \
     BLE_CUSTOM_ADV=%{BLE_CUSTOM_ADV} \
     BLE_DIVISION=%{BLE_DIVISION} \
-    BLE_TIZEN_30=%{BLE_TIZEN_30} \
+    BLE_TIZEN_40=%{BLE_TIZEN_40} \
     DISABLE_BLE_SERVER=%{DISABLE_BLE_SERVER} \
     MULTIPLE_OWNER=%{MULTIPLE_OWNER} \
     LOG_LEVEL=%{log_level} \
index 3d59fdb75538419cfff14be30258ca343da231ab..455838081759243117c297c5ba00a79ccfdb129a 100644 (file)
@@ -8,6 +8,7 @@ import os.path
 root_dir = os.pardir
 division = env.get('BLE_DIVISION')
 tizen30 = env.get('BLE_TIZEN_30')
+tizen40 = env.get('BLE_TIZEN_40')
 
 env.PrependUnique(CPPPATH = [ os.path.join(src_dir, 'tizen')])
 
@@ -17,7 +18,11 @@ if 'MCD' in division:
     env.AppendUnique(CPPDEFINES = ['BLE_MCD'])
     print "BLE for MCD"
 
-if tizen30 == 'True':
+if tizen40 == 'True':
+    env.AppendUnique(CPPDEFINES = ['BLE_TIZEN_30'])
+    env.AppendUnique(CPPDEFINES = ['BLE_TIZEN_40'])
+    print "Build using Tizen 4.0 BLE API"
+elif tizen30 == 'True':
     env.AppendUnique(CPPDEFINES = ['BLE_TIZEN_30'])
     print "Build using Tizen 3.0 BLE API"
 
index cf81f373806429219978f23e5f29eb995ed7a83d..5c36a0324f57703606fa73c7f70b728132e20f4c 100644 (file)
@@ -28,7 +28,7 @@
 #include "oic_malloc.h"
 #include "caleutil.h"
 #include "cautilinterface.h"
-#ifndef TIZEN_4
+#ifndef BLE_TIZEN_40
 #include <bluetooth_internal.h>
 #endif
 
@@ -1185,7 +1185,7 @@ uint16_t CALEServerGetMtuSize(const char* address)
     unsigned int mtu = CA_DEFAULT_BLE_MTU_SIZE;
     int ret = 0;
 
-#ifdef TIZEN_4
+#ifdef BLE_TIZEN_40
     bt_gatt_client_h client = NULL;
     ret = bt_gatt_client_create(address, &client);
     if (0 != ret)
@@ -1218,7 +1218,7 @@ uint16_t CALEServerGetMtuSize(const char* address)
                   "bt_device_get_att_mtu failed with return [%s]", CALEGetErrorMsg(ret));
         return CA_DEFAULT_BLE_MTU_SIZE;
     }
-#endif // TIZEN_4
+#endif // BLE_TIZEN_40
 
     OIC_LOG_V(INFO, TAG, "mtu size(including header) from bt_device_get_att_mtu is %d", mtu);
     OIC_LOG(DEBUG, TAG, "OUT");
index 901d3d8f6cb20aff8fca3b1217ea50a21a730afd..bcfe6d41f5b7aee8689a68e397003ad8fea4bfb8 100644 (file)
@@ -29,7 +29,7 @@
 #include "caleutil.h"
 #include "caadapterutils.h"
 #include <glib.h>
-#ifndef TIZEN_4
+#ifndef BLE_TIZEN_40
 #include <bluetooth_internal.h>
 #endif
 
@@ -1047,7 +1047,7 @@ uint16_t CALEServerGetMtuSize(const char* address)
     unsigned int mtu = CA_DEFAULT_BLE_MTU_SIZE;
     int ret = 0;
 
-#ifdef TIZEN_4
+#ifdef BLE_TIZEN_40
     bt_gatt_client_h client = NULL;
     ret = bt_gatt_client_create(address, &client);
     if (0 != ret)
@@ -1080,7 +1080,7 @@ uint16_t CALEServerGetMtuSize(const char* address)
                   "bt_device_get_att_mtu failed with return [%s]", CALEGetErrorMsg(ret));
         return CA_DEFAULT_BLE_MTU_SIZE;
     }
-#endif // TIZEN_4
+#endif // BLE_TIZEN_40
 
     OIC_LOG_V(INFO, TAG, "mtu size(including header) from bt_device_get_att_mtu is %d", mtu);
     OIC_LOG(DEBUG, TAG, "OUT");
index 2817e65ecd5fadc4adb5750ff5108f5c9c29b800..46e8332f48390824e0a1fb4dcb7b7c068d236407 100644 (file)
@@ -1,7 +1,7 @@
 Name: iotivity
 Version: 1.2.1
 Release: 0
-Summary: IoT Connectivity sponsored by the OCF
+Summary: Samsung IoT Connectivity
 Group: Network & Connectivity / IoT Connectivity
 License: Apache-2.0
 URL: https://www.iotivity.org/
@@ -57,6 +57,7 @@ Source1002: %{name}-test.manifest
 %define ex_lib_dir %{buildroot}%{_libdir}
 %define ex_include_dir %{buildroot}%{_includedir}
 
+%if 0%{?tizen_version_major} < 3
 %if ! 0%{?license:0}
 %define license %doc
 %endif
@@ -64,6 +65,7 @@ Source1002: %{name}-test.manifest
 %if ! 0%{?manifest:0}
 %define manifest %doc
 %endif
+%endif
 
 # Default values to be eventually overiden BEFORE or as gbs params:
 %{!?ES_TARGET_ENROLLEE: %define ES_TARGET_ENROLLEE tizen}
@@ -80,12 +82,13 @@ Source1002: %{name}-test.manifest
 %{!?WITH_MQ: %define WITH_MQ OFF}
 %{!?WITH_PROXY: %define WITH_PROXY 0}
 %{!?WITH_TCP: %define WITH_TCP 1}
+%{!?DISABLE_TCP_SERVER: %define DISABLE_TCP_SERVER 1}
 %{!?RD_MODE: %define RD_MODE CLIENT}
 %{!?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}
+%{!?MULTIPLE_OWNER: %define MULTIPLE_OWNER 1}
+%{!?BLE_TIZEN_40: %define BLE_TIZEN_40 True}
 
 BuildRequires:  expat-devel
 BuildRequires:  python, libcurl-devel
@@ -177,13 +180,13 @@ scons %{JOB} --prefix=%{_prefix} \
     WITH_MQ=%{WITH_MQ} \
     WITH_PROXY=%{WITH_PROXY} \
     WITH_TCP=%{WITH_TCP} \
+    DISABLE_TCP_SERVER=%{DISABLE_TCP_SERVER} \
     RD_MODE=%{RD_MODE} \
     BLE_CUSTOM_ADV=%{BLE_CUSTOM_ADV} \
     BLE_DIVISION=%{BLE_DIVISION} \
-    BLE_TIZEN_30=%{BLE_TIZEN_30} \
+    BLE_TIZEN_40=%{BLE_TIZEN_40} \
     DISABLE_BLE_SERVER=%{DISABLE_BLE_SERVER} \
     MULTIPLE_OWNER=%{MULTIPLE_OWNER} \
-    OIC_SUPPORT_TIZEN_TRACE=%{OIC_SUPPORT_TIZEN_TRACE} \
     LOG_LEVEL=%{log_level} \
     #eol
 
@@ -196,7 +199,6 @@ cp LICENSE %{buildroot}/%{_datadir}/license/%{name}
 mkdir -p %{ex_install_dir}
 mkdir -p %{ex_include_dir}
 mkdir -p %{ex_lib_dir}/pkgconfig
-
 #cp out/%{TARGET_OS}/%{TARGET_ARCH}/%{build_mode}/examples/OICMiddle/OICMiddle %{ex_install_dir}
 #cp out/%{TARGET_OS}/%{TARGET_ARCH}/%{build_mode}/resource/examples/devicediscoveryclient %{ex_install_dir}
 #cp out/%{TARGET_OS}/%{TARGET_ARCH}/%{build_mode}/resource/examples/devicediscoveryserver %{ex_install_dir}
@@ -246,6 +248,7 @@ mkdir -p %{ex_include_dir}/mbedtls
 cp -R ./extlibs/mbedtls/mbedtls/include/mbedtls/ %{buildroot}%{_includedir}
 cp ./resource/csdk/security/include/*.h %{buildroot}%{_includedir}
 cp ./resource/csdk/connectivity/api/*.h %{buildroot}%{_includedir}/
+cp ./resource/csdk/security/include/internal/*.h %{buildroot}%{_includedir}/
 cp ./resource/csdk/security/provisioning/include/oxm/*.h %{buildroot}%{_includedir}
 cp ./resource/csdk/security/provisioning/include/internal/*.h %{buildroot}%{_includedir}
 cp ./resource/csdk/security/provisioning/include/*.h %{buildroot}%{_includedir}
@@ -256,6 +259,7 @@ cp ./resource/csdk/security/provisioning/sample/oic_svr_db_server_randompin.dat
 
 %endif
 
+
 mkdir -p %{ex_include_dir}/service/resource-encapsulation
 mkdir -p %{ex_include_dir}/service/notification
 mkdir -p %{ex_include_dir}/service/easy-setup
@@ -280,7 +284,6 @@ cp resource/csdk/stack/include/*.h %{buildroot}%{_includedir}/resource
 cp resource/oc_logger/include/*.h %{buildroot}%{_includedir}/resource
 cp resource/oc_logger/include/*.hpp %{buildroot}%{_includedir}/resource
 
-
 cp resource/c_common/*.h %{buildroot}%{_includedir}
 cp resource/c_common/ocrandom/include/*.h %{buildroot}%{_includedir}
 cp resource/c_common/oic_string/include/*.h %{buildroot}%{_includedir}