From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from simark.ca by simark.ca with LMTP id MhlIEcBSlWqpChcAWB0awg (envelope-from ) for ; Mon, 31 Aug 2026 06:09:04 -0400 Authentication-Results: simark.ca; dkim=pass (1024-bit key; unprotected) header.d=suse.de header.i=@suse.de header.a=rsa-sha256 header.s=susede2_rsa header.b=vyqgh3dE; dkim=pass header.d=suse.de header.i=@suse.de header.a=ed25519-sha256 header.s=susede2_ed25519 header.b=MZXoErkH; dkim=pass (1024-bit key) header.d=suse.de header.i=@suse.de header.a=rsa-sha256 header.s=susede2_rsa header.b=AFwCIge6; dkim=neutral header.d=suse.de header.i=@suse.de header.a=ed25519-sha256 header.s=susede2_ed25519 header.b=UFwhIaeG; dkim-atps=neutral Received: by simark.ca (Postfix, from userid 112) id 298741E09B; Mon, 31 Aug 2026 06:09:04 -0400 (EDT) X-Spam-Checker-Version: SpamAssassin 4.0.1 (2024-03-25) on simark.ca X-Spam-Level: X-Spam-Status: No, score=-5.4 required=5.0 tests=ARC_SIGNED,ARC_VALID,BAYES_00, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,MAILING_LIST_MULTI, RCVD_IN_DNSWL_MED autolearn=ham autolearn_force=no version=4.0.1 Received: from vm01.sourceware.org (vm01.sourceware.org [IPv6:2620:52:6:3111::32]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange x25519 server-signature ECDSA (prime256v1) server-digest SHA256) (No client certificate requested) by simark.ca (Postfix) with ESMTPS id 0813E1E09B for ; Mon, 31 Aug 2026 06:09:03 -0400 (EDT) Received: from vm01.sourceware.org (localhost [IPv6:::1]) by sourceware.org (Postfix) with ESMTP id 157B44BA2E2F for ; Mon, 31 Aug 2026 10:09:02 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 157B44BA2E2F Authentication-Results: sourceware.org; dkim=pass (1024-bit key, unprotected) header.d=suse.de header.i=@suse.de header.a=rsa-sha256 header.s=susede2_rsa header.b=vyqgh3dE; dkim=pass header.d=suse.de header.i=@suse.de header.a=ed25519-sha256 header.s=susede2_ed25519 header.b=MZXoErkH; dkim=pass (1024-bit key) header.d=suse.de header.i=@suse.de header.a=rsa-sha256 header.s=susede2_rsa header.b=AFwCIge6; dkim=neutral header.d=suse.de header.i=@suse.de header.a=ed25519-sha256 header.s=susede2_ed25519 header.b=UFwhIaeG Received: from smtp-out1.suse.de (smtp-out1.suse.de [195.135.223.130]) by sourceware.org (Postfix) with ESMTPS id ABCDB4BA2E2B for ; Mon, 31 Aug 2026 10:08:36 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org ABCDB4BA2E2B Authentication-Results: sourceware.org; dmarc=pass (p=none dis=none) header.from=suse.de Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=suse.de ARC-Filter: OpenARC Filter v1.0.0 sourceware.org ABCDB4BA2E2B Authentication-Results: sourceware.org; arc=none smtp.remote-ip=195.135.223.130 ARC-Seal: i=1; a=rsa-sha256; d=sourceware.org; s=key; t=1788170916; cv=none; b=mqSWUF7IDV2uQYtQfWmLJxQbFMNWOsR1gFQQy7vCSLHoV4hpUmwe9wVUhl/Jt3uerXmMouxHxfzKp62cTWlTdQef/JEccfCC5CmpSSk3DM2xV4qYxvgkNAHKTCpega3knDcrnw9oGx3Ksld1N1ZWJ5HA0iRfA/s1iWQ893u8BC0= ARC-Message-Signature: i=1; a=rsa-sha256; d=sourceware.org; s=key; t=1788170916; c=relaxed/simple; bh=dr5nMbN9KKM9Fl34QxftjXbpEHSMj1Qo/WaM1Qr7uIU=; h=DKIM-Signature:DKIM-Signature:DKIM-Signature:DKIM-Signature:From: To:Subject:Date:Message-ID:MIME-Version; b=rDqsrFAudL5Cvz6IKoNT1kf39iJnLJPQko8J7FfvC9RhzJXCNGvrz3SASZN9hXi9yuETv297GOldfP8+xKjR00j/z1koAlpyeUSpRF1OGVpu9ECTRVKkNbdyJgollGAurnz680wR+Z3PJekUL5bLKupS4F9+vKujDdooYOFaRG4= ARC-Authentication-Results: i=1; sourceware.org; dkim=pass (1024-bit key, unprotected) header.d=suse.de header.i=@suse.de header.a=rsa-sha256 header.s=susede2_rsa header.b=vyqgh3dE; dkim=pass header.d=suse.de header.i=@suse.de header.a=ed25519-sha256 header.s=susede2_ed25519 header.b=MZXoErkH; dkim=pass (1024-bit key) header.d=suse.de header.i=@suse.de header.a=rsa-sha256 header.s=susede2_rsa header.b=AFwCIge6; dkim=neutral header.d=suse.de header.i=@suse.de header.a=ed25519-sha256 header.s=susede2_ed25519 header.b=UFwhIaeG DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org ABCDB4BA2E2B Received: from imap1.dmz-prg2.suse.org (imap1.dmz-prg2.suse.org [IPv6:2a07:de40:b281:104:10:150:64:97]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by smtp-out1.suse.de (Postfix) with ESMTPS id 8216722524 for ; Mon, 31 Aug 2026 10:08:27 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.de; s=susede2_rsa; t=1788170911; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc: mime-version:mime-version: content-transfer-encoding:content-transfer-encoding; bh=79TJfulgSTRWKpZM+jFV+nvZii4R5cOhu6LkqY3wqVQ=; b=vyqgh3dExiRf4J61gugct9rGnAxfXwqXKNu0BRyaDiE2gTXxAT5TEq5cC3OduLImGf467S DoiyurdOIdpu49VhPRIa2zvPRNkz8vbHpETiCsgTB9sJtLdsbKm0KGdC2Pr+isz5tS19SJ OzT8Z6V5Us1OhXjsr25aJw2pt+loMHw= DKIM-Signature: v=1; a=ed25519-sha256; c=relaxed/relaxed; d=suse.de; s=susede2_ed25519; t=1788170911; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc: mime-version:mime-version: content-transfer-encoding:content-transfer-encoding; bh=79TJfulgSTRWKpZM+jFV+nvZii4R5cOhu6LkqY3wqVQ=; b=MZXoErkHVmfdeX4WBQ1k+u9jzeRIx7jKTmgwTR29Kk/CqbKSkTheXE2DNxwd8oYabegGPh 68w5OMgs9m8hdeCg== Authentication-Results: smtp-out1.suse.de; dkim=pass header.d=suse.de header.s=susede2_rsa header.b=AFwCIge6; dkim=pass header.d=suse.de header.s=susede2_ed25519 header.b=UFwhIaeG DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.de; s=susede2_rsa; t=1788170907; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc: mime-version:mime-version: content-transfer-encoding:content-transfer-encoding; bh=79TJfulgSTRWKpZM+jFV+nvZii4R5cOhu6LkqY3wqVQ=; b=AFwCIge6Ews8jZVWZx2I4SCmKidaP9CGLWZhsEFRIIU/81dHoctSThCAEsPjGJLQA+tq/T rbJrkBo/rPhOShbpm/qU7tatjhAZIEM242HIjlWGwTIHNlmVhSh77eJ5q4g74bm38lUvIj QHKFKsdjT4F7DLAKUDnUTU9A3GFB94M= DKIM-Signature: v=1; a=ed25519-sha256; c=relaxed/relaxed; d=suse.de; s=susede2_ed25519; t=1788170907; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc: mime-version:mime-version: content-transfer-encoding:content-transfer-encoding; bh=79TJfulgSTRWKpZM+jFV+nvZii4R5cOhu6LkqY3wqVQ=; b=UFwhIaeGw7pYdwup6JzCwO2i9v8RN2uJXky4ODvP7lYDVAwIugCvSsU06wWghV7daUmu/k MD10rL0PuE9GXXDw== Received: from imap1.dmz-prg2.suse.org (localhost [127.0.0.1]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by imap1.dmz-prg2.suse.org (Postfix) with ESMTPS id 42B2313515 for ; Mon, 31 Aug 2026 10:08:27 +0000 (UTC) Received: from dovecot-director2.suse.de ([2a07:de40:b281:106:10:150:64:167]) by imap1.dmz-prg2.suse.org with ESMTPSA id +wMBD5tSlWpMeQAAD6G6ig (envelope-from ) for ; Mon, 31 Aug 2026 10:08:27 +0000 From: Tom de Vries To: gdb-patches@sourceware.org Subject: [PATCH] [gdb/config/djgpp] Make djcheck.sh and djconfig.sh shellcheck-clean Date: Mon, 31 Aug 2026 12:08:26 +0200 Message-ID: <20260831100826.3521717-1-tdevries@suse.de> X-Mailer: git-send-email 2.51.0 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Rspamd-Queue-Id: 8216722524 X-Rspamd-Server: rspamd1.dmz-prg2.suse.org X-Rspamd-Action: no action X-Spamd-Result: default: False [-3.01 / 50.00]; BAYES_HAM(-3.00)[100.00%]; NEURAL_HAM_LONG(-1.00)[-1.000]; MID_CONTAINS_FROM(1.00)[]; R_MISSING_CHARSET(0.50)[]; R_DKIM_ALLOW(-0.20)[suse.de:s=susede2_rsa,suse.de:s=susede2_ed25519]; NEURAL_HAM_SHORT(-0.20)[-1.000]; MIME_GOOD(-0.10)[text/plain]; MX_GOOD(-0.01)[]; TO_MATCH_ENVRCPT_ALL(0.00)[]; ARC_NA(0.00)[]; RCVD_VIA_SMTP_AUTH(0.00)[]; RCPT_COUNT_ONE(0.00)[1]; MIME_TRACE(0.00)[0:+]; FROM_HAS_DN(0.00)[]; PREVIOUSLY_DELIVERED(0.00)[gdb-patches@sourceware.org]; FROM_EQ_ENVFROM(0.00)[]; DBL_BLOCKED_OPENRESOLVER(0.00)[imap1.dmz-prg2.suse.org:helo,imap1.dmz-prg2.suse.org:rdns,suse.de:dkim,suse.de:mid]; DKIM_SIGNED(0.00)[suse.de:s=susede2_rsa,suse.de:s=susede2_ed25519]; RCVD_TLS_ALL(0.00)[]; TO_DN_NONE(0.00)[]; RCVD_COUNT_TWO(0.00)[2]; SPAMHAUS_XBL(0.00)[2a07:de40:b281:104:10:150:64:97:from]; DKIM_TRACE(0.00)[suse.de:+] X-BeenThere: gdb-patches@sourceware.org X-Mailman-Version: 2.1.30 Precedence: list List-Id: Gdb-patches mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: gdb-patches-bounces~public-inbox=simark.ca@sourceware.org Make gdb/config/djgpp/{djcheck.sh,djconfig.sh} shellcheck-clean: - add missing quotes - add "|| exit 1" after cd - use printf instead of echo -n - use $() - disable SC2016 for "configure --prefix='${DJDIR}' assuming ${DJDIR} is indeed not meant to be expanded I don't have a djgpp setup, so I can't test this. --- gdb/config/djgpp/djcheck.sh | 20 ++++----- gdb/config/djgpp/djconfig.sh | 81 ++++++++++++++++++------------------ gdb/contrib/shellcheck.sh | 4 +- 3 files changed, 52 insertions(+), 53 deletions(-) diff --git a/gdb/config/djgpp/djcheck.sh b/gdb/config/djgpp/djcheck.sh index fa75d03f776..24224ccc955 100644 --- a/gdb/config/djgpp/djcheck.sh +++ b/gdb/config/djgpp/djcheck.sh @@ -17,31 +17,31 @@ # You should have received a copy of the GNU General Public License # along with this program. If not, see . -ORIGDIR=`pwd` -GDB=${ORIGDIR}/../gdb.exe -SUBDIRS=`find $ORIGDIR -type d ! -ipath $ORIGDIR` +ORIGDIR=$(pwd) +GDB="${ORIGDIR}/../gdb.exe" +SUBDIRS=$(find "$ORIGDIR" -type d ! -ipath "$ORIGDIR") for d in $SUBDIRS do - cd $d + cd "$d" || exit 1 echo "Running tests in $d..." for f in *.out do - test -f $f || break - base=`basename $f .out` + test -f "$f" || break + base=$(basename "$f" .out) if test "${base}" = "dbx" ; then options=-dbx else options= fi - $GDB ${options} < ${base}.in 2>&1 \ + $GDB ${options} < "${base}.in" 2>&1 \ | sed -e '/GNU gdb /s/ [.0-9][.0-9]*//' \ -e '/^Copyright/s/[12][0-9][0-9][0-9]/XYZZY/g' \ -e '/Starting program: /s|[A-z]:/.*/||' \ -e '/main (/s/=0x[0-9a-f][0-9a-f]*/=XYZ/g' \ - > ${base}.tst - if diff --binary -u ${base}.out ${base}.tst ; then - rm -f ${base}.tst + > "${base}.tst" + if diff --binary -u "${base}.out" "${base}.tst" ; then + rm -f "${base}.tst" fi done done diff --git a/gdb/config/djgpp/djconfig.sh b/gdb/config/djgpp/djconfig.sh index b630e6c0c58..789509c92b8 100644 --- a/gdb/config/djgpp/djconfig.sh +++ b/gdb/config/djgpp/djconfig.sh @@ -40,30 +40,30 @@ unset CDPATH # then only forward slashes (/) in the directories. It should be # an absolute path. -if [ x$1 = x ]; then - srcdir=`pwd` +if [ "$1" = "" ]; then + srcdir=$(pwd) else - srcdir=`cd $1 && pwd` + srcdir=$(cd "$1" && pwd) || exit 1 shift fi # Make sure they don't have some file names mangled by untarring. -echo -n "Checking the unpacked distribution..." -if ( ! test -f ${srcdir}/bfd/ChangeLog.0203 || \ - ! test -f ${srcdir}/gdb/ChangeLog.002 || \ - ! test -f ${srcdir}/opcodes/ChangeLog.0203 || \ - ! test -f ${srcdir}/readline/config.h-in ) ; then - if ( ! test -f ${srcdir}/bfd/ChangeLog.0203 ) ; then - notfound=${srcdir}/bfd/ChangeLog.0203 +printf "Checking the unpacked distribution..." +if ! test -f "${srcdir}/bfd/ChangeLog.0203" || \ + ! test -f "${srcdir}/gdb/ChangeLog.002" || \ + ! test -f "${srcdir}/opcodes/ChangeLog.0203" || \ + ! test -f "${srcdir}/readline/config.h-in"; then + if ! test -f "${srcdir}/bfd/ChangeLog.0203"; then + notfound="${srcdir}/bfd/ChangeLog.0203" else - if ( ! test -f ${srcdir}/gdb/ChangeLog.002) ; then - notfound=${srcdir}/gdb/ChangeLog.002 + if ! test -f "${srcdir}/gdb/ChangeLog.002"; then + notfound="${srcdir}/gdb/ChangeLog.002" else - if ( ! test -f ${srcdir}/readline/config.h-in ) ; then - notfound=${srcdir}/readline/config.h-in + if ! test -f "${srcdir}/readline/config.h-in"; then + notfound="${srcdir}/readline/config.h-in" else - if ( ! test -f ${srcdir}/opcodes/ChangeLog.0203 ) ; then - notfound=${srcdir}/opcodes/ChangeLog.0203 + if ! test -f "${srcdir}/opcodes/ChangeLog.0203"; then + notfound="${srcdir}/opcodes/ChangeLog.0203" fi fi fi @@ -85,40 +85,40 @@ else fi # Where is the directory with DJGPP-specific scripts? -DJGPPDIR=${srcdir}/gdb/config/djgpp +DJGPPDIR="${srcdir}/gdb/config/djgpp" echo "Editing configure scripts for DJGPP..." TMPFILE="${TMPDIR-.}/cfg.tmp" # We need to skip the build directory if it is a subdirectory of $srcdir, # otherwise we will have an infinite recursion on our hands... -if test "`pwd`" = "${srcdir}" ; then +if test "$(pwd)" = "${srcdir}" ; then SKIPDIR="" SKIPFILES="" else - SKIPDIR=`pwd | sed -e "s|${srcdir}|.|"` + SKIPDIR=$(pwd | sed -e "s|${srcdir}|.|") SKIPFILES="${SKIPDIR}/*" fi # We use explicit /dev/env/DJDIR/bin/find to avoid catching # an incompatible DOS/Windows version that might be on their PATH. for fix_dir in \ - `cd $srcdir && /dev/env/DJDIR/bin/find . -type d ! -ipath "${SKIPDIR}" ! -ipath "${SKIPFILES}"` + $(cd "$srcdir" && /dev/env/DJDIR/bin/find . -type d ! -ipath "${SKIPDIR}" ! -ipath "${SKIPFILES}") do - if test ! -f ${fix_dir}/configure.orig ; then - if test -f ${srcdir}/${fix_dir}/configure ; then - mkdir -p ${fix_dir} - cp -p ${srcdir}/${fix_dir}/configure ${fix_dir}/configure.orig + if test ! -f "${fix_dir}/configure.orig" ; then + if test -f "${srcdir}/${fix_dir}/configure" ; then + mkdir -p "${fix_dir}" + cp -p "${srcdir}/${fix_dir}/configure" "${fix_dir}/configure.orig" fi fi - if test -f ${fix_dir}/configure.orig ; then - sed -f ${DJGPPDIR}/config.sed ${fix_dir}/configure.orig > $TMPFILE - update $TMPFILE ${fix_dir}/configure - touch ./${fix_dir}/configure -r ${fix_dir}/configure.orig - rm -f $TMPFILE + if test -f "${fix_dir}/configure.orig" ; then + sed -f "${DJGPPDIR}/config.sed" "${fix_dir}/configure.orig" > "$TMPFILE" + update "$TMPFILE" "${fix_dir}/configure" + touch "./${fix_dir}/configure" -r "${fix_dir}/configure.orig" + rm -f "$TMPFILE" fi - if test -f ${fix_dir}/INSTALL ; then - mv ${fix_dir}/INSTALL ${fix_dir}/INSTALL.txt + if test -f "${fix_dir}/INSTALL" ; then + mv "${fix_dir}/INSTALL" "${fix_dir}/INSTALL.txt" fi done @@ -134,7 +134,7 @@ export CONFIG_SHELL=/dev/env/DJDIR/bin/sh.exe # force to have the ltmain.sh script to be in DOS text format, # otherwise the resulting ltconfig script will have mixed # (UNIX/DOS) format and is unusable with Bash ports before v2.03. -utod $srcdir/ltmain.sh +utod "$srcdir/ltmain.sh" # Give the configure script some hints: export LD=ld @@ -159,16 +159,16 @@ export lt_cv_sys_max_cmd_len=12288 # Force depcomp to use _deps rather than .deps as the name of the # subdirectory where the *.Po dependency files are put. File names # with leading dots are invalid on DOS 8+3 filesystems. -export DEPDIR=${DEPDIR:-_deps} +export DEPDIR="${DEPDIR:-_deps}" # The configure script needs to see the `install-sh' script, otherwise # it decides the source installation is broken. But "make install" will # fail on 8+3 filesystems if it finds a file `install-', since there # are numerous "install-foo" targets in Makefile's. So we rename the # offending file after the configure step is done. -if test ! -f ${srcdir}/install-sh ; then - if test -f ${srcdir}/install-.sh ; then - mv ${srcdir}/install-.sh ${srcdir}/install-sh +if test ! -f "${srcdir}/install-sh" ; then + if test -f "${srcdir}/install-.sh" ; then + mv "${srcdir}/install-.sh" "${srcdir}/install-sh" fi fi @@ -176,10 +176,11 @@ fi # support, which is nearly impossible to be supported in the current way, # since it relies on file names which will never work on DOS. echo "Running the configure script..." -$srcdir/configure --srcdir="$srcdir" --prefix='${DJDIR}' \ +# shellcheck disable=SC2016 # $DJDIR is not expanded here. +"$srcdir/configure" --srcdir="$srcdir" --prefix='${DJDIR}' \ --disable-shared --disable-nls --verbose --enable-build-warnings=\ --Wimplicit,-Wcomment,-Wformat,-Wparentheses,-Wpointer-arith,-Wuninitialized $* +-Wimplicit,-Wcomment,-Wformat,-Wparentheses,-Wpointer-arith,-Wuninitialized "$@" -if test -f ${srcdir}/install- ; then - mv ${srcdir}/install- ${srcdir}/install-.sh +if test -f "${srcdir}/install-" ; then + mv "${srcdir}/install-" "${srcdir}/install-.sh" fi diff --git a/gdb/contrib/shellcheck.sh b/gdb/contrib/shellcheck.sh index 2a0425eeaab..2661d48b391 100755 --- a/gdb/contrib/shellcheck.sh +++ b/gdb/contrib/shellcheck.sh @@ -41,9 +41,7 @@ for f in "$@"; do # Skip generated files. continue ;; - gdb/config/djgpp/djcheck.sh \ - | gdb/config/djgpp/djconfig.sh \ - | gdb/contrib/gdb-add-index.sh \ + gdb/contrib/gdb-add-index.sh \ | gdb/gdb_buildall.sh \ | gdb/gdb_mbuild.sh \ | gdb/testsuite/lib/pdtrace.in) base-commit: 4d0228f6d409f0d74effbb19430c4c87d1762f55 -- 2.51.0