diff options
| author | Deepika Upadhyay <dupadhya@redhat.com> | 2021-10-04 10:10:48 +0200 |
|---|---|---|
| committer | Deepika Upadhyay <dupadhya@redhat.com> | 2021-10-04 10:22:49 +0200 |
| commit | 463193bcaf8b3b8a0e616315c8b650acdfe7b53a (patch) | |
| tree | 3208f3231319abceee844283b82c23c929dd6c40 /cmake | |
| parent | Merge pull request #43146 from jianwei1216/fix-partial-recovery-become-whole-... (diff) | |
| download | ceph-463193bcaf8b3b8a0e616315c8b650acdfe7b53a.tar.xz ceph-463193bcaf8b3b8a0e616315c8b650acdfe7b53a.zip | |
cmake,rgw: use jaeger-base target, as a dependency for building dbstore
* dbstore also has tracing dependencies and missing linking for libraries
* use `using` keyword in common/tracer.cc to silence unused method
warnings
* log building of external project jaegertracing, opentracing into build
file, (done to reduce external project build verbosity)
Signed-off-by: Deepika Upadhyay <dupadhya@redhat.com>
Diffstat (limited to 'cmake')
| -rw-r--r-- | cmake/modules/BuildJaeger.cmake | 2 | ||||
| -rw-r--r-- | cmake/modules/BuildOpenTracing.cmake | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/cmake/modules/BuildJaeger.cmake b/cmake/modules/BuildJaeger.cmake index ffd301b7b08..6a65a6bb42b 100644 --- a/cmake/modules/BuildJaeger.cmake +++ b/cmake/modules/BuildJaeger.cmake @@ -79,7 +79,7 @@ function(build_jaeger) INSTALL_COMMAND ${install_cmd} DEPENDS ${dependencies} BUILD_BYPRODUCTS ${CMAKE_BINARY_DIR}/external/lib/libjaegertracing.so - ) + LOG_BUILD ON) set_library_properties_for_external_project(opentracing::libopentracing opentracing) diff --git a/cmake/modules/BuildOpenTracing.cmake b/cmake/modules/BuildOpenTracing.cmake index 5098709dbdb..07a1ea44d1c 100644 --- a/cmake/modules/BuildOpenTracing.cmake +++ b/cmake/modules/BuildOpenTracing.cmake @@ -32,5 +32,5 @@ function(build_opentracing) BUILD_COMMAND ${make_cmd} INSTALL_COMMAND ${install_cmd} BUILD_BYPRODUCTS ${CMAKE_BINARY_DIR}/external/lib/libopentracing.so - ) + LOG_BUILD ON) endfunction() |