From: Kefu Chai <kchai@redhat.com>
Date: Fri, 23 Jul 2021 10:02:36 +0000 (+0800)
Subject: ceph.spec.in: build with header only fmt on RHEL
X-Git-Tag: v17.1.0~1304^2
X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=5419131499f4c4b5f6ba520342c39f9766e0cf4d;p=ceph.git

ceph.spec.in: build with header only fmt on RHEL

because fmt is packaged in EPEL, while librados is packaged
in RHEL, so we cannot have fmt as a runtime dependency of librados.
to address this issue, we should compile librados either with static library
or with header-only library of fmt. but because the fedora packaging
guideline does no encourage us to package static libraries, and it would
be complicated to package both static and dynamic library for fmt.

the simpler solution would be to compile Ceph with the header-only
version of fmt.

in this change, we compile ceph with the header-only version of fmt
on RHEL to address the runtime dependency issue.

Signed-off-by: Kefu Chai <kchai@redhat.com>
---

diff --git a/ceph.spec.in b/ceph.spec.in
index 041c5be48d1de..891369f339b1d 100644
--- a/ceph.spec.in
+++ b/ceph.spec.in
@@ -1347,6 +1347,9 @@ cmake .. \
     -DBOOST_J:STRING=%{jobs} \
 %else
     -DBOOST_J:STRING=%{_smp_build_ncpus} \
+%endif
+%if 0%{?rhel}
+    -DWITH_FMT_HEADER_ONLY:BOOL=ON \
 %endif
     -DWITH_GRAFANA:BOOL=ON