On Sun, May 14, 2006 at 04:27:31PM +0200, Matthias Kilian wrote:
> On Sun, May 14, 2006 at 12:37:15PM +0200, Daniel Hartmeier wrote:
> > If someone with ports-foo has any suggestions about how to split pfstatd
> > into a separate package (so a package containing only pfstatd could be
> > installed on firewalls without any dependancies), please speak up ;)
>
> Patch against your patch:
>
> diff -rNu ../pfstat.daniel/Makefile ./Makefile
> --- ../pfstat.daniel/Makefile Sun May 14 16:06:44 2006
> +++ ./Makefile Sun May 14 16:21:34 2006
> @@ -1,9 +1,12 @@
> # $OpenBSD: Makefile,v 1.11 2005/10/24 05:09:59 fgsch Exp $
>
> COMMENT= "packet filter statistics visualization"
> +COMMENT-daemon= "pfstat daemon"
>
> -DISTNAME= pfstat-2.0
> -PKGNAME= ${DISTNAME}
> +V= 2.0
> +DISTNAME= pfstat-${V}
> +PKGNAME= pfstat-${V}
> +PKGNAME-daemon= pfstat-daemon-${V}
How about making the PKGNAME for the daemon be pfstatd-${V}
Otherwise the rest looks fine to me.
> CATEGORIES= net
> MASTER_SITES= http://www.benzedrine.cx/
>
> @@ -15,11 +18,21 @@
> PERMIT_PACKAGE_FTP= Yes
> PERMIT_DISTFILES_CDROM= Yes
> PERMIT_DISTFILES_FTP= Yes
> -WANTLIB= c jpeg m png z
>
> +MULTI_PACKAGES= -daemon
> +SUBPACKAGE?=
> +
> +WANTLIB= c
> +
> +.if defined(PACKAGING)
> +. if ${SUBPACKAGE} != "-daemon"
> +WANTLIB+= jpeg m png z
> +
> MODULES= converters/libiconv
>
> LIB_DEPENDS= gd.18::graphics/gd
> +. endif
> +.endif
>
> do-install:
> ${INSTALL_PROGRAM} ${WRKSRC}/pfstat ${PREFIX}/bin
> diff -rNu ../pfstat.daniel/pkg/DESCR-daemon ./pkg/DESCR-daemon
> --- ../pfstat.daniel/pkg/DESCR-daemon Thu Jan 1 01:00:00 1970
> +++ ./pkg/DESCR-daemon Sun May 14 16:16:57 2006
> @@ -0,0 +1 @@
> +pfstatd is a daemon for remote access to pf statistics using pfstat.
> diff -rNu ../pfstat.daniel/pkg/PLIST ./pkg/PLIST
> --- ../pfstat.daniel/pkg/PLIST Sun May 14 16:06:44 2006
> +++ ./pkg/PLIST Sun May 14 16:15:57 2006
> @@ -1,5 +1,3 @@
> @comment $OpenBSD: PLIST,v 1.2 2004/09/15 18:17:44 espie Exp $
> bin/pfstat
> -bin/pfstatd
> @man man/man8/pfstat.8
> [EMAIL PROTECTED] man/man8/pfstatd.8
> diff -rNu ../pfstat.daniel/pkg/PLIST-daemon ./pkg/PLIST-daemon
> --- ../pfstat.daniel/pkg/PLIST-daemon Thu Jan 1 01:00:00 1970
> +++ ./pkg/PLIST-daemon Sun May 14 16:16:14 2006
> @@ -0,0 +1,3 @@
> [EMAIL PROTECTED] $OpenBSD$
> +bin/pfstatd
> [EMAIL PROTECTED] man/man8/pfstatd.8
>
