]> git.apps.os.sepia.ceph.com Git - ceph-ci.git/commitdiff
cmake: adds BUILD_BYPRODUCT for external build jaegertracing libs
authorDeepika Upadhyay <dupadhya@redhat.com>
Wed, 21 Jul 2021 16:03:42 +0000 (21:33 +0530)
committerDeepika Upadhyay <dupadhya@redhat.com>
Thu, 22 Jul 2021 05:39:09 +0000 (11:09 +0530)
* adds BUILD_BYPRODUCT which tells ninja which library will be generated
after the build(needed for dependent build libs)

Signed-off-by: Deepika Upadhyay <dupadhya@redhat.com>
cmake/modules/BuildJaeger.cmake

index 16e30dd3335734ee8c2d1aff4e91e50df5ee6b19..338013335a621b5f5ccc9e816cd72486e45ba0d5 100644 (file)
@@ -51,7 +51,8 @@ function(build_jaeger)
                        -Dthrift_HOME=${CMAKE_BINARY_DIR}/external
                        -DOpenTracing_HOME=${CMAKE_BINARY_DIR}/external)
 
-  set(dependencies opentracing thrift)
+  # build these libraries along with jaeger
+  set(dependencies opentracing)
   if(NOT WITH_SYSTEM_BOOST)
     list(APPEND dependencies Boost)
   endif()
@@ -78,7 +79,7 @@ function(build_jaeger)
     BINARY_DIR ${Jaeger_BINARY_DIR}
     BUILD_COMMAND ${make_cmd}
     INSTALL_COMMAND ${install_cmd}
-    DEPENDS "${dependencies}"
+    DEPENDS ${dependencies}
     BUILD_BYPRODUCTS ${CMAKE_BINARY_DIR}/external/lib/libjaegertracing.so
     )