Merge pull request #49 from neil-forks/master

Add DESTINATION for PUBLIC_HEADERS targets
2025-06-25 04:54:44 +00:00 · 2023-08-07 22:21:47 -05:00 · 2023-08-07 22:49:15 -04:00 · 2023-08-07 21:24:22 -05:00 · 2023-08-07 21:18:43 -04:00
1 changed files with 8 additions and 2 deletions
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@ -90,7 +90,13 @@ else()
    endif()
 endif()

-install(TARGETS libblisp libblisp_static DESTINATION lib)
+include(GNUInstallDirs)
+install(TARGETS libblisp libblisp_static 
+    RUNTIME DESTINATION ${CMAKE_INSTALL_BINDIR}
+    LIBRARY DESTINATION ${CMAKE_INSTALL_LIBDIR}
+    ARCHIVE DESTINATION ${CMAKE_INSTALL_LIBDIR}
+    PUBLIC_HEADER DESTINATION ${CMAKE_INSTALL_INCLUDEDIR}
+)

 if(BLISP_BUILD_CLI)
    add_subdirectory(tools/blisp)
Author	SHA1	Message	Date
Robert Lipe	d2fef0af22	Merge pull request #49 from neil-forks/master Add DESTINATION for PUBLIC_HEADERS targets	2023-08-07 22:21:47 -05:00
Neil Hanlon	2203a250e3	Add DESTINATION for PUBLIC_HEADERS targets	2023-08-07 22:49:15 -04:00
Robert Lipe	713e444656	Merge pull request #48 from neil-forks/master WIP: Use CMAKE_INSTALL_LIBDIR for install destination to respect build forges	2023-08-07 21:24:22 -05:00
Neil Hanlon	3b47993de7	Use CMAKE_INSTALL_LIBDIR for install destination to respect build forges CMAKE_INSTALL_LIBDIR should be popualted with either lib or lib64, depending on the architecture. This change makes it so that this flag is respected by blisp and the library, runtime, and archives are installed to the location requested by the build.	2023-08-07 21:18:43 -04:00