aboutsummaryrefslogtreecommitdiff
path: root/bin/vm
diff options
context:
space:
mode:
authorEuAndreh <eu@euandre.org>2023-02-17 15:07:30 -0300
committerEuAndreh <eu@euandre.org>2023-02-17 15:07:32 -0300
commit44b1e103b4d3a24aee89587ef302891e9f9c3807 (patch)
treea8e4045218bd1075b687dd5a6f1d023e6c407bff /bin/vm
parentetc/nix/configuration.nix: Add flutter2 and hover packages (diff)
downloaddotfiles-44b1e103b4d3a24aee89587ef302891e9f9c3807.tar.gz
dotfiles-44b1e103b4d3a24aee89587ef302891e9f9c3807.tar.xz
bin/: Use explicit "--" separator for assert-arg usages
In order to prevent legitimate cases to fail as errors, such as: NUM='-10' eval "$(assert-arg "$NUM" '-n NUM')" To prevent assert-arg(1) from treating "-10" as the options "-1 and -0, they need to be put after the "--" separator. This way, all usages of assert-arg(1) that had its first argument as a variable were adapted, and the usage and examples of assert-arg itself were updated to encourage this best practice.
Diffstat (limited to 'bin/vm')
-rwxr-xr-xbin/vm6
1 files changed, 3 insertions, 3 deletions
diff --git a/bin/vm b/bin/vm
index bfd182a..2296f72 100755
--- a/bin/vm
+++ b/bin/vm
@@ -214,7 +214,7 @@ write_ssh_config
ACTION="${1:-}"
OS="${2:-}"
-eval "$(assert-arg "$ACTION" 'ACTION')"
+eval "$(assert-arg -- "$ACTION" 'ACTION')"
shift
@@ -247,7 +247,7 @@ case "$ACTION" in
done
;;
up)
- eval "$(assert-arg "$OS" 'OS')"
+ eval "$(assert-arg -- "$OS" 'OS')"
shift
if [ "${1:-}" = '--' ]; then
shift
@@ -301,7 +301,7 @@ case "$ACTION" in
write_ssh_config
;;
down)
- eval "$(assert-arg "$OS" 'OS')"
+ eval "$(assert-arg -- "$OS" 'OS')"
shift
if [ "${1:-}" = '--' ]; then
shift