tests: apply suggestions from shellcheck

Re-cherry-pick for the files that were not in the last cherry-pick.

(cherry picked from commit be4b84635f)
This commit is contained in:
Jörg Thalheim
2023-02-19 08:55:11 +01:00
committed by John Ericson
parent aeaaa426ab
commit edec2a9ad7
2 changed files with 19 additions and 34 deletions

View File

@@ -1,44 +1,29 @@
#! /bin/sh -e
SCRATCH=scratch/$(basename $0 .sh)
SCRATCH=scratch/$(basename "$0" .sh)
rm -rf ${SCRATCH}
mkdir -p ${SCRATCH}
rm -rf "${SCRATCH}"
mkdir -p "${SCRATCH}"
cp simple-pie ${SCRATCH}/simple-pie
cp simple-pie "${SCRATCH}/simple-pie"
# Save the old OS ABI
OLDABI=`../src/patchelf --print-os-abi ${SCRATCH}/simple-pie`
OLDABI=$(../src/patchelf --print-os-abi "${SCRATCH}/simple-pie")
# Ensure it's not empty
test -n "$OLDABI"
# Change OS ABI and verify it has been changed
{
echo "System V"
echo "HP-UX"
echo "NetBSD"
echo "Linux"
echo "GNU Hurd"
echo "Solaris"
echo "AIX"
echo "IRIX"
echo "FreeBSD"
echo "Tru64"
echo "OpenBSD"
echo "OpenVMS"
} | {
while IFS="\n" read ABI; do
for ABI in "System V" "HP-UX" "NetBSD" "Linux" "GNU Hurd" "Solaris" "AIX" "IRIX" "FreeBSD" "Tru64" "OpenBSD" "OpenVMS"; do
echo "Set OS ABI to '$ABI'..."
../src/patchelf --set-os-abi "$ABI" ${SCRATCH}/simple-pie
../src/patchelf --set-os-abi "$ABI" "${SCRATCH}/simple-pie"
echo "Check is OS ABI is '$ABI'..."
NEWABI=`../src/patchelf --print-os-abi ${SCRATCH}/simple-pie`
NEWABI=$(../src/patchelf --print-os-abi "${SCRATCH}/simple-pie")
test "$NEWABI" = "$ABI"
done
}
# Reset OS ABI to the saved one
../src/patchelf --set-os-abi "$OLDABI" ${SCRATCH}/simple-pie
../src/patchelf --set-os-abi "$OLDABI" "${SCRATCH}/simple-pie"
# Verify we still can run the executable
${SCRATCH}/simple-pie
"${SCRATCH}/simple-pie"

View File

@@ -1,12 +1,12 @@
#! /bin/sh -e
SCRATCH=scratch/$(basename $0 .sh)
SCRATCH=scratch/$(basename "$0" .sh)
rm -rf ${SCRATCH}
mkdir -p ${SCRATCH}
cp $(dirname $(readlink -f $0))/empty-note ${SCRATCH}/
rm -rf "${SCRATCH}"
mkdir -p "${SCRATCH}"
cp "$(dirname "$(readlink -f "$0")")/empty-note" "${SCRATCH}/"
# Running --set-interpreter on this binary should not produce the following
# error:
# patchelf: cannot normalize PT_NOTE segment: non-contiguous SHT_NOTE sections
../src/patchelf --set-interpreter ld-linux-x86-64.so.2 ${SCRATCH}/empty-note
../src/patchelf --set-interpreter ld-linux-x86-64.so.2 "${SCRATCH}/empty-note"