svn commit: r240843 - head/usr.sbin/bsdconfig/timezone/share

[Devin Teske]

Author: dteske
Date: Sat Sep 22 22:16:07 2012
New Revision: 240843
URL: http://svn.freebsd.org/changeset/base/240843

Log:
  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)

Modified:
  head/usr.sbin/bsdconfig/timezone/share/zones.subr

Modified: head/usr.sbin/bsdconfig/timezone/share/zones.subr
==============================================================================
--- head/usr.sbin/bsdconfig/timezone/share/zones.subr	Sat Sep 22 22:04:17 2012	(r240842)
+++ head/usr.sbin/bsdconfig/timezone/share/zones.subr	Sat Sep 22 22:16:07 2012	(r240843)
@@ -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