cvs commit: ports/sysutils/LPRng Makefile distinfo pkg-descr
pkg-plist ports/sysutils/LPRng/files lprng.in pkg-message.in
Henrik Brix Andersen
brix at FreeBSD.org
Tue Jan 22 14:06:22 PST 2008
On Tue, Jan 22, 2008 at 09:51:08AM +0800, Rong-En Fan wrote:
> On Mon, Jan 21, 2008 at 12:45:02PM -0800, Doug Barton wrote:
> > The script looks good, thanks for doing this! The only thing I would point
> > out is that the default _enable=no is not needed, since rc.subr needs to
> > see one of the values for "yes" in order to do anything. No harm leaving it
> > in, I just wanted to point out that it's not strictly necessary.
> Thanks. I see most of our /etc/rc.d/* scripts does not have this.
> But most ports' have as our Porter's Handbook have such example.
I believe this is actually required for rc scripts supplied by
ports. Scripts included in the base wont need it, since their default
settings are already present in /etc/defaults/rc.conf.
I just tried removing the default setting for '$bitlbee_enable' in the
rc script for irc/bitlbee, I did recently. It then results in the
following error when '$bitlbee_enable' is undefined:
# /usr/local/etc/rc.d/bitlbee rcvar
/usr/local/etc/rc.d/bitlbee: WARNING: $bitlbee_enable is not set properly - see rc.conf(5).
Henrik Brix Andersen <brix at FreeBSD.org>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 209 bytes
Desc: not available
Url : http://lists.freebsd.org/pipermail/cvs-ports/attachments/20080122/f54029c3/attachment.pgp
More information about the cvs-ports