From fad021fa365bc3d885521d497a4463c37d5c9e94 Mon Sep 17 00:00:00 2001 From: Tommaso Tocci Date: Mon, 14 May 2018 16:32:22 +0200 Subject: [PATCH] cmake: Set include dirs property on exported target The exported `yaml-cpp` target misses the `INCLUDE_DIRECTORIES` property. This means that even if you use the yaml-cpp target as linked library you still need to add manually the yaml include dirs by using the `YAML_CPP_INCLUDE_DIR` variable. This commit fix the issue but setting properly the `INLCUDE_DIRECTORIES` property on the exported target. Fixes #587 [1] [1]: https://github.com/jbeder/yaml-cpp/issues/587 --- CMakeLists.txt | 1 + 1 file changed, 1 insertion(+) diff --git a/CMakeLists.txt b/CMakeLists.txt index 715c846..e3b3026 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -269,6 +269,7 @@ set(_INSTALL_DESTINATIONS RUNTIME DESTINATION bin LIBRARY DESTINATION ${LIB_INSTALL_DIR} ARCHIVE DESTINATION ${LIB_INSTALL_DIR} + INCLUDES DESTINATION ${INCLUDE_INSTALL_DIR} )