On 12/20, Stuart Henderson wrote:
 
> There are a couple of "/etc/riemann.config", /etc should be replaced
> with ${SYSCONFDIR} one way or another (either via SUBST_CMD or a simple
> sed -i).

Fixed.

> Similarly /var/log should be ${LOCALSTATEDIR}/log.
> 
> PLIST should have the next free uid from ports/infrastructure/db/user.list,

Fixed, the next free uid/gid 785, patch for user.list attached.
 
> And the @extraunexec is a bit wrong, you have
> 
> @extraunexec rmdir -rf /var/log/riemann
> 
> - should probably be
> 
> @extraunexec rm -rf /var/log/riemann/*

Fixed. Please see the updated port attached.
 
-- 
With best regards,
Pavel Korovin

Attachment: riemann.tar.gz
Description: application/tar-gz

Index: infrastructure/db/user.list
===================================================================
RCS file: /cvs/ports/infrastructure/db/user.list,v
retrieving revision 1.285
diff -u -p -r1.285 user.list
--- infrastructure/db/user.list 5 Dec 2016 13:58:14 -0000       1.285
+++ infrastructure/db/user.list 20 Dec 2016 21:35:41 -0000
@@ -293,3 +293,4 @@ id  user            group           port options
 782 _ympd              _ympd           audio/ympd
 783 _rmilter           _rmilter        mail/rmilter
 784 _hping             _hping          net/hping
+785 _riemann           _riemann        sysutils/riemann

Reply via email to