On Thu, Feb 15, 2018 at 08:30:08AM +0000, Gonzalo L. Rodriguez wrote:
> CVSROOT: /cvs
> Module name: ports
> Changes by: [email protected] 2018/02/15 01:30:08
>
> Log message:
> high performance network IDS, IPS and security monitoring OK jasper@
>
> Status:
>
> Vendor Tag: gonzalo
> Release Tags: gonzalo_20181502
>
> N ports/security/suricata/Makefile
> N ports/security/suricata/distinfo
> N ports/security/suricata/pkg/DESCR
> N ports/security/suricata/pkg/PLIST
> N ports/security/suricata/pkg/suricata.rc
> N ports/security/suricata/pkg/README
> N ports/security/suricata/patches/patch-suricata_yaml_in
>
> No conflicts created by this import
Hi.
That line in the rc.d script looks bogus, why is it needed?:
daemon_flags=""
I also don't understand the need for a specific rc_start() function.
You need to reset @owner and @group before the @rcscript marker in your PLIST.
Otherwise it will have wrong permissions.
In CONFIGURE_ENV you have:
CPPFLAGS="${CPPFLAGS} -I${LOCALBASE}/include"
LDFLAGS="${LDFLAGS} -L${LOCALBASE}/lib"
Is it necessary to have ${CPPFLAGS} and ${LDFLAGS} since they aren't defined
elsewhere?
--
Antoine
