Skip to content

Commit

Permalink
fixup 24ae2d7 (#178)
Browse files Browse the repository at this point in the history
Actually install headers and do not redundantly add the include dir to the target.
`INCLUDES DESTINATION` flags in `install(TARGETS` define additional paths that are
added to the INTERFACE_INCLUDE_DIRECTORIES, they do not install files.
  • Loading branch information
v4hn authored Jan 19, 2024
1 parent 9612aa4 commit bef286c
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -98,13 +98,15 @@ endif()
# Add support for installation
include(CMakePackageConfigHelpers)

# Install headers
install(DIRECTORY include/ruckig DESTINATION ${CMAKE_INSTALL_INCLUDEDIR})

# Install library
install(TARGETS ruckig
EXPORT ${PROJECT_NAME}-targets
ARCHIVE DESTINATION ${CMAKE_INSTALL_LIBDIR}
LIBRARY DESTINATION ${CMAKE_INSTALL_LIBDIR}
RUNTIME DESTINATION ${CMAKE_INSTALL_BINDIR}
INCLUDES DESTINATION ${CMAKE_INSTALL_INCLUDEDIR}
)

# Install CMake config files
Expand Down

0 comments on commit bef286c

Please sign in to comment.