Fixed minor bug in config file and updated instructions

zeek / zeek-af_packet-plugin

Plugin providing native AF_Packet support for Zeek.

Other

33 stars 30 forks source link

Fixed minor bug in config file and updated instructions #8

Closed grantcurell closed 5 years ago

grantcurell commented 5 years ago

Fixed a minor bug in the configuration file. It only included Bro's cmake folder. You need to include the plugin's as well otherwise it won't be able to find the FindKernelHeaders.cmake file.
Added instructions. Probably obvious to an experienced Bro user, but it took me a bit to figure out I not only needed the Bro source, but also needed to build it.

J-Gras commented 5 years ago

Thanks! Before I merge I will have a look whether we can make use of the upcoming devel package for Bro.

grantcurell commented 5 years ago

Sounds good!

J-Gras commented 5 years ago

As mentioned in #7, I switched to the new version of the configure script, which allows to build the plugin solely based on the headers shipped with Bro 2.6. Anyway, thanks a lot for pointing out the issues you encountered and putting work into the plugin!