Re: [PATCH net-next, v3] selftest: Add selftest for multicast address notifications

[Jakub Kicinski]

On Thu, 12 Jun 2025 11:05:14 +0900 Yuyang Huang wrote:
> +VERBOSE=0
> +PAUSE=no
> +PAUSE_ON_FAIL=no
> +
> +source lib.sh
> +
> +# set global exit status, but never reset nonzero one.
> +check_err()
> +{
> +	if [ $ret -eq 0 ]; then
> +		ret=$1
> +	fi
> +	[ -n "$2" ] && echo "$2"
> +}
> +
> +run_cmd_common()
> +{
> +	local cmd="$*"
> +	local out
> +	if [ "$VERBOSE" = "1" ]; then
> +		echo "COMMAND: ${cmd}"
> +	fi
> +	out=$($cmd 2>&1)
> +	rc=$?
> +	if [ "$VERBOSE" = "1" -a -n "$out" ]; then
> +		echo "    $out"
> +	fi
> +	return $rc
> +}
> +
> +run_cmd() {
> +	run_cmd_common "$@"
> +	rc=$?
> +	check_err $rc
> +	return $rc
> +}
> +
> +end_test()
> +{
> +	echo "$*"
> +	[ "${VERBOSE}" = "1" ] && echo
> +
> +	if [[ $ret -ne 0 ]] && [[ "${PAUSE_ON_FAIL}" = "yes" ]]; then
> +		echo "Hit enter to continue"
> +		read a
> +	fi;
> +
> +	if [ "${PAUSE}" = "yes" ]; then
> +		echo "Hit enter to continue"
> +		read a
> +	fi
> +
> +}

Perhaps move these to lib.sh as a separate commit?
They seem generic.

Please fix the shellcheck warnings. The "info"-level messages 
are up you, SC2317 can be ignored.
-- 
pw-bot: cr