cisagov / icsnpp-bacnet

Zeek BACnet Parser - CISA ICSNPP
BSD 3-Clause "New" or "Revised" License
15 stars 11 forks source link

Add zeek/zeek-version.h include #19

Closed awelzel closed 1 year ago

awelzel commented 1 year ago

πŸ—£ Description

I'd like to use this plugin for builtin-plugin integration testing within the Zeek CI pipeline (as an example of an external plugin using binpac). Due to zeek/zeek#2806 with the latest master version this requires an explicit zeek/zeek-version.h include.

That include is good practice anyway, so it'd be nice if that could be merged.

Reference zeek/zeek#2847.

πŸ’­ Motivation and context

See description.

πŸ§ͺ Testing

βœ… Pre-approval checklist

βœ… Pre-merge checklist

βœ… Post-merge checklist