[pacman-dev] [PATCH] parse_options: adjust error messages

[Allan McRae]

Provide consistent error messages for unknown long and short options.
Also get full string translation for the messages.

Signed-off-by: Allan McRae <allan at archlinux.org>
---
 scripts/library/parse_options.sh |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/scripts/library/parse_options.sh b/scripts/library/parse_options.sh
index 32aa0be..8057c28 100644
--- a/scripts/library/parse_options.sh
+++ b/scripts/library/parse_options.sh
@@ -25,7 +25,7 @@ parse_options() {
 							shift
 							printf " '%s'" "$1"
 						else
-							echo "@SCRIPTNAME@: option '$1' $(gettext "requires an argument")" >&2
+							printf "@SCRIPTNAME@: $(gettext "option %s requires an argument")" "'$1'" >&2
 							ret=1
 						fi
 					fi
@@ -50,7 +50,7 @@ parse_options() {
 								shift
 								printf " '%s'" "${1}"
 							else
-								echo "@SCRIPTNAME@: option $(gettext "requires an argument") -- '${1:i:1}'" >&2
+								printf "@SCRIPTNAME@: $(gettext "option %s requires an argument")" "'-${1:i:1}'" >&2
 								ret=1
 							fi
 						fi
@@ -59,7 +59,7 @@ parse_options() {
 						printf ' -%s' "${1:i:1}"
 					fi
 				else
-					echo "@SCRIPTNAME@: $(gettext "invalid option") -- '${1:i:1}'" >&2
+					echo "@SCRIPTNAME@: $(gettext "unrecognized option") '-${1:i:1}'" >&2
 					ret=1
 				fi
 			done
-- 
1.7.5.4