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

open-telemetry · Jul 26, 2022 · 60b0262 · 60b0262
1 parent 30b4fcd
commit 60b0262
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(