Make --no-echo default, and rename to --echo.

Reported-by: Paul Wise <pabs@debian.org>

1 files changed, 8 insertions, 8 deletions

diff --git a/src/password-store.sh b/src/password-store.sh
index 4a04fbe..8afc67e 100755
--- a/src/password-store.sh
+++ b/src/password-store.sh
@@ -38,9 +38,9 @@ Usage:
     $program [show] [--clip,-c] pass-name
         Show existing password and optionally put it on the clipboard.
         If put on the clipboard, it will be cleared in 45 seconds.
-    $program insert [--no-echo,-n | --multiline,-m] [--force,-f] pass-name
-        Insert new password. Optionally, the console can be enabled to not
-        echo the password back. Or, optionally, it may be multiline. Prompt
+    $program insert [--echo,-e | --multiline,-m] [--force,-f] pass-name
+        Insert new password. Optionally, the console can be enabled echo
+        the password back. Or, optionally, it may be multiline. Prompt
         before overwriting existing password unless forced.
     $program edit pass-name
         Insert a new password or edit an existing password using ${EDITOR:-vi}.
@@ -231,21 +231,21 @@ case "$command" in
 		;;
 	insert)
 		multiline=0
-		noecho=0
+		noecho=1
 		force=0
 
-		opts="$($GETOPT -o mnf -l multiline,no-echo,force -n "$program" -- "$@")"
+		opts="$($GETOPT -o mef -l multiline,echo,force -n "$program" -- "$@")"
 		err=$?
 		eval set -- "$opts"
 		while true; do case $1 in
 			-m|--multiline) multiline=1; shift ;;
-			-n|--no-echo) noecho=1; shift ;;
+			-e|--echo) noecho=0; shift ;;
 			-f|--force) force=1; shift ;;
 			--) shift; break ;;
 		esac done
 
-		if [[ $err -ne 0 || ( $multiline -eq 1 && $noecho -eq 1 ) || $# -ne 1 ]]; then
-			echo "Usage: $program $command [--no-echo,-n | --multiline,-m] [--force,-f] pass-name"
+		if [[ $err -ne 0 || $# -ne 1 ]]; then
+			echo "Usage: $program $command [--echo,-e | --multiline,-m] [--force,-f] pass-name"
 			exit 1
 		fi
 		path="$1"