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

build fat-mach-o binary for pygore #9

Merged
merged 2 commits into from
Apr 14, 2023

Conversation

ApexHack3r
Copy link
Contributor

Fixed an issue with pygore where the pip pygore package comes with amd64 compiled libgore.dylib and lacks support for Applie Silicon arm64 architecture.

The Makefile was changed to compile both an amd64 and an arm64 version and create a fat-mach-o universal libgore.dylib binary.

Tested to work on my MBP M1 14"

Copy link
Member

@TcM1911 TcM1911 left a comment

Choose a reason for hiding this comment

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

Thank you for contributing.

Makefile Outdated Show resolved Hide resolved
Makefile Show resolved Hide resolved
revert setting ARCH parameter, delete $(ARCH) from DARWIN_GO_ENV
@ApexHack3r ApexHack3r requested a review from TcM1911 April 9, 2023 17:35
@TcM1911 TcM1911 merged commit 0650fda into goretk:master Apr 14, 2023
@ApexHack3r ApexHack3r deleted the macos-multiarch-fat-mach-o-dylib branch April 14, 2023 21:56
@ApexHack3r
Copy link
Contributor Author

@TcM1911 - Hi, just checking by to see if this will be released as a patch version?

Would be cool if PyGoRE would be updated as well so users can enjoy this library on their Apple silicon MBPs with a simple pip install 😁.

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.

None yet

2 participants