Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 22 Sep 2012 04:04:02 +0000 (UTC)
From:      Devin Teske <dteske@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r240798 - in head/usr.sbin/bsdconfig: share startup
Message-ID:  <201209220404.q8M442gF080303@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: dteske
Date: Sat Sep 22 04:04:02 2012
New Revision: 240798
URL: http://svn.freebsd.org/changeset/base/240798

Log:
  Replace "( : ${var?} )" syntax with better "[ ${var+set} ]" syntax.
  
  Reviewed by:	jilles, adrian (co-mentor)
  Approved by:	jilles, adrian (co-mentor)

Modified:
  head/usr.sbin/bsdconfig/share/sysrc.subr
  head/usr.sbin/bsdconfig/startup/rcadd
  head/usr.sbin/bsdconfig/startup/rcconf
  head/usr.sbin/bsdconfig/startup/rcdelete
  head/usr.sbin/bsdconfig/startup/rcvar

Modified: head/usr.sbin/bsdconfig/share/sysrc.subr
==============================================================================
--- head/usr.sbin/bsdconfig/share/sysrc.subr	Sat Sep 22 03:11:35 2012	(r240797)
+++ head/usr.sbin/bsdconfig/share/sysrc.subr	Sat Sep 22 04:04:02 2012	(r240798)
@@ -132,7 +132,7 @@ f_sysrc_get()
 		# Clear the environment of all variables, preventing the
 		# expansion of normals such as `PS1', `TERM', etc.
 		#
-		f_clean_env --except RC_CONFS RC_DEFAULTS SUCCESS
+		f_clean_env --except RC_CONFS RC_DEFAULTS
 
 		. "$RC_DEFAULTS" > /dev/null 2>&1
 
@@ -153,15 +153,11 @@ f_sysrc_get()
 		# If RC_CONFS is defined, set $rc_conf_files to an explicit
 		# value, modifying the default behavior of source_rc_confs().
 		#
-		( : ${RC_CONFS?} ) > /dev/null 2>&1
-		if [ $? -eq ${SUCCESS:-0} ]; then
+		if [ "${RC_CONFS+set}" ]; then
 			rc_conf_files="$RC_CONFS"
 			_rc_confs_set=1
 		fi
 
-		unset SUCCESS
-			# no longer needed
-
 		source_rc_confs > /dev/null 2>&1
 
 		#

Modified: head/usr.sbin/bsdconfig/startup/rcadd
==============================================================================
--- head/usr.sbin/bsdconfig/startup/rcadd	Sat Sep 22 03:11:35 2012	(r240797)
+++ head/usr.sbin/bsdconfig/startup/rcadd	Sat Sep 22 04:04:02 2012	(r240798)
@@ -46,7 +46,7 @@ ipgm=$( f_index_menu_selection $BSDCFG_L
 # Options
 #
 # Inherit SHOW_DESC value if set, otherwise default to 1
-( : ${SHOW_DESC?} ) > /dev/null 2>&1 || SHOW_DESC=1
+[ "${SHOW_DESC+set}" ] || SHOW_DESC=1
 
 ############################################################ FUNCTIONS
 

Modified: head/usr.sbin/bsdconfig/startup/rcconf
==============================================================================
--- head/usr.sbin/bsdconfig/startup/rcconf	Sat Sep 22 03:11:35 2012	(r240797)
+++ head/usr.sbin/bsdconfig/startup/rcconf	Sat Sep 22 04:04:02 2012	(r240798)
@@ -54,7 +54,7 @@ RCCONF_MENU_LIST=
 # Options
 #
 # Inherit SHOW_DESC value if set, otherwise default to 1
-( : ${SHOW_DESC?} ) > /dev/null 2>&1 || SHOW_DESC=1
+[ "${SHOW_DESC+set}" ] || SHOW_DESC=1
 # Selectively inherit SHOW_* value (in order of preference)
 if [ "$SHOW_DEFAULT_VALUE" ]; then
 	SHOW_DEFAULT_VALUE=1

Modified: head/usr.sbin/bsdconfig/startup/rcdelete
==============================================================================
--- head/usr.sbin/bsdconfig/startup/rcdelete	Sat Sep 22 03:11:35 2012	(r240797)
+++ head/usr.sbin/bsdconfig/startup/rcdelete	Sat Sep 22 04:04:02 2012	(r240798)
@@ -54,7 +54,7 @@ RCCONF_MENU_LIST=
 # Options
 #
 # Inherit SHOW_DESC value if set, otherwise default to 1
-( : ${SHOW_DESC?} ) > /dev/null 2>&1 || SHOW_DESC=1
+[ "${SHOW_DESC+set}" ] || SHOW_DESC=1
 # Selectively inherit SHOW_* value (in order of preference)
 if [ "$SHOW_DEFAULT_VALUE" ]; then
 	SHOW_DEFAULT_VALUE=1
@@ -348,7 +348,7 @@ while :; do
 		var="${mtag# }"
 
 		# Toggle the state-variable and loop back to menu
-		if ( eval : \${_${var}_delete?} ) > /dev/null 2>&1; then
+		if eval [ \"\${_${var}_delete+set}\" ]; then
 			unset _${var}_delete
 		else
 			setvar _${var}_delete 1

Modified: head/usr.sbin/bsdconfig/startup/rcvar
==============================================================================
--- head/usr.sbin/bsdconfig/startup/rcvar	Sat Sep 22 03:11:35 2012	(r240797)
+++ head/usr.sbin/bsdconfig/startup/rcvar	Sat Sep 22 04:04:02 2012	(r240798)
@@ -54,7 +54,7 @@ RCVAR_MENU_LIST=
 # Options
 #
 # Inherit SHOW_DESC value if set, otherwise default to 1
-( ${SHOW_DESC?} ) > /dev/null 2>&1 || SHOW_DESC=1
+[ "${SHOW_DESC+set}" ] || SHOW_DESC=1
 
 ############################################################ FUNCTIONS
 



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201209220404.q8M442gF080303>