Skip to content

Conversation

@cho-m
Copy link
Member

@cho-m cho-m commented Oct 26, 2025

Experiment to see if same Sonoma failure happens on Tahoe/Sequoia.

@github-actions github-actions bot added legacy Relates to a versioned @ formula formula deprecated Formula deprecated long build Set a long timeout for formula testing icu4c ICU use is a significant feature of the PR or issue labels Oct 26, 2025
@github-actions

This comment was marked as outdated.

@cho-m cho-m added the CI-no-fail-fast Continue CI tests despite failing GitHub Actions matrix builds. label Oct 26, 2025
@github-actions github-actions bot added the automerge-skip `brew pr-automerge` will skip this pull request label Oct 26, 2025
@cho-m cho-m added the CI-skip-dependents Pass --skip-dependents to brew test-bot. label Oct 26, 2025
@github-actions

This comment was marked as outdated.

@github-actions github-actions bot removed the automerge-skip `brew pr-automerge` will skip this pull request label Oct 26, 2025
@github-actions

This comment was marked as outdated.

@cho-m
Copy link
Member Author

cho-m commented Oct 27, 2025

Same error on Tahoe so going to just maximum_macos this to make it more obvious that we aren't able to provide bottles anymore on Intel macOS.

@cho-m cho-m added the CI-syntax-only Change only affects brew syntax, not the install. Only run syntax CI. label Oct 27, 2025
@cho-m cho-m changed the title dotnet@8: try run on Sequoia and Tahoe dotnet@8: make brew aware of inability to build Intel Sonoma bottle Oct 27, 2025
@cho-m cho-m marked this pull request as ready for review October 27, 2025 12:54
Copilot AI review requested due to automatic review settings October 27, 2025 12:54
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 adds a build constraint for the dotnet@8 formula to prevent building on Intel-based macOS Sonoma or later due to a stack overflow issue during restore operations.

  • Adds depends_on maximum_macos constraint for Intel builds
  • Documents the issue with a reference to the upstream GitHub issue
  • Limits Intel builds to macOS Ventura and earlier

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@BrewTestBot BrewTestBot added this pull request to the merge queue Oct 27, 2025
Merged via the queue into main with commit af94cb9 Oct 27, 2025
27 checks passed
@BrewTestBot BrewTestBot deleted the dotnet@8-intel branch October 27, 2025 13:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI-no-fail-fast Continue CI tests despite failing GitHub Actions matrix builds. CI-skip-dependents Pass --skip-dependents to brew test-bot. CI-syntax-only Change only affects brew syntax, not the install. Only run syntax CI. formula deprecated Formula deprecated icu4c ICU use is a significant feature of the PR or issue legacy Relates to a versioned @ formula long build Set a long timeout for formula testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants