-
Notifications
You must be signed in to change notification settings - Fork 1.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
CMakeLists.txt does not export include paths. #444
Comments
That's excatly what I need in my project. Thumb up! |
Yep, that would be a really convenient feature. |
Great feature proposal! I`m also looking forward to use it in my project. |
Using target_include_directories requires cmake version at least 2.8.11. If you want to keep compatibility with the cmake 2.6, good approach to handle this can be seen in CMakeLists of googletest:
|
ghost
mentioned this issue
Jun 22, 2016
Merged. Thanks @kurylo |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Please add something similar to:
target_include_directories(tinyxml2 PUBLIC ${CMAKE_CURRENT_SOURCE_DIR}/.)
target_include_directories(tinyxml2_static PUBLIC ${CMAKE_CURRENT_SOURCE_DIR}/.)
to CMakeLists.txt. This will allow building and linking with tinyxml2 in a single build tree. Application using cmake would be able to just: add_subdirectory(tinyxml2) and then use tinyxml2 targets.
The text was updated successfully, but these errors were encountered: