scripts: run ipkg-make-index through shellcheck
authorPaul Spooren <mail@aparcar.org>
Tue, 9 May 2023 17:06:23 +0000 (19:06 +0200)
committerPaul Spooren <mail@aparcar.org>
Tue, 9 May 2023 17:06:23 +0000 (19:06 +0200)
Use quotes and replace `` by $() to pass the shellcheck tests.

Signed-off-by: Paul Spooren <mail@aparcar.org>
scripts/ipkg-make-index.sh

index 8965d0aa183db47c907beb659d5e00f5d2407182..7b209dca8ec181b2262b13c382f8f3ddddab0958 100755 (executable)
@@ -3,25 +3,25 @@ set -e
 
 pkg_dir=$1
 
-if [ -z $pkg_dir ] || [ ! -d $pkg_dir ]; then
+if [ -z "$pkg_dir" ] || [ ! -d "$pkg_dir" ]; then
        echo "Usage: ipkg-make-index <package_directory>" >&2
        exit 1
 fi
 
 empty=1
 
-for pkg in `find $pkg_dir -name '*.ipk' | sort`; do
+for pkg in $(find "$pkg_dir" -name '*.ipk' | sort); do
        empty=
        name="${pkg##*/}"
        name="${name%%_*}"
        [[ "$name" = "kernel" ]] && continue
        [[ "$name" = "libc" ]] && continue
        echo "Generating index for package $pkg" >&2
-       file_size=$(stat -L -c%s $pkg)
-       sha256sum=$($MKHASH sha256 $pkg)
+       file_size=$(stat -L -c%s "$pkg")
+       sha256sum=$($MKHASH sha256 "$pkg")
        # Take pains to make variable value sed-safe
-       sed_safe_pkg=`echo $pkg | sed -e 's/^\.\///g' -e 's/\\//\\\\\\//g'`
-       tar -xzOf $pkg ./control.tar.gz | tar xzOf - ./control | sed -e "s/^Description:/Filename: $sed_safe_pkg\\
+       sed_safe_pkg=$(echo "$pkg" | sed -e 's/^\.\///g' -e 's/\\//\\\\\\//g')
+       tar -xzOf "$pkg" ./control.tar.gz | tar xzOf - ./control | sed -e "s/^Description:/Filename: $sed_safe_pkg\\
 Size: $file_size\\
 SHA256sum: $sha256sum\\
 Description:/"