Bug 456242 - Review Request: gavl - A library for handling uncompressed audio and video data
Summary: Review Request: gavl - A library for handling uncompressed audio and video data
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Gwyn Ciesla
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On: 454888
Blocks: 456256 RussianFedoraRemix
TreeView+ depends on / blocked
 
Reported: 2008-07-22 13:39 UTC by Nicolas Chauvet (kwizart)
Modified: 2009-05-02 04:21 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2009-04-10 10:33:31 UTC
Type: ---
Embargoed:
gwync: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Nicolas Chauvet (kwizart) 2008-07-22 13:39:20 UTC
Spec URL:
http://kwizart.fedorapeople.org/SPECS/gavl.spechttp://kwizart.fedorapeople.org/SPECS/gavl.spec
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

Comment 1 Nicolas Chauvet (kwizart) 2008-07-22 13:41:56 UTC
Spec URL:
http://kwizart.fedorapeople.org/SPECS/gavl.spec


Comment 2 Nicolas Chauvet (kwizart) 2008-07-29 14:02:08 UTC
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

Comment 3 Gwyn Ciesla 2008-10-10 16:56:16 UTC
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.

Comment 4 Zarko (grof) 2009-03-19 07:55:36 UTC
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

Comment 5 Nicolas Chauvet (kwizart) 2009-03-24 02:08:41 UTC
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.

Comment 6 Gwyn Ciesla 2009-03-24 13:00:27 UTC
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?

Comment 7 Gwyn Ciesla 2009-03-31 13:30:04 UTC
Mock build good.  Does the explanation from 456256 apply here WRT ldconfig?

Comment 8 Gwyn Ciesla 2009-04-07 14:59:04 UTC
Ping?

Comment 9 Nicolas Chauvet (kwizart) 2009-04-07 15:15:46 UTC
(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.

Comment 10 Gwyn Ciesla 2009-04-07 15:25:07 UTC
Ok, then.

APPROVED.

Comment 11 Nicolas Chauvet (kwizart) 2009-04-07 15:32:21 UTC
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

Comment 12 Kevin Fenzi 2009-04-08 02:31:54 UTC
cvs done.

Comment 13 Gwyn Ciesla 2009-04-09 15:15:20 UTC
Imported and built for rawhide and F-10.  Going to build for F-9?

Comment 14 Nicolas Chauvet (kwizart) 2009-04-09 15:42:21 UTC
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.

Comment 15 Gwyn Ciesla 2009-04-09 15:47:04 UTC
Gotcha.  The same thing with gavl is blocking my rawhide mock build for frei0r-plugins. :)

Comment 16 Zarko (grof) 2009-04-09 20:29:19 UTC
(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...??!!

Comment 17 Nicolas Chauvet (kwizart) 2009-04-09 23:26:57 UTC
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.

Comment 18 Zarko (grof) 2009-04-10 05:43:38 UTC
(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

Comment 19 Nicolas Chauvet (kwizart) 2009-04-10 10:33:31 UTC
packages built and requested to updates-testing for Fedora stable releases.
Thx for the reviews


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