-
Notifications
You must be signed in to change notification settings - Fork 21
Handle Homebrew conflicts: Install cmake only if not present #754
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #754 +/- ##
=======================================
Coverage 96.82% 96.82%
=======================================
Files 122 122
Lines 7496 7496
=======================================
Hits 7258 7258
Misses 238 238 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
* try with macos14 * uninstall cmake if exists * uninstallcmake * try with condiiton (cherry picked from commit d60ab7c)
|
Successfully created backport PR for |
* try with macos14 * uninstall cmake if exists * uninstallcmake * try with condiiton (cherry picked from commit d60ab7c)
|
Successfully created backport PR for |
|
Successfully created backport PR for |
* try with macos14 * uninstall cmake if exists * uninstallcmake * try with condiiton (cherry picked from commit d60ab7c)
* try with macos14 * uninstall cmake if exists * uninstallcmake * try with condiiton (cherry picked from commit d60ab7c)
|
Successfully created backport PR for |
|
Successfully created backport PR for |
* try with macos14 * uninstall cmake if exists * uninstallcmake * try with condiiton (cherry picked from commit d60ab7c)
Due to the change introduced in Homebrew/brew#20304, Homebrew now prevents installing formulae with the same name. This means that workflows attempting to install a formula already present will fail.
This PR fixes the error message failing macos WF by installing cmake only if it isn’t already present.