Minor indentation adjustment.

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

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

Merge pull request #410 from apache/feature/rsa_doc_improvement

Minor doxygen document improvement.

Minor doxygen document improvement.

Merge pull request #407 from xuzhenbao/fix_configurable_listening_ip

make the  listening ip of RSA  configurable

Fixes some cmake config and re-enables C++ RSA tests

Fixes some naming issues in the embedded Find<pkg>.cmake files.

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.

Updates FindZMQ.cmake and FindCZMQ.cmake conform the versions generated by conan.

Merge pull request #401 from apache/feature/conan_support

Add basic Conan support

Add version to recipe, add executable linking workaround for test_package.

Reverts version handling in conanfile, applies undefined linker setup for macos and fixes openssl version

Updates wrong link target and update conan cmake configure if no version is found

Updates conanfile.py and jansson linking for macos setup

Add missing -ldl

add option for binding to the RSA_IP

Add code coverage support in conan, and eliminate gcc warnings when using cpputest with build-in memory leak detector.

Revert "Remove comments."

This reverts commit 9198422dd2eee7ef402f844e45c9af268b10dfa3.

Remove comments.

Remove unnecessary -DCPPUTEST_MEM_LEAK_DETECTION_DISABLED, and fix conan zeromq override warning.

Disable cpputest's build-in memory_leak_detection completely when ASAN is enabled.

Make clang sanitizer work with cpputest.

Make clang sanitizer work with cpputest.

Add virtualrunenv generator to facilitate integration with CLion.

Merge branch 'apache:master' into fix_configurable_listening_ip

make the rsa listening ip configurable

Following recommended approach of using FindThreads.

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

Minor comment update.

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.

Revert "Fix transitive private shared library dependency."

This reverts commit 08c590c2b0fa218642a660331529f0b48d42c552.

Fix transitive private shared library dependency.

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

Merge pull request #404 from apache/hotfix/array_list_equality

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

Add missing DEPS.

Use old version range syntax to increase compatibility.

Merge branch 'hotfix/array_list_equality' into conan_support

Fix add/remove bugs reintroduced by 6975a4e1.

Merge branch 'master' into hotfix/array_list_equality

Merge branch 'hotfix/array_list_equality' into conan_support

Fix -Wunused-variable warnings.

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.

Make usage of conan in test_package completely non-intrusive, move test_package under examples, and add minimum documentation.

Turn off experimental CXX RSA by default.

Merge pull request #403 from apache/hotfix/psa_memory_leak

Fix memory leaks in psa, reported by CI.

Adjust option order of test_package to be consistent with that of Celix, add test_package for RSA.

Merge branch 'hotfix/psa_memory_leak' into conan_support

Fix memory leaks in psa, reported by CI.

Add test_package for remaining pubsub components.

Add test_package for pubsub and cxx_shell, add missing requirement for rapidjson.

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

Add test_package for pushstreams, fix compilation error, and make FindCppUTest.cmake compatible with the one generated by conan.

Add missing find_dependency in CelixConfig.cmake, and add test_package for etcdlib, launcher, and promises.

Merge pull request #402 from apache/hotfix/framework_shutdown_deadlock

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

Add test_package for shell-related bundles, and remove irrelevant options from conan package ID calculation.

Merge branch 'hotfix/framework_shutdown_deadlock' into conan_support

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

Add test_package for syslog_writer.

Fix incorrect order of cmake.definitions and cmake.configure, avoid intrusive usage of conan in CMakeLists.txt.

Add test_package for the framework, http_admin and log_admin.

Add missing license.

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.

Fix bonjour shell compilation failure.

Package modules improvements.

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

Add missing options and loosen libzip requirement.

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).

Use CelixConfig.cmake instead of conan's own mechanism, and add missing dependency.

Add initial conan support.

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

Revert "cxx_remote_service_admin as a separate subproject."

This reverts commit 58a2f881d8f76647e8349ecb24f7c8dd55165ce1.

Merge branch 'master' into conan_support

Merge pull request #399 from PengZheng/tracker-improvements

Service tracker improvements

Merge branch 'master' into conan_support

Minor code comment cleanup.

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.

Merge branch 'master' into tracker-improvements

cxx_remote_service_admin as a separate subproject.

It used GPLv3ed zmq by default, which cannot be included in most proprietary softwares deployed in embedded device.

Merge pull request #400 from xuzhenbao/rs_error_code_fix

Improved error codes for remote services

Use reference counting to protect against concurrently unregistering the same serviceId multiple times.

Reference counting together with atomic compare-and-swap make celix_serviceRegistry_unregisterService MT-safe.

rename CELIX_FACILITY_NULL to CELIX_FACILITY_FRAMEWORK

Use flags instead of bool to accommodate future API extension.

Performance improvements for celix_bundleContext_useService(s)WithOptions.

Remove unnecessary celix_framework_waitForEmptyEventQueue from the indirect call path, while replace it with more efficient celix_framework_waitUntilNoPendingRegistration  for the direct call path. Added framework->dispatcher.stats can also be used for diagnostic purpose.

Service tracker performance optimization and bug fix.

Add options to allow call use callback directly from the caller thread instead of delegating to the Celix event loop. The default behaviour is kept to avoid introducing various ABBA deadlocks into existing bundles. A race condition, which leads to resource leak, is also fixed.

Convert jsonRpc Error to Celix default error

Further service tracker performance optimization preliminary.

celix_bundleContext_useService(s)WithOptions invoke service in the main event loop, which means any blocking service is blocking the whole world. Also, usleep(10) polling is expensive if high-resolution timer is enabled on Linux. This commit add support out of loop service calling to inner tracker implementation without modify current API behaviour.

Revert "Service tracker performance optimization."

This reverts commit 11c1ca1eda59c78e5bd3499973c91793fbeabcab.

change celix_errno.h according to code review

Service tracker performance optimization.

Previously celix_bundleContext_useService(s)WithOptions invoke service in the main event loop, which means any blocking service is blocking the whole world. Also, usleep(10) polling is expensive if high-resolution timer is enabled on Linux. This commit fixes both of them.

Merge remote-tracking branch 'origin/tracker-improvements' into tracker-improvements

fix thread-safety bug introduced by 691dc6cb and broken tracker set callback.

One intent of 691dc6cb is to make bundleContext_retainServiceReference and bundleContext_retainServiceReference lockless. Unfortunately it introduced a race condition: if a dying reference (still in registry) get revived by another call of bundleContext_getServiceReferences before it's actually removed from the registry and destroyed, hazard happens.

The set callback of an all-service tracker, whose service name is empty, did not work as expected by CelixBundleContextServicesTests.TrackerOfAllServicesSetTest. Unnecessary set callbacks when closing trackers are eliminated.

fix thread-safety bug introduced by 691dc6cb and broken tracker set callback.

One intent of 691dc6cb is to make bundleContext_retainServiceReference and bundleContext_retainServiceReference lockless. Unfortunately it introduced a race condition: if a dying reference (still in registry) get revived by another call of bundleContext_getServiceReferences before it's actually removed from the registry and destroyed, hazard happens.

The set callback of an all-service tracker, whose service name is empty, did not work as expected by CelixBundleContextServicesTests.TrackerOfAllServicesSetTest. Unnecessary set callbacks are eliminated.

remove redundant code

resloved conflict

fix memory leak for interceptor callback return false

Eliminate unnecessary locking and fix race condition when unregistering.

There is no need of locking when accessing read-only fields of serviceRegistration and serviceReference. Also a first-check-then-do race condition in serviceRegistration_unregister is fixed using atomic compare-and-swap.

Simplify serviceReference implementation by making the embedded registration always accessible during its lifetime.

service proxy handle interceptor exception

record CELIX_START_USERERR deprecated

Modify error code comments format

resolve pull conflicts

fix pull conflicts

Get the invocation error of remote service function in client