Bug 447457 - Review Request: gupnp-av - UPnP-AV is a collection of helpers for building AV upnp apps
Review Request: gupnp-av - UPnP-AV is a collection of helpers for building AV...
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
All Linux
low Severity low
: ---
: ---
Assigned To: Denis Leroy
Fedora Extras Quality Assurance
Depends On: 446639
Blocks: 447456 454668
  Show dependency treegraph
Reported: 2008-05-19 19:35 EDT by Peter Robinson
Modified: 2008-09-06 12:09 EDT (History)
3 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2008-09-06 12:09:25 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
denis: fedora‑review+
kevin: fedora‑cvs+

Attachments (Terms of Use)

  None (edit)
Description Peter Robinson 2008-05-19 19:35:59 EDT
SPEC: http://fedora.roving-it.com/rawhide/gupnp-av.spec
SRPM: http://fedora.roving-it.com/rawhide/gupnp-av-0.2-1.fc9.src.rpm

GUPnP is an object-oriented open source framework for creating UPnP
devices and control points, written in C using GObject and libsoup.
The GUPnP API is intended to be easy to use, efficient and flexible.

GUPnP-AV is a collection of helpers for building AV (audio/video)
applications using GUPnP.
Comment 1 Peter Robinson 2008-06-17 05:37:44 EDT
New SRPM. Spec file is the same location

SRPM: http://fedora.roving-it.com/rawhide/gupnp-av-0.2.1-1.fc9.src.rpm
Comment 3 Zeeshan Ali 2008-08-13 18:14:03 EDT
Sorry for being a nitpicker but one small correction to the description: AV upnp apps -> UPnP AV applications.
Comment 4 Denis Leroy 2008-08-29 09:27:19 EDT
Some items before the format review:

- URL source link is incorrect (s/gupnp/gupnp-av/)

- Same mistake in devel package summary

- %doc statement not necessary for gtk-doc dir (as with pangomm)

- Redundant entries in the %files section. The %dir entries are not strictly necessary, you could simply specify:


this is pretty minor, almost a matter of style or preference.
Comment 5 Peter Robinson 2008-08-29 09:44:14 EDT
Fixed. New spec at same spot, new srpm here

Comment 6 Peter Robinson 2008-08-30 09:19:58 EDT
gupnp that this depends on is now built and should hit rawhide tomorrow. Along with the srpm above all should be good to go now. I tested a scratch build in koji just now http://koji.fedoraproject.org/koji/taskinfo?taskID=795141
Comment 7 Peter Robinson 2008-09-03 10:00:04 EDT
Hi Denis,

With rawhide on the move again this now builds in koji. 

Comment 8 Denis Leroy 2008-09-04 03:41:27 EDT
- rpmlint clean
- name ok
- license ok (LGPLv2+)
- source md5sum ok
- deps ok
- dir ownership ok
- devel package split ok
- test programs compile and work ok

Package is APPROVED.
Comment 9 Peter Robinson 2008-09-04 05:21:47 EDT
New Package CVS Request
Package Name: gupnp-av
Short Description: GUPnP-AV is a collection of helpers for building UPnP AV applications
Owners: pbrobinson
Branches: F-8 F-9
InitialCC: pbrobinson
Cvsextras Commits: yes
Comment 10 Kevin Fenzi 2008-09-05 13:02:43 EDT
cvs done.
Comment 11 Peter Robinson 2008-09-06 12:09:25 EDT
Built and in rawhide. Thanks for your help Denis.

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