Avoid outputting Rosetta 2 caveats on Intel machines

[giapoldo]

Verification

• This issue's title and/or description do not reference a single formula e.g. brew install wget. If they do, open an issue at https://github.com/Homebrew/homebrew-core/issues/new/choose instead.

Provide a detailed description of the proposed feature

Check for system type (Intel or Silicon) to filter which Caveats are shown. I'm on an Intel mac and I get the following Caveat:

"{cask} is built for Intel macOS and so requires Rosetta 2 to be installed.
You can install Rosetta 2 with:
softwareupdate --install-rosetta --agree-to-license
Note that it is very difficult to remove Rosetta 2 once it is installed."

This particular time it was for the "session" cask, but I have gotten it for other installs too.

What is the motivation for the feature?

This caveat is a warning that is simply unnecessary for Intel systems.

How will the feature be relevant to at least 90% of Homebrew users?

Less visual clutter and cleaner user experience. The first time I got it I had to investigate a little bit as to why I was getting this warning since I am on an Intel system and I had no clue what "Rosetta 2" was, nor did it imply anything about the warning being only for Silicon systems. My first though was that something in my system was misconfigured since the warning showed up.

What alternatives to the feature have been considered?

An alternative is to modify the caveat text to:

"For Apple Silicon systems:
{cask} is built for Intel macOS and so requires Rosetta 2 to be installed.
You can install Rosetta 2 with:
softwareupdate --install-rosetta --agree-to-license
Note that it is very difficult to remove Rosetta 2 once it is installed."

[apainintheneck]

It looks like there is already a check that should ensure that this only gets displayed on arm machines.

brew/Library/Homebrew/cask/dsl/caveats.rb

Lines 143 to 152 in 29c22e0

	caveat :requires_rosetta do
	next if Homebrew::SimulateSystem.current_arch != :arm
	<<~EOS
	#{@cask} is built for Intel macOS and so requires Rosetta 2 to be installed.
	You can install Rosetta 2 with:
	softwareupdate --install-rosetta --agree-to-license
	Note that it is very difficult to remove Rosetta 2 once it is installed.
	EOS
	end

Could you run the following diagnostic commands locally and post the output here?

brew doctor

brew config

brew ruby -e 'p Homebrew::SimulateSystem.current_arch'

[apainintheneck]

Actually, I figured this out. It's based on pre-computing the caveats when serializing formula casks for the API.

From the JSON API:

$ brew info Sleipnir
==> sleipnir: 4.6.6
https://www.fenrir-inc.com/jp/sleipnir/
Not installed
From: https://github.com/Homebrew/homebrew-cask/blob/HEAD/Casks/s/sleipnir.rb
==> Name
Sleipnir
==> Description
Web browser
==> Artifacts
Sleipnir.app (App)
==> Caveats
sleipnir is built for Intel macOS and so requires Rosetta 2 to be installed.
You can install Rosetta 2 with:
  softwareupdate --install-rosetta --agree-to-license
Note that it is very difficult to remove Rosetta 2 once it is installed.

From the Ruby cask:

$ HOMEBREW_NO_INSTALL_FROM_API=true brew info Sleipnir
==> sleipnir: 4.6.6
https://www.fenrir-inc.com/jp/sleipnir/
Not installed
From: https://github.com/Homebrew/homebrew-cask/blob/HEAD/Casks/s/sleipnir.rb
==> Name
Sleipnir
==> Description
Web browser
==> Artifacts
Sleipnir.app (App)

It seems like this issue is already covered by #17288 so maybe we can close this one.

[Bo98]

Yeah Ruby logic in caveats is not supported, and even if it theoretically were we would still avoid them wherever possible for speed reasons. If it were just a few casks then maybe. But not 1.1k casks.

The proper fix here instead of #17288 would be for us to provide better serialisation of "built-in caveats" or alternatively get rid of the Rosetta caveat in favour of a requires_rosetta true DSL (like auto_updates) or similar. In either case, brew could then add the template text to the caveats dynamically.

Formulae already have a similar notion of depends_on arch: :x86_64. I think it would make sense for casks to have a similar pre-install check - instead of a caveat saying you need Rosetta, we have a Requirement that actually checks if you have Rosetta.

[Bo98]

Above point still applies but reading this again:

It's based on pre-computing the caveats when serializing formula for the API.

Why does it not differ in variations? That would be the quick fix here (though could bloat the JSON a little bit).

[apainintheneck]

Above point still applies but reading this again:

It's based on pre-computing the caveats when serializing formula for the API.

Why does it not differ in variations? That would be the quick fix here (though could bloat the JSON a little bit).

My bad I wrote formula above but this is a cask-only issue.

brew/Library/Homebrew/cask/cask.rb

Line 458 in 9160445

if @dsl.on_system_blocks_exist?

What's happening here is that we only calculate variations if the cask has on system blocks which isn't the case here.

It looks like there are 1227 casks in the main cask repo that have requires_rosetta set in the caveats section so I'd be a bit worried about it bloating the JSON size but I haven't tested the size locally.

[apainintheneck]

Making Rosetta a requirement for installing these casks on arm machines makes sense to me too assuming it's a straightforward change.

[giapoldo]

It looks like there is already a check that should ensure that this only gets displayed on arm machines.

brew/Library/Homebrew/cask/dsl/caveats.rb

Lines 143 to 152 in 29c22e0

	caveat :requires_rosetta do
	next if Homebrew::SimulateSystem.current_arch != :arm
	<<~EOS
	#{@cask} is built for Intel macOS and so requires Rosetta 2 to be installed.
	You can install Rosetta 2 with:
	softwareupdate --install-rosetta --agree-to-license
	Note that it is very difficult to remove Rosetta 2 once it is installed.
	EOS
	end

Could you run the following diagnostic commands locally and post the output here?

brew doctor

brew config

brew ruby -e 'p Homebrew::SimulateSystem.current_arch'

Thanks everyone for taking the time to check this. I'm guessing from the discusssion that I don't need to run the diagnostic commands anymore.

What you're discussing is absolutely out of my knowledge, but don't hesitate to tag me if in need of testers for this.

Thanks a lot again. (Hopefuly I didn't open a pandora's box :P)

[MikeMcQuaid]

@garvamel all good, thanks for the helpful issue!

[MikeMcQuaid]

It looks like there are 1227 casks in the main cask repo that have requires_rosetta set in the caveats section so I'd be a bit worried about it bloating the JSON size but I haven't tested the size locally.

We should prioritise correctness over JSON size. I think having something like caveats_rosetta: true or another minimal addition rather than duplicating the text would make the most sense here.

[EricFromCanada]

Formulae already have a similar notion of depends_on arch: :x86_64

Casks do as well, for example amd-power-gadget.

One solution might be to add :rosetta or :x86_64_or_rosetta as a possible value for depends_on arch: to mean "runs on x86_64 or on ARM with translation installed", which leaves depends_on arch: :x86_64 to mean "requires x86_64 hardware", as is the case above.

[carlocab]

Are there any instances of depends_on arch: :x86_64 that won't work on Rosetta? If not, then we could probably just infer the Rosetta requirement from depends_on arch: :x86_64.

[EricFromCanada]

Probably only casks that tap directly into the hardware like amd-power-gadget, cuda-z, turbo-boost-switcher, and their ilk. (They might be rare enough to not worry about special-casing them.)