Skip to content

Fix deprecated depends_on :macos usage in tart and softnet formulas - #19

Closed
alokdhir wants to merge 1 commit into
cirruslabs:masterfrom
alokdhir:fix-tart-macos-depends-on-deprecation
Closed

Fix deprecated depends_on :macos usage in tart and softnet formulas#19
alokdhir wants to merge 1 commit into
cirruslabs:masterfrom
alokdhir:fix-tart-macos-depends-on-deprecation

Conversation

@alokdhir

Copy link
Copy Markdown

Summary

brew install cirruslabs/cli/tart currently prints this warning:

Warning: Calling `depends_on :macos` with `depends_on macos:` is deprecated!
Use `depends_on :macos` with `depends_on macos:` inside an `on_macos` block instead.

The formula sets a minimum macOS version via depends_on :macos => :ventura
outside of an on_macos block, which Homebrew now considers deprecated
syntax. softnet.rb (a dependency of tart) has the same issue with
depends_on :macos => :sequoia.

Fix

Wrap the version-specific depends_on :macos call in an on_macos block in
both tart.rb and softnet.rb, matching the syntax Homebrew expects.

Note: these formulas are marked "generated by GoReleaser, DO NOT EDIT", but
the upstream tart repository has moved to openai/tart and now publishes
to a different tap, so this tap is no longer auto-updated for tart/softnet
releases. This same fix has already been applied upstream in the newer
.goreleaser.yml, so this PR just brings it here directly.

Homebrew deprecated passing a minimum OS version directly to
depends_on :macos outside of an on_macos block, which produces a
warning on every brew command. Wrap the version-specific dependency in
an on_macos block in both formulas to silence the warning.
@alokdhir

Copy link
Copy Markdown
Author

Closing as duplicate of #16 and #18. Confirmed the actual fix already exists upstream: both tart and softnet have been fixed and released in openai/homebrew-tools (tart 2.34.0, softnet 0.21.1), since the tart/softnet projects moved from cirruslabs to openai and now publish there instead of this tap. This tap appears to no longer be actively updated for these two formulas. Recommending users switch taps rather than waiting on this stale one: brew untap cirruslabs/cli && brew tap openai/tools && brew install openai/tools/tart.

@lra

lra commented Jul 29, 2026

Copy link
Copy Markdown

@alokdhir thanks that makes sense, I was about to do the same PR.
I think people are confused because the tart website still mentions the old tap despite the source being fixed.
I created an issue: openai/tart#1298

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants