On Tue, 8 Jan 2002 11:02:06 +0100
Thomas Quinot <[EMAIL PROTECTED]> wrote:
[snip]
> + # Handle absent nfs client support
> + if sysctl vfs.nfs >/dev/null 2>&1; then
> + nfsclient_in_kernel=1
> + else
> + nfsclient_in_kernel=0
> + fi
> +
This should be handled inside the case statement for ${nfs_client_enable}, as you want
to load the nfsclient kld only if the nfs client is enabled.
> case ${nfs_client_enable} in
> [Yy][Ee][Ss])
> + kldload nfsclient && nfsclient_in_kernel=1
> +
> + case $nfsclient_in_kernel in
> + 1)
> + ;;
> + *)
> + echo 'Warning: NFS client kernel module failed to load'
> + nfs_client_enable=NO
> + ;;
> + esac
> + ;;
> + esac
> +
> + case "${nfs_client_enable}" in
> + [Yy][Ee][Ss])
> if [ -n "${nfs_access_cache}" ]; then
> echo -n " NFS access cache time=${nfs_access_cache}"
> sysctl
>vfs.nfs.access_cache_timeout=${nfs_access_cache} >/dev/null
> @@ -732,6 +754,27 @@
> echo -n ' rpc.lockd'; rpc.lockd
> ;;
> esac
You should clean this up so there is only *one* 'case "${nfs_client_enable}"'.
Look at the implementation of case ${nfs_server_enable}, earlier in rc.network for an
example of how it should be done.
Cheers,
mike makonnen
To Unsubscribe: send mail to [EMAIL PROTECTED]
with "unsubscribe freebsd-current" in the body of the message
