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

Apple silicon plus x264 commit #71

Merged

Conversation

rtgoodwin
Copy link
Contributor

Same changes as apple_silicon branch but switched to the x264 commit to see if it passes PR testing.

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
* Define M1 variable once
* Grab x264 archive vs git
* configure vs Configure 
* bonus: output binary type. Feel free to take or leave :)
branched from "apple_silicon" branch.
@markus-perl
Copy link
Owner

Great, all checks have passed. I create a new pre-release of the script tomorrow, which will include your patches and the new additional library libsvtav1. Afterward, It would be good if you can verify that ffmpeg still builds on an m1 mac.

@rtgoodwin
Copy link
Contributor Author

rtgoodwin commented Jan 20, 2021 via email

@markus-perl markus-perl merged commit cd5c294 into markus-perl:master Jan 20, 2021
@markus-perl
Copy link
Owner

I've merged all outstanding changes into the master branch. Tests passed :-)
Can you check out the master and make sure that everything still works on the M1?
Unfortunately, Github Actions do not yet support this environment: actions/runner-images#2187

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