Skip to content

cmake: Use CMAKE_INSTALL_INCLUDEDIR indirection - #6440

Closed
gjasny wants to merge 1 commit into
curl:masterfrom
gjasny:gnu-include-dir
Closed

cmake: Use CMAKE_INSTALL_INCLUDEDIR indirection#6440
gjasny wants to merge 1 commit into
curl:masterfrom
gjasny:gnu-include-dir

Conversation

@gjasny

@gjasny gjasny commented Jan 12, 2021

Copy link
Copy Markdown
Contributor

Like the header installation uses CMAKE_INSTALL_INCLUDEDIR:

curl/CMakeLists.txt

Lines 1516 to 1518 in 3831043

install(DIRECTORY "${CMAKE_CURRENT_SOURCE_DIR}/include/curl"
DESTINATION ${CMAKE_INSTALL_INCLUDEDIR}
FILES_MATCHING PATTERN "*.h")

the exported CMake configuration must use it as well to lookup the installed files in the same place.

@bagder bagder added the cmake label Jan 12, 2021
@bagder
bagder requested a review from snikulov February 23, 2021 15:58

@snikulov snikulov left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@bagder bagder closed this in 811acd1 Feb 24, 2021
@bagder

bagder commented Feb 24, 2021

Copy link
Copy Markdown
Member

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

3 participants