From: Kefu Chai Date: Fri, 24 May 2024 09:51:55 +0000 (+0800) Subject: cmake: : link shec_utils against legacy-option-headers X-Git-Tag: v20.0.0~1868^2 X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=refs%2Fpull%2F57693%2Fhead;p=ceph.git cmake: : link shec_utils against legacy-option-headers in c24a6ffe20, we tried to link all target dependent on legacy option headers against legacy-option-headers, but we missed some of them. in our CI, we spotted build failure like: ``` In file included from /home/jenkins-build/build/workspace/ceph-pull-requests/src/erasure-code/ErasureCode.cc:26: In file included from /home/jenkins-build/build/workspace/ceph-pull-requests/src/osd/osd_types.h:41: In file included from /home/jenkins-build/build/workspace/ceph-pull-requests/src/common/ceph_context.h:41: In file included from /home/jenkins-build/build/workspace/ceph-pull-requests/src/common/config_proxy.h:6: In file included from /home/jenkins-build/build/workspace/ceph-pull-requests/src/common/config.h:27: In file included from /home/jenkins-build/build/workspace/ceph-pull-requests/src/common/config_values.h:59: /home/jenkins-build/build/workspace/ceph-pull-requests/src/common/options/legacy_config_opts.h:1:10: fatal error: 'global_legacy_options.h' file not found ^~~~~~~~~~~~~~~~~~~~~~~~~ 1 error generated. ``` so in this change, we link the shec_utils to `legacy-option-headers` as well to fulfill the build dependency. Signed-off-by: Kefu Chai --- diff --git a/src/erasure-code/shec/CMakeLists.txt b/src/erasure-code/shec/CMakeLists.txt index 0e699203d7c2..e7521542e311 100644 --- a/src/erasure-code/shec/CMakeLists.txt +++ b/src/erasure-code/shec/CMakeLists.txt @@ -10,6 +10,8 @@ set(shec_utils_srcs determinant.c) add_library(shec_utils OBJECT ${shec_utils_srcs}) +target_link_libraries(shec_utils + PRIVATE legacy-option-headers) set(ec_shec_objs $