Skip to content

Conversation

@AkashiSN
Copy link
Contributor

build test

  • add AMF support
  • add vaapi support
  • With the addition of vaapi support, Python3 is now required for Linux.
  • Downloading failed on some sources, so we changed the download destination to GitHub or GitLab.
  • I added autotools because I needed them for that.

Best regards.

@AkashiSN
Copy link
Contributor Author

I found a case where it doesn't work properly with vaapi, so I'll revert to the draft until it's fixed.

@AkashiSN AkashiSN marked this pull request as draft October 25, 2020 11:51
@markus-perl
Copy link
Owner

Vaapi has a few odd dependencies like python and I have no AMD graphics card to fully test the final build. That's why I haven't integrated it yet. But if all checks pass and the build works on your machine, I see no problems merging the changes into the script.

@AkashiSN
Copy link
Contributor Author

AkashiSN commented Oct 26, 2020

To be precise, Python is required because meson, the build tool for building libdrm, a dependency of vaapi's library, libva, is written in Python. So, if you have a policy of enabling vaapi when libdrm is present in the environment you are building, you will not have a Python dependency.
I don't have an AMD graphics card either, so I can't do a full test. I'll be removing support for AMF for this PR.
I am currently testing and modifying vaapi on actual machines and will open the PR again as soon as we finish.

@markus-perl markus-perl merged commit 30d2961 into markus-perl:master Oct 27, 2020
@AkashiSN
Copy link
Contributor Author

Thank you very much fix my terrible English , and merge it. 🙏

@AkashiSN AkashiSN deleted the vaapi branch October 28, 2020 01:58
@markus-perl
Copy link
Owner

No Problem and thanks for all the work and time you've spent on this topic!

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