Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
https://bugzilla.redhat.com/show_bug.cgi?id=456242
Summary: Review Request: gavl - A library for handling uncompressed audio and video data Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: kwizart@gmail.com QAContact: extras-qa@fedoraproject.org CC: fedora-package-review@redhat.com,notting@redhat.com
Spec URL: http://kwizart.fedorapeople.org/SPECS/gavl.spechttp://kwizart.fedorapeople.o... SRPM URL: http://kwizart.fedorapeople.org/SRPMS/gavl-1.0.0-2.fc8.kwizart.src.rpm Description: A library for handling uncompressed audio and video data
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: gavl - A library for handling uncompressed audio and video data
https://bugzilla.redhat.com/show_bug.cgi?id=456242
kwizart@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- BugsThisDependsOn| |454888
------- Additional Comments From kwizart@gmail.com 2008-07-22 09:41 EST ------- Spec URL: http://kwizart.fedorapeople.org/SPECS/gavl.spec
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: gavl - A library for handling uncompressed audio and video data
https://bugzilla.redhat.com/show_bug.cgi?id=456242
kwizart@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- OtherBugsDependingO| |456256 nThis| |
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: gavl - A library for handling uncompressed audio and video data
https://bugzilla.redhat.com/show_bug.cgi?id=456242
------- Additional Comments From kwizart@gmail.com 2008-07-29 10:02 EST ------- Spec URL: http://kwizart.fedorapeople.org/SPECS/gavl.spec SRPM URL: http://kwizart.fedorapeople.org/SRPMS/gavl-1.0.1-1.fc8.kwizart.src.rpm Description: A library for handling uncompressed audio and video data
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=456242
Jon Ciesla limb@jcomserv.net changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED CC| |limb@jcomserv.net AssignedTo|nobody@fedoraproject.org |limb@jcomserv.net Flag| |fedora-review?
--- Comment #3 from Jon Ciesla limb@jcomserv.net 2008-10-10 12:56:16 EDT --- rpmlint: not even crickets.
You have ldconfig in %post and %postun, do you not need them for -devel as well?
COPYING is GPLv3. Code all says GPLv2+. Web site doesn't really specify. ?? Needs clarification.
Using a patched bundled library, which is documented. OK.
Looks great otherwise, full review very nearly done. Just need libgdither to be in rawhide so I can mock-test the BuildRequires.
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=456242
Zarko zarko.pintar@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |zarko.pintar@gmail.com
--- Comment #4 from Zarko zarko.pintar@gmail.com 2009-03-19 03:55:36 EDT --- Hello,
What happened with this packet?
I want prepare frei0r-plugins for Fedora but I need this packet as dependency.
I prepare this packet for Fedora 10 so please, if you want to see my spec and src:
http://dl.getdropbox.com/u/728801/kdenlive/sources/gavl.spec
http://dl.getdropbox.com/u/728801/kdenlive/sources/gavl-1.1.0-1.fc10.src.rpm
Please see them.
kind regards, Zarko
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=456242
--- Comment #5 from Nicolas Chauvet (kwizart) kwizart@gmail.com 2009-03-23 22:08:41 EDT --- Spec URL: http://kwizart.fedorapeople.org/SPECS/gavl.spec SRPM URL: http://kwizart.fedorapeople.org/SRPMS/gavl-1.1.0-1.fc10.src.rpm Description: A library for handling uncompressed audio and video data
Changelog: - Update to 1.1.0 - Disable buildtime CPU detection.
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=456242
--- Comment #6 from Jon Ciesla limb@jcomserv.net 2009-03-24 09:00:27 EDT --- rpmlint still silent.
Code says GPLv2+, COPYING is GPLv3+, tag is GPLv3+. OK.
Still awaiting libgdither in rawhide for mock build.
What about ldconfig in %post and %postun for -devel?
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=456242
Bug 456242 depends on bug 454888, which changed state.
Bug 454888 Summary: Review Request: libgdither - Library for applying dithering to PCM audio sources https://bugzilla.redhat.com/show_bug.cgi?id=454888
What |Old Value |New Value ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution| |NEXTRELEASE
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=456242
--- Comment #7 from Jon Ciesla limb@jcomserv.net 2009-03-31 09:30:04 EDT --- Mock build good. Does the explanation from 456256 apply here WRT ldconfig?
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=456242
--- Comment #8 from Jon Ciesla limb@jcomserv.net 2009-04-07 10:59:04 EDT --- Ping?
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=456242
--- Comment #9 from Nicolas Chauvet (kwizart) kwizart@gmail.com 2009-04-07 11:15:46 EDT --- (In reply to comment #7)
Mock build good. Does the explanation from 456256 apply here WRT ldconfig?
Sorry, I tought to have already answeared. But yes, no needs for ldconfig call for -devel.
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=456242
Jon Ciesla limb@jcomserv.net changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-review? |fedora-review+
--- Comment #10 from Jon Ciesla limb@jcomserv.net 2009-04-07 11:25:07 EDT --- Ok, then.
APPROVED.
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=456242
Nicolas Chauvet (kwizart) kwizart@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-cvs?
--- Comment #11 from Nicolas Chauvet (kwizart) kwizart@gmail.com 2009-04-07 11:32:21 EDT --- New Package CVS Request ======================= Package Name: gavl Short Description: A library for handling uncompressed audio and video data Owners: kwizart Branches: F-10 F-9 Cvsextras Commits: yes
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=456242
Kevin Fenzi kevin@tummy.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs? |fedora-cvs+
--- Comment #12 from Kevin Fenzi kevin@tummy.com 2009-04-07 22:31:54 EDT --- cvs done.
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=456242
--- Comment #13 from Jon Ciesla limb@jcomserv.net 2009-04-09 11:15:20 EDT --- Imported and built for rawhide and F-10. Going to build for F-9?
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=456242
--- Comment #14 from Nicolas Chauvet (kwizart) kwizart@gmail.com 2009-04-09 11:42:21 EDT --- build failed on F-10 because libgdither wasn't within the buildroot. I hope it will be fixed on next push. I will close the bug once F-9/F-10 gavl got built.
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=456242
--- Comment #15 from Jon Ciesla limb@jcomserv.net 2009-04-09 11:47:04 EDT --- Gotcha. The same thing with gavl is blocking my rawhide mock build for frei0r-plugins. :)
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=456242
--- Comment #16 from Zarko zarko.pintar@gmail.com 2009-04-09 16:29:19 EDT --- (In reply to comment #14)
build failed on F-10 because libgdither wasn't within the buildroot. I hope it will be fixed on next push. I will close the bug once F-9/F-10 gavl got built.
I do not understand well what was happening here...
I succesfully build libgdither, gavl and frei0r-plugins with mock and put all of these packages on my private repo...??!!
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=456242
--- Comment #17 from Nicolas Chauvet (kwizart) kwizart@gmail.com 2009-04-09 19:26:57 EDT --- newly built packages don't get automatically available to others build in stable Fedora branches. They need to be either requested to release engineer, either pushed to stable release. As libgdither is still in updates-testing... gavl cannot build on F-10/F-9. But it was released for rawhide. So other review can continue.
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=456242
--- Comment #18 from Zarko zarko.pintar@gmail.com 2009-04-10 01:43:38 EDT --- (In reply to comment #17)
newly built packages don't get automatically available to others build in stable Fedora branches. They need to be either requested to release engineer, either pushed to stable release.
Ah, now it is clear.... :)
Thx, Zarko
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=456242
Nicolas Chauvet (kwizart) kwizart@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution| |NEXTRELEASE
--- Comment #19 from Nicolas Chauvet (kwizart) kwizart@gmail.com 2009-04-10 06:33:31 EDT --- packages built and requested to updates-testing for Fedora stable releases. Thx for the reviews
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=456242
Peter Lemenkov lemenkov@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Blocks| |496433(RussianFedora)
package-review@lists.fedoraproject.org