Browse Source

[ci] license-header.sh: Fix some shellcheck warnings

Signed-off-by: Caio Oliveira <caiooliveirafarias0@gmail.com>
pull/2579/head
Caio Oliveira 5 months ago
parent
commit
a7f932438b
No known key found for this signature in database GPG Key ID: 362DA3DC1901E080
  1. 6
      .ci/license-header.sh

6
.ci/license-header.sh

@ -140,10 +140,10 @@ cat << EOF
other files have been reviewed and addressed. other files have been reviewed and addressed.
EOF 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 if [ "$FIX" = "true" ] || [ "$UPDATE" = "true" ]; then
echo 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 for file in $SRC_FILES $OTHER_FILES; do
BASENAME=$(basename "$file") BASENAME=$(basename "$file")
@ -163,7 +163,7 @@ if [ "$FIX" = "true" ] || [ "$UPDATE" = "true" ]; then
TMP="$TMP_DIR/$BASENAME.tmp" TMP="$TMP_DIR/$BASENAME.tmp"
UPDATED=0 UPDATED=0
cp -p "$file" "$TMP" cp -p "$file" "$TMP"
> "$TMP"
: > "$TMP"
# this logic is bit hacky but sed don't work well with $VARIABLES # 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 # it's this or complete remove this logic and keep only the old way

Loading…
Cancel
Save