-
Notifications
You must be signed in to change notification settings - Fork 736
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
Intel QSV with ffmpeg #485
Conversation
Thanks! Would it be possible to add it to the other platforms as well? |
Yes, linux can be. |
That's alright, Travis CI will test it :) Thanks |
So it builds fine on Windows, please update for the other platforms! Thanks |
Could you try to add the following lines for the other platforms as well and see what happens? cd ../mfx_dispatch-$MFX_VERSION
autoreconf -fiv
PKG_CONFIG_PATH="../lib/pkgconfig" ./configure --prefix="`pwd`/.." --disable-shared --enable-static --enable-fast-install
make
make install |
I only have windows-x86_64 platform, so I can't try on other platforms. The only modification on windows can't be merged? |
We can build libmfx on other platforms as easily as on Windows. Just add the same kind of build commands as you did for Windows, and then we can check if there are any builds issues here: |
Really? That's a good news. Anyway, I'll test for all platforms and look forward to the result of Travis CI. |
Yes, like that, thanks! It looks like we'll need to resolve the conflict before Travis CI runs your pull request though. |
Unfortunately, it fails on all platforms except Windows :( |
I knew it... Anyway, I think it cannot be helped. |
No, the errors are just normal build errors. We don't need any special
hardware or software...
|
But it looks like it wouldn't do anything on other platforms than linux-x86 and linux-x86_64. So let's get it working on at least those 2? |
I'd like to build
libmfx
into ffmpeg on windows platform.libmfx
enables Intel QSV codec and I've checked this pull request without any errors on windows-x86_64.Can you merge it?