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 ARM64 MacOS #854

Open
wants to merge 12 commits into
base: develop/2.4.0
Choose a base branch
from

Conversation

FazanaJ
Copy link
Collaborator

@FazanaJ FazanaJ commented Jan 6, 2025

Simply added the measures to build on Mac. Have tested on both WSL and MacOS, both are building.

Flips has been stubbed out for Mac because I'm too stupid to figure out how to finagle make into doing my bidding. I don't consider this a blocking issue honestly.

@gheskett gheskett changed the base branch from master to develop/2.4.0 January 6, 2025 15:00
Copy link
Collaborator

@gheskett gheskett left a comment

Choose a reason for hiding this comment

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

Repointed to 2.4, please fix merge conflict

@gheskett gheskett added the enhancement New feature or request label Jan 6, 2025
@gheskett gheskett added this to the 2.4 milestone Jan 6, 2025
@FazanaJ
Copy link
Collaborator Author

FazanaJ commented Jan 6, 2025

Donezo

@FazanaJ
Copy link
Collaborator Author

FazanaJ commented Jan 6, 2025

the lz4 tool errors out on mac, so I'll fix that in the meantime, too

Copy link
Collaborator

@gheskett gheskett left a comment

Choose a reason for hiding this comment

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

Wait actually, can you throw a more meaningful error at make patch for MAC?

@FazanaJ
Copy link
Collaborator Author

FazanaJ commented Jan 6, 2025

Addresses issued for both. lz4t no longer errors and make patch throws an assert

@FazanaJ FazanaJ requested a review from gheskett January 6, 2025 15:28
Makefile Outdated Show resolved Hide resolved
Copy link
Collaborator

@gheskett gheskett left a comment

Choose a reason for hiding this comment

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

Have you confirmed for sure make patch works as intended on both operating systems?

@FazanaJ
Copy link
Collaborator Author

FazanaJ commented Jan 6, 2025

Yeah, patches on Linux, tells me to go get a better computer on mac

Makefile Outdated Show resolved Hide resolved
Copy link
Collaborator

@gheskett gheskett left a comment

Choose a reason for hiding this comment

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

Very cool

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Needs Review
Development

Successfully merging this pull request may close these issues.

2 participants