Commit 081508af authored by Daniel Vrátil's avatar Daniel Vrátil 🤖
Browse files

Clean up the CMake build system a bit

Most porting to use ecm_add_test and target_include_directories,
couple fixes of include paths and linking libraries. We try to make
sure that libs and apps don't have any other include paths other than
their own and those provided by their dependencies.

This will be useful in future if we try to split the kdepim repo.
parent f9f86bb4
include_directories(
${CMAKE_SOURCE_DIR}/akregator/interfaces
${CMAKE_SOURCE_DIR}
${CMAKE_CURRENT_SOURCE_DIR}
${CMAKE_CURRENT_BINARY_DIR}
)
......
......@@ -17,6 +17,8 @@ target_link_libraries(akregatorinterfaces
KF5::Syndication
)
target_include_directories(akregatorinterfaces PUBLIC "$<BUILD_INTERFACE:${akregator_SOURCE_DIR}/interfaces;${akregator_BINARY_DIR}/interfaces>")
set_target_properties(akregatorinterfaces PROPERTIES VERSION ${KDEPIM_LIB_VERSION} SOVERSION ${KDEPIM_LIB_SOVERSION}
)
......
......@@ -3,10 +3,6 @@
add_subdirectory(about)
add_subdirectory(icons)
include_directories(
${CMAKE_SOURCE_DIR}/akregator/interfaces
${CMAKE_BINARY_DIR}/akregator/interfaces
)
########### next target ###############
......@@ -70,6 +66,8 @@ PRIVATE
pimcommon
)
target_include_directories(akregatorprivate PUBLIC "$<BUILD_INTERFACE:${akregator_SOURCE_DIR}/src;${akregator_BINARY_DIR}/src>")
set_target_properties(akregatorprivate
PROPERTIES VERSION ${KDEPIM_LIB_VERSION}
SOVERSION ${KDEPIM_LIB_SOVERSION}
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment