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

Add support for building on Apple Silicon #183

Closed
wants to merge 1 commit into from

Conversation

areese
Copy link

@areese areese commented Jun 8, 2021

Don't build logging_arm64 when on darwin, MacOS on arm64 does not have a dup3 syscall
Remove !arm64 from logging_other, so builds work on MacOS on arm64 (Apple Silicon)

Don't build logging_arm64 when on darwin, MacOS on arm64 does not have a dup3 syscall
Remove \!arm64 from logging_other, so builds work on MacOS on arm64 (Apple Silicon)
areese pushed a commit to areese/homebrew-core that referenced this pull request Jun 14, 2021
BrewTestBot pushed a commit to Homebrew/homebrew-core that referenced this pull request Jun 14, 2021
…pple Silicon

Closes #79314.

Signed-off-by: Carlo Cabrera <30379873+carlocab@users.noreply.github.com>
Signed-off-by: Michael Cho <20700669+cho-m@users.noreply.github.com>
Signed-off-by: BrewTestBot <1589480+BrewTestBot@users.noreply.github.com>
@xxxserxxx
Copy link
Owner

This will be released with v4.1.2 -- dispite the long delay, thanks for your patch. Accepting this required finding and integrating a newer cross-compile CGO build chain, and because I can't test this myelf I've been cautious about making the change.

@xxxserxxx xxxserxxx closed this Jul 20, 2021
@areese areese deleted the m1-support branch July 20, 2021 18:05
@areese
Copy link
Author

areese commented Jul 20, 2021

@xxxserxxx thank you, I'll update the homebrew formula to stop patching this when v4.1.2 is released

@xxxserxxx xxxserxxx added this to the v4.1.2 milestone Jul 20, 2021
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