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
libva: 1.7.3 -> 2.0.0 #32599
libva: 1.7.3 -> 2.0.0 #32599
Conversation
Please check the eval failure |
Messed up the final commit cleanup - thanks for noticing @globin. Sorted now. |
@GrahamcOfBorg eval |
oh damn wrong PR ... i want to merge this :) |
Why merge to master directly when it's auto-tagged by 10.rebuild-linux: 501+? |
@vcunat oh shit! i must be more careful, i'll revert and push to staging. would that be ok? |
Yes. |
This reverts commit 15d3d68. The PR is moved from master to staging.
Moved to staging. |
I see the |
gst_all_1.gst-vaapi too: |
Upstream is recommending carrying both the old and new version - I'll update this to allow for both. |
It seems that mpv was made compatible with libva 2 in mpv-player/mpv@2ecf240 (not released yet) |
And gst-vaapi 1.12.4 should already be compatible: GStreamer/gstreamer-vaapi@0d710ed (https://github.com/GStreamer/gstreamer-vaapi/commits/1.12.4) |
Updated gst-vaapi in b29c78d, this fixed the build. |
Fixed mpv by applying the aforementioned patch in 745be0c. |
So... we don't need both v1 and v2 (i.e. PR #32796) anymore, right? |
I don't know why v1 would be needed.
@peterhoeg Could you post a link? |
|
Thanks! At this point at least free software we package does not need old libva. |
Motivation for this change
The
vaapi-intel
driver is released in accordance withlibva
releases, so this PR sets up the dependency.Additionally, the tools previously part of
libva
are now a separatelibva-utils
.This triggers a lot of rebuilding so I haven't tried it out here yet.
Cc: @garbas
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)