Merge pull request ceph#61592 from hasan4791/issue-69732

Isolate the absl flags to ceph-mon & ceph-nvmeof-monitor-client targets

Reviewed-by: Radoslaw Zarzynski <[email protected]>
Reviewed-by: Brad Hubbard <[email protected]>

Author: yuriw

src/CMakeLists.txt

@@ -316,12 +316,6 @@ endif(WITH_BLKIN)
 
 if(WITH_JAEGER)
   find_package(thrift 0.13.0 REQUIRED)
-
-  if(EXISTS "/etc/redhat-release" OR EXISTS "/etc/fedora-release")
-    # absl is installed as grpc build dependency on RPM based systems
-    add_definitions(-DHAVE_ABSEIL)
-  endif()
-
   include(BuildOpentelemetry)
   build_opentelemetry()
   add_library(jaeger_base INTERFACE)
@@ -993,6 +987,12 @@ if(WITH_NVMEOF_GATEWAY_MONITOR_CLIENT)
     nvmeof/NVMeofGwMonitorClient.cc)
   add_executable(ceph-nvmeof-monitor-client ${ceph_nvmeof_monitor_client_srcs})
   add_dependencies(ceph-nvmeof-monitor-client ceph-common)
+  # absl is installed as grpc build dependency on RPM based systems
+  # Also isolate this flag to specific targets which needs this package
+  if(EXISTS "/etc/redhat-release" OR EXISTS "/etc/fedora-release")
+    target_compile_definitions(ceph-nvmeof-monitor-client PRIVATE HAVE_ABSEIL)
+    target_compile_definitions(ceph-mon PRIVATE HAVE_ABSEIL)
+  endif()
   target_link_libraries(ceph-nvmeof-monitor-client
     client
     mon