Skip to content

Commit

Permalink
Updates FindJansson.cmake to a lower case package name.
Browse files Browse the repository at this point in the history
Conan generates a Findjansson.cmake (lower case), so the embedded FindJansson.cmake is updated to align with this.
  • Loading branch information
pnoltes committed Apr 4, 2022
1 parent f02b651 commit 3f93d42
Show file tree
Hide file tree
Showing 14 changed files with 24 additions and 24 deletions.
2 changes: 1 addition & 1 deletion bundles/http_admin/gtest/CMakeLists.txt
Expand Up @@ -15,7 +15,7 @@
# specific language governing permissions and limitations
# under the License.

find_package(Jansson REQUIRED)
find_package(jansson REQUIRED)

add_celix_bundle(http_admin_sut
#"Vanilla" bundle which is under test
Expand Down
8 changes: 4 additions & 4 deletions bundles/pubsub/examples/CMakeLists.txt
Expand Up @@ -22,7 +22,7 @@ if (PUBSUB_EXAMPLES)
find_program(ETCD_CMD NAMES etcd)
find_program(XTERM_CMD NAMES xterm)

find_package(Jansson REQUIRED)
find_package(jansson REQUIRED)

set(PUBSUB_CONTAINER_LIBS jansson::jansson Celix::dfi)

Expand Down Expand Up @@ -176,10 +176,10 @@ if (PUBSUB_EXAMPLES)
endif()

if (BUILD_PUBSUB_PSA_ZMQ)
find_package(ZMQ REQUIRED)
find_package(CZMQ REQUIRED)
find_package(ZeroMQ REQUIRED)
find_package(czmq REQUIRED)

set(PUBSUB_CONTAINER_LIBS ${PUBSUB_CONTAINER_LIBS} ZMQ::lib CZMQ::lib ${OPTIONAL_OPENSSL_LIB})
set(PUBSUB_CONTAINER_LIBS ${PUBSUB_CONTAINER_LIBS} ZeroMQ::ZeroMQ czmq::czmq ${OPTIONAL_OPENSSL_LIB})

if (BUILD_PUBSUB_PSA_TCP AND BUILD_PUBSUB_PSA_UDP_MC)
# Dynamic ZMQ / UDP / TCP admin
Expand Down
16 changes: 8 additions & 8 deletions bundles/pubsub/integration/CMakeLists.txt
Expand Up @@ -17,7 +17,7 @@

celix_subproject(PUBSUB_INTEGRATION "Option to build the pubsub integration" ON DEPS SHELL SHELL_TUI)
if (PUBSUB_INTEGRATION)
find_package(Jansson REQUIRED)
find_package(jansson REQUIRED)

add_celix_bundle(pubsub_endpoint_sut
#"Vanilla" bundle which is under test
Expand Down Expand Up @@ -376,8 +376,8 @@ if (PUBSUB_INTEGRATION)
endif()

if (BUILD_PUBSUB_PSA_ZMQ)
find_package(ZMQ REQUIRED)
find_package(CZMQ REQUIRED)
find_package(ZeroMQ REQUIRED)
find_package(czmq REQUIRED)

add_celix_container(pubsub_zmq_v2_tests
USE_CONFIG #ensures that a config.properties will be created with the launch bundles.
Expand All @@ -396,7 +396,7 @@ if (PUBSUB_INTEGRATION)
pubsub_serializer
)

target_link_libraries(pubsub_zmq_v2_tests PRIVATE Celix::pubsub_api Celix::dfi ZMQ::lib CZMQ::lib GTest::gtest GTest::gtest_main)
target_link_libraries(pubsub_zmq_v2_tests PRIVATE Celix::pubsub_api Celix::dfi ZeroMQ::ZeroMQ czmq::czmq GTest::gtest GTest::gtest_main)
target_include_directories(pubsub_zmq_v2_tests SYSTEM PRIVATE gtest)
add_test(NAME pubsub_zmq_v2_tests COMMAND pubsub_zmq_v2_tests WORKING_DIRECTORY $<TARGET_PROPERTY:pubsub_zmq_v2_tests,CONTAINER_LOC>)
setup_target_for_coverage(pubsub_zmq_v2_tests SCAN_DIR ..)
Expand All @@ -421,7 +421,7 @@ if (PUBSUB_INTEGRATION)
pubsub_serializer
)

target_link_libraries(pubsub_zmq_v2_zerocopy_tests PRIVATE Celix::pubsub_api Celix::dfi ZMQ::lib CZMQ::lib GTest::gtest GTest::gtest_main)
target_link_libraries(pubsub_zmq_v2_zerocopy_tests PRIVATE Celix::pubsub_api Celix::dfi ZeroMQ::ZeroMQ czmq::czmq GTest::gtest GTest::gtest_main)
target_include_directories(pubsub_zmq_v2_zerocopy_tests SYSTEM PRIVATE gtest)

add_test(NAME pubsub_zmq_v2_zerocopy_tests COMMAND pubsub_zmq_v2_zerocopy_tests WORKING_DIRECTORY $<TARGET_PROPERTY:pubsub_zmq_v2_zerocopy_tests,CONTAINER_LOC>)
Expand All @@ -442,7 +442,7 @@ if (PUBSUB_INTEGRATION)
)

target_compile_definitions(pstm_deadlock_zmq_v2_test PRIVATE -DDEADLOCK_SUT_BUNDLE_FILE=\"${DEADLOCK_SUT_BUNDLE_FILE}\")
target_link_libraries(pstm_deadlock_zmq_v2_test PRIVATE Celix::pubsub_api GTest::gtest GTest::gtest_main Celix::dfi ZMQ::lib CZMQ::lib)
target_link_libraries(pstm_deadlock_zmq_v2_test PRIVATE Celix::pubsub_api GTest::gtest GTest::gtest_main Celix::dfi ZeroMQ::ZeroMQ czmq::czmq)
target_include_directories(pstm_deadlock_zmq_v2_test SYSTEM PRIVATE pstm_deadlock_zmq_v2_test)

#Note we do not link to bundles, as result (to ensure a bundle zip file is created) an dependency on the bundle is needed.
Expand Down Expand Up @@ -535,8 +535,8 @@ if (PUBSUB_INTEGRATION)
endif ()

#Linking against zmq if present
if (TARGET ZMQ::lib AND TARGET CZMQ::lib)
target_link_libraries(${TEST_TARGET_NAME} PRIVATE ZMQ::lib CZMQ::lib)
if (TARGET ZeroMQ::ZeroMQ AND TARGET czmq::czmq)
target_link_libraries(${TEST_TARGET_NAME} PRIVATE ZeroMQ::ZeroMQ czmq::czmq)
endif ()
endfunction()

Expand Down
2 changes: 1 addition & 1 deletion bundles/pubsub/pubsub_admin_udp_mc/CMakeLists.txt
Expand Up @@ -17,7 +17,7 @@

celix_subproject(PUBSUB_PSA_UDP_MC "Build UDP MC PubSub Admin" ON)
if (PUBSUB_PSA_UDP_MC)
find_package(Jansson REQUIRED)
find_package(jansson REQUIRED)

add_celix_bundle(celix_pubsub_admin_udp_multicast
BUNDLE_SYMBOLICNAME "apache_celix_pubsub_admin_udp_multicast"
Expand Down
2 changes: 1 addition & 1 deletion bundles/pubsub/pubsub_admin_websocket/CMakeLists.txt
Expand Up @@ -17,7 +17,7 @@

celix_subproject(PUBSUB_PSA_WS "Build WebSocket PubSub Admin" ON DEPS HTTP_ADMIN)
if (PUBSUB_PSA_WS)
find_package(Jansson REQUIRED)
find_package(jansson REQUIRED)
find_package(UUID REQUIRED)

add_celix_bundle(celix_pubsub_admin_websocket
Expand Down
2 changes: 1 addition & 1 deletion bundles/pubsub/pubsub_discovery/CMakeLists.txt
Expand Up @@ -18,7 +18,7 @@
celix_subproject(PUBSUB_DISCOVERY_ETCD "Option to enable building the PubSub Discovery (ETCD) bundle" ON DEPS CELIX_ETCDLIB)
if (PUBSUB_DISCOVERY_ETCD)
find_package(CURL REQUIRED)
find_package(Jansson REQUIRED)
find_package(jansson REQUIRED)

add_celix_bundle(celix_pubsub_discovery_etcd
BUNDLE_SYMBOLICNAME "apache_celix_pubsub_discovery_etcd"
Expand Down
4 changes: 2 additions & 2 deletions bundles/pubsub/pubsub_serializer_avrobin/CMakeLists.txt
Expand Up @@ -15,7 +15,7 @@
# specific language governing permissions and limitations
# under the License.

find_package(Jansson REQUIRED)
find_package(jansson REQUIRED)

add_celix_bundle(celix_pubsub_serializer_avrobin
BUNDLE_SYMBOLICNAME "apache_celix_pubsub_serializer_avrobin"
Expand All @@ -40,4 +40,4 @@ add_library(Celix::celix_pubsub_serializer_avrobin ALIAS celix_pubsub_serializer

if (ENABLE_TESTING)
add_subdirectory(gtest)
endif(ENABLE_TESTING)
endif(ENABLE_TESTING)
2 changes: 1 addition & 1 deletion bundles/pubsub/pubsub_serializer_json/CMakeLists.txt
Expand Up @@ -15,7 +15,7 @@
# specific language governing permissions and limitations
# under the License.

find_package(Jansson REQUIRED)
find_package(jansson REQUIRED)


add_celix_bundle(celix_pubsub_serializer_json
Expand Down
2 changes: 1 addition & 1 deletion bundles/remote_services/discovery_etcd/CMakeLists.txt
Expand Up @@ -19,7 +19,7 @@ celix_subproject(RSA_DISCOVERY_ETCD "Option to enable building the Discovery (ET
if (RSA_DISCOVERY_ETCD)
find_package(CURL REQUIRED)
find_package(LibXml2 REQUIRED)
find_package(Jansson REQUIRED)
find_package(jansson REQUIRED)

add_celix_bundle(rsa_discovery_etcd
VERSION 0.9.0
Expand Down
Expand Up @@ -18,7 +18,7 @@
celix_subproject(RSA_REMOTE_SERVICE_ADMIN_DFI "Option to enable building the Remote Service Admin Service DFI" ON)
if (RSA_REMOTE_SERVICE_ADMIN_DFI)
find_package(CURL REQUIRED)
find_package(Jansson REQUIRED)
find_package(jansson REQUIRED)
find_package(UUID REQUIRED)

add_celix_bundle(rsa_dfi
Expand Down
2 changes: 1 addition & 1 deletion bundles/remote_services/topology_manager/CMakeLists.txt
Expand Up @@ -34,7 +34,7 @@ install_celix_bundle(rsa_topology_manager EXPORT celix COMPONENT rsa)
add_library(Celix::rsa_topology_manager ALIAS rsa_topology_manager)

if (ENABLE_TESTING AND BUILD_RSA_REMOTE_SERVICE_ADMIN_DFI)
find_package(Jansson REQUIRED)
find_package(jansson REQUIRED)
add_subdirectory(tms_tst)
endif ()

File renamed without changes.
2 changes: 1 addition & 1 deletion libs/dfi/CMakeLists.txt
Expand Up @@ -16,7 +16,7 @@
# under the License.

find_package(FFI REQUIRED)
find_package(Jansson REQUIRED)
find_package(jansson REQUIRED)

set(SOURCES
src/dyn_common.c
Expand Down
2 changes: 1 addition & 1 deletion libs/etcdlib/CMakeLists.txt
Expand Up @@ -37,7 +37,7 @@ if (NOT COMMAND celix_subproject)
set(ETCDLIB_CMP ${CMAKE_INSTALL_DEFAULT_COMPONENT_NAME})

find_package(CURL REQUIRED)
find_package(Jansson REQUIRED)
find_package(jansson REQUIRED)

if (NOT TARGET CURL::libcurl)
#Note more recent curl will create CURL::libcurl target
Expand Down

0 comments on commit 3f93d42

Please sign in to comment.