]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commit
ceph.spec.in: s/%cmake/cmake/ 39590/head
authorKefu Chai <kchai@redhat.com>
Sat, 20 Feb 2021 03:43:21 +0000 (11:43 +0800)
committerKefu Chai <kchai@redhat.com>
Sat, 20 Feb 2021 03:43:30 +0000 (11:43 +0800)
commit71aea12bac1620a45ad58b8721952f2bcabf27c8
tree886c4179638aef6cf92cdeec2ef48e9b5fc20e2c
parentea373c1fa1b4d3769250c377092ceaa697654d2c
ceph.spec.in: s/%cmake/cmake/

this change partially reverts da7030db79c5dbd480040eb249a76638caf52707
which use %cmake rpm macro in the place of "cmake". but

%cmake sets BUILD_SHARED_LIBS=ON. so quite a few internal libraries
defined using add_library() are now compiled into shared libraries which
are not installed or packagesd. when we are installing the rpm packages
compiled with this option, rpm compiles because the linked libraries are
missing, for instance, `libgmock.so.1.10.0` was compiled as a static
library before da7030db79c5dbd480040eb249a76638caf52707, and was
included by the test executables. but after that change it's compiled
as a shared library.

so we need to either package the linked shared libraries or just link
against them statically. at this moment, the latter approach is simpler,
albeit larger size of exectuable and dbg symbols.

Fixes: https://tracker.ceph.com/issues/49395
Signed-off-by: Kefu Chai <kchai@redhat.com>
ceph.spec.in