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/net/socat/files/socat.in b/net/socat/files/socat.in
index b3cd35c02e10..55694ca0bf06 100644
--- a/net/socat/files/socat.in
+++ b/net/socat/files/socat.in
@@ -9,6 +9,10 @@
 # REQUIRE: LOGIN
 # KEYWORD: shutdown
 
+# Add the following lines to /etc/rc.conf to enable socat:
+# socat_enable="YES"
+# socat_flags="<set as needed>"
+
 . %%RC_SUBR%%
 
 name="socat"
@@ -20,6 +24,5 @@ command_args="&"
 load_rc_config $name
 
 : ${socat_enable="NO"}
-: ${socat_flags=""}
 
 run_rc_command "$1"