diff options
| -rw-r--r-- | ChangeLog | 6 | ||||
| -rw-r--r-- | Completion/Unix/Command/_subversion | 63 | ||||
| -rw-r--r-- | Completion/Unix/Type/_urls | 4 |
3 files changed, 51 insertions, 22 deletions
diff --git a/ChangeLog b/ChangeLog index aa23fd863..395a80f4d 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,9 @@ 2009-11-15 Peter Stephenson <p.w.stephenson@ntlworld.com> + * Greg Klanderman: 27409: Completion/Unix/Command/_subversion, + Completion/Unix/Type/_urls: improve subversion remote paths + and fix minor URL issues. + * Frank Terbeck: 27407: Completion/Unix/Command/_tmux: switch-client subcommand. @@ -12344,5 +12348,5 @@ ***************************************************** * This is used by the shell to define $ZSH_PATCHLEVEL -* $Revision: 1.4813 $ +* $Revision: 1.4814 $ ***************************************************** diff --git a/Completion/Unix/Command/_subversion b/Completion/Unix/Command/_subversion index b4e47d330..9d6babed5 100644 --- a/Completion/Unix/Command/_subversion +++ b/Completion/Unix/Command/_subversion @@ -205,32 +205,57 @@ _svn_status() { (( ${(M)#${(f)_cache_svn_status[$key]}:#(#s)${~pat}*$REPLY} )) } -(( $+functions[_svn_urls] )) || -_svn_urls() { +(( $+functions[_svn_remote_paths] )) || +_svn_remote_paths() { local expl remfiles remdispf remdispd suf ret=1 - if [[ -prefix *: ]] && ! _urls && - zstyle -T ":completion:${curcontext}:" remote-access - then - remfiles=( ${(f)"$(svn list $IPREFIX${PREFIX%%[^./][^/]#} 2>/dev/null)"} ) - compset -P '*/' - compset -S '/*' || suf=file - remdispf=(${remfiles:#*/}) - remdispd=(${(M)remfiles:#*/}) - _tags files - while _tags; do - while _next_label files expl ${suf:-directory}; do - [[ -n $suf ]] && compadd -S ' ' -q "$@" "$expl[@]" -d remdispf $remdispf && ret=0 - compadd -S "${suf:+/}" -q "$@" "$expl[@]" -d remdispd ${remdispd%/} && ret=0 - done - (( ret )) || return 0 + [[ -prefix *://*/ ]] || return 1 + zstyle -T ":completion:${curcontext}:" remote-access || return 1 + + remfiles=( ${(f)"$(svn list $IPREFIX${PREFIX%%[^./][^/]#} 2>/dev/null)"} ) + (( $? == 0 )) || return 1 + + # you might consider trying to return early if $#remfiles is zero, + # but for whatever reason remfiles will always contain at least a + # single empty string; that case is handled correctly below. + + compset -P '*/' + compset -S '/*' || suf=file + remdispf=(${remfiles:#*/}) + remdispd=(${(M)remfiles:#*/}) + _tags files + while _tags; do + while _next_label files expl ${suf:-directory}; do + # add files, unless there is a '/' immediately to the right + [[ -n $suf ]] && + compadd -S ' ' -q "$@" "$expl[@]" -d remdispf $remdispf && ret=0 + # add directories; use empty suffix if there is a '/' immediately to the right + compadd -S "${suf:+/}" -q "$@" "$expl[@]" -d remdispd ${remdispd%/} && ret=0 done - else + (( ret )) || return 0 + done + + return 1 +} + +(( $+functions[_svn_urls] )) || +_svn_urls() { + local expl ret=1 + + # first try completing a remote path; if successful, we are all done.. + _svn_remote_paths && return 0 + + # allow configuring svn repository locations using the 'urls' zstyle. + # always attempt completion of these because then matcher-list styles + # which do substring matching will work correctly. + _urls -S/ && ret=0 + + if [[ ! -prefix *://? ]] ; then compset -S '[^:]*' _wanted url-schemas expl 'URL schema' compadd -S '' - \ file:// http:// https:// svn:// svn+ssh:// && ret=0 fi - + return ret } diff --git a/Completion/Unix/Type/_urls b/Completion/Unix/Type/_urls index 853a48553..b53f5a040 100644 --- a/Completion/Unix/Type/_urls +++ b/Completion/Unix/Type/_urls @@ -48,7 +48,7 @@ zstyle -a ":completion:${curcontext}:urls" urls urls if [[ $#urls -gt 1 || ( $#urls -eq 1 && ! -d $urls[1] ) ]]; then [[ $#urls -eq 1 && -f $urls[1] ]] && urls=( $(< $urls[1]) ) - _wanted urls expl 'URL' compadd -a urls && return 0 + _wanted urls expl 'URL' compadd "$@" -a urls && return 0 urls=() fi @@ -82,7 +82,7 @@ case "$scheme" in fi ;; file) - [[ -prefix //(127.0.0.1|localhost)/ ]] && compset -P //(127.0.0.1|localhost) + [[ -prefix //(127.0.0.1|localhost)/ ]] && compset -P '//(127.0.0.1|localhost)' [[ -prefix /// ]] && compset -P // if ! compset -P //; then _tags -C file files |