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.
- Changelog
* Rejected mail now are updated in totality (eg vrfy rejection IS a rejection).
* Support for Policyd-weigth added by Bagolu
* Fixed imprecises totals like in mailgraph
* Fixed update of rrd_virus when nobody logs in (Bug and fix
reported by Claudius Herder)
* Added Bagolu's patches
* Fix dovecot detection
PR: ports/125142
Submitted by: Xavier Beaudouin <kiwi@oav.net> (maintainer)
handled after load_rc_config per Handbook. For example, if user sets
name_pidfile in rc.conf, it does not correctly pass to name_flags because
when we set name_flags, it uses our default name_pidfile, but not user's.
- Bump PORTREVISION as rc scripts changed.
Approved by: portmgr (pav), Xavier Beaudouin <kiwi at oav.net> (ovs maintainer)
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