links-chmod: mode parameter is mandatoryv2.0

2 files changed, 26 insertions, 9 deletions

diff --git a/links-chmod b/links-chmod
index 8eb8849..9810cf6 100755
--- a/links-chmod
+++ b/links-chmod
@@ -5,7 +5,7 @@
 # For details, see https://github.com/egor-tensin/config-links.
 # Distributed under the MIT License.
 
-# usage: ./links-chmod [-h|--help] [-d|--database PATH] [-s|--shared-dir DIR] [-m|--mode MODE] [-n|--dry-run]
+# usage: ./links-chmod [-h|--help] [-d|--database PATH] [-s|--shared-dir DIR] [-n|--dry-run] MODE
 
 set -o errexit
 set -o nounset
@@ -32,16 +32,18 @@ script_usage() {
         echo "$script_name: $msg"
     done
 
-    echo "usage: $script_name [-h|--help] [-d|--database PATH] [-s|--shared-dir DIR] [-n|--dry-run]
+    echo "usage: $script_name [-h|--help] [-d|--database PATH] [-s|--shared-dir DIR] [-n|--dry-run] MODE
+  MODE               shared files mode (as in chmod)
   -h,--help          show this message and exit
   -d,--database      set database file path
   -s,--shared-dir    set top-level shared directory path
                      (current working directory by default)
-  -m,--mode          shared files mode (as in chmod)
   -n,--dry-run       don't actually do anything intrusive"
 }
 
 parse_script_options() {
+    local -a args=()
+
     while [ "$#" -gt 0 ]; do
         local key="$1"
         shift
@@ -55,12 +57,19 @@ parse_script_options() {
                 set_dry_run
                 continue
                 ;;
-            -d|--database|-s|--shared-dir|-m|--mode)
+            -d|--database|-s|--shared-dir)
                 ;;
-            *)
+            --)
+                break
+                ;;
+            -*)
                 script_usage "unrecognized parameter: $key" >&2
                 exit 1
                 ;;
+            *)
+                args+=("$key")
+                continue
+                ;;
         esac
 
         if [ "$#" -eq 0 ]; then
@@ -78,17 +87,25 @@ parse_script_options() {
             -s|--shared-dir)
                 update_shared_dir "$value"
                 ;;
-            -m|--mode)
-                mode="$( parse_mode "$value" )"
-                ;;
             *)
                 script_usage "unrecognized parameter: $key" >&2
                 exit 1
                 ;;
         esac
     done
+
+    args+=("$@")
+
+    if [ "${#args[@]}" -ne 1 ]; then
+        script_usage "missing parameter MODE" >&2
+        exit 1
+    fi
+
+    mode="$( parse_mode "${args[0]}" )"
 }
 
+mode=''
+
 main() {
     parse_script_options "$@"
     read_database
diff --git a/test/test.sh b/test/test.sh
index de4dd85..0e02480 100755
--- a/test/test.sh
+++ b/test/test.sh
@@ -359,7 +359,7 @@ $test_dest_dir/foo/2.txt->$test_src_dir/%DEST%/foo/2.txt"
     echo 'Verifying 1.txt (the shared file) permissions...'
 
     verify_mode 0644 "$test_src_dir/%DEST%/1.txt"
-    call_chmod --mode 0600
+    call_chmod 0600
     verify_mode 0600 "$test_src_dir/%DEST%/1.txt"
 }