Allow for cases where user might've aliased some programs

The $(type -p | head -1) construct expects to always get a path, so need to ignore such aliases with -P

Closes https://github.com/pyenv/pyenv/issues/1662
This commit is contained in:
Ivan Pozdeev 2021-05-17 10:40:35 +03:00
parent fdbcdd38f1
commit ddf952ab2d
8 changed files with 9 additions and 9 deletions

View file

@ -29,7 +29,7 @@ if enable -f "${BASH_SOURCE%/*}"/../libexec/pyenv-realpath.dylib realpath 2>/dev
else
[ -z "$PYENV_NATIVE_EXT" ] || abort "failed to load \`realpath' builtin"
READLINK=$(type -p greadlink readlink | head -1)
READLINK=$(type -P greadlink readlink | head -1)
[ -n "$READLINK" ] || abort "cannot find readlink - are you missing GNU coreutils?"
resolve_link() {

View file

@ -45,7 +45,7 @@ extract_initial_comment_block() {
collect_documentation() {
# shellcheck disable=SC2016
$(type -p gawk awk | head -1) '
$(type -P gawk awk | head -1) '
/^Summary:/ {
summary = substr($0, 10)
next

View file

@ -26,7 +26,7 @@ if ! enable -f "${BASH_SOURCE%/*}"/pyenv-realpath.dylib realpath 2>/dev/null; th
echo "pyenv: failed to load \`realpath' builtin" >&2
exit 1
fi
READLINK=$(type -p greadlink readlink | head -1)
READLINK=$(type -P greadlink readlink | head -1)
if [ -z "$READLINK" ]; then
echo "pyenv: cannot find readlink - are you missing GNU coreutils?" >&2
exit 1

View file

@ -33,7 +33,7 @@ if ! enable -f "${BASH_SOURCE%/*}"/pyenv-realpath.dylib realpath 2>/dev/null; th
exit 1
fi
READLINK=$(type -p greadlink readlink | head -1)
READLINK=$(type -P greadlink readlink | head -1)
if [ -z "$READLINK" ]; then
echo "pyenv: cannot find readlink - are you missing GNU coreutils?" >&2
exit 1

View file

@ -59,7 +59,7 @@ usage() {
definitions() {
local query="$1"
python-build --definitions | $(type -ap ggrep grep | head -1) -F "$query" || true
python-build --definitions | $(type -P ggrep grep | head -1) -F "$query" || true
}
indent() {

View file

@ -58,7 +58,7 @@ lib "$1"
resolve_link() {
$(type -p greadlink readlink | head -1) "$1"
$(type -P greadlink readlink | head -1) "$1"
}
abs_dirname() {
@ -1494,7 +1494,7 @@ build_package_verify_openssl() {
RbConfig::CONFIG.fetch("configure_args").shellsplit.each { |arg| $stderr.puts " #{arg}" }
exit 1
end
' "$(basename "$(type -p yum apt-get | head -1)")" >&4 2>&1
' "$(basename "$(type -P yum apt-get | head -1)")" >&4 2>&1
}
use_homebrew_zlib() {

View file

@ -89,7 +89,7 @@ install_tmp_fixture() {
}
resolve_link() {
$(type -p greadlink readlink | head -1) "$1"
$(type -P greadlink readlink | head -1) "$1"
}
run_inline_definition_with_name() {

View file

@ -130,7 +130,7 @@ assert_output_contains() {
echo "assert_output_contains needs an argument" >&2
return 1
fi
echo "$output" | $(type -p ggrep grep | head -1) -F "$expected" >/dev/null || {
echo "$output" | $(type -P ggrep grep | head -1) -F "$expected" >/dev/null || {
{ echo "expected output to contain $expected"
echo "actual: $output"
} | flunk