diff options
author | EuAndreh <eu@euandre.org> | 2023-02-17 15:07:30 -0300 |
---|---|---|
committer | EuAndreh <eu@euandre.org> | 2023-02-17 15:07:32 -0300 |
commit | 44b1e103b4d3a24aee89587ef302891e9f9c3807 (patch) | |
tree | a8e4045218bd1075b687dd5a6f1d023e6c407bff /bin/vm | |
parent | etc/nix/configuration.nix: Add flutter2 and hover packages (diff) | |
download | dotfiles-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-x | bin/vm | 6 |
1 files changed, 3 insertions, 3 deletions
@@ -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 |