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).
