Skip to content

Commit

Permalink
FindZeek: Split --include_dirs on colon instead of space
Browse files Browse the repository at this point in the history
The zeek-config output is colon separated. Using ZEEK_INCLUDE_DIRS for
include_directories() previously did not work correctly as
ZEEK_INCLUDE_DIRS was treated as a single directory, resulting in:

    -I/opt/zeek/include:/usr/include
  • Loading branch information
awelzel committed Jan 5, 2024
1 parent 82b1c81 commit 6a8ddc2
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion FindZeek.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -107,7 +107,7 @@ else ()
run_zeek_config(ZEEK_VERSION "--version")
run_zeek_config(ZEEK_BUILD_TYPE "--build_type")

string(REPLACE " " ";" ZEEK_INCLUDE_DIRS "${ZEEK_INCLUDE_DIRS}")
string(REPLACE ":" ";" ZEEK_INCLUDE_DIRS "${ZEEK_INCLUDE_DIRS}")

# Copied from Zeek to generate numeric version number.
string(REGEX REPLACE "[.-]" " " version_numbers "${ZEEK_VERSION}")
Expand Down

0 comments on commit 6a8ddc2

Please sign in to comment.