Don't require applications using jaeger exporter to know about libcurl (

#1518)
open-telemetry · Jul 26, 2022 · 3a8f913 · 3a8f913 · github-actions · Jul 26, 2022
1 parent 30b4fcd
commit 3a8f913
Showing 1 changed file with 8 additions and 4 deletions.
diff --git a/ext/src/http/client/curl/CMakeLists.txt b/ext/src/http/client/curl/CMakeLists.txt
@@ -8,13 +8,17 @@ if(CURL_FOUND)
  PROPERTIES EXPORT_NAME http_client_curl)
 
  if(TARGET CURL::libcurl)
- target_link_libraries(opentelemetry_http_client_curl
- PUBLIC opentelemetry_ext CURL::libcurl)
+ target_link_libraries(
+ opentelemetry_http_client_curl
+ PUBLIC opentelemetry_ext
+ PRIVATE CURL::libcurl)
  else()
  target_include_directories(opentelemetry_http_client_curl
  INTERFACE "${CURL_INCLUDE_DIRS}")
- target_link_libraries(opentelemetry_http_client_curl
- PUBLIC opentelemetry_ext ${CURL_LIBRARIES})
+ target_link_libraries(
+ opentelemetry_http_client_curl
+ PUBLIC opentelemetry_ext
+ PRIVATE ${CURL_LIBRARIES})
  endif()
 
  install(