command=foo >/dev/null 2>&1
|
v
command=foo
command_args='>/dev/null 2>&1'
This is clearly what should have been done, for several reasons.
No PORTREVISION bump because the old version simply ignored everything
after the space, and does not seem to have done any harm. However
it's good to clean these up so that similar errors aren't pasted into
a new script where they might actually matter.
literal name_enable wherever possible, and ${name}_enable
when it's not, to prepare for the demise of set_rcvar().
In cases where I had to hand-edit unusual instances also
modify formatting slightly to be more uniform (and in
some cases, correct). This includes adding some $FreeBSD$
tags, and most importantly moving rcvar= to right after
name= so it's clear that one is derived from the other.
perl unconditonally, or conditionally. To be able to conditionalize the
inclusion of bsd.perl.mk, they now need to be defined before the inclusion
of bsd.port.pre.mk.
Hat: portmgr
Move net/rrdtool to databases/rrdtool.
It's an itch which needs to be scratched: net/rrdtool came from
net/mrtg, which was a good location for it. net/mrtg has later
been moved to net-mgmt/mrtg. net/rrdtool is "Round Robin Database
Tools", therefor it's better if it moves to databases/rrdtool.
Same with net/rrdtool10.
PR: ports/112942
Submitted by: Edwin Groothuis <edwin@mavetju.org>
We have not checked for this KEYWORD for a long time now, so this
is a complete noop, and thus no PORTREVISION bump. Removing it at
this point is mostly for pedantic reasons, and partly to avoid
perpetuating this anachronism by copy and paste to future scripts.
Couriergraph is a very simple mail statistics RRDtool
frontend for Postfix that produces daily, weekly, monthly
and yearly graphs of Courier's POP3, IMAP, POP3s and IMAPs
logins.
WWW: http://www.stahl.bau.tu-bs.de/~hildeb/postfix/couriergraph/
PR: ports/85985
Submitted by: Xavier Beaudouin <kiwi@oav.net>
