Page MenuHomeFreeBSD

New port security/zeek - Network Intrusion Detection System (formally known as Bro)
ClosedPublic

Authored by leres on Thu, Nov 14, 11:42 PM.

Details

Summary

Note: I created this review using something silimar to:

cd security
svn copy bro zeek
cd zeek/files
svn rename bro.in zeek.in
[then updated files in security/zeek with ones from the new zeek port]
[edited security/Makefile to add security/zeek]
[edited UIDs and GIDs to add the zeek user and group]
arc diff --create

Once the new port has been committed I will add something like this
to security/bro:

DEPRECATED= Please migrate to security/zeek
EXPIRATION_DATE= 2020-06-01

Proposed commit message:

This adds security/zeek, the new version of security/bro. This
is being done as svn copy instead of rename so that users of
security/bro can have some time to migrate. It also allows for
possible security updates to the old bro port which upstream
has indicated is possible for at least a few months.

Reviewed by: ? (mentor)
Approved by: ? (mentor)
Differential Revision: ?

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

leres created this revision.Thu, Nov 14, 11:42 PM
matthew added inline comments.Fri, Nov 15, 12:07 AM
security/zeek/Makefile
65 ↗(On Diff #64347)

Is that 'BroControl' or 'ZeekControl' now?

If it's still called BroControl, then shouldn't the option name be BROCTL to match?

security/zeek/pkg-descr
3 ↗(On Diff #64347)

s/Bro/Zeek/

12 ↗(On Diff #64347)

And again s/Bro/Zeek/

security/zeek/pkg-plist
1896 ↗(On Diff #64347)

Is it %%BROUSER%% still, or %%ZEEKUSER%% now?

leres updated this revision to Diff 64350.Fri, Nov 15, 1:23 AM

Fix four missing bro -> zeek changes Matthew identified.

leres updated this revision to Diff 64358.Fri, Nov 15, 4:31 AM

Installing and probably building conflicts with bro

ler requested changes to this revision.Sat, Nov 16, 6:30 PM
ler added inline comments.
security/zeek/pkg-descr
1 ↗(On Diff #64358)

s/formally/formerly/

This revision now requires changes to proceed.Sat, Nov 16, 6:30 PM
leres updated this revision to Diff 64470.Sat, Nov 16, 9:04 PM

Fix new spelling error Larry found.

ler accepted this revision.Sun, Nov 17, 12:37 AM

LGTM now.

This revision is now accepted and ready to land.Sun, Nov 17, 12:37 AM