path: root/extras/hook-scripts/stop/pre/
diff options
authorAnoop C S <>2017-10-19 13:40:35 +0530
committerJeff Darcy <>2017-10-30 15:49:02 +0000
commit39ac511a5d6dbc141bb3e1d8b4b4b8922c71b5a4 (patch)
tree3ee8da099f6c511d8abeccebbf7546b2f5c8a787 /extras/hook-scripts/stop/pre/
parent9aa574a51b84717c1f3949ed2e28a49e49840a93 (diff)
extras/hooks: Fix getopt usage
`getopt` does not have an optional argument as '-name'. It should be either '-n' or '--name'(see man getopt(1)). This wrong usage resulted in setting the script name as 'ame' instead of $PROGNAME in most of the hook-scripts. Additionally the following line from DESCRIPTION given for `getopt` shell command expects short options for almost every kind of usage mentioned in SYNOPSIS: . . . If no '-o' or '--options' option is found in the first part, the first parameter of the second part is used as the short options string. . . . Refer for more clarity on its usage. Change-Id: I95baf5fa8c99025e66b2d83656dd838d4f6048ce BUG: 1503983 Signed-off-by: Anoop C S <>
Diffstat (limited to 'extras/hook-scripts/stop/pre/')
1 files changed, 1 insertions, 1 deletions
diff --git a/extras/hook-scripts/stop/pre/ b/extras/hook-scripts/stop/pre/
index e9116c8cfdb..1435de626b8 100755
--- a/extras/hook-scripts/stop/pre/
+++ b/extras/hook-scripts/stop/pre/
@@ -12,7 +12,7 @@ VOL=
function parse_args () {
- ARGS=$(getopt -l $OPTSPEC -name $PROGNAME "$@")
+ ARGS=$(getopt -o '' -l $OPTSPEC -n $PROGNAME -- "$@")
eval set -- "$ARGS"
while true; do