Merge pull request #682 from albers/completion-service-options

Fix and simplify bash completion for service env, mounts and labels

contrib/completion/bash/docker

@@ -3354,7 +3354,6 @@ _docker_service_update_and_create() {
  local options_with_args="
  --endpoint-mode
  --entrypoint
- --env -e
  --force
  --health-cmd
  --health-interval
@@ -3363,12 +3362,10 @@ _docker_service_update_and_create() {
  --health-timeout
  --hostname
  --isolation
- --label -l
  --limit-cpu
  --limit-memory
  --log-driver
  --log-opt
- --mount
  --replicas
  --reserve-cpu
  --reserve-memory
@@ -3416,11 +3413,14 @@ _docker_service_update_and_create() {
  --dns
  --dns-option
  --dns-search
+ --env -e
  --env-file
  --generic-resource
  --group
  --host
+ --label -l
  --mode
+ --mount
  --name
  --network
  --placement-pref
@@ -3429,39 +3429,14 @@ _docker_service_update_and_create() {
  "
 
  case "$prev" in
- --config)
- __docker_complete_configs
- return
- ;;
  --env-file)
  _filedir
  return
  ;;
- --group)
- COMPREPLY=( $(compgen -g -- "$cur") )
- return
- ;;
- --host)
- case "$cur" in
- *:)
- __docker_complete_resolved_hostname
- return
- ;;
- esac
- ;;
  --mode)
  COMPREPLY=( $( compgen -W "global replicated" -- "$cur" ) )
  return
  ;;
- --placement-pref)
- COMPREPLY=( $( compgen -W "spread" -S = -- "$cur" ) )
- __docker_nospace
- return
- ;;
- --secret)
- __docker_complete_secrets
- return
- ;;
  esac
  fi
  if [ "$subcommand" = "update" ] ; then
@@ -3479,13 +3454,19 @@ _docker_service_update_and_create() {
  --dns-rm
  --dns-search-add
  --dns-search-rm
+ --env-add
+ --env-rm
  --generic-resource-add
  --generic-resource-rm
  --group-add
  --group-rm
  --host-add
  --host-rm
  --image
+ --label-add
+ --label-rm
+ --mount-add
+ --mount-rm
  --network-add
  --network-rm
  --placement-pref-add
@@ -3498,39 +3479,14 @@ _docker_service_update_and_create() {
  "
 
  case "$prev" in
- --config-add|--config-rm)
- __docker_complete_configs
- return
- ;;
- --group-add|--group-rm)
- COMPREPLY=( $(compgen -g -- "$cur") )
+ --env-rm)
+ COMPREPLY=( $( compgen -e -- "$cur" ) )
  return
  ;;
- --host-add|--host-rm)
- case "$cur" in
- *:)
- __docker_complete_resolved_hostname
- return
- ;;
- esac
- ;;
  --image)
  __docker_complete_images --repo --tag --id
  return
  ;;
- --network-add|--network-rm)
- __docker_complete_networks
- return
- ;;
- --placement-pref-add|--placement-pref-rm)
- COMPREPLY=( $( compgen -W "spread" -S = -- "$cur" ) )
- __docker_nospace
- return
- ;;
- --secret-add|--secret-rm)
- __docker_complete_secrets
- return
- ;;
  esac
  fi
 
@@ -3544,16 +3500,32 @@ _docker_service_update_and_create() {
  esac
 
  case "$prev" in
+ --config|--config-add|--config-rm)
+ __docker_complete_configs
+ return
+ ;;
  --endpoint-mode)
  COMPREPLY=( $( compgen -W "dnsrr vip" -- "$cur" ) )
  return
  ;;
- --env|-e)
+ --env|-e|--env-add)
  # we do not append a "=" here because "-e VARNAME" is legal systax, too
  COMPREPLY=( $( compgen -e -- "$cur" ) )
  __docker_nospace
  return
  ;;
+ --group|--group-add|--group-rm)
+ COMPREPLY=( $(compgen -g -- "$cur") )
+ return
+ ;;
+ --host|--host-add|--host-rm)
+ case "$cur" in
+ *:)
+ __docker_complete_resolved_hostname
+ return
+ ;;
+ esac
+ ;;
  --isolation)
  __docker_complete_isolation
  return
@@ -3566,10 +3538,15 @@ _docker_service_update_and_create() {
  __docker_complete_log_options
  return
  ;;
- --network)
+ --network|--network-add|--network-rm)
  __docker_complete_networks
  return
  ;;
+ --placement-pref|--placement-pref-add|--placement-pref-rm)
+ COMPREPLY=( $( compgen -W "spread" -S = -- "$cur" ) )
+ __docker_nospace
+ return
+ ;;
  --restart-condition)
  COMPREPLY=( $( compgen -W "any none on-failure" -- "$cur" ) )
  return
@@ -3578,6 +3555,10 @@ _docker_service_update_and_create() {
  COMPREPLY=( $( compgen -W "continue pause" -- "$cur" ) )
  return
  ;;
+ --secret|--secret-add|--secret-rm)
+ __docker_complete_secrets
+ return
+ ;;
  --stop-signal)
  __docker_complete_signals
  return