.pct-helpers

@@ -1,8 +1,5 @@
 #!/usr/bin/bash
 #----------------------------------------------------------------------
-#
-#
-#----------------------------------------------------------------------
 
 CT_DIR=${CT_DIR:=/etc/pve/lxc/}
 
@@ -181,10 +178,6 @@ xread(){
 		shift
 		local non_empty=1
 	fi
-	local prefix=
-	if [ $SCRIPTING ] ; then
-		prefix='# '
-	fi
 	# skip...
 	if [[ "${!2}" == "SKIP" ]] \
 			|| [[ "$(eval "echo \$DFL_$2")" == "SKIP" ]] ; then
@@ -192,7 +185,7 @@ xread(){
 		return
 	fi
 	if [ -z ${!2} ] ; then
-		eval 'read -ep "'$prefix''$1'" -i "$DFL_'$2'" '${2}''
+		eval 'read -ep "'$1'" -i "$DFL_'$2'" '${2}''
 		XREAD_VARS+=(${2})
 	fi
 	if [ -z $non_empty ] ; then
@@ -207,16 +200,12 @@ xread(){
 #
 xreadYes(){
 	if [ -z ${2} ] ; then
-		local var=__LOCAL
-		local __LOCAL
+		local var=__X
+		local __X
 	else
 		local var=${2}
 		local mode=
 	fi
-	local prefix=
-	if [ $SCRIPTING ] ; then
-		prefix='# '
-	fi
 	# XXX check DFL_..???
 	if [[ "${!var}" == "SKIP" ]] ; then
 		eval "$var="
@@ -232,7 +221,7 @@ xreadYes(){
 			local no=n
 			local dfl=1
 		fi
-		eval 'read -ep "'$prefix''$1' ('$yes'/'$no') " '${var}''
+		eval 'read -ep "'$1' ('$yes'/'$no') " '${var}''
 		XREAD_VARS+=(${var})
 		# normalize...
 		eval "${var}=${!var,,}"
@@ -246,7 +235,6 @@ xreadYes(){
 		fi
 	fi
 	[ $SCRIPTING ] \
-		&& [[ "$var" != '__LOCAL' ]] \
 		&& echo "$var=${!var}"
 
 	if [ -z ${!var} ] ; then
@@ -261,22 +249,18 @@ xreadpass(){
 	if [[ ${!1} == 'SKIP' ]] ; then
 		return
 	fi
-	local prefix=
-	if [ $SCRIPTING ] ; then
-		prefix='# '
-	fi
 	local PASS1
 	local PASS2
 	for attempt in 1 2 3 ; do
-		read -sep "${prefix}password (Enter to skip): " PASS1
+		read -sep "password (Enter to skip): " PASS1
 		echo
 		if [ -z $PASS1 ] ; then
 			return
 		fi
-		read -sep "${prefix}retype password: " PASS2
+		read -sep "retype password: " PASS2
 		echo
 		if [[ $PASS1 != $PASS2 ]] ; then
-			echo "ERROR: passwords do not match." >&2
+			echo "ERR: passwords do not match."
 			continue
 		fi
 		eval ''$1'='${PASS1}''

host/make.sh

@@ -49,7 +49,7 @@ if xreadYes "# Update system?" UPDATE ; then
 	@ apt upgrade
 fi
 if xreadYes "# Install additional apps?" APPS ; then
-	@ apt install ${SOFTWARE[@]}
+	@ apt install $(SOFTWARE[@])
 fi
 
 # Networking
@@ -67,26 +67,11 @@ if xreadYes "# Create bridges?" BRIDGES ; then
 	xread "Gate ADMIN IP: " GATE_ADMIN_IP
 	readBridgeVars
 
-	# check if new bridges already exist in interfaces...
-	if [ -e /etc/network/interfaces ] \
-			&& grep -q \
-				"vmbr\(${WAN_BRIDGE}\|${LAN_BRIDGE}\|${ADMIN_BRIDGE}\)" \
-				/etc/network/interfaces ; then
-		conflict=
-		for br in WAN_BRIDGE LAN_BRIDGE ADMIN_BRIDGE ; do
-			if grep -q "vmbr${!br}" /etc/network/interfaces ; then
-				conflict="${conflict}, vmbr${!br} (${br})"
-			fi
-		done
-		echo "ERROR: will not overwrite existing bridges: ${conflict:2}" >&2
-		exit 1
-	fi
-
 	@ cp /etc/network/interfaces{,.new}
 
-	BRIDGES="$(\
+	BRIDGES="${\
 		cat bridges.tpl \
-			| expandPCTTemplate WAN_PORT ADMIN_PORT)"
+			| expandPCTTemplate WAN_PORT ADMIN_PORT}"
 
 	# XXX add $BRIDGES to /etc/network/interfaces either before the 
 	#		source command or at the end...