-
Notifications
You must be signed in to change notification settings - Fork 337
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
Random patches and tricks for building on M1 #69
Conversation
Needs newer code in some cases, and some conditionals to check platform. The external patches possibly should be brought in? Seems to work for now if anyone is in need. Learnings from https://github.com/ssut/ffmpeg-on-apple-silicon
@rtgoodwin, thank you for your work. I do not own an M1 Mac, so I'm currently unable to test the build process. But with some small improvements, I can merge the pull request into the main branch, as all other systems seem to build without any problems. Things I would improve:
Best, |
* Define M1 variable once * Grab x264 archive vs git * configure vs Configure * bonus: output binary type. Feel free to take or leave :)
Tail of build with current script commit. (Kind of makes me want to add logging 🤔 😄 )
|
Hmm.. either tests are queued/borked (still haven't completed), or my addition of using "file" causes issues. Definitely optional if so. Edit: All completed fine! Will do a separate PR for the x264 commit to see if it passes all platforms. |
Looks like #71 (this branch plus using the tagged x264 commit) is fine on all platforms. Thanks as always! 😄 |
Preview only
I suspect several things in here are questionable, open to ideas/better ways. But it builds! (On Big Sur, on my machine, etc...)
Notes
Learnings from https://github.com/ssut/ffmpeg-on-apple-silicon
Output analysis