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, 4 insertions, 1 deletions

diff --git a/dns/dns_balance/files/dns_balance.sh.in b/dns/dns_balance/files/dns_balance.sh.in
index 1dbe5dd..e7add69 100644
--- a/dns/dns_balance/files/dns_balance.sh.in
+++ b/dns/dns_balance/files/dns_balance.sh.in
@@ -5,6 +5,10 @@
 # PROVIDE: dns_balance
 # REQUIRE: DAEMON
 
+# Add the following line to /etc/rc.conf to enable dns_balance
+# dns_balance_enable="YES"
+# dns_balance_flags="<set as needed>"
+
 . %%RC_SUBR%%
 
 name=dns_balance
@@ -14,7 +18,6 @@ load_rc_config ${name}
 : ${dns_balance_enable="NO"}
 : ${dns_balance_logfile="/var/log/${name}.log"}
 : ${dns_balance_pidfile="/var/log/${name}.pid"}
-: ${dns_balance_flags=""}
 
 command="%%PREFIX%%/sbin/${name}"
 command_args="-l ${dns_balance_logfile} -p ${dns_balance_pidfile}"