UPDATE: net/snort 2.9.12

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

UPDATE: net/snort 2.9.12

Markus Lude-3
Hello,

attached is an update to snort-2.9.12.

Tested on amd64.

Please test, comment and eventually commit.

Regards
Markus


snort-2.9.12.diff (4K) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: UPDATE: net/snort 2.9.12

Stuart Henderson
On 2019/02/10 20:29, Markus Lude wrote:

> Hello,
>
> attached is an update to snort-2.9.12.
>
> Tested on amd64.
>
> Please test, comment and eventually commit.
>
> Regards
> Markus
>

> Index: Makefile
> ===================================================================
> RCS file: /cvs/ports/net/snort/Makefile,v
> retrieving revision 1.100
> diff -u -p -u -p -r1.100 Makefile
> --- Makefile 4 Sep 2018 12:46:18 -0000 1.100
> +++ Makefile 10 Feb 2019 19:24:33 -0000
> @@ -2,8 +2,8 @@
>  
>  COMMENT = highly flexible sniffer/NIDS
>  
> -VERSION = 2.9.11.1
> -DISTNAME = snort-${VERSION}
> +VERSION = 2.9.12.0
> +DISTNAME = snort-${VERSION:S/.0$//g}

that seems a bit of a fiddly way to deal with this - how about this
to make it cleaner and a bit more self-documenting?

DISTNAME = snort-2.9.12
RULESV = 29120
SUBST_VARS = RULESV

(and change README to match, and remove the V lower down which is also
confusing portbump - hence REVISION being in the wrong place :-)

>  
>  CATEGORIES = net security
>  
> @@ -25,6 +25,7 @@ CONFIGURE_ARGS += --disable-static-daq \
>  
>  LIB_DEPENDS = archivers/xz \
>   devel/pcre \
> + lang/luajit \

Note this:

$ make package
===>  Building package for snort-2.9.12
Create /usr/ports/packages/amd64/all/snort-2.9.12.tgz
LIB_DEPENDS lang/luajit not needed for net/snort ?
Creating package snort-2.9.12
Link to /usr/ports/packages/amd64/ftp/snort-2.9.12.tgz
Link to /usr/ports/packages/amd64/cdrom/snort-2.9.12.tgz

see "man bsd.port.mk | less +/not.needed.for" and port-lib-depends-check
(in this case it's a missing addition to WANTLIB).

Reply | Threaded
Open this post in threaded view
|

Re: UPDATE: net/snort 2.9.12

Markus Lude-3
Hello,

thanks Stuart for your suggestions.

Updated diff attached.
- straight DISTNAME
- use RULESV instead of V
- fix WANTLIB

Regards
Markus


snort-2.9.12.diff2 (5K) Download Attachment