Skip to content

Revert "[release/9.0] Remove Homebrew LLVM during macOS CI setup"#125786

Closed
rbhanda wants to merge 1 commit intorelease/9.0from
revert-125785-backport/remove-brew-llvm-9.0-release
Closed

Revert "[release/9.0] Remove Homebrew LLVM during macOS CI setup"#125786
rbhanda wants to merge 1 commit intorelease/9.0from
revert-125785-backport/remove-brew-llvm-9.0-release

Conversation

@rbhanda
Copy link
Collaborator

@rbhanda rbhanda commented Mar 19, 2026

Reverts #125785

Copilot AI review requested due to automatic review settings March 19, 2026 17:22
@rbhanda rbhanda enabled auto-merge (squash) March 19, 2026 17:26
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR reverts a prior change in the macOS CI dependency installation script by removing the step that uninstalls a preinstalled Homebrew LLVM from the runner environment.

Changes:

  • Removed the brew uninstall llvm workaround from macOS/iOS/tvOS CI dependency setup.
  • macOS dependency setup now only runs brew bundle from eng/Brewfile.

You can also share your feedback on Copilot code review. Take the survey.

# The build uses Apple clang from /usr/bin/clang exclusively.
brew uninstall --ignore-dependencies llvm 2>/dev/null || true

brew bundle --no-upgrade --file "$(dirname "$0")/Brewfile"
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants