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
dev-cmd: disable when broken on Apple Silicon. #10375
Conversation
This is better than users getting a weird error.
Review period will end on 2021-01-21 at 14:08:38 UTC. |
Review period skipped due to |
EOS | ||
|
||
named_args :none | ||
end | ||
end | ||
|
||
def install_bundler_gems | ||
# TODO: update description above if removing this. | ||
raise UsageError, "not (yet) working on Apple Silicon!" if Hardware::CPU.arm? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe we can allow this command but block the commands we know don't work? Using the system ruby, install-bundler-gems
does install gems, it's just that some don't work.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall I like this but maybe we can allow the command I commented on. Then again, it doesn't matter too much until we decide on a path forward overall for how to run these developer commands on Apple Silicon, so up to you if you want to change it or not.
Thanks @dtrodrigues! I reckon it's worth keeping this minimal for now until we've figured out fixes. |
This broke ARM CI, I think: https://github.com/Homebrew/homebrew-core/pull/69460/checks?check_run_id=1740727396#step:6:32 |
This is better than users getting a weird error.
Partially addresses #10210