On Tue, Jul 30, 2013 at 03:48:13AM -0300, Lucas De Marchi wrote: > From: Lucas De Marchi <lucas.demarchi@xxxxxxxxx> > > 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 majordomo@xxxxxxxxxxxxxxx > 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 majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
