fmt: remove FMT_HEADER_ONLY defines

a WITH_FMT_HEADER_ONLY cmake option controls which version of the
library to use. individual sources/targets shouldn't override that, or
they'll end up including/linking against both versions

Fixes: https://tracker.ceph.com/issues/58953
Signed-off-by: Casey Bodley <cbodley@redhat.com>
(cherry picked from commit d4ae3d0db857766ba38656d2c5c523883e518879)

diff --git a/src/cls/fifo/cls_fifo.cc b/src/cls/fifo/cls_fifo.cc
index 3cbb4636c9d9f28e027c1eac98948bf5f85ab060..85022eeb061a1728ae9801fe4006b13ea2617fcb 100644 (file)
--- a/src/cls/fifo/cls_fifo.cc
+++ b/src/cls/fifo/cls_fifo.cc
@@ -12,8 +12,6 @@
 #include <optional>
 #include <string>
 
-#undef FMT_HEADER_ONLY
-#define FMT_HEADER_ONLY 1
 #include <fmt/format.h>
 
 #include "include/buffer.h"

diff --git a/src/cls/fifo/cls_fifo_types.h b/src/cls/fifo/cls_fifo_types.h
index 2b9fec26af9b1671c4b7571133020394a0dea147..1c69c1f08718a9725fad3f09b190153b8424f0d9 100644 (file)
--- a/src/cls/fifo/cls_fifo_types.h
+++ b/src/cls/fifo/cls_fifo_types.h
@@ -25,8 +25,6 @@
 
 #include <boost/container/flat_set.hpp>
 
-#undef FMT_HEADER_ONLY
-#define FMT_HEADER_ONLY 1
 #include <fmt/format.h>
 #if FMT_VERSION >= 90000
 #include <fmt/ostream.h>

diff --git a/src/cls/rgw/cls_rgw_types.h b/src/cls/rgw/cls_rgw_types.h
index c65526c06c59dba65d7acaf42382174cb6ccdb83..37a06defc31fbc967e63d07f2e7fbc648d67f435 100644 (file)
--- a/src/cls/rgw/cls_rgw_types.h
+++ b/src/cls/rgw/cls_rgw_types.h
@@ -9,8 +9,6 @@
 #include "common/ceph_time.h"
 #include "common/Formatter.h"
 
-#undef FMT_HEADER_ONLY
-#define FMT_HEADER_ONLY 1
 #include <fmt/format.h>
 
 #include "rgw/rgw_basic_types.h"

diff --git a/src/librbd/migration/S3Stream.cc b/src/librbd/migration/S3Stream.cc
index 3b4db0cefe19226a91e6840c7849288fa4c050c7..a611e274ae131b09ad55abb23071cdf242379e01 100644 (file)
--- a/src/librbd/migration/S3Stream.cc
+++ b/src/librbd/migration/S3Stream.cc
@@ -17,8 +17,6 @@
 #include "librbd/migration/HttpProcessorInterface.h"
 #include <boost/beast/http.hpp>
 
-#undef FMT_HEADER_ONLY
-#define FMT_HEADER_ONLY 1
 #include <fmt/chrono.h>
 #include <fmt/format.h>
 

diff --git a/src/mds/cephfs_features.cc b/src/mds/cephfs_features.cc
index 6102a9b1652c22c0678dd3c93c804431018c170a..0df6123e937422c47484afbb4539bd06c217d3c2 100644 (file)
--- a/src/mds/cephfs_features.cc
+++ b/src/mds/cephfs_features.cc
@@ -5,8 +5,6 @@
 #include "cephfs_features.h"
 #include "mdstypes.h"
 
-#undef FMT_HEADER_ONLY
-#define FMT_HEADER_ONLY 1
 #include <fmt/format.h>
 
 static const std::array feature_names

diff --git a/src/rgw/CMakeLists.txt b/src/rgw/CMakeLists.txt
index 34a346281850f49caf4d6f089bb1ccac8bf00f2c..69bfac23778316db1fb5a4e3288a8668d56edcc8 100644 (file)
--- a/src/rgw/CMakeLists.txt
+++ b/src/rgw/CMakeLists.txt
@@ -231,8 +231,6 @@ endif(WITH_RADOSGW_ARROW_FLIGHT)
 
 
 add_library(rgw_common STATIC ${librgw_common_srcs})
-target_compile_definitions(rgw_common
-  PUBLIC "FMT_HEADER_ONLY")
 
 include(CheckCXXCompilerFlag)
 check_cxx_compiler_flag("-Wimplicit-const-int-float-conversion"

diff --git a/src/rgw/rgw_iam_policy.h b/src/rgw/rgw_iam_policy.h
index ff6061c7550ea5621f9feae3de2ecb0da25439e9..c0a7e51b5fd7e8f7fdce253f97e89575c2b2f0ab 100644 (file)
--- a/src/rgw/rgw_iam_policy.h
+++ b/src/rgw/rgw_iam_policy.h
@@ -17,8 +17,6 @@
 #include <boost/thread/shared_mutex.hpp>
 #include <boost/variant.hpp>
 
-#undef FMT_HEADER_ONLY
-#define FMT_HEADER_ONLY 1
 #include <fmt/format.h>
 
 #include "common/ceph_time.h"

diff --git a/src/test/fio/fio_librgw.cc b/src/test/fio/fio_librgw.cc
index b088b68f96c5ceb1296edc59a0e7d2fc1e700a3f..bac4ff2daac671e974d04c5c6f3bf5aba86d66bc 100644 (file)
--- a/src/test/fio/fio_librgw.cc
+++ b/src/test/fio/fio_librgw.cc
@@ -21,8 +21,6 @@
 
 #include <semaphore.h> // XXX kill this?
 
-#undef FMT_HEADER_ONLY
-#define FMT_HEADER_ONLY 1
 #include "fmt/include/fmt/format.h"
 
 #include "include/rados/librgw.h"

diff --git a/src/test/rgw/rgw_cr_test.cc b/src/test/rgw/rgw_cr_test.cc
index b53d0655923b4b1ea2037d3aabc65a93e047d1b8..dc5d25d23aeddc024672011bb723a14b8c19b608 100644 (file)
--- a/src/test/rgw/rgw_cr_test.cc
+++ b/src/test/rgw/rgw_cr_test.cc
@@ -6,8 +6,6 @@
 #include <sstream>
 #include <string>
 
-#undef FMT_HEADER_ONLY
-#define FMT_HEADER_ONLY 1
 #include <fmt/format.h>
 
 #include "include/rados/librados.hpp"

diff --git a/src/test/rgw/test_log_backing.cc b/src/test/rgw/test_log_backing.cc
index 4c81b2d57d2cf4b4b81fc74014177966b89c99a7..e4109d535d10cc7693b5dafd288b78c39ea750a4 100644 (file)
--- a/src/test/rgw/test_log_backing.cc
+++ b/src/test/rgw/test_log_backing.cc
@@ -18,8 +18,6 @@
 #include <iostream>
 #include <string_view>
 
-#undef FMT_HEADER_ONLY
-#define FMT_HEADER_ONLY 1
 #include <fmt/format.h>
 
 #include "include/types.h"