[CMAKE] remove global include_directories usage and rely on target properties (open-telemetry#3426)

Author: dbarker

CHANGELOG.md

@@ -55,6 +55,9 @@ Increment the:
 * [CMAKE] Add generated protobuf headers to the opentelemetry_proto target
   [#3400](https://github.com/open-telemetry/opentelemetry-cpp/pull/3400)
 
+* [CMAKE] Remove include_directories usage and rely on target properties
+  [#3426](https://github.com/open-telemetry/opentelemetry-cpp/pull/3426)
+
 ## [1.20 2025-04-01]
 
 * [BUILD] Update opentelemetry-proto version

CMakeLists.txt

@@ -859,8 +859,6 @@ if(DEFINED OPENTELEMETRY_BUILD_DLL)
   add_definitions(-DOPENTELEMETRY_BUILD_EXPORT_DLL)
 endif()
 
-include_directories(api/include)
-
 add_subdirectory(api)
 
 if(WITH_OPENTRACING)
@@ -893,9 +891,6 @@ endif()
 
 if(NOT WITH_API_ONLY)
   set(BUILD_TESTING ${BUILD_TESTING})
-  include_directories(sdk/include)
-  include_directories(sdk)
-  include_directories(ext/include)
 
   add_subdirectory(sdk)
   add_subdirectory(ext)

examples/common/foo_library/foo_library.cc

@@ -2,7 +2,6 @@
 // SPDX-License-Identifier: Apache-2.0
 
 #include "opentelemetry/nostd/shared_ptr.h"
-#include "opentelemetry/sdk/version/version.h"
 #include "opentelemetry/trace/provider.h"
 #include "opentelemetry/trace/scope.h"
 #include "opentelemetry/trace/tracer.h"
@@ -16,7 +15,7 @@ namespace
 nostd::shared_ptr<trace::Tracer> get_tracer()
 {
   auto provider = trace::Provider::GetTracerProvider();
-  return provider->GetTracer("foo_library", OPENTELEMETRY_SDK_VERSION);
+  return provider->GetTracer("foo_library");
 }
 
 void f1()

examples/common/logs_foo_library/foo_library.cc

@@ -5,7 +5,6 @@
 #include "opentelemetry/logs/logger_provider.h"
 #include "opentelemetry/logs/provider.h"
 #include "opentelemetry/nostd/shared_ptr.h"
-#include "opentelemetry/sdk/version/version.h"
 #include "opentelemetry/trace/provider.h"
 #include "opentelemetry/trace/scope.h"
 #include "opentelemetry/trace/span.h"
@@ -21,7 +20,7 @@ namespace
 opentelemetry::nostd::shared_ptr<trace::Tracer> get_tracer()
 {
   auto provider = trace::Provider::GetTracerProvider();
-  return provider->GetTracer("foo_library", OPENTELEMETRY_SDK_VERSION);
+  return provider->GetTracer("foo_library");
 }
 
 opentelemetry::nostd::shared_ptr<logs::Logger> get_logger()

exporters/prometheus/CMakeLists.txt

@@ -1,7 +1,6 @@
 # Copyright The OpenTelemetry Authors
 # SPDX-License-Identifier: Apache-2.0
 
-include_directories(include)
 if(NOT TARGET prometheus-cpp::core)
   find_package(prometheus-cpp CONFIG REQUIRED)
 endif()

exporters/zipkin/CMakeLists.txt

@@ -1,8 +1,6 @@
 # Copyright The OpenTelemetry Authors
 # SPDX-License-Identifier: Apache-2.0
 
-include_directories(include)
-add_definitions(-DWITH_CURL)
 add_library(
   opentelemetry_exporter_zipkin_trace
   src/zipkin_exporter.cc src/zipkin_exporter_factory.cc src/recordable.cc)

ext/test/http/CMakeLists.txt

@@ -22,7 +22,7 @@ if(WITH_HTTP_CLIENT_CURL)
     target_link_libraries(${FILENAME} opentelemetry_http_client_curl
                           opentelemetry_common ${CURL_IMPORTED_TARGET_NAME})
   else()
-    include_directories(${CURL_INCLUDE_DIRS})
+    target_include_directories(${FILENAME} PRIVATE ${CURL_INCLUDE_DIRS})
     target_link_libraries(${FILENAME} ${CURL_LIBRARIES}
                           opentelemetry_http_client_curl opentelemetry_common)
   endif()
@@ -35,7 +35,7 @@ endif()
 
 set(URL_PARSER_FILENAME url_parser_test)
 add_executable(${URL_PARSER_FILENAME} ${URL_PARSER_FILENAME}.cc)
-target_link_libraries(${URL_PARSER_FILENAME} opentelemetry_api ${GMOCK_LIB}
+target_link_libraries(${URL_PARSER_FILENAME} opentelemetry_ext ${GMOCK_LIB}
                       ${GTEST_BOTH_LIBRARIES} ${CMAKE_THREAD_LIBS_INIT})
 gtest_add_tests(
   TARGET ${URL_PARSER_FILENAME}

ext/test/w3c_tracecontext_http_test_server/CMakeLists.txt

@@ -1,8 +1,6 @@
 # Copyright The OpenTelemetry Authors
 # SPDX-License-Identifier: Apache-2.0
 
-include_directories(${CMAKE_SOURCE_DIR}/exporters/ostream/include)
-
 add_executable(w3c_tracecontext_http_test_server main.cc)
 target_link_libraries(
   w3c_tracecontext_http_test_server

functional/otlp/CMakeLists.txt

@@ -1,8 +1,6 @@
 # Copyright The OpenTelemetry Authors
 # SPDX-License-Identifier: Apache-2.0
 
-include_directories(${CMAKE_SOURCE_DIR}/exporters/otlp/include)
-
 if(WITH_OTLP_GRPC)
   add_executable(func_otlp_grpc func_grpc_main.cc)
   target_link_libraries(func_otlp_grpc ${CMAKE_THREAD_LIBS_INIT}

sdk/CMakeLists.txt

@@ -9,6 +9,9 @@ target_include_directories(
 
 set_target_properties(opentelemetry_sdk PROPERTIES EXPORT_NAME sdk)
 
+target_link_libraries(opentelemetry_sdk INTERFACE opentelemetry_api)
+
+set(OTEL_SDK_DIR ${CMAKE_CURRENT_SOURCE_DIR})
 add_subdirectory(src)
 
 otel_add_component(