brew.sh: hide ignorable permission errors.

`GIT_DESCRIBE_CACHE` is a performance optimisation that doesn't clean
up consistently when run with different Homebrew users. Instead of
doing user detection: let's just hide these errors that we don't care
about.

Reported in:
https://github.com/orgs/Homebrew/discussions/5479
This commit is contained in:
Mike McQuaid 2024-08-19 09:17:21 +01:00
parent 3e725e747d
commit c6ddb9cedb
No known key found for this signature in database

View File

@ -446,10 +446,13 @@ then
then then
HOMEBREW_VERSION="$(cat "${GIT_DESCRIBE_CACHE_FILE}")" HOMEBREW_VERSION="$(cat "${GIT_DESCRIBE_CACHE_FILE}")"
else else
rm -rf "${GIT_DESCRIBE_CACHE}"
HOMEBREW_VERSION="$("${HOMEBREW_GIT}" -C "${HOMEBREW_REPOSITORY}" describe --tags --dirty --abbrev=7 2>/dev/null)" HOMEBREW_VERSION="$("${HOMEBREW_GIT}" -C "${HOMEBREW_REPOSITORY}" describe --tags --dirty --abbrev=7 2>/dev/null)"
mkdir -p "${GIT_DESCRIBE_CACHE}" # Don't output any permissions errors here.
echo "${HOMEBREW_VERSION}" >"${GIT_DESCRIBE_CACHE_FILE}" # The user may not have write permissions to the cache but we don't care
# because it's an optional performance improvement.
rm -rf "${GIT_DESCRIBE_CACHE}" 2>/dev/null
mkdir -p "${GIT_DESCRIBE_CACHE}" 2>/dev/null
echo "${HOMEBREW_VERSION}" >"${GIT_DESCRIBE_CACHE_FILE}" 2>/dev/null
fi fi
unset GIT_DESCRIBE_CACHE_FILE unset GIT_DESCRIBE_CACHE_FILE
else else