Fix a few "bad example" problems in the rc.d scripts that have been

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.

1 files changed, 1 insertions, 1 deletions

diff --git a/games/pvpgn/files/bnetd.sh.in b/games/pvpgn/files/bnetd.sh.in
index b285620ee6f7..38f5d1e1152a 100644
--- a/games/pvpgn/files/bnetd.sh.in
+++ b/games/pvpgn/files/bnetd.sh.in
@@ -12,6 +12,7 @@
 # Add the following lines to /etc/rc.conf to enable bnetd:
 #
 # bnetd_enable="YES"
+# bnetd_flags="<set as needed>"
 #
 # See bnetd(1) for flags
 #
@@ -35,6 +36,5 @@ stop_postcmd()
 load_rc_config $name
 
 : ${bnetd_enable="NO"}
-: ${bnetd_flags=""}
 
 run_rc_command "$1"