scripts(run-docker): style cleanup and explicit runtime guards

Author: playday3008

scripts/run-docker.sh

@@ -16,15 +16,15 @@ _detect_runtime() {
 	if [ -n "${TERMUX_CONTAINER_RUNTIME:-}" ]; then
 		case "$TERMUX_CONTAINER_RUNTIME" in
 			docker|podman) echo "$TERMUX_CONTAINER_RUNTIME"; return;;
-			*) echo "Error: TERMUX_CONTAINER_RUNTIME must be 'docker' or 'podman'" 1>&2; exit 1;;
+			*) echo "Error: TERMUX_CONTAINER_RUNTIME must be 'docker' or 'podman'" >&2; exit 1;;
 		esac
 	fi
-	if command -v docker >/dev/null 2>&1; then
+	if command -v docker &>/dev/null; then
 		echo "docker"
-	elif command -v podman >/dev/null 2>&1; then
+	elif command -v podman &>/dev/null; then
 		echo "podman"
 	else
-		echo "Error: Neither docker nor podman found in PATH" 1>&2
+		echo "Error: Neither docker nor podman found in PATH" >&2
 		exit 1
 	fi
 }
@@ -34,12 +34,12 @@ _detect_runtime() {
 _detect_sudo() {
 	if [ "$(id -u)" = "0" ]; then
 		echo ""
-	elif command -v sudo >/dev/null 2>&1; then
+	elif command -v sudo &>/dev/null; then
 		echo "sudo"
-	elif command -v doas >/dev/null 2>&1; then
+	elif command -v doas &>/dev/null; then
 		echo "doas"
 	else
-		echo "Error: This script must be run as root or with sudo/doas available in PATH" 1>&2
+		echo "Error: This script must be run as root or with sudo/doas available in PATH" >&2
 		exit 1
 	fi
 }
@@ -100,7 +100,7 @@ while (( $# != 0 )); do
 			TERMUX_DOCKER_RUN_EXTRA_ARGS="--volume /data:/data --volume $HOME/.termux-build:$CONTAINER_HOME_DIR/.termux-build $TERMUX_DOCKER_RUN_EXTRA_ARGS"
 			shift 1;;
 		--) shift 1; break;;
-		-*) echo "Error: Unknown option '$1'" 1>&2; shift 1; exit 1;;
+		-*) echo "Error: Unknown option '$1'" >&2; shift 1; exit 1;;
 		*) break;;
 	esac
 done
@@ -117,7 +117,7 @@ if [ "${dry_run}" = "true" ]; then
 			RETURN_VALUE=0
 			OUTPUT="$("$TERMUX_SCRIPTDIR/scripts/bin/build-package-dry-run-simulation.sh" "$@" 2>&1)" || RETURN_VALUE=$?
 			if [ $RETURN_VALUE -ne 0 ]; then
-				echo "$OUTPUT" 1>&2
+				echo "$OUTPUT" >&2
 				if [ $RETURN_VALUE -eq 85 ]; then # EX_C__NOOP
 					echo "$0: Exiting since '$BUILDSCRIPT_NAME' would not have built any packages"
 					exit 0
@@ -148,7 +148,7 @@ elif [ "$RUNTIME" = "docker" ]; then
 		SEC_OPT=" --security-opt seccomp=$REPOROOT/scripts/profile.json --security-opt apparmor=_custom-termux-package-builder-$CONTAINER_NAME --cap-add CAP_SYS_ADMIN --device /dev/fuse"
 	fi
 else
-	echo "Error: Unsupported runtime '$RUNTIME'" 1>&2
+	echo "Error: Unsupported runtime '$RUNTIME'" >&2
 	exit 1
 fi
 
@@ -214,7 +214,7 @@ if [ "$RUNTIME" = "docker" ]; then
 	load_apparmor_profile ./scripts/profile-relaxed.apparmor
 fi
 
-$SUDO $RUNTIME start $CONTAINER_NAME >/dev/null 2>&1 || {
+$SUDO $RUNTIME start $CONTAINER_NAME &>/dev/null || {
 	echo "Creating new container..."
 	if [ "$RUNTIME" = "podman" ]; then
 		# In rootless Podman the default user-namespace mapping is:
@@ -240,7 +240,7 @@ $SUDO $RUNTIME start $CONTAINER_NAME >/dev/null 2>&1 || {
 			--tty \
 			$TERMUX_DOCKER_RUN_EXTRA_ARGS \
 			$TERMUX_BUILDER_IMAGE_NAME
-	else
+	elif [ "$RUNTIME" = "docker" ]; then
 		$SUDO $RUNTIME run \
 			--detach \
 			--init \
@@ -259,6 +259,9 @@ $SUDO $RUNTIME start $CONTAINER_NAME >/dev/null 2>&1 || {
 				$SUDO $RUNTIME exec $DOCKER_TTY $TERMUX_DOCKER_EXEC_EXTRA_ARGS $CONTAINER_NAME sudo groupmod -g $(id -g) builder
 			fi
 		fi
+	else
+		echo "Error: Unsupported runtime '$RUNTIME'" >&2
+		exit 1
 	fi
 }
 
@@ -284,11 +287,14 @@ if [ "$RUNTIME" = "podman" ]; then
 		$TERMUX_DOCKER_EXEC_EXTRA_ARGS \
 		$CONTAINER_NAME \
 		"$@"
-else
+elif [ "$RUNTIME" = "docker" ]; then
 	$SUDO $RUNTIME exec $CI_OPT \
 		--env "DOCKER_EXEC_PID_FILE_PATH=$DOCKER_EXEC_PID_FILE_PATH" \
 		--interactive $DOCKER_TTY \
 		$TERMUX_DOCKER_EXEC_EXTRA_ARGS \
 		$CONTAINER_NAME \
 		"$@"
+else
+	echo "Error: Unsupported runtime '$RUNTIME'" >&2
+	exit 1
 fi