[libgpiod,v2,6/8] tools: tests: check exit code directly

Message ID	20240603115628.102616-7-warthog618@gmail.com
State	New
Headers	show Received: from mail-pg1-f180.google.com (mail-pg1-f180.google.com [209.85.215.180]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 72ADE86ADC for <linux-gpio@vger.kernel.org>; Mon, 3 Jun 2024 11:57:35 +0000 (UTC) From: Kent Gibson <warthog618@gmail.com> To: linux-gpio@vger.kernel.org, brgl@bgdev.pl Cc: Kent Gibson <warthog618@gmail.com> Subject: [libgpiod][PATCH v2 6/8] tools: tests: check exit code directly Date: Mon, 3 Jun 2024 19:56:26 +0800 Message-Id: <20240603115628.102616-7-warthog618@gmail.com> In-Reply-To: <20240603115628.102616-1-warthog618@gmail.com> References: <20240603115628.102616-1-warthog618@gmail.com> Precedence: bulk MIME-Version: 1.0 Content-Transfer-Encoding: 8bit
Series	tools: tests: fix shellcheck warnings \| expand [libgpiod,v2,0/8] tools: tests: fix shellcheck warnings [libgpiod,v2,1/8] tools: tests: don't mix string and array [libgpiod,v2,2/8] tools: tests: declare and assign separately [libgpiod,v2,3/8] tools: tests: fix unused variables [libgpiod,v2,4/8] tools: tests: use read -r to avoid mangling backslashes [libgpiod,v2,5/8] tools: tests: don't use variables in printf format string [libgpiod,v2,6/8] tools: tests: check exit code directly [libgpiod,v2,7/8] tools: tests: shellcheck don't follow sourced file [libgpiod,v2,8/8] tools: tests: avoid splitting and globbing

Message ID

20240603115628.102616-7-warthog618@gmail.com

State

New

Headers

From: Kent Gibson <warthog618@gmail.com>
To: linux-gpio@vger.kernel.org,
	brgl@bgdev.pl
Cc: Kent Gibson <warthog618@gmail.com>
Subject: [libgpiod][PATCH v2 6/8] tools: tests: check exit code directly
Date: Mon,  3 Jun 2024 19:56:26 +0800
Message-Id: <20240603115628.102616-7-warthog618@gmail.com>
In-Reply-To: <20240603115628.102616-1-warthog618@gmail.com>
References: <20240603115628.102616-1-warthog618@gmail.com>
Precedence: bulk
MIME-Version: 1.0
Content-Transfer-Encoding: 8bit

Series

tools: tests: fix shellcheck warnings | expand

Commit Message

Kent Gibson June 3, 2024, 11:56 a.m. UTC

Fix shellcheck SC2181[1] - check exit code directly.

[1] https://www.shellcheck.net/wiki/SC2181

Signed-off-by: Kent Gibson <warthog618@gmail.com>
---
 tools/gpio-tools-test.bash | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/tools/gpio-tools-test.bash b/tools/gpio-tools-test.bash
index d9930f1..83b05ec 100755
--- a/tools/gpio-tools-test.bash
+++ b/tools/gpio-tools-test.bash
@@ -3069,8 +3069,7 @@  check_kernel() {
 check_prog() {
 	local PROG=$1
 
-	which "$PROG" > /dev/null
-	if [ "$?" -ne "0" ]
+	if ! which "$PROG" > /dev/null
 	then
 		die "$PROG not found - needed to run the tests"
 	fi

[libgpiod,v2,6/8] tools: tests: check exit code directly

Commit Message

Patch