search

zeek / cmake

CMake scripts used in Zeek
https://www.zeek.org
Other
48 stars 59 forks source link

Remove the search for CAF from ZeekPluginDynamic.cmake #39

Closed dopheide-esnet closed 2 years ago

dopheide-esnet commented 2 years ago

With the removal of the public CAF dependency it appears we also need to remove looking for CAF from ZeekPluginDynamic.cmake.

I also noticed that zeek-config still has a "--caf_root" option, but I don't know what downstream effects removing it may have.

timwoj commented 2 years ago

@Neverlord Was this a leftover from the CAF dependency work? Can it be removed?

Neverlord commented 2 years ago

@timwoj yes, looks like I didn't catch those when filing the Zeek-side PR. @dopheide-esnet thanks for cleaning those up! 🙏