github/plexupdate
1
0
Fork 0
mirror of https://github.com/mrworf/plexupdate.git synced 2025-08-20 13:23:21 -07:00
Code Issues Projects Releases Packages Wiki Activity Actions

Fix to allow combined command-line options

Browse source 
Due to a feature/bug in getopt, calling plexupdate.sh with concatenated
options (e.g. `plexupdate.sh -cq`), causes an error even though modern
versions of getopt have support for that format. The culprit seems to be
the "getopt compatibility mode". With these changes, a modern
implementation of getopt will work correctly with more complex options,
while old versions will continue to work as they currently do (requiring
multiple options to be given independently e.g. `plexupdate.sh -c -q`)
This commit is contained in:
Alex Malinovich 2016-07-26 15:18:46 -07:00
commit 96d115de0e
1 changed files with 6 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

7
plexupdate.sh
View file

@ -165,7 +165,12 @@ usage() {
# Parse commandline # Parse commandline
ALLARGS=( "$@" ) ALLARGS=( "$@" )
set -- $(getopt acCdfhkopqruU: -- "$@") optstring="acCdfhkopqruU"
getopt -T >/dev/null
if [ $? -eq 4 ]; then
optstring="-o $optstring"
fi
set -- $(getopt $optstring -- "$@")
while true; while true;
do do
case "$1" in case "$1" in