svn commit: r271262 - in head/etc: defaults rc.d

[Dag-Erling Smørgrav]

Author: des
Date: Mon Sep  8 12:26:52 2014
New Revision: 271262
URL: http://svnweb.freebsd.org/changeset/base/271262

Log:
  Revert r271257 after several issues were pointed out.  An updated patch
  will be committed at a later date.

Modified:
  head/etc/defaults/rc.conf
  head/etc/rc.d/local_unbound

Modified: head/etc/defaults/rc.conf
==============================================================================
--- head/etc/defaults/rc.conf	Mon Sep  8 12:11:49 2014	(r271261)
+++ head/etc/defaults/rc.conf	Mon Sep  8 12:26:52 2014	(r271262)
@@ -267,13 +267,6 @@ hastd_program="/sbin/hastd"	# path to ha
 hastd_flags=""			# Optional flags to hastd.
 ctld_enable="NO"		# CAM Target Layer / iSCSI target daemon.
 local_unbound_enable="NO"	# local caching resolver
-local_unbound_program="/usr/sbin/unbound"
-local_unbound_workdir=/var/unbound
-local_unbound_config="${local_unbound_workdir}/unbound.conf"
-local_unbound_flags="-c${local_unbound_config}"
-local_unbound_forwardconf="${local_unbound_workdir}/forward.conf}"
-local_unbound_anchor="${local_unbound_workdir}/root.key"
-local_unbound_forwarders=""
 
 #
 # kerberos. Do not run the admin daemons on slave servers

Modified: head/etc/rc.d/local_unbound
==============================================================================
--- head/etc/rc.d/local_unbound	Mon Sep  8 12:11:49 2014	(r271261)
+++ head/etc/rc.d/local_unbound	Mon Sep  8 12:26:52 2014	(r271262)
@@ -13,6 +13,7 @@ name="local_unbound"
 desc="local caching forwarding resolver"
 rcvar="local_unbound_enable"
 
+command="/usr/sbin/unbound"
 extra_commands="anchor configtest reload setup"
 start_precmd="local_unbound_prestart"
 reload_precmd="local_unbound_configtest"
@@ -21,9 +22,18 @@ configtest_cmd="local_unbound_configtest
 setup_cmd="local_unbound_setup"
 pidfile="/var/run/${name}.pid"
 
+: ${local_unbound_workdir:=/var/unbound}
+: ${local_unbound_config:=${local_unbound_workdir}/unbound.conf}
+: ${local_unbound_flags:=-c${local_unbound_config}}
+: ${local_unbound_forwardconf:=${local_unbound_workdir}/forward.conf}
+: ${local_unbound_anchor:=${local_unbound_workdir}/root.key}
+: ${local_unbound_forwarders:=}
+
+load_rc_config $name
+
 do_as_unbound()
 {
-	echo "$@" | /usr/bin/su -m unbound
+	echo "$@" | su -m unbound
 }
 
 #
@@ -31,8 +41,7 @@ do_as_unbound()
 #
 local_unbound_anchor()
 {
-	do_as_unbound ${local_unbound_program%/*}/unbound-anchor \
-	    -a ${local_unbound_anchor}
+	do_as_unbound /usr/sbin/unbound-anchor -a ${local_unbound_anchor}
 	# we can't trust the exit code - check if the file exists
 	[ -f ${local_unbound_anchor} ]
 }
@@ -42,8 +51,7 @@ local_unbound_anchor()
 #
 local_unbound_configtest()
 {
-	do_as_unbound ${local_unbound_program%/*}/unbound-checkconf \
-	    ${local_unbound_config}
+	do_as_unbound /usr/sbin/unbound-checkconf ${local_unbound_config}
 }
 
 #
@@ -53,7 +61,7 @@ local_unbound_configtest()
 local_unbound_setup()
 {
 	echo "Performing initial setup."
-	${local_unbound_program%/*}/local-unbound-setup -n \
+	/usr/sbin/local-unbound-setup -n \
 	    -u unbound \
 	    -w ${local_unbound_workdir} \
 	    -c ${local_unbound_config} \