Re: [UPDATE] mail/dkim-milter

2014-01-14 Thread Remi Locherer
On Tue, Jan 14, 2014 at 11:34:06AM +, Stuart Henderson wrote:
> On 2014/01/14 12:21, Remi Locherer wrote:
> > On Tue, Jan 14, 2014 at 11:05:04AM +, Stuart Henderson wrote:
> > > On 2014/01/14 08:19, Remi Locherer wrote:
> > > > ping
> > > 
> > > groff is picked up by configure and used at the end of build, I think this
> > > should be disabled, which can be done with a tweak to site.config.m4,
> > > 
> > >  post-configure:
> > > @(echo "define(\`confCC', \`${CC}')"; \
> > > - echo "define(\`confOPTIMIZE', \`${CFLAGS}')") \
> > > + echo "define(\`confOPTIMIZE', \`${CFLAGS}')"; \
> > > + echo "define(\`confNO_MAN_BUILD')") \
> > > >${WRKSRC}/devtools/Site/site.config.m4
> > > 
> > > I'm a bit unsure about the reload support, it does have some support, but
> > > a number of config options aren't handled, and it loses command line 
> > > values
> > > (which may be passed in via dkim_filter_flags), so I think maybe it's 
> > > better
> > > not to have that exposed by rc.d?
> > > 
> > 
> > This is true. Command line flags get lost on reload. Maybe it's better
> > to not support reload and add rc_reload=NO to the rc script. 
> > 
> 
> With additional tweaks: @sample the configuration file into
> /etc/dkim-filter.conf, and adjust the default flags to use it.
> Does this still work for you Remi?

Yes, this works for me and I like this addidional tweak. I also compared 
the man pages again with an installation on a 5.4 system now that the
port really builds without groff. They still look fine.

> 
> Index: Makefile
> ===
> RCS file: /cvs/ports/mail/dkim-milter/Makefile,v
> retrieving revision 1.37
> diff -u -p -r1.37 Makefile
> --- Makefile  23 Oct 2013 16:33:20 -  1.37
> +++ Makefile  14 Jan 2014 11:32:54 -
> @@ -3,7 +3,7 @@
>  COMMENT= DKIM milter
>  
>  DISTNAME=dkim-milter-2.8.3
> -REVISION =   2
> +REVISION=3
>  CATEGORIES=  mail
>  HOMEPAGE=http://sourceforge.net/projects/dkim-milter/
>  
> @@ -18,11 +18,11 @@ PERMIT_PACKAGE_FTP=   Yes
>  WANTLIB= c crypto milter pthread ssl
>  
>  EXAMPLESDIR= share/examples/dkim-milter
> -USE_GROFF =  Yes
>  
>  post-configure:
>   @(echo "define(\`confCC', \`${CC}')"; \
> -   echo "define(\`confOPTIMIZE', \`${CFLAGS}')") \
> +   echo "define(\`confOPTIMIZE', \`${CFLAGS}')"; \
> +   echo "define(\`confNO_MAN_BUILD')") \
>   >${WRKSRC}/devtools/Site/site.config.m4
>  
>  do-install:
> Index: distinfo
> ===
> RCS file: /cvs/ports/mail/dkim-milter/distinfo,v
> retrieving revision 1.27
> diff -u -p -r1.27 distinfo
> --- distinfo  1 Jun 2009 12:15:57 -   1.27
> +++ distinfo  14 Jan 2014 11:32:54 -
> @@ -1,5 +1,2 @@
> -MD5 (dkim-milter-2.8.3.tar.gz) = 0gQ8Jp8XIMwJWptPFjzz3w==
> -RMD160 (dkim-milter-2.8.3.tar.gz) = M5iGqOwCYVp4W1pWBbK3UujpRrs=
> -SHA1 (dkim-milter-2.8.3.tar.gz) = 7o96oUKfdj9qQBLpAilnM98PoOY=
>  SHA256 (dkim-milter-2.8.3.tar.gz) = 
> Iir8k2twuCMono4yfAbCkgpfM99D8HAktfkxhWQGBI8=
>  SIZE (dkim-milter-2.8.3.tar.gz) = 748415
> Index: pkg/PLIST
> ===
> RCS file: /cvs/ports/mail/dkim-milter/pkg/PLIST,v
> retrieving revision 1.5
> diff -u -p -r1.5 PLIST
> --- pkg/PLIST 8 Mar 2013 06:42:09 -   1.5
> +++ pkg/PLIST 14 Jan 2014 11:32:54 -
> @@ -13,3 +13,5 @@ bin/dkim-genkey
>  share/doc/pkg-readmes/${FULLPKGNAME}
>  share/examples/dkim-milter/
>  share/examples/dkim-milter/dkim-filter.conf.sample
> +@sample ${SYSCONFDIR}/dkim-filter.conf
> +@rcscript ${RCDIR}/dkim_filter
> Index: pkg/dkim_filter.rc
> ===
> RCS file: pkg/dkim_filter.rc
> diff -N pkg/dkim_filter.rc
> --- /dev/null 1 Jan 1970 00:00:00 -
> +++ pkg/dkim_filter.rc14 Jan 2014 11:32:54 -
> @@ -0,0 +1,11 @@
> +#!/bin/sh
> +#
> +# $OpenBSD$
> +
> +daemon="${TRUEPREFIX}/libexec/dkim-filter"
> +daemon_flags="-x ${SYSCONFDIR}/dkim-filter.conf"
> +
> +. /etc/rc.d/rc.subr
> +
> +rc_reload=NO
> +rc_cmd $1
> 

-- 
Remi Locherer
Brunnwiesenstrasse 92 :: 8049 Zürich :: CH
remi.loche...@relo.ch :: +41 76 583 32 84



Re: [UPDATE] mail/dkim-milter

2014-01-14 Thread Stuart Henderson
On 2014/01/14 12:21, Remi Locherer wrote:
> On Tue, Jan 14, 2014 at 11:05:04AM +, Stuart Henderson wrote:
> > On 2014/01/14 08:19, Remi Locherer wrote:
> > > ping
> > 
> > groff is picked up by configure and used at the end of build, I think this
> > should be disabled, which can be done with a tweak to site.config.m4,
> > 
> >  post-configure:
> > @(echo "define(\`confCC', \`${CC}')"; \
> > - echo "define(\`confOPTIMIZE', \`${CFLAGS}')") \
> > + echo "define(\`confOPTIMIZE', \`${CFLAGS}')"; \
> > + echo "define(\`confNO_MAN_BUILD')") \
> > >${WRKSRC}/devtools/Site/site.config.m4
> > 
> > I'm a bit unsure about the reload support, it does have some support, but
> > a number of config options aren't handled, and it loses command line values
> > (which may be passed in via dkim_filter_flags), so I think maybe it's better
> > not to have that exposed by rc.d?
> > 
> 
> This is true. Command line flags get lost on reload. Maybe it's better
> to not support reload and add rc_reload=NO to the rc script. 
> 

With additional tweaks: @sample the configuration file into
/etc/dkim-filter.conf, and adjust the default flags to use it.
Does this still work for you Remi?

Index: Makefile
===
RCS file: /cvs/ports/mail/dkim-milter/Makefile,v
retrieving revision 1.37
diff -u -p -r1.37 Makefile
--- Makefile23 Oct 2013 16:33:20 -  1.37
+++ Makefile14 Jan 2014 11:32:54 -
@@ -3,7 +3,7 @@
 COMMENT=   DKIM milter
 
 DISTNAME=  dkim-milter-2.8.3
-REVISION = 2
+REVISION=  3
 CATEGORIES=mail
 HOMEPAGE=  http://sourceforge.net/projects/dkim-milter/
 
@@ -18,11 +18,11 @@ PERMIT_PACKAGE_FTP= Yes
 WANTLIB=   c crypto milter pthread ssl
 
 EXAMPLESDIR=   share/examples/dkim-milter
-USE_GROFF =Yes
 
 post-configure:
@(echo "define(\`confCC', \`${CC}')"; \
- echo "define(\`confOPTIMIZE', \`${CFLAGS}')") \
+ echo "define(\`confOPTIMIZE', \`${CFLAGS}')"; \
+ echo "define(\`confNO_MAN_BUILD')") \
>${WRKSRC}/devtools/Site/site.config.m4
 
 do-install:
Index: distinfo
===
RCS file: /cvs/ports/mail/dkim-milter/distinfo,v
retrieving revision 1.27
diff -u -p -r1.27 distinfo
--- distinfo1 Jun 2009 12:15:57 -   1.27
+++ distinfo14 Jan 2014 11:32:54 -
@@ -1,5 +1,2 @@
-MD5 (dkim-milter-2.8.3.tar.gz) = 0gQ8Jp8XIMwJWptPFjzz3w==
-RMD160 (dkim-milter-2.8.3.tar.gz) = M5iGqOwCYVp4W1pWBbK3UujpRrs=
-SHA1 (dkim-milter-2.8.3.tar.gz) = 7o96oUKfdj9qQBLpAilnM98PoOY=
 SHA256 (dkim-milter-2.8.3.tar.gz) = 
Iir8k2twuCMono4yfAbCkgpfM99D8HAktfkxhWQGBI8=
 SIZE (dkim-milter-2.8.3.tar.gz) = 748415
Index: pkg/PLIST
===
RCS file: /cvs/ports/mail/dkim-milter/pkg/PLIST,v
retrieving revision 1.5
diff -u -p -r1.5 PLIST
--- pkg/PLIST   8 Mar 2013 06:42:09 -   1.5
+++ pkg/PLIST   14 Jan 2014 11:32:54 -
@@ -13,3 +13,5 @@ bin/dkim-genkey
 share/doc/pkg-readmes/${FULLPKGNAME}
 share/examples/dkim-milter/
 share/examples/dkim-milter/dkim-filter.conf.sample
+@sample ${SYSCONFDIR}/dkim-filter.conf
+@rcscript ${RCDIR}/dkim_filter
Index: pkg/dkim_filter.rc
===
RCS file: pkg/dkim_filter.rc
diff -N pkg/dkim_filter.rc
--- /dev/null   1 Jan 1970 00:00:00 -
+++ pkg/dkim_filter.rc  14 Jan 2014 11:32:54 -
@@ -0,0 +1,11 @@
+#!/bin/sh
+#
+# $OpenBSD$
+
+daemon="${TRUEPREFIX}/libexec/dkim-filter"
+daemon_flags="-x ${SYSCONFDIR}/dkim-filter.conf"
+
+. /etc/rc.d/rc.subr
+
+rc_reload=NO
+rc_cmd $1



Re: [UPDATE] mail/dkim-milter

2014-01-14 Thread Remi Locherer
On Tue, Jan 14, 2014 at 11:05:04AM +, Stuart Henderson wrote:
> On 2014/01/14 08:19, Remi Locherer wrote:
> > ping
> 
> groff is picked up by configure and used at the end of build, I think this
> should be disabled, which can be done with a tweak to site.config.m4,
> 
>  post-configure:
> @(echo "define(\`confCC', \`${CC}')"; \
> - echo "define(\`confOPTIMIZE', \`${CFLAGS}')") \
> + echo "define(\`confOPTIMIZE', \`${CFLAGS}')"; \
> + echo "define(\`confNO_MAN_BUILD')") \
> >${WRKSRC}/devtools/Site/site.config.m4
> 
> I'm a bit unsure about the reload support, it does have some support, but
> a number of config options aren't handled, and it loses command line values
> (which may be passed in via dkim_filter_flags), so I think maybe it's better
> not to have that exposed by rc.d?
> 

This is true. Command line flags get lost on reload. Maybe it's better
to not support reload and add rc_reload=NO to the rc script. 



Re: [UPDATE] mail/dkim-milter

2014-01-14 Thread Stuart Henderson
On 2014/01/14 08:19, Remi Locherer wrote:
> ping

groff is picked up by configure and used at the end of build, I think this
should be disabled, which can be done with a tweak to site.config.m4,

 post-configure:
@(echo "define(\`confCC', \`${CC}')"; \
- echo "define(\`confOPTIMIZE', \`${CFLAGS}')") \
+ echo "define(\`confOPTIMIZE', \`${CFLAGS}')"; \
+ echo "define(\`confNO_MAN_BUILD')") \
>${WRKSRC}/devtools/Site/site.config.m4

I'm a bit unsure about the reload support, it does have some support, but
a number of config options aren't handled, and it loses command line values
(which may be passed in via dkim_filter_flags), so I think maybe it's better
not to have that exposed by rc.d?



Re: [UPDATE] mail/dkim-milter

2014-01-13 Thread Remi Locherer
ping

On Wed, Jan 08, 2014 at 07:39:59AM +0100, Remi Locherer wrote:
> On Tue, Jan 07, 2014 at 09:27:26PM -0300, Gleydson Soares wrote:
> > Em 07/01/2014 20:30, "Antoine Jacoutot"  escreveu:
> > >
> > > On Tue, Jan 07, 2014 at 11:13:24PM +0100, Remi Locherer wrote:
> > > > This adds an rc script do dkim-milter. Manuals look good without
> > > > using groff.
> > > >
> > > > This is not an update of dkim-milter itself, 2.8.3 is likely the
> > > > last version. Development continues within the OpenDKIM project
> > > > (http://www.opendkim.org/).
> > > >
> > > > Remi
> 
> > > >
> > > > +daemon="/usr/local/libexec/dkim-filter"
> > >
> > > ${TRUEPREFIX} instead of /usr/local
> > >
> > >
> > 
> > plus:
> > add RCS id in RC script.
> 
> new diff:
> 
> 
> cvs diff: Diffing .
> Index: Makefile
> ===
> RCS file: /cvs/ports/mail/dkim-milter/Makefile,v
> retrieving revision 1.37
> diff -u -p -r1.37 Makefile
> --- Makefile  23 Oct 2013 16:33:20 -  1.37
> +++ Makefile  7 Jan 2014 21:30:28 -
> @@ -3,7 +3,7 @@
>  COMMENT= DKIM milter
>  
>  DISTNAME=dkim-milter-2.8.3
> -REVISION =   2
> +REVISION =   3
>  CATEGORIES=  mail
>  HOMEPAGE=http://sourceforge.net/projects/dkim-milter/
>  
> @@ -18,7 +18,6 @@ PERMIT_PACKAGE_FTP= Yes
>  WANTLIB= c crypto milter pthread ssl
>  
>  EXAMPLESDIR= share/examples/dkim-milter
> -USE_GROFF =  Yes
>  
>  post-configure:
>   @(echo "define(\`confCC', \`${CC}')"; \
> cvs diff: Diffing patches
> cvs diff: Diffing pkg
> Index: pkg/PLIST
> ===
> RCS file: /cvs/ports/mail/dkim-milter/pkg/PLIST,v
> retrieving revision 1.5
> diff -u -p -r1.5 PLIST
> --- pkg/PLIST 8 Mar 2013 06:42:09 -   1.5
> +++ pkg/PLIST 7 Jan 2014 19:40:36 -
> @@ -13,3 +13,4 @@ bin/dkim-genkey
>  share/doc/pkg-readmes/${FULLPKGNAME}
>  share/examples/dkim-milter/
>  share/examples/dkim-milter/dkim-filter.conf.sample
> +@rcscript ${RCDIR}/dkim_filter
> Index: pkg/dkim_filter.rc
> ===
> RCS file: pkg/dkim_filter.rc
> diff -N pkg/dkim_filter.rc
> --- /dev/null 1 Jan 1970 00:00:00 -
> +++ pkg/dkim_filter.rc7 Jan 2014 23:52:03 -
> @@ -0,0 +1,13 @@
> +#!/bin/sh
> +#
> +# $OpenBSD$
> +
> +daemon="${TRUEPREFIX}/libexec/dkim-filter"
> +
> +. /etc/rc.d/rc.subr
> +
> +rc_reload() {
> + pkill -USR1 -f "^${pexp}"
> +}
> +
> +rc_cmd $1
> 



Re: [UPDATE] mail/dkim-milter

2014-01-07 Thread Remi Locherer
On Tue, Jan 07, 2014 at 09:27:26PM -0300, Gleydson Soares wrote:
> Em 07/01/2014 20:30, "Antoine Jacoutot"  escreveu:
> >
> > On Tue, Jan 07, 2014 at 11:13:24PM +0100, Remi Locherer wrote:
> > > This adds an rc script do dkim-milter. Manuals look good without
> > > using groff.
> > >
> > > This is not an update of dkim-milter itself, 2.8.3 is likely the
> > > last version. Development continues within the OpenDKIM project
> > > (http://www.opendkim.org/).
> > >
> > > Remi

> > >
> > > +daemon="/usr/local/libexec/dkim-filter"
> >
> > ${TRUEPREFIX} instead of /usr/local
> >
> >
> 
> plus:
> add RCS id in RC script.

new diff:


cvs diff: Diffing .
Index: Makefile
===
RCS file: /cvs/ports/mail/dkim-milter/Makefile,v
retrieving revision 1.37
diff -u -p -r1.37 Makefile
--- Makefile23 Oct 2013 16:33:20 -  1.37
+++ Makefile7 Jan 2014 21:30:28 -
@@ -3,7 +3,7 @@
 COMMENT=   DKIM milter
 
 DISTNAME=  dkim-milter-2.8.3
-REVISION = 2
+REVISION = 3
 CATEGORIES=mail
 HOMEPAGE=  http://sourceforge.net/projects/dkim-milter/
 
@@ -18,7 +18,6 @@ PERMIT_PACKAGE_FTP=   Yes
 WANTLIB=   c crypto milter pthread ssl
 
 EXAMPLESDIR=   share/examples/dkim-milter
-USE_GROFF =Yes
 
 post-configure:
@(echo "define(\`confCC', \`${CC}')"; \
cvs diff: Diffing patches
cvs diff: Diffing pkg
Index: pkg/PLIST
===
RCS file: /cvs/ports/mail/dkim-milter/pkg/PLIST,v
retrieving revision 1.5
diff -u -p -r1.5 PLIST
--- pkg/PLIST   8 Mar 2013 06:42:09 -   1.5
+++ pkg/PLIST   7 Jan 2014 19:40:36 -
@@ -13,3 +13,4 @@ bin/dkim-genkey
 share/doc/pkg-readmes/${FULLPKGNAME}
 share/examples/dkim-milter/
 share/examples/dkim-milter/dkim-filter.conf.sample
+@rcscript ${RCDIR}/dkim_filter
Index: pkg/dkim_filter.rc
===
RCS file: pkg/dkim_filter.rc
diff -N pkg/dkim_filter.rc
--- /dev/null   1 Jan 1970 00:00:00 -
+++ pkg/dkim_filter.rc  7 Jan 2014 23:52:03 -
@@ -0,0 +1,13 @@
+#!/bin/sh
+#
+# $OpenBSD$
+
+daemon="${TRUEPREFIX}/libexec/dkim-filter"
+
+. /etc/rc.d/rc.subr
+
+rc_reload() {
+   pkill -USR1 -f "^${pexp}"
+}
+
+rc_cmd $1



Re: [UPDATE] mail/dkim-milter

2014-01-07 Thread Gleydson Soares
Em 07/01/2014 20:30, "Antoine Jacoutot"  escreveu:
>
> On Tue, Jan 07, 2014 at 11:13:24PM +0100, Remi Locherer wrote:
> > This adds an rc script do dkim-milter. Manuals look good without
> > using groff.
> >
> > This is not an update of dkim-milter itself, 2.8.3 is likely the
> > last version. Development continues within the OpenDKIM project
> > (http://www.opendkim.org/).
> >
> > Remi
> >
> >
> > cvs diff: Diffing .
> > Index: Makefile
> > ===
> > RCS file: /cvs/ports/mail/dkim-milter/Makefile,v
> > retrieving revision 1.37
> > diff -u -p -r1.37 Makefile
> > --- Makefile  23 Oct 2013 16:33:20 -  1.37
> > +++ Makefile  7 Jan 2014 21:30:28 -
> > @@ -3,7 +3,7 @@
> >  COMMENT= DKIM milter
> >
> >  DISTNAME=dkim-milter-2.8.3
> > -REVISION =   2
> > +REVISION =   3
> >  CATEGORIES=  mail
> >  HOMEPAGE=http://sourceforge.net/projects/dkim-milter/
> >
> > @@ -18,7 +18,6 @@ PERMIT_PACKAGE_FTP= Yes
> >  WANTLIB= c crypto milter pthread ssl
> >
> >  EXAMPLESDIR= share/examples/dkim-milter
> > -USE_GROFF =  Yes
> >
> >  post-configure:
> >   @(echo "define(\`confCC', \`${CC}')"; \
> > cvs diff: Diffing patches
> > cvs diff: Diffing pkg
> > Index: pkg/PLIST
> > ===
> > RCS file: /cvs/ports/mail/dkim-milter/pkg/PLIST,v
> > retrieving revision 1.5
> > diff -u -p -r1.5 PLIST
> > --- pkg/PLIST 8 Mar 2013 06:42:09 -   1.5
> > +++ pkg/PLIST 7 Jan 2014 19:40:36 -
> > @@ -13,3 +13,4 @@ bin/dkim-genkey
> >  share/doc/pkg-readmes/${FULLPKGNAME}
> >  share/examples/dkim-milter/
> >  share/examples/dkim-milter/dkim-filter.conf.sample
> > +@rcscript ${RCDIR}/dkim_filter
> > Index: pkg/dkim_filter.rc
> > ===
> > RCS file: pkg/dkim_filter.rc
> > diff -N pkg/dkim_filter.rc
> > --- /dev/null 1 Jan 1970 00:00:00 -
> > +++ pkg/dkim_filter.rc7 Jan 2014 19:36:41 -
> > @@ -0,0 +1,11 @@
> > +#!/bin/sh
> > +
> > +daemon="/usr/local/libexec/dkim-filter"
>
> ${TRUEPREFIX} instead of /usr/local
>
>
> > +
> > +. /etc/rc.d/rc.subr
> > +
> > +rc_reload() {
> > +pkill -USR1 -f "^${pexp}"
> > +}
> > +
> > +rc_cmd $1
> >
>
> --
> Antoine
>

plus:
add RCS id in RC script.
Em 07/01/2014 20:30, "Antoine Jacoutot"  escreveu:

> On Tue, Jan 07, 2014 at 11:13:24PM +0100, Remi Locherer wrote:
> > This adds an rc script do dkim-milter. Manuals look good without
> > using groff.
> >
> > This is not an update of dkim-milter itself, 2.8.3 is likely the
> > last version. Development continues within the OpenDKIM project
> > (http://www.opendkim.org/).
> >
> > Remi
> >
> >
> > cvs diff: Diffing .
> > Index: Makefile
> > ===
> > RCS file: /cvs/ports/mail/dkim-milter/Makefile,v
> > retrieving revision 1.37
> > diff -u -p -r1.37 Makefile
> > --- Makefile  23 Oct 2013 16:33:20 -  1.37
> > +++ Makefile  7 Jan 2014 21:30:28 -
> > @@ -3,7 +3,7 @@
> >  COMMENT= DKIM milter
> >
> >  DISTNAME=dkim-milter-2.8.3
> > -REVISION =   2
> > +REVISION =   3
> >  CATEGORIES=  mail
> >  HOMEPAGE=http://sourceforge.net/projects/dkim-milter/
> >
> > @@ -18,7 +18,6 @@ PERMIT_PACKAGE_FTP= Yes
> >  WANTLIB= c crypto milter pthread ssl
> >
> >  EXAMPLESDIR= share/examples/dkim-milter
> > -USE_GROFF =  Yes
> >
> >  post-configure:
> >   @(echo "define(\`confCC', \`${CC}')"; \
> > cvs diff: Diffing patches
> > cvs diff: Diffing pkg
> > Index: pkg/PLIST
> > ===
> > RCS file: /cvs/ports/mail/dkim-milter/pkg/PLIST,v
> > retrieving revision 1.5
> > diff -u -p -r1.5 PLIST
> > --- pkg/PLIST 8 Mar 2013 06:42:09 -   1.5
> > +++ pkg/PLIST 7 Jan 2014 19:40:36 -
> > @@ -13,3 +13,4 @@ bin/dkim-genkey
> >  share/doc/pkg-readmes/${FULLPKGNAME}
> >  share/examples/dkim-milter/
> >  share/examples/dkim-milter/dkim-filter.conf.sample
> > +@rcscript ${RCDIR}/dkim_filter
> > Index: pkg/dkim_filter.rc
> > ===
> > RCS file: pkg/dkim_filter.rc
> > diff -N pkg/dkim_filter.rc
> > --- /dev/null 1 Jan 1970 00:00:00 -
> > +++ pkg/dkim_filter.rc7 Jan 2014 19:36:41 -
> > @@ -0,0 +1,11 @@
> > +#!/bin/sh
> > +
> > +daemon="/usr/local/libexec/dkim-filter"
>
> ${TRUEPREFIX} instead of /usr/local
>
>
> > +
> > +. /etc/rc.d/rc.subr
> > +
> > +rc_reload() {
> > +pkill -USR1 -f "^${pexp}"
> > +}
> > +
> > +rc_cmd $1
> >
>
> --
> Antoine
>
>


Re: [UPDATE] mail/dkim-milter

2014-01-07 Thread Antoine Jacoutot
On Tue, Jan 07, 2014 at 11:13:24PM +0100, Remi Locherer wrote:
> This adds an rc script do dkim-milter. Manuals look good without
> using groff.
> 
> This is not an update of dkim-milter itself, 2.8.3 is likely the
> last version. Development continues within the OpenDKIM project
> (http://www.opendkim.org/).
> 
> Remi
> 
> 
> cvs diff: Diffing .
> Index: Makefile
> ===
> RCS file: /cvs/ports/mail/dkim-milter/Makefile,v
> retrieving revision 1.37
> diff -u -p -r1.37 Makefile
> --- Makefile  23 Oct 2013 16:33:20 -  1.37
> +++ Makefile  7 Jan 2014 21:30:28 -
> @@ -3,7 +3,7 @@
>  COMMENT= DKIM milter
>  
>  DISTNAME=dkim-milter-2.8.3
> -REVISION =   2
> +REVISION =   3
>  CATEGORIES=  mail
>  HOMEPAGE=http://sourceforge.net/projects/dkim-milter/
>  
> @@ -18,7 +18,6 @@ PERMIT_PACKAGE_FTP= Yes
>  WANTLIB= c crypto milter pthread ssl
>  
>  EXAMPLESDIR= share/examples/dkim-milter
> -USE_GROFF =  Yes
>  
>  post-configure:
>   @(echo "define(\`confCC', \`${CC}')"; \
> cvs diff: Diffing patches
> cvs diff: Diffing pkg
> Index: pkg/PLIST
> ===
> RCS file: /cvs/ports/mail/dkim-milter/pkg/PLIST,v
> retrieving revision 1.5
> diff -u -p -r1.5 PLIST
> --- pkg/PLIST 8 Mar 2013 06:42:09 -   1.5
> +++ pkg/PLIST 7 Jan 2014 19:40:36 -
> @@ -13,3 +13,4 @@ bin/dkim-genkey
>  share/doc/pkg-readmes/${FULLPKGNAME}
>  share/examples/dkim-milter/
>  share/examples/dkim-milter/dkim-filter.conf.sample
> +@rcscript ${RCDIR}/dkim_filter
> Index: pkg/dkim_filter.rc
> ===
> RCS file: pkg/dkim_filter.rc
> diff -N pkg/dkim_filter.rc
> --- /dev/null 1 Jan 1970 00:00:00 -
> +++ pkg/dkim_filter.rc7 Jan 2014 19:36:41 -
> @@ -0,0 +1,11 @@
> +#!/bin/sh
> +
> +daemon="/usr/local/libexec/dkim-filter"

${TRUEPREFIX} instead of /usr/local


> +
> +. /etc/rc.d/rc.subr
> +
> +rc_reload() {
> +pkill -USR1 -f "^${pexp}"
> +}
> +
> +rc_cmd $1
> 

-- 
Antoine



[UPDATE] mail/dkim-milter

2014-01-07 Thread Remi Locherer
This adds an rc script do dkim-milter. Manuals look good without
using groff.

This is not an update of dkim-milter itself, 2.8.3 is likely the
last version. Development continues within the OpenDKIM project
(http://www.opendkim.org/).

Remi


cvs diff: Diffing .
Index: Makefile
===
RCS file: /cvs/ports/mail/dkim-milter/Makefile,v
retrieving revision 1.37
diff -u -p -r1.37 Makefile
--- Makefile23 Oct 2013 16:33:20 -  1.37
+++ Makefile7 Jan 2014 21:30:28 -
@@ -3,7 +3,7 @@
 COMMENT=   DKIM milter
 
 DISTNAME=  dkim-milter-2.8.3
-REVISION = 2
+REVISION = 3
 CATEGORIES=mail
 HOMEPAGE=  http://sourceforge.net/projects/dkim-milter/
 
@@ -18,7 +18,6 @@ PERMIT_PACKAGE_FTP=   Yes
 WANTLIB=   c crypto milter pthread ssl
 
 EXAMPLESDIR=   share/examples/dkim-milter
-USE_GROFF =Yes
 
 post-configure:
@(echo "define(\`confCC', \`${CC}')"; \
cvs diff: Diffing patches
cvs diff: Diffing pkg
Index: pkg/PLIST
===
RCS file: /cvs/ports/mail/dkim-milter/pkg/PLIST,v
retrieving revision 1.5
diff -u -p -r1.5 PLIST
--- pkg/PLIST   8 Mar 2013 06:42:09 -   1.5
+++ pkg/PLIST   7 Jan 2014 19:40:36 -
@@ -13,3 +13,4 @@ bin/dkim-genkey
 share/doc/pkg-readmes/${FULLPKGNAME}
 share/examples/dkim-milter/
 share/examples/dkim-milter/dkim-filter.conf.sample
+@rcscript ${RCDIR}/dkim_filter
Index: pkg/dkim_filter.rc
===
RCS file: pkg/dkim_filter.rc
diff -N pkg/dkim_filter.rc
--- /dev/null   1 Jan 1970 00:00:00 -
+++ pkg/dkim_filter.rc  7 Jan 2014 19:36:41 -
@@ -0,0 +1,11 @@
+#!/bin/sh
+
+daemon="/usr/local/libexec/dkim-filter"
+
+. /etc/rc.d/rc.subr
+
+rc_reload() {
+pkill -USR1 -f "^${pexp}"
+}
+
+rc_cmd $1