Skip to content
Snippets Groups Projects
Commit 41a32cb9 authored by Jesse Mapel's avatar Jesse Mapel Committed by Jesse Mapel
Browse files

Fixed installing headers

parent e5f210da
No related branches found
No related tags found
No related merge requests found
...@@ -24,16 +24,17 @@ find_package(nlohmann_json REQUIRED) ...@@ -24,16 +24,17 @@ find_package(nlohmann_json REQUIRED)
add_library(ale SHARED add_library(ale SHARED
${CMAKE_CURRENT_SOURCE_DIR}/src/ale.cpp ${CMAKE_CURRENT_SOURCE_DIR}/src/ale.cpp
${CMAKE_CURRENT_SOURCE_DIR}/src/Rotation.cpp) ${CMAKE_CURRENT_SOURCE_DIR}/src/Rotation.cpp)
# Alias a scoped target for safer linking in downstream projects set(ALE_BUILD_INCLUDE_DIR "${CMAKE_CURRENT_SOURCE_DIR}/include/")
set(ALE_HEADERS "include/ale.h, include/Rotation.h") set(ALE_HEADERS "${ALE_BUILD_INCLUDE_DIR}/ale.h"
"${ALE_BUILD_INCLUDE_DIR}/Rotation.h")
set(ALE_INSTALL_INCLUDE_DIR "include/ale")
set_target_properties(ale PROPERTIES set_target_properties(ale PROPERTIES
VERSION ${PROJECT_VERSION} VERSION ${PROJECT_VERSION}
SOVERSION 0 SOVERSION 0)
PUBLIC_HEADER ${ALE_HEADERS})
# Use generator expressions so that downstream projects can use this target # Use generator expressions so that downstream projects can use this target
target_include_directories(ale target_include_directories(ale
PUBLIC PUBLIC
$<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/include> $<BUILD_INTERFACE:${ALE_BUILD_INCLUDE_DIR}>
$<INSTALL_INTERFACE:include>) $<INSTALL_INTERFACE:include>)
target_link_libraries(ale target_link_libraries(ale
...@@ -73,11 +74,14 @@ configure_file(cmake/config.cmake.in ...@@ -73,11 +74,14 @@ configure_file(cmake/config.cmake.in
install(FILES ${CMAKE_CURRENT_BINARY_DIR}/${PROJECT_NAME}-config.cmake install(FILES ${CMAKE_CURRENT_BINARY_DIR}/${PROJECT_NAME}-config.cmake
DESTINATION ${CMAKE_INSTALL_LIBDIR}/cmake/${PROJECT_NAME}) DESTINATION ${CMAKE_INSTALL_LIBDIR}/cmake/${PROJECT_NAME})
# Install the headers
install(FILES ${ALE_HEADERS} DESTINATION ${ALE_INSTALL_INCLUDE_DIR})
# Install the library # Install the library
install(TARGETS ale install(TARGETS ale
EXPORT aleTargets EXPORT aleTargets
LIBRARY DESTINATION ${CMAKE_INSTALL_LIBDIR} LIBRARY DESTINATION ${CMAKE_INSTALL_LIBDIR}
PUBLIC_HEADER DESTINATION ${CMAKE_INSTALL_INCLUDEDIR}) INCLUDES DESTINATION ${ALE_INSTALL_INCLUDE_DIR})
# Install the target # Install the target
install(EXPORT aleTargets install(EXPORT aleTargets
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment