Skip to content

Commit 7eab11a

Browse files
ThomasDevoogdtpatrick-stephens
authored andcommitted
plugins: kafka: fix cmake cross compile error
KAFKA_INCLUDEDIR is not set if FLB_PREFER_SYSTEM_LIB_KAFKA is not used, when cross-compiling, it just translates to -I/librdkafka, which is not allowed. Fix this by only including KAFKA_INCLUDEDIR if really set. x86_64-linux-gcc: ERROR: unsafe header/library path used in cross-compilation: '-I/librdkafka' Signed-off-by: Thomas Devoogdt <thomas@devoogdt.com>
1 parent c7a707e commit 7eab11a

File tree

2 files changed

+6
-2
lines changed

2 files changed

+6
-2
lines changed

plugins/in_kafka/CMakeLists.txt

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,5 +3,7 @@ set(src
33
)
44

55
FLB_PLUGIN(in_kafka "${src}" ${KAFKA_LIBRARIES} flb-aws)
6-
target_include_directories(flb-plugin-in_kafka PUBLIC ${KAFKA_INCLUDEDIR}/librdkafka)
6+
if(DEFINED KAFKA_INCLUDEDIR)
7+
target_include_directories(flb-plugin-in_kafka PUBLIC ${KAFKA_INCLUDEDIR}/librdkafka)
8+
endif()
79
target_link_libraries(flb-plugin-in_kafka -lpthread)

plugins/out_kafka/CMakeLists.txt

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,5 +5,7 @@ set(src
55
kafka.c)
66

77
FLB_PLUGIN(out_kafka "${src}" ${KAFKA_LIBRARIES} flb-aws)
8-
target_include_directories(flb-plugin-out_kafka PUBLIC ${KAFKA_INCLUDEDIR}/librdkafka)
8+
if(DEFINED KAFKA_INCLUDEDIR)
9+
target_include_directories(flb-plugin-out_kafka PUBLIC ${KAFKA_INCLUDEDIR}/librdkafka)
10+
endif()
911
target_link_libraries(flb-plugin-out_kafka -lpthread)

0 commit comments

Comments
 (0)