utils-code/pyenv.pyenv
1
0
Fork 0
mirror of https://github.com/pyenv/pyenv.git synced 2026-09-10 07:36:16 -04:00
Code Issues Actions 2 Packages Projects Releases Wiki Activity

versions: fix --executables portability regression

Browse source 
Replace the use of `xargs -r` and `basename -a` in the
`--executables` fast path with Bash 3.2-compatible parameter
expansion.

The previous implementation relied on non-portable utility options,
causing `pyenv rehash` and shell initialization to fail on illumos,
OpenBSD, and other non-GNU userlands.

Preserve the optimized executable discovery path by continuing to
discover executables directly from the versions directory and
deduplicate their basenames.
This commit is contained in:
Tim Hawes 2026-08-02 21:39:49 -04:00
parent bdf8d254fe
commit a33b30fa2d
No known key found for this signature in database
GPG key ID: 7927112CC0EB3597
1 changed files with 9 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
libexec/pyenv-versions
View file

@ -42,9 +42,15 @@ versions_dir="${PYENV_ROOT}/versions"
if [[ -n "$executables" ]]; then
if [ -d "$versions_dir" ]; then
shopt -s dotglob nullglob
# MacOS 12+ and FreeBSD 15 support `xargs -r -0' and `basename -a'
# `sort -u` is simpler and a bit faster than `awk '!seen[$0]++'`, with the same result for rehash purposes
printf '%s\0' "$versions_dir"/*/bin/* "$versions_dir"/*/envs/*/bin/* | xargs -0 -r basename -a | sort -u
# Fast path for rehash: skip filtering and link resolution.
# Discover executables directly from the versions directory and
# deduplicate their basenames. Use bash 3.2 conventions to keep code portable.
for path in \
"$versions_dir"/*/bin/* \
"$versions_dir"/*/envs/*/bin/*
do
printf '%s\n' "${path##*/}"
done | sort -u
shopt -u dotglob nullglob
fi
exit 0