Skip to content

Commit

Permalink
Disable ShellCheck warnings for old getopt(1)
Browse files Browse the repository at this point in the history
The use of $* is according to getopt's documentation. Using $@ instead
(SC2048) is explicitly discouraged by the manpage.
  • Loading branch information
neverpanic committed Jul 2, 2015
1 parent a1affea commit 514075a
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions google-font-download
Original file line number Diff line number Diff line change
Expand Up @@ -147,6 +147,7 @@ else
EOF

ret=0
# shellcheck disable=SC2048,SC2086
temp=$(getopt f:hl:o: $*) || ret=$?
if [ $ret -ne 0 ]; then
echo >&2
Expand Down

0 comments on commit 514075a

Please sign in to comment.