From 4300f65dc045c1d88a85c079ea14c931c82cd535 Mon Sep 17 00:00:00 2001 From: Caio Oliveira Date: Sun, 28 Sep 2025 19:52:28 -0300 Subject: [PATCH] [ci] license-header.sh: Fix some shellcheck warnings Signed-off-by: Caio Oliveira --- .ci/license-header.sh | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/.ci/license-header.sh b/.ci/license-header.sh index 870555d533..ee775db01e 100755 --- a/.ci/license-header.sh +++ b/.ci/license-header.sh @@ -140,10 +140,10 @@ cat << EOF other files have been reviewed and addressed. EOF -TMP_DIR=$(mktemp -d /tmp/license-header.XXXXXX) || exit 1 +TMP_DIR=$(mktemp -d "/tmp/license-header.XXXXXX") || exit 1 if [ "$FIX" = "true" ] || [ "$UPDATE" = "true" ]; then echo - echo "license-header.sh: FIX set to true, fixing headers..." + echo "license-header.sh: FIX or UPDATE set to true, fixing headers..." for file in $SRC_FILES $OTHER_FILES; do BASENAME=$(basename "$file") @@ -163,7 +163,7 @@ if [ "$FIX" = "true" ] || [ "$UPDATE" = "true" ]; then TMP="$TMP_DIR/$BASENAME.tmp" UPDATED=0 cp -p "$file" "$TMP" - > "$TMP" + : > "$TMP" # this logic is bit hacky but sed don't work well with $VARIABLES # it's this or complete remove this logic and keep only the old way