Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

os/mac/hardware: remove TODO. #15938

Merged
merged 1 commit into from Sep 3, 2023

Conversation

MikeMcQuaid
Copy link
Member

@MikeMcQuaid MikeMcQuaid commented Sep 2, 2023

As discussed in comments, this TODO can pretty much never be done so doesn't make sense to leave as-is.

@Bo98
Copy link
Member

Bo98 commented Sep 2, 2023

Rosetta probably got a spike in usage really with the Gaming Porting Toolkit that came out. If I'm reading analytics correctly, Rosetta looks like it might cover about 800,000, or about 10% of arm64 installs, in the last month which is relatively high.

This change feels problematic given some formulae (e.g. pkg-config, compilers) need per-OS bottles and we will not be offering them anymore with this change, and we'll also be stuck if Rosetta is still used 2 years from now when we drop Ventura which seems likely with the way Apple have moved recently.

@fxcoudert
Copy link
Member

I tend to agree with @Bo98: this will break some formulas, while we know there is still significant Rosetta 2 usage. On the other hand, the AVX optimisation is, well, probably negligible for most formulas anyway. And I hope people who have CPU-bound apps or heavily computational needs compile from source rather than use a distro, anyway.

As discussed in #15938, this TODO can pretty much never be done so
doesn't make sense to leave as-is.
@MikeMcQuaid MikeMcQuaid changed the title os/mac: don't use Sonoma bottles under Rosetta 2. os/mac/hardware: remove TODO. Sep 3, 2023
@MikeMcQuaid
Copy link
Member Author

@Bo98 @fxcoudert good points, I agree. As a result, I've rescoped this PR to remove the TODO instead as we can never really implement it.

@MikeMcQuaid MikeMcQuaid added the critical Critical change which should be shipped as soon as possible. label Sep 3, 2023
@MikeMcQuaid MikeMcQuaid merged commit dbc4406 into Homebrew:master Sep 3, 2023
25 checks passed
@MikeMcQuaid MikeMcQuaid deleted the sonoma_rosetta_2_bottles branch September 3, 2023 13:09
Copy link

@medoasiri medoasiri left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

dduugg pushed a commit to dduugg/brew that referenced this pull request Sep 30, 2023
As discussed in Homebrew#15938, this TODO can pretty much never be done so
doesn't make sense to leave as-is.
@github-actions github-actions bot added the outdated PR was locked due to age label Oct 4, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 4, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
critical Critical change which should be shipped as soon as possible. outdated PR was locked due to age
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants