In the rc.d scripts, change assignments to rcvar to use the

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.

1 files changed, 4 insertions, 9 deletions

diff --git a/news/inn/files/innd.sh.in b/news/inn/files/innd.sh.in
index ef7b056ba982..a86073304c93 100644
--- a/news/inn/files/innd.sh.in
+++ b/news/inn/files/innd.sh.in
@@ -1,17 +1,17 @@
 #!/bin/sh
+
+# $FreeBSD$
 #
 # PROVIDE: innd
 # REQUIRE: DAEMON
 # KEYWORD: shutdown
-#
-# $FreeBSD$
-#
 
 . /etc/rc.subr
 
 name=innd
+rcvar=innd_enable
 
-PREFIX=%%PREFIX%%
+load_rc_config ${name}
 
 innd_enable=${innd_enable-"NO"}
 innd_user=${innd_user-"news"}
@@ -19,13 +19,9 @@ innd_group=${innd_group-"news"}
 
 required_files="%%DBDIR%%/history.dir"
 
-load_rc_config ${name}
-
 start_cmd=${name}_start
 stop_cmd=${name}_stop
 
-rcvar=`set_rcvar`
-
 command=%%NEWSBASE%%/bin/rc.news
 
 innd_start() {
@@ -36,5 +32,4 @@ innd_stop() {
 	su -fm ${innd_user} -c "${command} stop"
 }
 
-load_rc_config ${name}
 run_rc_command "$1"