-
Notifications
You must be signed in to change notification settings - Fork 123
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 macOS on Apple Silicon #164
base: master
Are you sure you want to change the base?
Conversation
…o can locate them
@skidder 👋 A gentle ping. If I can do anything to make this easier to review and incorporate, let me know. |
hi, any updates on this PR? |
|
||
# TODO: Fix this in the upstream deps repo | ||
cd "$SRCDIR" | ||
wget -q -O giflib-5.2.2.tar.gz https://sourceforge.net/projects/giflib/files/giflib-5.2.2.tar.gz/download |
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.
seems like 5.2.2 has been added in the upstream deps repo already and that discord has added a patch for giflib as well
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.
@remiolivier-discord I noticed you updated the deps repo, could you also update the build scripts in the main branch for lilliput? 🙏
As discussed in #157, this PR contributes the following:
I tried to break things up into small commits for easier review. The last ~5 commits are the key changes -- adding arm64 support, adding CI, and rebuilding the dependencies. The rest are fixes for the build.
If you'd like me to make a separate PR for the CI stuff, I'm happy to do that. There's a test run of the main CI workflow here and one for the dependency workflow here.
Happy to talk through any questions or concerns. Thanks!
Resolves #157. Resolves #138. Resolves #139.