From 37e396cab4edd01384916e8310fae01ab67b53c3 Mon Sep 17 00:00:00 2001 From: wileyhy <84648683+wileyhy@users.noreply.github.com> Date: Fri, 28 Apr 2023 19:57:20 -0700 Subject: alsa-info.sh: uname - reduce execs by four, and eliminate a logic bug Per `info uname`, `uname` always prints its data in the same order. Also, "note that A && B || C is not if-then-else. C may run when A is true." https://www.shellcheck.net/wiki/SC2015 set -x; read -r KERNEL_VERSION < <(uname -v); smp=x; \ [[ "$KERNEL_VERSION" = *SMP* ]] && \ { smp=y; readonly smp; false;} || smp=n; set - ... https://github.com/koalaman/shellcheck Fixes: https://github.com/alsa-project/alsa-utils/pull/207 From: wileyhy @ github Signed-off-by: Jaroslav Kysela --- alsa-info/alsa-info.sh | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/alsa-info/alsa-info.sh b/alsa-info/alsa-info.sh index 4934d70..1979686 100755 --- a/alsa-info/alsa-info.sh +++ b/alsa-info/alsa-info.sh @@ -415,11 +415,9 @@ fi # Fetch the info and store in temp files/variables TSTAMP=$(LANG=C TZ=UTC date) DISTRO=$(grep -ihs "buntu\|SUSE\|Fedora\|PCLinuxOS\|MEPIS\|Mandriva\|Debian\|Damn\|Sabayon\|Slackware\|KNOPPIX\|Gentoo\|Zenwalk\|Mint\|Kubuntu\|FreeBSD\|Puppy\|Freespire\|Vector\|Dreamlinux\|CentOS\|Arch\|Xandros\|Elive\|SLAX\|Red\|BSD\|KANOTIX\|Nexenta\|Foresight\|GeeXboX\|Frugalware\|64\|SystemRescue\|Novell\|Solaris\|BackTrack\|KateOS\|Pardus\|ALT" /etc/{issue,*release,*version}) -KERNEL_VERSION=$(uname -r) -KERNEL_PROCESSOR=$(uname -p) -KERNEL_MACHINE=$(uname -m) -KERNEL_OS=$(uname -o) -[[ $(uname -v | grep SMP) ]] && KERNEL_SMP="Yes" || KERNEL_SMP="No" +read -r KERNEL_RELEASE KERNEL_MACHINE KERNEL_PROCESSOR KERNEL_OS < <(uname -rpmo) +read -r KERNEL_VERSION < <(uname -v) +if [[ "$KERNEL_VERSION" = *SMP* ]]; then KERNEL_SMP=Yes; else KERNEL_SMP=No; fi ALSA_DRIVER_VERSION=$(cat /proc/asound/version | head -n1 | awk '{ print $7 }' | sed 's/\.$//') get_alsa_library_version ALSA_UTILS_VERSION=$(amixer -v | awk '{ print $3 }') -- cgit v1.2.1