From 69ada95acd9cb5c5ea51277f23739affe13cc1fb Mon Sep 17 00:00:00 2001 From: Daniel Shahaf Date: Sat, 7 Mar 2015 15:31:14 +0000 Subject: 34693: sudo completion: Don't false positive 'sudo -e' detection --- ChangeLog | 3 +++ Completion/Unix/Command/_sudo | 9 ++++++++- 2 files changed, 11 insertions(+), 1 deletion(-) diff --git a/ChangeLog b/ChangeLog index c99f726b0..109027b34 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,8 @@ 2015-03-09 Daniel Shahaf + * 34693: Completion/Unix/Command/_sudo: sudo completion: Don't + false positive 'sudo -e' detection + * 34693: Completion/Linux/Command/_strace: _strace: Fix variable leakage of $sys_calls diff --git a/Completion/Unix/Command/_sudo b/Completion/Unix/Command/_sudo index ee238b462..63ac37f62 100644 --- a/Completion/Unix/Command/_sudo +++ b/Completion/Unix/Command/_sudo @@ -1,5 +1,7 @@ #compdef sudo sudoedit +setopt localoptions extended_glob + local environ e local -a args @@ -29,7 +31,12 @@ args=( '(-v --validate)'{-v,--validate}"[update user's timestamp without running a command]" ) -if [[ $service = sudoedit || -n ${words[(r)-e]} ]]; then +# Does -e appears before the first word that doesn't begin with a hyphen? +# The way (i) works in subscripts, the test will always be true if all the +# words begin with a hyphen. +# +# TODO: use _arguments' $opt_args to detect the cases '-u jrandom -e' and '-Ae' +if [[ $service = sudoedit ]] || (( $words[(i)-e] < $words[(i)^(*sudo|-[^-]*)] )) ; then args=( -A "-*" $args '!(-V --version -h --help)-e' '*:file:_files' ) else args+=( -- cgit 1.4.1