-
Notifications
You must be signed in to change notification settings - Fork 69
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
Update FFmpeg to 4.4.1 and add libaom, librist & svt-av1 deps for Windows #86
Conversation
For the Nvidia headers:
This bumps it from the previous minimum of 418.81 (Feb 2019) or newer |
Wanted to ask about this. Does this exclude any graphics cards? If so, which ones? |
caf78ee
to
c09c8e0
Compare
i'm reverting to nvidia sdk 9 since we haven't found any info about which devices are supported by the SDK 11 and the relevant drivers. |
c09c8e0
to
0dd1109
Compare
As far as I can tell, NVIDIA dropped Fermi support in April 2018. It seems that the R390 driver series was the last to support Fermi, with R410 supporting only Kepler and newer. R470 was the last driver series to support Kepler. If I'm understanding this correctly, R410 through R470 should support the same hardware, with Kepler remaining as the oldest supported. Once the requirement moves beyond R470, we would be dropping Kepler support, which should only happen on a major release and should be coordinated and communicated ahead of time. As an aside, nv-codec-headers tag n11.1.5.0 says:
While tag n11.0.10.1 says:
I'm guessing the former is a slight typo, and it meant to say "Corresponds to Video Codec SDK version 11.1.5.". Indeed, "Video Codec SDK 11.1 is supported on R470 and above drivers on Windows and Linux". |
3750a8c
to
69b2849
Compare
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.
I've requested some changes. I've also left some "thinking aloud" comments possibly to address at a later time, but I wanted to write down to better commit them to memory.
57eb67b
to
a71f9c5
Compare
481979f
to
f5fa58d
Compare
3662cf5
to
a9474af
Compare
So I've been checking the configuration scripts for Required adding additional lines to the patches for https://gist.github.com/PatTheMav/5708ff44c40ff8591cde3b7f2fe8e24d |
So I did a cross compile run today with my patch applied and checked the DLLs - looks like no stray mingw DLLs being linked to anymore: https://gist.github.com/PatTheMav/0d399fba9d059ca6d8cd1db0d9091f78 |
Update (28 dec 2021):
|
30e125e
to
7cdd8c2
Compare
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.
The first commit message subject should be: "windows: Update FFmpeg to 4.4.1 and add aom, librist, svt-av1". Please fix the bulleted list in that commit message.
The third commit message should mention that it's on Linux when setting up the cross-compile tools. It doesn't actually take place on Windows. If you include the changes in win-install-tools.sh, it doesn't need the CI or windows prefix.
All other review comments are nitpicks or clarifying questions which hopefully will not result in major changes.
cfc8b00
to
935d1c1
Compare
New push after removing the cherry picks from FFmpeg. |
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.
Checking over this again. Hopefully my last review, and anything else can be left to follow-ups.
935d1c1
to
d0c3599
Compare
I checked on windows 10 x64 with obs master that:
|
d0c3599
to
c6c4156
Compare
For Windows obs-deps: - Add librist, libaom (x64 only), and SVT-AV1 (x64 only) to FFmpeg - Update FFmpeg from 4.2.4 to 4.4.1 - Use a shim instead of winpthreads for libaom and libvpx (PatTheMav) Co-authored-by: pkv <pkv@obsproject.com> Co-authored-by: PatTheMav <PatTheMav@users.noreply.github.com> Co-authored-by: WizardCM <git@wizardcm.com>
Allows cherry picking in the future and aligns implementation with Windows. Co-authored-by: PatTheMav <PatTheMav@users.noreply.github.com> Co-authored-by: pkv <pkv@obsproject.com>
@pkviet |
It is required by svt-av1 according to its build instructions. |
So i checked the CMakeLists.txt for svt-av1 and nasm can be used but it requires a cmake flag to be set. So yasm is their recommended choice on default |
Description
For windows obs-deps, this:
For mac FFmpeg dep, cherry picks are added and instead od downloading an archive one uses git.
Motivation and Context
Update FFmpeg dll.
How Has This Been Tested?
This was tested by running the cross-compile script for windows deps.
Runs successfully and creates an archive.
I haven't tested if x86 builds correctly. I'll let CI check it for us.
Types of changes
Checklist: