-
-
Notifications
You must be signed in to change notification settings - Fork 12.5k
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
simg2img: patch formula to support apple silicon installations #201149
Conversation
Viable alternatives to dropping arm64 support on MacOS would be to either:
|
I didn't want to open an issue as to clutter up the repo, so i figured this was the best way to bring up this issue. Changing the source repo would be the best solution at least in my-case, even though the fork is not maintained either (past the single patch for apple silicon). |
This seems like the best solution then. |
Thanks for contributing to Homebrew! 🎉 It looks like you're having trouble with a CI failure. See our contribution guide for help. You may be most interested in the section on dealing with CI failures. You can find the CI logs in the Checks tab of your pull request. |
Uses a patch to fix the execution of the compiled binaries on arm64 anestisb/android-simg2img@931df9d
@daeho-ro This is my first time working with a formula patch, but I think it should now meet the requirements. |
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.
Thank you, @SoCuul.
Now it looks fine to go!
🤖 An automated task has requested bottles to be published to this PR. |
The compiled binaries do not work properly when executed on apple silicon, only when they are ran on intel-based macs.
This has been solved by another fork (anestisb/android-simg2img#38), but the upstream maintainer of simg2img is inactive and has not merged it.
HOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>
, where<formula>
is the name of the formula you're submitting?brew test <formula>
, where<formula>
is the name of the formula you're submitting?brew audit --strict <formula>
(after doingHOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>
)? If this is a new formula, does it passbrew audit --new <formula>
?