From 5d76a761f44691ce4a8c7f5b2703b0bc005d224e 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 `INTERFACE_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 `INTERFACE_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..16259df 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_ROOT_DIR} )