mountd and reload

[John Baldwin]

On Friday 26 March 2010 3:18:00 pm Xin LI wrote:
> On Fri, Mar 26, 2010 at 11:10 AM, John Baldwin <jhb at freebsd.org> wrote:
> > FYI, it would be nice if someone could fix it so that /etc/rc.d/mountd reload
> > works when nfs_server_enable="YES" is set in /etc/rc.conf but
> > mountd_enable="YES" is not explicitly set.
> 
> How would you think about something like this (hackish, though...  I
> think we should perhaps make a 'rcvars' list):
> 
> Index: mountd
> ===================================================================
> --- mountd	(revision 205651)
> +++ mountd	(working copy)
> @@ -10,7 +10,15 @@
>  . /etc/rc.subr
> 
>  name="mountd"
> -rcvar=`set_rcvar`
> +if ! checkyesno mountd_enable 1>/dev/null 2>&1 ; then
> +	if ! checkyesno nfsd_enable 1>/dev/null 2>&1; then
> +		rcvar=nfs_server_enable
> +	else
> +		rcvar=nfsd_enable
> +	fi
> +else
> +	rcvar=`set_rcvar`
> +fi
>  command="/usr/sbin/${name}"
>  pidfile="/var/run/${name}.pid"
>  required_files="/etc/exports"

Hmm, that might work.  I think you can drop the stdio redirection for
checkyesno as other parts of this script don't do that when invoking
checkyesno.

-- 
John Baldwin