maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

[Support] [Debuginfod] Use libcurl imported library. 

A [[ https://lists.llvm.org/pipermail/llvm-dev/2021-December/154203.html | report on llvm-dev ]] indicated that `curl.h` could be found during build configuration but not during compilation. This diff uses the libcurl imported library created by findcurl, so that includes and libs are automatically managed correctly by cmake.

Reviewed By: phosek

Differential Revision: https://reviews.llvm.org/D115189
This commit is contained in:
Noah Shutty 2021-12-09 00:51:33 +00:00
parent d0022a7250
commit 54b35c09b2
1 changed files with 15 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
llvm/lib/Debuginfod/CMakeLists.txt
View File

@ -1,6 +1,6 @@
# Link LibCURL if the user wants it
if (LLVM_ENABLE_CURL)
set(imported_libs ${CURL_LIBRARIES})
set(imported_libs CURL::libcurl)
endif()
add_llvm_component_library(LLVMDebuginfod
@ -17,4 +17,17 @@ add_llvm_component_library(LLVMDebuginfod
Support
)
set(llvm_system_libs ${system_libs})
# This block is only needed for llvm-config. When we deprecate llvm-config and
# move to using CMake export, this block can be removed.
if(LLVM_ENABLE_CURL)
# CMAKE_BUILD_TYPE is only meaningful to single-configuration generators.
if(CMAKE_BUILD_TYPE)
string(TOUPPER ${CMAKE_BUILD_TYPE} build_type)
get_property(curl_library TARGET CURL::libcurl PROPERTY LOCATION_${build_type})
endif()
if(NOT curl_library)
get_property(curl_library TARGET CURL::libcurl PROPERTY LOCATION)
endif()
get_library_name(${curl_library} curl_library)
set(llvm_system_libs ${llvm_system_libs} "${curl_library}")
endif()