Bug 1392917

Summary: Review Request: libvdpau-va-gl - VDPAU driver with OpenGL/VAAPI back-end
Product: [Fedora] Fedora Reporter: Nicolas Chauvet (kwizart) <kwizart>
Component: Package ReviewAssignee: Igor Gnatenko <ignatenko>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: kwizart, package-review
Target Milestone: ---Flags: ignatenko: fedora-review+
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2017-03-13 21:28:24 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description Nicolas Chauvet (kwizart) 2016-11-08 13:30:39 UTC
Spec URL: http://dl.kwizart.net/review/libvdpau-va.spec
SRPM URL: http://dl.kwizart.net/review/libvdpau-va-gl-0.4.2-2.fc26.src.rpm
Description: VDPAU driver with OpenGL/VAAPI back-end
Fedora Account System Username: kwizart

Comment 1 Nicolas Chauvet (kwizart) 2016-11-08 13:44:29 UTC
koji scratch build for f26:
http://koji.fedoraproject.org/koji/taskinfo?taskID=16352969

rpmlint is clean

This package was previously in a 3rd part repository, but since it has dropped a dependency on a component not allowed in Fedora, it can now be submitted here.

This package is a bridge between a vdpau player and vaapi backend driver.
There are currently several native vaapi backend:
- radeonsi_drv_video , r600_drv_video, nouveau_drv_video (which last requires an external firmware). Thoses are from the mesa-dri-drivers package. But theses also have a native vdpau backend.
- libva-intel-driver which is not in fedora because it may implement forbidden items in software.

Comment 2 Igor Gnatenko 2016-11-18 17:52:51 UTC
link to spec gives 404. proper link is: http://dl.kwizart.net/review/libvdpau-va-gl.spec

Comment 3 Igor Gnatenko 2016-11-18 17:57:08 UTC
* Missing BuildRequires: gcc-c++
* Missing BuildRequires: make
* would be nice to add `-print` to find invocation to see what actually it deletes (actually it doesn't delete anything, so can be dropped)
* why do you exclude unversioned .so?

Comment 4 Nicolas Chauvet (kwizart) 2017-01-17 14:40:46 UTC
Spec URL: http://dl.kwizart.net/review/libvdpau-va-gl.spec
SRPM URL: http://dl.kwizart.net/review/libvdpau-va-gl-0.4.2-3.fc26.src.rpm
Description: VDPAU driver with OpenGL/VAAPI back-end

Changelog:
- Drop ExclusiveArch
- Add comment about why to drop un-versioned symlink
- Add missing BR

koji scratch build for f26:
https://koji.fedoraproject.org/koji/taskinfo?taskID=17312520

Sorry for the lack of answer. So far Makefile is still the default buildroot.
I've added gcc-c++

Comment 5 Igor Gnatenko 2017-03-03 10:09:32 UTC
> * would be nice to add `-print` to find invocation to see what actually it deletes (actually it doesn't delete anything, so can be dropped)

This is still not fixed, but not a blocker.

Comment 6 Gwyn Ciesla 2017-03-06 14:19:47 UTC
Package request has been approved: https://admin.fedoraproject.org/pkgdb/package/rpms/libvdpau-va-gl

Comment 7 Igor Gnatenko 2017-03-11 19:33:11 UTC
ping?