summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authordteske <dteske@FreeBSD.org>2012-09-22 22:16:07 +0000
committerdteske <dteske@FreeBSD.org>2012-09-22 22:16:07 +0000
commitec17dd03e0b43e9e9f9b96fc7e6ed1072b7a9154 (patch)
tree4636d7592cbf13b89129b653993a3c2a9b830601
parentfa9edb685ba4be52ba2b3c6364bdf03cd100c070 (diff)
downloadFreeBSD-src-ec17dd03e0b43e9e9f9b96fc7e6ed1072b7a9154.zip
FreeBSD-src-ec17dd03e0b43e9e9f9b96fc7e6ed1072b7a9154.tar.gz
jilles accurately advises that, in the context of shell redirection, the `:'
builtin is processed specially and thus the `: > file' syntax for example will cause premature termination of the current shell on redirection-error. The `true' builtin on the other-hand is not included in this special processing (for compatibility reasons to satisfy legacy scripts programmed for systems where `true' is not a builtin). Change bare `: > file' syntax into `true > file' syntax to prevent premature shell termination in the event of redirection-error. NOTE: Instances of `: > file' that appear within a sub-shell have been left unmodified as these will not cause premature termination of the main script. Reviewed by: jilles, adrian (co-mentor) Approved by: adrian (co-mentor)
-rw-r--r--usr.sbin/bsdconfig/timezone/share/zones.subr2
1 files changed, 1 insertions, 1 deletions
diff --git a/usr.sbin/bsdconfig/timezone/share/zones.subr b/usr.sbin/bsdconfig/timezone/share/zones.subr
index 874b272..81008a9 100644
--- a/usr.sbin/bsdconfig/timezone/share/zones.subr
+++ b/usr.sbin/bsdconfig/timezone/share/zones.subr
@@ -610,7 +610,7 @@ f_install_zoneinfo()
# Save knowledge for later
if [ "$REALLYDOIT" -a $rv -eq $SUCCESS ]; then
- if : 2> /dev/null > "$_PATH_DB"; then
+ if true 2> /dev/null > "$_PATH_DB"; then
cat <<-EOF > "$_PATH_DB"
$zoneinfo
EOF
OpenPOWER on IntegriCloud