Reject redundant minimum macOS dependencies #24020

Merged
p-linnane opened 8:37am on September 17, 2026 wanted to merge 210 Ξ” into homebrew/brew main from
redundant-macos-minimum
Diff Delta:
210
About 42 Diff Delta/hour
Classified as:  General

p-linnane's Description of Work

depends_on macos: :big_sur requires a release every supported macOS already meets. Homebrew/OSDependsOn now reports minimum macOS versions at or below the runtime floor and autocorrects the plain top-level form to depends_on :macos; OS-block, multi-key, tagged and commented forms get guidance instead. The floor comes from MacOSVersion::SYMBOLS and a spec pins it to HOMEBREW_MACOS_OLDEST_ALLOWED.

A bare depends_on :macos in on_arm/on_intel now means no minimum on that architecture and Cask/NoOverrides accepts it opposite a versioned block. Both cookbooks are updated.

Current taps have 14 casks and 5 formulae to clean up.



  • [x] Have you followed our Contributing guidelines?

  • [x] Have you checked for other open Pull Requests for the same change?

  • [x] Have you explained what your changes do? Performance claims (e.g. "this is faster") must include brew benchmark results.

  • [x] Have you explained why you'd like these changes included, not just what they do?

  • [ ] For bug fixes, have you given step-by-step brew commands to reproduce the bug?

  • [x] Have you written new tests (excluding integration tests)? Here's an example.

  • [x] Have you successfully run brew lgtm (style, typechecking and tests) locally?



  • [x] I did not use AI/LLM to create this PR, or I disclosed the tool/model below and reviewed its output; I did not attribute commits to AI and will answer maintainer questions and review comments myself without AI/LLM.

Claude Code (Fable 5.1) drafted the implementation and tests; I reviewed the diff, verified the new tests fail without the change and pass with it, and ran brew lgtm --online plus targeted specs.


6 total changed files
Loading changes...
Loading changes...
Loading changes...
Loading changes...
Review complete πŸ’ͺ
LGTM-land is officially in view