diff options
| author | Peter Stephenson <pws@users.sourceforge.net> | 2008-03-26 17:22:17 +0000 |
|---|---|---|
| committer | Peter Stephenson <pws@users.sourceforge.net> | 2008-03-26 17:22:17 +0000 |
| commit | b6d1d7414f90576409509c45d95961ab6e292bd2 (patch) | |
| tree | 077074b6c2409b464ca5be8d95a4f1b543cb45d0 | |
| parent | 461a9bdf9f4e8dcefcb1316d5ddef61f214e3740 (diff) | |
| download | zsh-b6d1d7414f90576409509c45d95961ab6e292bd2.tar.gz zsh-b6d1d7414f90576409509c45d95961ab6e292bd2.tar.xz zsh-b6d1d7414f90576409509c45d95961ab6e292bd2.zip | |
25735: use zstat to get canonical name for symbolic links
| -rw-r--r-- | ChangeLog | 4 | ||||
| -rw-r--r-- | Completion/Unix/Type/_canonical_paths | 39 |
2 files changed, 35 insertions, 8 deletions
diff --git a/ChangeLog b/ChangeLog index 1b4108802..2ca77bf9c 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,9 @@ 2008-03-26 Peter Stephenson <pws@csr.com> + * 25753 (after various previous attempts): + Completion/Unix/Type/_canonical_paths: use zstat to find + canonical name for symbolic links. + * c.f. 24741: Doc/Zsh/mod_stat.yo: loading zstat builtin was misdocumented. diff --git a/Completion/Unix/Type/_canonical_paths b/Completion/Unix/Type/_canonical_paths index c33892c84..188b1d997 100644 --- a/Completion/Unix/Type/_canonical_paths +++ b/Completion/Unix/Type/_canonical_paths @@ -27,18 +27,38 @@ local expl ret=1 tag=$1 desc=$2 shift 2 -if (( ! $+commands[readlink] )); then +if ! zmodload -F zsh/stat b:zstat 2>/dev/null; then _wanted "$tag" expl "$desc" compadd $__gopts $@ && ret=0 return ret fi +typeset REPLY typeset -a matches files +_canonical_paths_get_canonical_path() { + typeset newfile + typeset -A seen + + REPLY=$1 + # Guard against loops. + while [[ -z ${seen[$REPLY]} ]]; do + seen[$REPLY]=1 + newfile=$(zstat +link $REPLY 2>/dev/null) + if [[ -n $newfile ]]; then + REPLY=$newfile + else + break + fi + done +} + + if (( $__opts[(I)-N] )); then files=($@) else for __index in $@; do - files+=$(readlink -qf $__index) + _canonical_paths_get_canonical_path $__index + files+=($REPLY) done fi @@ -48,13 +68,16 @@ _canonical_paths_add_paths () { expref=${~origpref} [[ $origpref == (|*/). ]] && rltrim=. curpref=${${expref%$rltrim}:-./} - canpref=$(readlink -qf $curpref) - if [[ $? -eq 0 ]]; then - [[ $curpref == */ && $canpref == *[^/] ]] && canpref+=/ - canpref+=$rltrim - [[ $expref == *[^/] && $canpref == */ ]] && origpref+=/ - matches+=(${${(M)files:#$canpref*}/$canpref/$origpref}) + if zstat $curpref >&/dev/null; then + _canonical_paths_get_canonical_path $curpref + canpref=$REPLY + else + canpref=$curpref fi + [[ $curpref == */ && $canpref == *[^/] ]] && canpref+=/ + canpref+=$rltrim + [[ $expref == *[^/] && $canpref == */ ]] && origpref+=/ + matches+=(${${(M)files:#$canpref*}/$canpref/$origpref}) for subdir in $expref?*(@); do _canonical_paths_add_paths ${subdir/$expref/$origpref} add done |