diff options
author | dteske <dteske@FreeBSD.org> | 2013-11-21 19:43:45 +0000 |
---|---|---|
committer | dteske <dteske@FreeBSD.org> | 2013-11-21 19:43:45 +0000 |
commit | a3300300c5ffe60632e7edb17ed68a0501b6c9b2 (patch) | |
tree | 5f4a1cef4a35784fa98a319a0150e5f2180fcd2a /usr.sbin/bsdconfig/includes | |
parent | f360fec59cc4c33b765f0fcb88b6be28fa0df079 (diff) | |
download | FreeBSD-src-a3300300c5ffe60632e7edb17ed68a0501b6c9b2.zip FreeBSD-src-a3300300c5ffe60632e7edb17ed68a0501b6c9b2.tar.gz |
f_die() (see `bsdconfig includes -dF die') uses a dialog box (and has been
documented as such; I just forgot). These utilities are command-line only
and as such should stick to either using f_die without arguments or printf)
Diffstat (limited to 'usr.sbin/bsdconfig/includes')
-rwxr-xr-x | usr.sbin/bsdconfig/includes/includes | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/usr.sbin/bsdconfig/includes/includes b/usr.sbin/bsdconfig/includes/includes index 338a4f5..28802a0 100755 --- a/usr.sbin/bsdconfig/includes/includes +++ b/usr.sbin/bsdconfig/includes/includes @@ -137,7 +137,7 @@ done shift $(( $OPTIND - 1 )) # cd(1) to `share' dir so relative paths work for find and positional args -cd $BSDCFG_SHARE || f_die 1 "$msg_directory_not_found" "$BSDCFG_SHARE" +cd $BSDCFG_SHARE || f_die # Pedantic # # If given an argument, operate on it specifically (implied `-f') and exit @@ -147,9 +147,11 @@ for include in "$@"; do # See if they've just omitted the `*.subr' suffix [ -f "$include.subr" -a ! -f "$include" ] && include="$include.subr" if [ ! -f "$include" ]; then - f_die 1 "$msg_no_such_file_or_directory" "$0" "$include" + printf "$msg_no_such_file_or_directory" "$0" "$include" + exit $FAILURE elif [ ! -r "$include" ]; then - f_die 1 "$msg_permission_denied" "$0" "$include" + printf "$msg_permission_denied" "$0" "$include" + exit $FAILURE fi show_include "$include" || f_die done |