-n, but it's not handled by this
case.while getopts "vrn" n
do
case "$n" in
v) echo "Verbose" ;;
r) echo "Recursive" ;;
\?) usage;;
esac
donewhile getopts "vrn" n
do
case "$n" in
v) echo "Verbose" ;;
r) echo "Recursive" ;;
n) echo "Dry-run" ;; # -n handled here
\?) usage;;
esac
doneYou have a while getopts loop where the corresponding
case statement fails to handle one of the flags.
Either add a case to handle the flag, or remove it from the
getopts option string.
ShellCheck may not correctly recognize less canonical uses of
while getopts ..; do case ..;, such as when modifying the
variable before using it:
while getopts "rf-:" OPT; do
if [ "$OPT" = "-" ]; then # long option: reformulate OPT and OPTARG
OPT="${OPTARG%%=*}" # extract long option name
OPTARG="${OPTARG#$OPT}" # extract long option argument (may be empty)
OPTARG="${OPTARG#=}" # if long option argument, remove assigning `=`
fi
case "$OPT" in
r) ... ;;
f) ... ;;
my-long-option) ... ;;
esac
doneIn such cases you can do one of:
getopt (no "s") which supports
long options natively.-) branch.ShellCheck is a static analysis tool for shell scripts. This page is part of its documentation.