Fix open-telemetry#740 open-telemetry#741

Signed-off-by: owent <admin@owent.net>

CMakeLists.txt

@@ -49,8 +49,6 @@ endif()
 
 if(WITH_STL)
   # Require at least C++17. C++20 is needed to avoid gsl::span
-  add_definitions(-DHAVE_CPP_STDLIB -DHAVE_GSL)
-
   if(CMAKE_MINOR_VERSION VERSION_GREATER "3.18")
     # Ask for 20, may get anything below
     set(CMAKE_CXX_STANDARD 20)
@@ -130,7 +128,7 @@ if(WITH_OTLP)
   set(protobuf_MODULE_COMPATIBLE ON)
   find_package(Protobuf)
   find_package(gRPC)
-  if((NOT protobuf_FOUND) OR (NOT gRPC_FOUND))
+  if((NOT Protobuf_FOUND) OR (NOT gRPC_FOUND))
     if(WIN32 AND (NOT DEFINED CMAKE_TOOLCHAIN_FILE))
       install_windows_deps()
     endif()
@@ -142,7 +140,7 @@ if(WITH_OTLP)
       include(${CMAKE_TOOLCHAIN_FILE})
     endif()
 
-    if(NOT protobuf_FOUND)
+    if(NOT Protobuf_FOUND)
       find_package(Protobuf REQUIRED)
     endif()
     if(NOT gRPC_FOUND)

api/CMakeLists.txt

@@ -25,6 +25,8 @@ endif()
 
 if(WITH_STL)
   message("Building with standard library types...")
+  target_compile_definitions(opentelemetry_api INTERFACE HAVE_CPP_STDLIB
+                                                         HAVE_GSL)
 else()
   message("Building with nostd types...")
 endif()