celix.git
3 weeks agoMerge pull request #414 from xuzhenbao/fix_rsalist_race master
PengZheng [Sun, 24 Apr 2022 06:00:24 +0000 (14:00 +0800)] 
Merge pull request #414 from xuzhenbao/fix_rsalist_race

Fix rsalist race

3 weeks agoremodify log level 414/head
xuzhenbao [Sun, 24 Apr 2022 04:20:00 +0000 (12:20 +0800)] 
remodify log level

4 weeks agochange a log level
xuzhenbao [Wed, 20 Apr 2022 16:04:22 +0000 (00:04 +0800)] 
change a log level

4 weeks agoreplace with celix_ prefixed array list api, and add some comments
xuzhenbao [Wed, 20 Apr 2022 13:50:29 +0000 (21:50 +0800)] 
replace with celix_ prefixed array list api, and add some comments

4 weeks agoMerge pull request #412 from pnoltes/feature/update_documentation
Pepijn Noltes [Tue, 19 Apr 2022 20:09:24 +0000 (22:09 +0200)] 
Merge pull request #412 from pnoltes/feature/update_documentation

Feature/update documentation

4 weeks agoRemoves TODOs, will be addressed in separate pull request 412/head
Pepijn Noltes [Tue, 19 Apr 2022 19:07:41 +0000 (21:07 +0200)] 
Removes TODOs, will be addressed in separate pull request

4 weeks agouse a single mutex to refactor topology manager
xuzhenbao [Mon, 18 Apr 2022 11:11:18 +0000 (19:11 +0800)] 
use a single mutex to refactor topology manager

5 weeks agoMerge branch 'apache:master' into fix_rsalist_race
xuzhenbao [Thu, 14 Apr 2022 12:59:10 +0000 (20:59 +0800)] 
Merge branch 'apache:master' into fix_rsalist_race

5 weeks agoMerge pull request #398 from rlenferink/add-asf-yaml
Roy Lenferink [Wed, 13 Apr 2022 19:00:25 +0000 (21:00 +0200)] 
Merge pull request #398 from rlenferink/add-asf-yaml

Add .asf.yaml for Celix repo

5 weeks agoEnable require pull requests 398/head
Roy Lenferink [Wed, 13 Apr 2022 18:59:19 +0000 (20:59 +0200)] 
Enable require pull requests

5 weeks agokeep rsaRemoved and rsaAdded strictly reverse order
xuzhenbao [Wed, 13 Apr 2022 06:24:10 +0000 (14:24 +0800)] 
keep rsaRemoved and rsaAdded strictly reverse order

5 weeks agoUpdates documentation based on review comments
Pepijn Noltes [Tue, 12 Apr 2022 19:04:35 +0000 (21:04 +0200)] 
Updates documentation based on review comments

5 weeks agoMerge pull request #413 from xuzhenbao/fix_RSA_use_after_free
Pepijn Noltes [Tue, 12 Apr 2022 17:49:36 +0000 (19:49 +0200)] 
Merge pull request #413 from xuzhenbao/fix_RSA_use_after_free

Fix rsa use after free

5 weeks agofix rsalist race
xuzhenbao [Tue, 12 Apr 2022 15:57:30 +0000 (23:57 +0800)] 
fix rsalist race

5 weeks agorename variable name 413/head
xuzhenbao [Tue, 12 Apr 2022 09:09:31 +0000 (17:09 +0800)] 
rename variable name

5 weeks agorename variable name
xuzhenbao [Tue, 12 Apr 2022 02:58:25 +0000 (10:58 +0800)] 
rename variable name

5 weeks agofix rsalist race
xuzhenbao [Mon, 11 Apr 2022 07:52:49 +0000 (15:52 +0800)] 
fix rsalist race

5 weeks agoFixes some small types in the Celix documentation
Pepijn Noltes [Sun, 10 Apr 2022 21:36:26 +0000 (23:36 +0200)] 
Fixes some small types in the Celix documentation

5 weeks agoAdds container documentation and moves component doc
Pepijn Noltes [Sun, 10 Apr 2022 19:34:42 +0000 (21:34 +0200)] 
Adds container documentation and moves component doc

5 weeks agoUpdates build, framework and dev documentation
Pepijn Noltes [Sun, 10 Apr 2022 19:04:01 +0000 (21:04 +0200)] 
Updates build, framework and dev documentation

5 weeks agoMerge remote-tracking branch 'origin/master' into feature/update_documentation
Pepijn Noltes [Sun, 10 Apr 2022 08:21:31 +0000 (10:21 +0200)] 
Merge remote-tracking branch 'origin/master' into feature/update_documentation

# Conflicts:
# CMakeLists.txt
# libs/utils/CMakeLists.txt

6 weeks agoMerge pull request #411 from apache/feature/conan_find_module_unification
PengZheng [Sat, 9 Apr 2022 02:58:03 +0000 (10:58 +0800)] 
Merge pull request #411 from apache/feature/conan_find_module_unification

Unify conan autogenerated find modules with Celix's own find modules.

6 weeks agoMinor indentation adjustment. 411/head
PengZheng [Thu, 7 Apr 2022 10:30:37 +0000 (18:30 +0800)] 
Minor indentation adjustment.

6 weeks agoUnify conan autogenerated find modules with Celix's own find modules.
PengZheng [Thu, 7 Apr 2022 10:25:58 +0000 (18:25 +0800)] 
Unify conan autogenerated find modules with Celix's own find modules.

6 weeks agoMerge pull request #409 from apache/feature/rename_zmq_and_jansson_targets_to_align_w...
Pepijn Noltes [Wed, 6 Apr 2022 18:00:54 +0000 (20:00 +0200)] 
Merge pull request #409 from apache/feature/rename_zmq_and_jansson_targets_to_align_with_conan

Feature/rename zmq and jansson targets to align with conan

6 weeks agoMerge pull request #410 from apache/feature/rsa_doc_improvement
PengZheng [Wed, 6 Apr 2022 07:03:10 +0000 (15:03 +0800)] 
Merge pull request #410 from apache/feature/rsa_doc_improvement

Minor doxygen document improvement.

6 weeks agoMinor doxygen document improvement. 410/head
PengZheng [Wed, 6 Apr 2022 07:00:41 +0000 (15:00 +0800)] 
Minor doxygen document improvement.

6 weeks agoMerge pull request #407 from xuzhenbao/fix_configurable_listening_ip
PengZheng [Wed, 6 Apr 2022 03:04:57 +0000 (11:04 +0800)] 
Merge pull request #407 from xuzhenbao/fix_configurable_listening_ip

make the  listening ip of RSA  configurable

6 weeks agoFixes some cmake config and re-enables C++ RSA tests 409/head
Pepijn Noltes [Tue, 5 Apr 2022 19:25:55 +0000 (21:25 +0200)] 
Fixes some cmake config and re-enables C++ RSA tests

6 weeks agoFixes some naming issues in the embedded Find<pkg>.cmake files.
Pepijn Noltes [Mon, 4 Apr 2022 17:20:10 +0000 (19:20 +0200)] 
Fixes some naming issues in the embedded Find<pkg>.cmake files.

6 weeks agoUpdates FindJansson.cmake to a lower case package name.
Pepijn Noltes [Mon, 4 Apr 2022 16:58:51 +0000 (18:58 +0200)] 
Updates FindJansson.cmake to a lower case package name.

Conan generates a Findjansson.cmake (lower case), so the embedded FindJansson.cmake is updated to align with this.

6 weeks agoUpdates FindZMQ.cmake and FindCZMQ.cmake conform the versions generated by conan.
Pepijn Noltes [Mon, 4 Apr 2022 16:56:15 +0000 (18:56 +0200)] 
Updates FindZMQ.cmake and FindCZMQ.cmake conform the versions generated by conan.

6 weeks agoMerge pull request #401 from apache/feature/conan_support
PengZheng [Mon, 4 Apr 2022 08:05:41 +0000 (16:05 +0800)] 
Merge pull request #401 from apache/feature/conan_support

Add basic Conan support

6 weeks agoAdd version to recipe, add executable linking workaround for test_package. 401/head
PengZheng [Mon, 4 Apr 2022 06:57:13 +0000 (14:57 +0800)] 
Add version to recipe, add executable linking workaround for test_package.

6 weeks agoReverts version handling in conanfile, applies undefined linker setup for macos and...
Pepijn Noltes [Sun, 3 Apr 2022 17:40:08 +0000 (19:40 +0200)] 
Reverts version handling in conanfile, applies undefined linker setup for macos and fixes openssl version

6 weeks agoUpdates wrong link target and update conan cmake configure if no version is found
Pepijn Noltes [Sun, 3 Apr 2022 12:09:53 +0000 (14:09 +0200)] 
Updates wrong link target and update conan cmake configure if no version is found

6 weeks agoUpdates conanfile.py and jansson linking for macos setup 408/head
Pepijn Noltes [Sun, 3 Apr 2022 11:56:31 +0000 (13:56 +0200)] 
Updates conanfile.py and jansson linking for macos setup

7 weeks agofix use-after-free when topology manager close imported services
xuzhenbao [Fri, 1 Apr 2022 14:16:21 +0000 (22:16 +0800)] 
fix use-after-free when topology manager close imported services

7 weeks agoAdd missing -ldl
PengZheng [Wed, 30 Mar 2022 12:51:52 +0000 (20:51 +0800)] 
Add missing -ldl

7 weeks agoadd option for binding to the RSA_IP 407/head
xuzhenbao [Wed, 30 Mar 2022 12:19:44 +0000 (20:19 +0800)] 
add option for binding to the RSA_IP

7 weeks agoAdd code coverage support in conan, and eliminate gcc warnings when using cpputest...
PengZheng [Mon, 28 Mar 2022 05:57:08 +0000 (13:57 +0800)] 
Add code coverage support in conan, and eliminate gcc warnings when using cpputest with build-in memory leak detector.

7 weeks agoRevert "Remove comments."
PengZheng [Mon, 28 Mar 2022 05:07:35 +0000 (13:07 +0800)] 
Revert "Remove comments."

This reverts commit 9198422dd2eee7ef402f844e45c9af268b10dfa3.

7 weeks agoRemove comments.
PengZheng [Mon, 28 Mar 2022 03:19:12 +0000 (11:19 +0800)] 
Remove comments.

7 weeks agoRemove unnecessary -DCPPUTEST_MEM_LEAK_DETECTION_DISABLED, and fix conan zeromq overr...
PengZheng [Mon, 28 Mar 2022 03:18:12 +0000 (11:18 +0800)] 
Remove unnecessary -DCPPUTEST_MEM_LEAK_DETECTION_DISABLED, and fix conan zeromq override warning.

7 weeks agoDisable cpputest's build-in memory_leak_detection completely when ASAN is enabled.
PengZheng [Sun, 27 Mar 2022 10:07:29 +0000 (18:07 +0800)] 
Disable cpputest's build-in memory_leak_detection completely when ASAN is enabled.

7 weeks agoMake clang sanitizer work with cpputest.
PengZheng [Sun, 27 Mar 2022 09:58:59 +0000 (17:58 +0800)] 
Make clang sanitizer work with cpputest.

7 weeks agoMake clang sanitizer work with cpputest.
PengZheng [Sun, 27 Mar 2022 09:22:36 +0000 (17:22 +0800)] 
Make clang sanitizer work with cpputest.

8 weeks agoAdd virtualrunenv generator to facilitate integration with CLion.
PengZheng [Sat, 26 Mar 2022 14:14:42 +0000 (22:14 +0800)] 
Add virtualrunenv generator to facilitate integration with CLion.

8 weeks agoMerge branch 'apache:master' into fix_configurable_listening_ip
xuzhenbao [Sat, 26 Mar 2022 13:08:21 +0000 (21:08 +0800)] 
Merge branch 'apache:master' into fix_configurable_listening_ip

8 weeks agomake the rsa listening ip configurable
xuzhenbao [Sat, 26 Mar 2022 10:44:43 +0000 (18:44 +0800)] 
make the rsa listening ip configurable

8 weeks agoFollowing recommended approach of using FindThreads.
PengZheng [Sat, 26 Mar 2022 06:43:19 +0000 (14:43 +0800)] 
Following recommended approach of using FindThreads.

See https://cmake.org/cmake/help/v3.2/module/FindThreads.html

8 weeks agoMinor comment update.
PengZheng [Fri, 25 Mar 2022 15:27:45 +0000 (23:27 +0800)] 
Minor comment update.

8 weeks agoAdd back workaround of transitive private shared library dependency issue.
PengZheng [Fri, 25 Mar 2022 10:59:45 +0000 (18:59 +0800)] 
Add back workaround of transitive private shared library dependency issue.

See https://github.com/conan-io/conan/issues/7192 for details. virtualrunenv doesn't help for cross-compilation.

8 weeks agoRevert "Fix transitive private shared library dependency."
PengZheng [Fri, 25 Mar 2022 10:51:22 +0000 (18:51 +0800)] 
Revert "Fix transitive private shared library dependency."

This reverts commit 08c590c2b0fa218642a660331529f0b48d42c552.

8 weeks agoFix transitive private shared library dependency.
PengZheng [Fri, 25 Mar 2022 08:54:12 +0000 (16:54 +0800)] 
Fix transitive private shared library dependency.

See https://github.com/conan-io/conan/issues/7192 for details.

8 weeks agoMerge pull request #404 from apache/hotfix/array_list_equality
PengZheng [Fri, 25 Mar 2022 07:48:29 +0000 (15:48 +0800)] 
Merge pull request #404 from apache/hotfix/array_list_equality

Fix broken array list equality on some 32-bit platforms where sizeof(…

8 weeks agoAdd missing DEPS.
PengZheng [Thu, 24 Mar 2022 13:15:15 +0000 (21:15 +0800)] 
Add missing DEPS.

8 weeks agoUse old version range syntax to increase compatibility.
PengZheng [Tue, 22 Mar 2022 13:05:21 +0000 (21:05 +0800)] 
Use old version range syntax to increase compatibility.

8 weeks agoMerge branch 'hotfix/array_list_equality' into conan_support
PengZheng [Tue, 22 Mar 2022 12:51:30 +0000 (20:51 +0800)] 
Merge branch 'hotfix/array_list_equality' into conan_support

8 weeks agoFix add/remove bugs reintroduced by 6975a4e1. 404/head
PengZheng [Tue, 22 Mar 2022 12:33:46 +0000 (20:33 +0800)] 
Fix add/remove bugs reintroduced by 6975a4e1.

8 weeks agoMerge branch 'master' into hotfix/array_list_equality
PengZheng [Tue, 22 Mar 2022 12:04:59 +0000 (20:04 +0800)] 
Merge branch 'master' into hotfix/array_list_equality

8 weeks agoMerge branch 'hotfix/array_list_equality' into conan_support
PengZheng [Tue, 22 Mar 2022 11:02:05 +0000 (19:02 +0800)] 
Merge branch 'hotfix/array_list_equality' into conan_support

8 weeks agoFix -Wunused-variable warnings.
PengZheng [Tue, 22 Mar 2022 10:58:41 +0000 (18:58 +0800)] 
Fix -Wunused-variable warnings.

8 weeks agoFix broken array list equality on some 32-bit platforms where sizeof(double)==8.
PengZheng [Tue, 22 Mar 2022 10:54:29 +0000 (18:54 +0800)] 
Fix broken array list equality on some 32-bit platforms where sizeof(double)==8.

Unnecessary memset calls are also removed. celix_arrayList_sort by design cannot be used with array of doubles, for which a warning is added.

8 weeks agoMake usage of conan in test_package completely non-intrusive, move test_package under...
PengZheng [Tue, 22 Mar 2022 04:43:03 +0000 (12:43 +0800)] 
Make usage of conan in test_package completely non-intrusive, move test_package under examples, and add minimum documentation.

8 weeks agoTurn off experimental CXX RSA by default.
PengZheng [Tue, 22 Mar 2022 02:35:04 +0000 (10:35 +0800)] 
Turn off experimental CXX RSA by default.

8 weeks agoMerge pull request #403 from apache/hotfix/psa_memory_leak
PengZheng [Tue, 22 Mar 2022 01:53:37 +0000 (09:53 +0800)] 
Merge pull request #403 from apache/hotfix/psa_memory_leak

Fix memory leaks in psa, reported by CI.

2 months agoAdjust option order of test_package to be consistent with that of Celix, add test_pac...
PengZheng [Mon, 21 Mar 2022 14:43:51 +0000 (22:43 +0800)] 
Adjust option order of test_package to be consistent with that of Celix, add test_package for RSA.

2 months agoMerge branch 'hotfix/psa_memory_leak' into conan_support
PengZheng [Mon, 21 Mar 2022 05:33:41 +0000 (13:33 +0800)] 
Merge branch 'hotfix/psa_memory_leak' into conan_support

2 months agoFix memory leaks in psa, reported by CI. 403/head
PengZheng [Mon, 21 Mar 2022 05:27:32 +0000 (13:27 +0800)] 
Fix memory leaks in psa, reported by CI.

2 months agoAdd test_package for remaining pubsub components.
PengZheng [Mon, 21 Mar 2022 04:14:44 +0000 (12:14 +0800)] 
Add test_package for remaining pubsub components.

2 months agoAdd test_package for pubsub and cxx_shell, add missing requirement for rapidjson.
PengZheng [Sun, 20 Mar 2022 13:58:44 +0000 (21:58 +0800)] 
Add test_package for pubsub and cxx_shell, add missing requirement for rapidjson.

2 months agoAdd build_all option, remove irrelevant packaging options, add workaround for CMake...
PengZheng [Sat, 19 Mar 2022 14:41:48 +0000 (22:41 +0800)] 
Add build_all option, remove irrelevant packaging options, add workaround for CMake shared library private linking issue.

For the CMake private linking issue, see https://github.com/conan-io/conan/issues/7192

2 months agoAdd test_package for pushstreams, fix compilation error, and make FindCppUTest.cmake...
PengZheng [Sat, 19 Mar 2022 08:36:20 +0000 (16:36 +0800)] 
Add test_package for pushstreams, fix compilation error, and make FindCppUTest.cmake compatible with the one generated by conan.

2 months agoAdd missing find_dependency in CelixConfig.cmake, and add test_package for etcdlib...
PengZheng [Fri, 18 Mar 2022 14:07:48 +0000 (22:07 +0800)] 
Add missing find_dependency in CelixConfig.cmake, and add test_package for etcdlib, launcher, and promises.

2 months agoMerge pull request #402 from apache/hotfix/framework_shutdown_deadlock
PengZheng [Fri, 18 Mar 2022 01:44:59 +0000 (09:44 +0800)] 
Merge pull request #402 from apache/hotfix/framework_shutdown_deadlock

Fix a first-check-then-do race condition during framework shutdown.

2 months agoAdd test_package for shell-related bundles, and remove irrelevant options from conan...
PengZheng [Thu, 17 Mar 2022 14:53:34 +0000 (22:53 +0800)] 
Add test_package for shell-related bundles, and remove irrelevant options from conan package ID calculation.

2 months agoMerge branch 'hotfix/framework_shutdown_deadlock' into conan_support
PengZheng [Thu, 17 Mar 2022 04:59:13 +0000 (12:59 +0800)] 
Merge branch 'hotfix/framework_shutdown_deadlock' into conan_support

2 months agoFix a first-check-then-do race condition during framework shutdown. 402/head
PengZheng [Thu, 17 Mar 2022 04:49:45 +0000 (12:49 +0800)] 
Fix a first-check-then-do race condition during framework shutdown.

2 months agoAdd test_package for syslog_writer.
PengZheng [Thu, 17 Mar 2022 04:22:26 +0000 (12:22 +0800)] 
Add test_package for syslog_writer.

2 months agoFix incorrect order of cmake.definitions and cmake.configure, avoid intrusive usage...
PengZheng [Thu, 17 Mar 2022 02:23:41 +0000 (10:23 +0800)] 
Fix incorrect order of cmake.definitions and cmake.configure, avoid intrusive usage of conan in CMakeLists.txt.

2 months agoAdd test_package for the framework, http_admin and log_admin.
PengZheng [Wed, 16 Mar 2022 14:29:07 +0000 (22:29 +0800)] 
Add test_package for the framework, http_admin and log_admin.

2 months agoAdd missing license.
PengZheng [Wed, 16 Mar 2022 03:10:39 +0000 (11:10 +0800)] 
Add missing license.

2 months agoImprove the modular structure of cmake project, update conanfile accordingly.
PengZheng [Tue, 15 Mar 2022 04:55:18 +0000 (12:55 +0800)] 
Improve the modular structure of cmake project, update conanfile accordingly.

Replace the broken automatic dependency deduction in celix_subproject with automatic error detection. Use celix_subproject systematically to reflect the current modular structure, and thus allowing more to be opted out. Essential and non-essential usages are carefully differentiated: e.g., PUBSUB_PSA_WS uses HTTP_ADMIN essentially, while all bundles use log_helper non-essentially. Now it's possible to turn off all bundles.

Minor cmake style improvements, including: replacing add_dependencies with add_celix_bundle_dependencies; use ${CMAKE_INSTALL_INCLUDEDIR} instead of fixed `include`; replacing $<INSTALL_INTERFACE:> with INCLUDES DESTINATION.

conanfile.py is also updated according to the current project structure. The default options reflect a reasonably minimal and functionally stable configuration: framework without etcd, log_admin, http_admin, pubsub_pas_ws with no need of external discovery mechanism, rsa with preconfigured discovery.

2 months agoFix bonjour shell compilation failure.
PengZheng [Fri, 11 Mar 2022 14:51:15 +0000 (22:51 +0800)] 
Fix bonjour shell compilation failure.

2 months agoDisables default extra compiler warnings
Pepijn Noltes [Thu, 10 Mar 2022 08:04:01 +0000 (09:04 +0100)] 
Disables default extra compiler warnings

2 months agoMerge remote-tracking branch 'origin/master' into feature/update_documentation
Pepijn Noltes [Thu, 10 Mar 2022 07:57:42 +0000 (08:57 +0100)] 
Merge remote-tracking branch 'origin/master' into feature/update_documentation

# Conflicts:
# examples/CMakeLists.txt

2 months agoPackage modules improvements.
PengZheng [Wed, 9 Mar 2022 14:47:13 +0000 (22:47 +0800)] 
Package modules improvements.

Avoid redefining libzip::libzip and support Jansson target for etcdlib.

2 months agoAdd suppress warning for cpputest code
Pepijn Noltes [Wed, 9 Mar 2022 09:53:19 +0000 (10:53 +0100)] 
Add suppress warning for cpputest code

2 months agoUpdates documentation and adds svc tracking intro and exmpl code.
Pepijn Noltes [Wed, 9 Mar 2022 09:45:23 +0000 (10:45 +0100)] 
Updates documentation and adds svc tracking intro and exmpl code.

2 months agoAdd missing options and loosen libzip requirement.
PengZheng [Tue, 8 Mar 2022 10:08:59 +0000 (18:08 +0800)] 
Add missing options and loosen libzip requirement.

2 months agoFix misuse of find_package and find_dependency in CelixConfig.cmake.
PengZheng [Tue, 8 Mar 2022 03:17:46 +0000 (11:17 +0800)] 
Fix misuse of find_package and find_dependency in CelixConfig.cmake.

Use find_dependency without REQUIRED to handle find_package(Celix) correctly (i.e., use Celix as an optional package).

2 months agoUse CelixConfig.cmake instead of conan's own mechanism, and add missing dependency.
PengZheng [Fri, 4 Mar 2022 13:56:20 +0000 (21:56 +0800)] 
Use CelixConfig.cmake instead of conan's own mechanism, and add missing dependency.

2 months agoAdd initial conan support.
PengZheng [Fri, 4 Mar 2022 04:51:03 +0000 (12:51 +0800)] 
Add initial conan support.

There are still some rough edges in CMake in-project dependency management.

2 months agoRevert "cxx_remote_service_admin as a separate subproject."
PengZheng [Fri, 4 Mar 2022 02:58:52 +0000 (10:58 +0800)] 
Revert "cxx_remote_service_admin as a separate subproject."

This reverts commit 58a2f881d8f76647e8349ecb24f7c8dd55165ce1.

2 months agoMerge branch 'master' into conan_support
PengZheng [Thu, 3 Mar 2022 15:11:11 +0000 (23:11 +0800)] 
Merge branch 'master' into conan_support

2 months agoMerge pull request #399 from PengZheng/tracker-improvements
PengZheng [Thu, 3 Mar 2022 15:04:20 +0000 (23:04 +0800)] 
Merge pull request #399 from PengZheng/tracker-improvements

Service tracker improvements

2 months agoMerge branch 'master' into conan_support
PengZheng [Tue, 1 Mar 2022 12:04:47 +0000 (20:04 +0800)] 
Merge branch 'master' into conan_support

2 months agoMinor code comment cleanup. 399/head
PengZheng [Tue, 1 Mar 2022 10:03:56 +0000 (18:03 +0800)] 
Minor code comment cleanup.

2 months agoUsing service directly does not support SOD by default.
PengZheng [Tue, 1 Mar 2022 08:28:32 +0000 (16:28 +0800)] 
Using service directly does not support SOD by default.

To request such support, the caller has to explicitly set CELIX_SERVICE_USE_SOD flags. Other code review issues are also addressed.