Don't use extglob in bash/docker-machine.bash

extglob is not needed here - the only call is:
_docker_machine_map_key_of_current_option '--filter'

So $glob is always '--filter', so there isn't every a pattern-list to match.

According to my testing completion for docker-machine
ls --filter completion works for both driver and state

Fixes #4126
Introduced in fd9a0a683c

Signed-off-by: Ben Woosley <ben.woosley@gmail.com>
This commit is contained in:
Ben Woosley
2017-06-16 16:23:30 -07:00
parent c604d46dc5
commit d983b844e9

View File

@@ -62,7 +62,7 @@ _docker_machine_value_of_option() {
}
# Returns `key` if we are currently completing the value of a map option
# (`key=value`) which matches the extglob passed in as an argument.
# (`key=value`) which matches the glob passed in as an argument.
# This function is needed for key-specific argument completions.
_docker_machine_map_key_of_current_option() {
local glob="$1"
@@ -83,7 +83,7 @@ _docker_machine_map_key_of_current_option() {
[ "${words[$glob_pos]}" = "=" ] && ((glob_pos--)) # --option=key=value syntax
[[ ${words[$glob_pos]} == @($glob) ]] && echo "$key"
[[ ${words[$glob_pos]} == $glob ]] && echo "$key"
}
# --- completion functions ---------------------------------------------------