On Tue, Jul 30, 2013 at 03:48:13AM -0300, Lucas De Marchi wrote:
> From: Lucas De Marchi <[email protected]>
>
> Skeleton pulled from udevadm in systemd and adapted to kmod needs.
> ---
> NEWS | 2 +-
> shell-completion/bash/kmod | 53
> ++++++++++++++++++++++++++++++++++++++++++++++
> 2 files changed, 54 insertions(+), 1 deletion(-)
> create mode 100644 shell-completion/bash/kmod
>
> diff --git a/NEWS b/NEWS
> index 1dff366..491146d 100644
> --- a/NEWS
> +++ b/NEWS
> @@ -6,7 +6,7 @@ kmod 15
>
> - New features:
> - kmod static-nodes creates parent directories if given a -o option
> - - kmod binary doesn't statically link to libkmod
> + - kmod binary statically link to libkmod
>
> kmod 14
> =======
> diff --git a/shell-completion/bash/kmod b/shell-completion/bash/kmod
> new file mode 100644
> index 0000000..81dbf46
> --- /dev/null
> +++ b/shell-completion/bash/kmod
> @@ -0,0 +1,53 @@
> +# kmod completion -*- shell-script
> -*-
> +#
> +# This file is part of systemd.
> +#
> +# Copyright 2010 Ran Benita
> +# Copyright (C) 2013 Intel Corporation. All rights reserved.
> +#
> +# systemd is free software; you can redistribute it and/or modify it
> +# under the terms of the GNU Lesser General Public License as published by
> +# the Free Software Foundation; either version 2.1 of the License, or
> +# (at your option) any later version.
> +#
> +# systemd is distributed in the hope that it will be useful, but
> +# WITHOUT ANY WARRANTY; without even the implied warranty of
> +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
> +# General Public License for more details.
> +#
> +# You should have received a copy of the GNU Lesser General Public License
> +# along with systemd; If not, see <http://www.gnu.org/licenses/>.
> +
> +__contains_word () {
> + local word=$1; shift
> + for w in $*; do [[ $w = $word ]] && return 0; done
Should be "$@", not $*, and "$word", not $word.
> + return 1
> +}
> +
> +_kmod() {
> + local cur=${COMP_WORDS[COMP_CWORD]} prev=${COMP_WORDS[COMP_CWORD-1]}
> + local verb comps
> +
> + local -A VERBS=(
> + [HELP]='help'
> + [LIST]='list'
> + [STATIC-NODES]='static-nodes'
It seems you have tabs here instead of spaces. I think you probably want
the top level actions in a single list, not broken out like this.
Subsequent options for the toplevel actions would want to be broken out
in an associative array like you have here.
> + )
> +
> + for ((i=0; $i <= $COMP_CWORD; i++)); do
> + if __contains_word "${COMP_WORDS[i]}" ${VERBS[*]} &&
> + ! __contains_word "${COMP_WORDS[i-1]}" ${OPTS[ARG]}; then
> + verb=${COMP_WORDS[i]}
> + break
> + fi
> + done
> +
> + if [[ -z $verb ]]; then
> + comps=${VERBS[*]}
Tabs here too.
> + fi
> +
> + COMPREPLY=( $(compgen -W '$comps' -- "$cur" ) )
> + return 0
> +}
> +
> +complete -F _kmod kmod
> --
> 1.8.3.4
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-modules" in
> the body of a message to [email protected]
> More majordomo info at http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-modules" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
