Skip to content

[release/9.0] Remove Homebrew LLVM during macOS CI setup - Part 2#125789

Merged
lewing merged 2 commits intodotnet:release/9.0from
steveisok:backport/remove-brew-llvm-9.0-release
Mar 19, 2026
Merged

[release/9.0] Remove Homebrew LLVM during macOS CI setup - Part 2#125789
lewing merged 2 commits intodotnet:release/9.0from
steveisok:backport/remove-brew-llvm-9.0-release

Conversation

@steveisok
Copy link
Member

The CI runner image may ship with a Homebrew LLVM whose libraries (e.g., an x86_64-only libunwind.dylib in /usr/local/lib) conflict with the Apple SDK and break native linking. The build uses Apple clang from /usr/bin/clang exclusively and does not need Homebrew LLVM.

Backport of #125763

The CI runner image may ship with a Homebrew LLVM whose libraries
(e.g., an x86_64-only libunwind.dylib in /usr/local/lib) conflict with
the Apple SDK and break native linking. The build uses Apple clang from
/usr/bin/clang exclusively and does not need Homebrew LLVM.

Backport of dotnet#125763
Copilot AI review requested due to automatic review settings March 19, 2026 17:31
@dotnet-policy-service
Copy link
Contributor

Tagging subscribers to this area: @dotnet/runtime-infrastructure
See info in area-owners.md if you want to be subscribed.

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

Updates macOS dependency installation to proactively remove a preinstalled Homebrew LLVM that can conflict with Apple SDK libraries and break native linking in CI (backport of #125763).

Changes:

  • Add a Homebrew llvm@18 uninstall step during macOS CI dependency setup.
  • Document why Homebrew LLVM is removed (wrong-arch/conflicting libraries; Apple clang is used).

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

@steveisok steveisok requested a review from a team March 19, 2026 17:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants