diff options
author | mtm <mtm@FreeBSD.org> | 2008-06-19 06:11:34 +0000 |
---|---|---|
committer | mtm <mtm@FreeBSD.org> | 2008-06-19 06:11:34 +0000 |
commit | 7a52073f5b5a8882a3604c6a2166017d64dddf22 (patch) | |
tree | bb62af31b4182c24c48da917d137b96202f2d3c1 /etc/rc.subr | |
parent | 6acc22fcf7b38044376e58f542a2872e27c7b18d (diff) | |
download | FreeBSD-src-7a52073f5b5a8882a3604c6a2166017d64dddf22.zip FreeBSD-src-7a52073f5b5a8882a3604c6a2166017d64dddf22.tar.gz |
Move the check for enabled knobs further down in run_rc_command() so
that bogus commands cause usage information to be printed instead of
diagnostics about enabling the knob.
Diffstat (limited to 'etc/rc.subr')
-rw-r--r-- | etc/rc.subr | 28 |
1 files changed, 14 insertions, 14 deletions
diff --git a/etc/rc.subr b/etc/rc.subr index f5c616c..341f3c8 100644 --- a/etc/rc.subr +++ b/etc/rc.subr @@ -612,29 +612,29 @@ run_rc_command() fi fi + eval $_pidcmd # determine the pid if necessary + + for _elem in $_keywords; do + if [ "$_elem" != "$rc_arg" ]; then + continue + fi # if ${rcvar} is set, and $1 is not # "rcvar", then run # checkyesno ${rcvar} # and return if that failed # - if [ -n "${rcvar}" -a "$rc_arg" != "rcvar" ]; then - if ! checkyesno ${rcvar}; then - if [ -n "${rc_quiet}" ]; then + if [ -n "${rcvar}" -a "$rc_arg" != "rcvar" ]; then + if ! checkyesno ${rcvar}; then + if [ -n "${rc_quiet}" ]; then + return 0 + fi + echo -n "Cannot '${rc_arg}' $name. Set ${rcvar} to " + echo -n "YES in /etc/rc.conf or use 'one${rc_arg}' " + echo "instead of '${rc_arg}'." return 0 fi - echo -n "Cannot '${rc_arg}' $name. Set ${rcvar} to " - echo -n "YES in /etc/rc.conf or use 'one${rc_arg}' " - echo "instead of '${rc_arg}'." - return 0 fi - fi - - eval $_pidcmd # determine the pid if necessary - for _elem in $_keywords; do - if [ "$_elem" != "$rc_arg" ]; then - continue - fi # if there's a custom ${XXX_cmd}, # run that instead of the default # |