Fix comments, remove sed bkp files, fix bash compatiblity

diff --git a/bin/run_xgettext.sh b/bin/run_xgettext.sh
index 933682da58847be2984ea0fd7a2d2bba48afb158..427325e027985f026cc9df531a74b566f32f53d8 100755 (executable)
--- a/bin/run_xgettext.sh
+++ b/bin/run_xgettext.sh
@@ -36,7 +36,6 @@ then
        MODE='single'
 fi
 
-
 case "$MODE" in
        'addon')
                cd "$FULLPATH/../addon/$ADDONNAME"
@@ -72,24 +71,25 @@ KEYWORDS="-k -kt -ktt:1,2"
 echo "Extract strings to $OUTFILE.."
 rm "$OUTFILE"; touch "$OUTFILE"
 
-# shellcheck disable=SC2086  # FINDOPTS is meant to be splitted
+# shellcheck disable=SC2086  # $FINDOPTS is meant to be split
 find_result=$(find "$FINDSTARTDIR" $FINDOPTS -name "*.php" -type f | LC_ALL=C sort --stable)
 
 total_files=$(wc -l <<< "${find_result}")
 
+count=1
 for file in $find_result
 do
-       ((count++))
        echo -ne "                                            \r"
        echo -ne "Reading file $count/$total_files..."
 
        # On Windows, find still outputs the name of pruned folders
        if [ ! -d "$file" ]
        then
-               # shellcheck disable=SC2086  # KEYWORDS is meant to be splitted
+               # shellcheck disable=SC2086  # $KEYWORDS is meant to be split
                xgettext $KEYWORDS -j -o "$OUTFILE" --from-code=UTF-8 "$file" || exit 1
                sed -i.bkp "s/CHARSET/UTF-8/g" "$OUTFILE"
        fi
+       (( count++ ))
 done
 echo -ne "\n"
 
@@ -121,4 +121,6 @@ then
        msgmerge -U "$OUTFILE" "$UPDATEFILE"
 fi
 
+[ -f "$OUTFILE.bkp" ] && rm "$OUTFILE.bkp" 
+
 echo "Done."