Bug 1392917 - Review Request: libvdpau-va-gl - VDPAU driver with OpenGL/VAAPI back-end
Summary: Review Request: libvdpau-va-gl - VDPAU driver with OpenGL/VAAPI back-end
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Igor Gnatenko
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-11-08 13:30 UTC by Nicolas Chauvet (kwizart)
Modified: 2017-03-13 21:28 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2017-03-13 21:28:24 UTC
Type: ---
ignatenko: fedora-review+


Attachments (Terms of Use)

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?


Note You need to log in before you can comment on or make changes to this bug.