averello/macvim-mirror
1
0
Fork 0
mirror of https://github.com/macvim-dev/macvim.git synced 2026-06-11 15:37:29 +02:00
Code Issues Packages Projects Releases Wiki Activity

ci: Minor fix so that if gettext/libsodium is cached, don't warn

Browse Source 
Previously when we try to set up a cached package, Homebrew generates an
annoying "already installed, it's just not linked" warning which is
distracting when parsing CI logs. Just make sure to run `brew install`
if we don't have the package cached.

Note that when a package is not cached and we have to rebuild it,
Homebrew will still warn needlessly because we have
HOMEBREW_NO_INSTALLED_DEPENDENTS_CHECK set. This is mostly ok because
most of the time our packages should be cached in CI.
This commit is contained in:
Yee Cheng Chin
2025-01-18 18:18:35 -08:00
parent fd1c3fc996
commit 7fa7edd405
1 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
.github/actions/universal-package/action.yml
+3 -2
View File
@@ -72,8 +72,9 @@ runs:
# which cause brew install to return non-zero and fail the build.
export HOMEBREW_NO_INSTALLED_DEPENDENTS_CHECK=1
# This will be a no-op if formula was cached
brew install --quiet --formula -s ./${formula}.rb
# This will be a no-op if formula was cached. We check if the package
# exists first just to avoid an "already installed" warning.
brew list ${formula} &>/dev/null || brew install --quiet --formula -s ./${formula}.rb
# If formula was cached, this step is necessary to relink it to brew prefix (e.g. /usr/local)
brew unlink ${formula} && brew link ${formula}