Page MenuHomeFreeBSD

security/bro: Add a NETMAP option to build and install the bro netmap plugin
ClosedPublic

Authored by leres on Feb 14 2018, 11:37 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Mar 22, 10:00 PM
Unknown Object (File)
Fri, Mar 22, 10:00 PM
Unknown Object (File)
Fri, Mar 22, 10:00 PM
Unknown Object (File)
Fri, Mar 22, 10:00 PM
Unknown Object (File)
Fri, Mar 22, 10:00 PM
Unknown Object (File)
Fri, Mar 22, 9:46 PM
Unknown Object (File)
Fri, Mar 8, 3:22 PM
Unknown Object (File)
Feb 21 2024, 10:22 AM
Subscribers

Details

Summary

Proposed commit message:

Add a NETMAP option to build and install the bro netmap plugin.

PR: 224918
Reported by: Shane Peters
Reviewed by: ? (mentor)
Approved by: ? (mentor)
Differential Revision: ?

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 15066
Build 15162: arc lint + arc unit

Event Timeline

Looks ok to me, but I'd like to see @matthew look at it too before you commit it.

This revision is now accepted and ready to land.Feb 14 2018, 11:55 PM
mat requested changes to this revision.Feb 15 2018, 1:20 PM
mat added inline comments.
security/bro/Makefile
7–8

USE_GITHUB for both of those.

This revision now requires changes to proceed.Feb 15 2018, 1:20 PM

Redo NETMAP changes to use GH_TUPLE (thanks to @mat for providing
an example by changing BROKER to work this way). Only download caf
and bro-netmap when they are enabled.

security/bro/Makefile
22–23

Those should go with the other options helpers.
And USE_GITHUB should probably only be defined when one of the option is enabled.

BROKER_USE= GITHUB=nodefault
NETMAP_USE= GITHUB=nodefault

Consolidate options helpers.
Only USE_GITHUB when BROKER or NETMAP is enabled.

This revision was not accepted when it landed; it landed in state Needs Review.Feb 19 2018, 10:04 PM
This revision was automatically updated to reflect the committed changes.
leres marked an inline comment as done.