Azure / Azure/azure-storage-cpplite

Use curl::curl target when available

Open
#122 0 comments 0 reactions 0 assignees View on GitHub
feature request
Dominant language
C++
Stars
26
Forks
50
PR merge metrics
No merged PRs in 30d

Description

Build system integration could be simplified a bit if this library would conditionally link against the `curl::curl` target provided by curl's own CMake config files.

Currently the linkage here is unconditionally against `CURL_LIBRARIES` and `CURL_INCLUDE_DIRS`, which are only provided by the CMake built-in FindCurl.cmake. (using curl's CMake can be disabled by setting `CURL_NO_CURL_CMAKE`, but this is only available in CMake 3.17+).

(I'm happy to provide a patch to this effect, but unclear if patches are being accepted/merged here)

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.