]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
cmake: WITH_SYSTEM_UTF8PROC defaults to OFF 45766/head
authorCasey Bodley <cbodley@redhat.com>
Tue, 29 Mar 2022 21:46:04 +0000 (17:46 -0400)
committerCasey Bodley <cbodley@redhat.com>
Mon, 4 Apr 2022 14:24:35 +0000 (10:24 -0400)
change the default value of WITH_SYSTEM_UTF8PROC from ON to OFF, so that
centos/rhel users can build with the default cmake configuration. no other
WITH_SYSTEM_* variable in ceph defaults to ON, so this is consistent
with other bundled libraries like boost and rocksdb

unfortunately, this also means that users that do have system packages
must opt-in to using them with -DWITH_SYSTEM_UTF8PROC=ON

both deb and rpm builds dependended on the previous default value, so
their logic was negated to match the new default

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

CMakeLists.txt
ceph.spec.in
debian/rules

index e46655beb983e442e4cc8063cc0f3273649618f5..3c44a18555d0eaf4f64ce31e49fd7d202536d4f0 100644 (file)
@@ -403,7 +403,7 @@ option(WITH_RADOSGW_DBSTORE "DBStore backend for Rados Gateway" ON)
 option(WITH_RADOSGW_SELECT_PARQUET "Support for s3 select on parquet objects" ON)
 
 option(WITH_SYSTEM_ARROW "Use system-provided arrow" OFF)
-option(WITH_SYSTEM_UTF8PROC "Use system-provided utf8proc" ON)
+option(WITH_SYSTEM_UTF8PROC "Use system-provided utf8proc" OFF)
 
 if(WITH_RADOSGW)
   find_package(EXPAT REQUIRED)
index cb3fa2d3677401eff01c04513197e480689979d9..79ca827c7599fabba71f549794d5129136397f76 100644 (file)
@@ -1351,8 +1351,8 @@ cmake .. \
 %if 0%{with system_arrow}
     -DWITH_SYSTEM_ARROW:BOOL=ON \
 %endif
-%if 0%{without system_utf8proc}
-    -DWITH_SYSTEM_UTF8PROC:BOOL=OFF \
+%if 0%{with system_utf8proc}
+    -DWITH_SYSTEM_UTF8PROC:BOOL=ON \
 %endif
     -DWITH_GRAFANA:BOOL=ON
 
index 8904342750f48dc7b7d5fea3656ccd12772935fc..9f49deabb0e0128d24ce0ca1f3dc08d562bcce58 100755 (executable)
@@ -25,6 +25,7 @@ ifneq ($(filter pkg.ceph.arrow,$(DEB_BUILD_PROFILES)),)
   extraopts += -DWITH_SYSTEM_ARROW=ON
 endif
 
+extraopts += -DWITH_SYSTEM_UTF8PROC=ON
 extraopts += -DWITH_OCF=ON -DWITH_LTTNG=ON
 extraopts += -DWITH_MGR_DASHBOARD_FRONTEND=OFF
 extraopts += -DWITH_PYTHON3=3