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.
logged into system it segfaults (signal 11).
There is a bug in the 'chk_maxuser' function. When it receives as a
prameter 'user==NULL' following condition:
if (user->mgroup != 0) {
causes a segfault. Bug can be very easy reproduced and tracked by
gdb (core dumps have to be enabled).
Tested fix is:
if (user != NULL && user->mgroup != 0) {
Obtained from: doinkd bug ID at Sourceforge: 1652293
http://sourceforge.net/tracker/?func=detail&aid=1652293&group_id=168453&atid=846828
propogated by copy and paste.
1. Primarily the "empty variable" default assignment, which is mostly
${name}_flags="", but fix a few others as well.
2. Where they are not already documented, add the existence of the _flags
(or other deleted empties) option to the comments, and in some cases add
comments from scratch.
3. Replace things that look like:
prefix=%%PREFIX%%
command=${prefix}/sbin/foo
to just use %%PREFIX%%. In many cases the $prefix variable is only used
once, and in some cases it is not used at all.
4. In a few cases remove ${name}_flags from command_args
5. Remove a long-stale comment about putting the port's rc.d script in
/etc/rc.d (which is no longer necessary).
No PORTREVISION bumps because all of these changes are noops.
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.
[Has anyone figured-out what makes the number 393 so interesting to PW, now?]
I wonder what was going through Jordan's head during his infamous
$Id$-smashing commit.
Before I forget....
Thanks to naddy@mips.rhein-neckar.de (Christian Weisgerber) for prompting
this commit. See msg-id: 7geokh$tje$1@mips.rhein-neckar.de
(1) NOMANCOMPRESS is set by user, not porter. To indicate that MAN? variables
are referring to already compressed manpages, use MANCOMPRESSED.
(2) Remove second address from MAINTAINER line, I don't want anything to break.
