Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

add vaapi support #62

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

Merged
markus-perl merged 28 commits into markus-perl:master from AkashiSN:vaapi
Oct 27, 2020
Merged

add vaapi support #62

markus-perl merged 28 commits into markus-perl:master from AkashiSN:vaapi
Oct 27, 2020

Conversation

@AkashiSN
Copy link
Contributor

@AkashiSN AkashiSN commented Oct 25, 2020

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.

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
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 reacted with thumbs up emoji

Copy link
Contributor Author

AkashiSN commented Oct 26, 2020
edited
Loading

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
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
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

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /