From fcbee1b981b9f53430f03d414f6e17d489453fe6 Mon Sep 17 00:00:00 2001 From: Mathieu Duponchelle Date: Tue, 25 Oct 2016 21:13:39 +0200 Subject: AudioManagerUtilities: use LINK_DIRECTORIES The -L flags were not retrieved from pkg_check_modules, except for automotive-dlt, where LDFLAGS was added to the LIBS variable, which was then passed to TARGET_LINK_LIBRARIES. This led to compile-time errors when compiling in a custom prefix. We also propagate the LIB_DIRECTORIES and use them in AmControlInterfaceTest and AudioManagerDaemon. --- AudioManagerUtilities/CMakeLists.txt | 31 +++++++++++++++++++++++++++---- 1 file changed, 27 insertions(+), 4 deletions(-) (limited to 'AudioManagerUtilities/CMakeLists.txt') diff --git a/AudioManagerUtilities/CMakeLists.txt b/AudioManagerUtilities/CMakeLists.txt index ea3f142..6e9a06f 100644 --- a/AudioManagerUtilities/CMakeLists.txt +++ b/AudioManagerUtilities/CMakeLists.txt @@ -23,6 +23,8 @@ SET(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -fPIC") SET(AUDIOMANAGER_UTILITIES_EXTERNAL_INCLUDE "") +SET(AUDIOMANAGER_UTILITIES_EXTERNAL_LIB_DIRECTORIES "") + SET(AUDIO_MANAGER_UTILITIES_LIBS ${CMAKE_DL_LIBS} CACHE INTERNAL "AUDIO_MANAGER_UTILITIES_LIBS libs" FORCE) @@ -46,6 +48,9 @@ if(WITH_SYSTEMD_WATCHDOG) ${AUDIO_MANAGER_UTILITIES_LIBS} ${SYSTEMD_LIBRARIES} CACHE INTERNAL "AUDIO_MANAGER_UTILITIES_LIBS libs" FORCE) + set(AUDIOMANAGER_UTILITIES_EXTERNAL_LIB_DIRECTORIES + ${AUDIOMANAGER_UTILITIES_EXTERNAL_LIB_DIRECTORIES} + ${SYSTEMD_LIBRARY_DIRS}) set(AUDIOMANAGER_UTILITIES_EXTERNAL_INCLUDE ${AUDIOMANAGER_UTILITIES_EXTERNAL_INCLUDE} ${SYSTEMD_INCLUDEDIR}) @@ -67,6 +72,10 @@ IF (WITH_DBUS_WRAPPER) ${DBUS_LIBRARIES} CACHE INTERNAL "AUDIO_MANAGER_UTILITIES_LIBS libs" FORCE) + set(AUDIOMANAGER_UTILITIES_EXTERNAL_LIB_DIRECTORIES + ${AUDIOMANAGER_UTILITIES_EXTERNAL_LIB_DIRECTORIES} + ${DBUS_LIBRARY_DIRS}) + set(AUDIO_MANAGER_UTILITIES_SRCS_CXX ${AUDIO_MANAGER_UTILITIES_SRCS_CXX} src/CAmDbusWrapper.cpp) @@ -81,6 +90,11 @@ IF (WITH_CAPI_WRAPPER) ${COMMONAPI_DBUS_LIBRARIES} CACHE INTERNAL "AUDIO_MANAGER_UTILITIES_LIBS libs" FORCE) + set(AUDIOMANAGER_UTILITIES_EXTERNAL_LIB_DIRECTORIES + ${AUDIOMANAGER_UTILITIES_EXTERNAL_LIB_DIRECTORIES} + ${COMMONAPI_LIBRARY_DIRS} + ${COMMONAPI_DBUS_LIBRARY_DIRS}) + set(AUDIO_MANAGER_UTILITIES_SRCS_CXX ${AUDIO_MANAGER_UTILITIES_SRCS_CXX} src/CAmCommonAPIWrapper.cpp) @@ -90,20 +104,24 @@ IF (WITH_CAPI_WRAPPER) ${COMMONAPI_INCLUDE_DIRS}) ENDIF (WITH_CAPI_WRAPPER) -if(WITH_DLT) - pkg_check_modules(DLT REQUIRED "automotive-dlt >= 2.2.0") +IF (WITH_DLT) + pkg_check_modules(DLT REQUIRED "automotive-dlt >= 2.2.0") add_definitions(${DLT_CFLAGS_OTHER}) set(AUDIO_MANAGER_UTILITIES_LIBS ${AUDIO_MANAGER_UTILITIES_LIBS} - ${DLT_LDFLAGS} + ${DLT_LIBRARIES} CACHE INTERNAL "AUDIO_MANAGER_UTILITIES_LIBS libs" FORCE) + set(AUDIOMANAGER_UTILITIES_EXTERNAL_LIB_DIRECTORIES + ${AUDIOMANAGER_UTILITIES_EXTERNAL_LIB_DIRECTORIES} + ${DLT_LIBRARY_DIRS}) + SET(AUDIOMANAGER_UTILITIES_EXTERNAL_INCLUDE ${AUDIOMANAGER_UTILITIES_EXTERNAL_INCLUDE} ${DLT_INCLUDE_DIRS}) -endif(WITH_DLT) +endif (WITH_DLT) SET(AUDIOMANAGER_UTILITIES_INCLUDE ${AUDIOMANAGER_UTILITIES}/include @@ -113,9 +131,14 @@ SET(AUDIOMANAGER_UTILITIES_INCLUDE ${AUDIOMANAGER_UTILITIES_EXTERNAL_INCLUDE} CACHE INTERNAL "AUDIOMANAGER_INCLUDE directories" FORCE) +SET(AUDIOMANAGER_UTILITIES_LIB_DIRECTORIES + ${AUDIOMANAGER_UTILITIES_EXTERNAL_LIB_DIRECTORIES} + CACHE INTERNAL "AUDIOMANAGER_UTILITIES_LIB_DIRECTORIES directories" FORCE) INCLUDE_DIRECTORIES(${AUDIOMANAGER_UTILITIES_INCLUDE}) +LINK_DIRECTORIES(${AUDIOMANAGER_UTILITIES_LIB_DIRECTORIES}) + ADD_LIBRARY(AudioManagerUtilities ${LIBRARY_TYPE} ${AUDIO_MANAGER_UTILITIES_SRCS_CXX}) TARGET_LINK_LIBRARIES(AudioManagerUtilities ${AUDIO_MANAGER_UTILITIES_LIBS}) -- cgit v1.2.1