Skip to content
This repository was archived by the owner on Aug 30, 2022. It is now read-only.

Commit 9521a8e

Browse files
authored
Fix dynamic load test to include macros in Constants header (#84)
Signed-off-by: Isaac Hier <[email protected]>
1 parent 1b8ed62 commit 9521a8e

File tree

2 files changed

+5
-1
lines changed

2 files changed

+5
-1
lines changed

CMakeLists.txt

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -291,9 +291,11 @@ if(BUILD_TESTING)
291291
add_test(NAME UnitTest COMMAND UnitTest)
292292

293293
add_executable(DynamicallyLoadTracerTest src/jaegertracing/DynamicallyLoadTracerTest.cpp)
294+
target_include_directories(DynamicallyLoadTracerTest PUBLIC
295+
$<BUILD_INTERFACE:${CMAKE_CURRENT_BINARY_DIR}/src>)
294296
target_link_libraries(
295297
DynamicallyLoadTracerTest OpenTracing::opentracing-static)
296-
add_test(NAME DynamicallyLoadTracerTest COMMAND DynamicallyLoadTracerTest
298+
add_test(NAME DynamicallyLoadTracerTest COMMAND DynamicallyLoadTracerTest
297299
${CMAKE_CURRENT_BINARY_DIR}/${CMAKE_SHARED_LIBRARY_PREFIX}jaegertracing${CMAKE_SHARED_LIBRARY_SUFFIX})
298300
if(JAEGERTRACING_COVERAGE)
299301
setup_target_for_coverage(NAME UnitTestCoverage

src/jaegertracing/DynamicallyLoadTracerTest.cpp

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,8 @@
1818

1919
#include <opentracing/dynamic_load.h>
2020

21+
#include "jaegertracing/Constants.h"
22+
2123
// Verify that Jaeger's shared library can be dynamically loaded and used
2224
// as a plugin.
2325
int main(int argc, char* argv[])

0 commit comments

Comments
 (0)