diff options
| author | John Southworth <john.southworth@vyatta.com> | 2011-10-06 16:20:12 -0500 |
|---|---|---|
| committer | John Southworth <john.southworth@vyatta.com> | 2011-10-06 16:20:35 -0500 |
| commit | 5de548e5ee8fd5fc9873963ae4a9d477b2bfdcb3 (patch) | |
| tree | 89a14aace737cf68e2e519529cd79f9a2b676c8b | |
| parent | 9e07fa4ddef970f4e6370d6513051239811f0534 (diff) | |
| download | vyatta-op-5de548e5ee8fd5fc9873963ae4a9d477b2bfdcb3.tar.gz vyatta-op-5de548e5ee8fd5fc9873963ae4a9d477b2bfdcb3.zip | |
Bugfix 6747: use bash conditional expressions instead of posix test expressions when doing comparisons. The conditional expressions don't do as much expansion as the test expression does, and are the modern variant
| -rw-r--r-- | etc/bash_completion.d/vyatta-op | 2 | ||||
| -rw-r--r-- | functions/interpreter/vyatta-common | 4 |
2 files changed, 3 insertions, 3 deletions
diff --git a/etc/bash_completion.d/vyatta-op b/etc/bash_completion.d/vyatta-op index 48074dd..815b264 100644 --- a/etc/bash_completion.d/vyatta-op +++ b/etc/bash_completion.d/vyatta-op @@ -514,7 +514,7 @@ _vyatta_pipe_completion () _vyatta_pipe_noncompletions=() for word in "$@"; do - if [ "$found" == "1" -o "$word" == "|" ]; then + if [[ "$found" == "1" || "$word" == "|" ]]; then pipe_cmd+=( "$word" ) found=1 fi diff --git a/functions/interpreter/vyatta-common b/functions/interpreter/vyatta-common index 0a98d56..c6e645f 100644 --- a/functions/interpreter/vyatta-common +++ b/functions/interpreter/vyatta-common @@ -34,7 +34,7 @@ get_prefix_filtered_list () local idx=0 for elem in "${olist[@]}"; do local sub=${elem#$pfx} - if [ "$elem" == "$sub" ] && [ -n "$pfx" ]; then + if [[ "$elem" == "$sub" ]] && [[ -n "$pfx" ]]; then continue fi eval "$3[$idx]=\"$elem\"" @@ -60,7 +60,7 @@ get_prefix_filtered_list2 () eval "local elem=\${$2[$orig_idx]}" eval "local elem2=\${$4[$orig_idx]}" local sub=${elem#$pfx} - if [ "$elem" == "$sub" ] && [ -n "$pfx" ]; then + if [[ "$elem" == "$sub" ]] && [[ -n "$pfx" ]]; then continue fi eval "$3[$idx]=\"$elem\"" |