[Bug 771254] Review Request: libva-vdpau-driver - HW video decode support for VDPAU platforms
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=771254 Nicolas Chauvet (kwizart) changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution|--- |NEXTRELEASE Last Closed||2013-03-28 12:06:03 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=IK4MNGeaRV&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 771254] Review Request: libva-vdpau-driver - HW video decode support for VDPAU platforms
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=771254 --- Comment #23 from Jon Ciesla --- Git done (by process-git-requests). -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=Qw4z19OXqn&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 771254] Review Request: libva-vdpau-driver - HW video decode support for VDPAU platforms
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=771254 Nicolas Chauvet (kwizart) changed: What|Removed |Added Flags|fedora-cvs+ | Flags||fedora-cvs? --- Comment #22 from Nicolas Chauvet (kwizart) --- Package Change Request == Package Name: libva-vdpau-driver New Branches: f19 Owners: kwizart InitialCC: Sorry, I should have requested a f19 branch also -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=bMvxMNDIlv&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 771254] Review Request: libva-vdpau-driver - HW video decode support for VDPAU platforms
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=771254 --- Comment #21 from Jon Ciesla --- Git done (by process-git-requests). -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=p4m9qllbvK&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 771254] Review Request: libva-vdpau-driver - HW video decode support for VDPAU platforms
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=771254 Nicolas Chauvet (kwizart) changed: What|Removed |Added Flags||fedora-cvs? --- Comment #20 from Nicolas Chauvet (kwizart) --- New Package SCM Request === Package Name: libva-vdpau-driver Short Description: HW video decode support for VDPAU platforms Owners: kwizart Branches: el6 f18 f17 InitialCC: -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=wcgMJoSQhN&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 771254] Review Request: libva-vdpau-driver - HW video decode support for VDPAU platforms
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=771254 Nicolas Chauvet (kwizart) changed: What|Removed |Added Flags|needinfo?(kwiz...@gmail.com | |) | --- Comment #19 from Nicolas Chauvet (kwizart) --- Sorry, yes I will add Requires: mesa-dri-filesystem -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=nRellUI2Rf&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 771254] Review Request: libva-vdpau-driver - HW video decode support for VDPAU platforms
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=771254 Mamoru TASAKA changed: What|Removed |Added Flags||needinfo?(kwiz...@gmail.com ||) --- Comment #18 from Mamoru TASAKA --- ping? -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=s9utI0x1oK&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 771254] Review Request: libva-vdpau-driver - HW video decode support for VDPAU platforms
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=771254 Mamoru TASAKA changed: What|Removed |Added Flags|fedora-review? | Flags||fedora-review+ --- Comment #17 from Mamoru TASAKA --- For 0.7.4-3: * Directory ownership - It seems that it is better that this rpm has "Requires: mesa-dri-filesystem" for the ownership of %_libdir/dri . Please fix the above issue before importing this into git. - This package (libva-vdpau-driver) is APPROVED by mtasaka - -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=B8yAgMK104&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 771254] Review Request: libva-vdpau-driver - HW video decode support for VDPAU platforms
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=771254 Nicolas Chauvet (kwizart) changed: What|Removed |Added Flags|needinfo?(kwiz...@gmail.com | |) | --- Comment #16 from Nicolas Chauvet (kwizart) --- Spec URL: http://kwizart.fedorapeople.org/review/libva-vdpau-driver.spec SRPM URL: http://kwizart.fedorapeople.org/review/libva-vdpau-driver-0.7.4-3.fc19.src.rpm Description: HW video decode support for VDPAU platforms Changelog: - Add --disable-silent-rules - Clean-up spec -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=b4s4ga6Zmc&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 771254] Review Request: libva-vdpau-driver - HW video decode support for VDPAU platforms
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=771254 Mamoru TASAKA changed: What|Removed |Added Flags||needinfo?(kwiz...@gmail.com ||) --- Comment #15 from Mamoru TASAKA --- Again ping? -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=P1DplMbchw&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 771254] Review Request: libva-vdpau-driver - HW video decode support for VDPAU platforms
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=771254 --- Comment #14 from Mamoru TASAKA --- ping again? -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=CZ5AyAvuGE&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 771254] Review Request: libva-vdpau-driver - HW video decode support for VDPAU platforms
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=771254 --- Comment #13 from Mamoru TASAKA --- Okay, thank you for response. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=MjphVjmeVN&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 771254] Review Request: libva-vdpau-driver - HW video decode support for VDPAU platforms
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=771254 Nicolas Chauvet (kwizart) changed: What|Removed |Added Flags|needinfo?(kwiz...@gmail.com | |) | --- Comment #12 from Nicolas Chauvet (kwizart) --- I will probably have more time this week. (I will do your review) -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=DLCKVOtO6x&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 771254] Review Request: libva-vdpau-driver - HW video decode support for VDPAU platforms
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=771254 Mamoru TASAKA changed: What|Removed |Added Flags||needinfo?(kwiz...@gmail.com ||) -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=GoEuhEqvNY&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 771254] Review Request: libva-vdpau-driver - HW video decode support for VDPAU platforms
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=771254 --- Comment #11 from Mamoru TASAKA --- ping? -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=digg2RoGyN&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 771254] Review Request: libva-vdpau-driver - HW video decode support for VDPAU platforms
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=771254
--- Comment #10 from Mamoru TASAKA ---
For 0.7.4-2:
* Make build.log verbose
- Currently build.log just shows
-
make[2]: Entering directory
`/builddir/build/BUILD/libva-vdpau-driver-0.7.4/src'
CC debug.lo
CC object_heap.lo
CC uasyncqueue.lo
CC ulist.lo
CC uqueue.lo
CC utils.lo
-
and I cannot test some more detailed issues, for example
if Fedora specific compilation flags are passed correctly
or not.
Please add "--disable-silent-rules" to %configure
! No longer needed items (not blockers)
- The following items are no longer needed, unless
you want to import this also on EPEL5.
* BuildRoot: line
* "rm -rf %{buildroot}" at the top of %install
* %clean section
* %defattr
! Note
I don't know how to check if this driver works or not,
so I just check packaging issues on this rpm.
--
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug
https://bugzilla.redhat.com/token.cgi?t=TmUwgGlOyJ&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 771254] Review Request: libva-vdpau-driver - HW video decode support for VDPAU platforms
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=771254 Mamoru TASAKA changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|nob...@fedoraproject.org|mtas...@fedoraproject.org Flags||fedora-review? --- Comment #9 from Mamoru TASAKA --- Taking. I would appreciate it if you would review my review request bug 902024 . -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=07qOPPhiKz&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 771254] Review Request: libva-vdpau-driver - HW video decode support for VDPAU platforms
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=771254 --- Comment #8 from Nicolas Chauvet (kwizart) --- Spec URL: http://kwizart.fedorapeople.org/review/libva-vdpau-driver.spec SRPM URL: http://kwizart.fedorapeople.org/review/libva-vdpau-driver-0.7.4-2.fc18.src.rpm Description: HW video decode support for VDPAU platforms Changelog: - Fix build with recent mesa The patch was sent upstream (no answear yet). koji scratch build for f18 http://koji.fedoraproject.org/koji/taskinfo?taskID=4874068 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=vxQfFWKUR4&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 771254] Review Request: libva-vdpau-driver - HW video decode support for VDPAU platforms
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=771254 --- Comment #7 from Mamoru TASAKA --- Nicolas, would you try if gentoo patch works? If build gets successful, I may want to review this ticket. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=d6M0HZzwL5&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 771254] Review Request: libva-vdpau-driver - HW video decode support for VDPAU platforms
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=771254 --- Comment #6 from Mamoru TASAKA --- By the way, at least please make build.log more verbose so that we can check easily if Fedora specific compilation flags are honored correctly (just checked build.log only, not checked spec file or so yet) -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=hCZwVSe9wL&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 771254] Review Request: libva-vdpau-driver - HW video decode support for VDPAU platforms
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=771254 Mamoru TASAKA changed: What|Removed |Added CC||mtas...@fedoraproject.org --- Comment #5 from Mamoru TASAKA --- (In reply to comment #4) > Sounds like the package fails with recent mesa. > I will probably ask upstream about a patch similar to this: > http://gitorious.org/vaapi/gstreamer-vaapi/commit/ > 5db2e93d3c930fd8470cfdcd26f514d1cebc59a0/diffs Gentoo seems to have applied similar patch to their package: https://bugs.gentoo.org/show_bug.cgi?id=438180 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=goqKoxXh2B&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 771254] Review Request: libva-vdpau-driver - HW video decode support for VDPAU platforms
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=771254 --- Comment #4 from Nicolas Chauvet (kwizart) --- Sounds like the package fails with recent mesa. I will probably ask upstream about a patch similar to this: http://gitorious.org/vaapi/gstreamer-vaapi/commit/5db2e93d3c930fd8470cfdcd26f514d1cebc59a0/diffs -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=1RM58rvxaB&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 771254] Review Request: libva-vdpau-driver - HW video decode support for VDPAU platforms
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=771254 --- Comment #3 from Nicolas Chauvet (kwizart) --- Spec URL: http://kwizart.fedorapeople.org/review/libva-vdpau-driver.spec SRPM URL: http://kwizart.fedorapeople.org/review/libva-vdpau-driver-0.7.4-1.fc19.src.rpm Description: HW video decode support for VDPAU platforms Changelog: Update to 0.7.4 Koji scratch build for rawhide: http://koji.fedoraproject.org/koji/taskinfo?taskID=4833120 rpmlint is clean -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=Q0jWpj3fGD&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 771254] Review Request: libva-vdpau-driver - HW video decode support for VDPAU platforms
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=771254 --- Comment #2 from Nicolas Chauvet (kwizart) 2012-01-15 17:48:16 EST --- Hello Mattia and thanks for your interest in this review. This is a good remark, but actually this is a special case here. This library is not intended to be used 'system wide' but only from a dedicated library (libva), which last is installed with ldconfig. So libva is expected to 'dlopen' the right backend %name_drv_video.so from the special directory ( %_libdir/dri ) given the current driver used by the Xorg server. For the same reason, this is usually expected to have the shared object un-versioned in this case. So to sum-up, you can consider this library as a module or a plugin like the one provided by mesa-dri-drivers from the mesa case. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 771254] Review Request: libva-vdpau-driver - HW video decode support for VDPAU platforms
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=771254 Mattia Verga changed: What|Removed |Added CC||mattia.ve...@tiscali.it --- Comment #1 from Mattia Verga 2012-01-15 11:08:30 EST --- Seems to miss %post and %postun sections required for libraries: https://fedoraproject.org/wiki/Packaging:Guidelines#Shared_Libraries -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
