Red Hat Bugzilla – Bug 207685
Review Request: gstreamer-plugins-farsight - GStreamer plug-ins for farsight protocol
Last modified: 2008-04-23 12:20:59 EDT
Spec URL: http://piedmont.homelinux.org/fedora/telepathy/gstreamer-plugins-farsight.spec
SRPM URL: http://piedmont.homelinux.org/fedora/telepathy/gstreamer-plugins-farsight-0.10.2-1.src.rpm
Description: This is a set of plugins for GStreamer that will be used by Farsight for Audio/Video conferencing.
The above urls don't seem to work. Can you give me updated url info for these?
(In reply to comment #1)
> The above urls don't seem to work. Can you give me updated url info for these?
I just retired my web server this week, and haven't got around to obtaining any
new space to host this at.
email them to me and I can put them up for you.
Here we go.
Let me grab a beer and I'll start a mock build and if that passes I'll start a
One thing, is there any way to test the functionality of this plugin currently?
One minor quibble.
Would it make sense to use the majorminor define in the version tag, like you do
later in the file location strings? It reduces the potential for out of sync
strings if/when upstream moves to 0.11
A couple of questions with regard to the configure options
jasper is in Extras currently. Is there a specific problem associated with
Are there other optional plugins to explore?
(In reply to comment #6)
> A couple of questions with regard to the configure options
> jasper is in Extras currently. Is there a specific problem associated with
> enabling it?
> Are there other optional plugins to explore?
Most of the other plugins are still fairly experimental, and probably not stable
enought to build currently.
The best way to test this is with Stream Engine, which I haven't yet submitted
to FE (September was fairly busy, and it slipped my mind). I'll send you the
spec & SRPM for it once I update it to the latest version.
Once you have gstreamer-plugins-farsight & Stream Engine installed you can test
it by running 'STREAM_ENGINE_PERSIST=1 telepathy-stream-engine'
Here's my review request for telepathy-stream-engine:
BTW, I've brought my server back online for now for these packages.
*Built in mock on i386 against fedora-development
*rpmlint runs cleanly against it
*follows naming guidelines for addon packages
*licnese is GPL and spec tag matches
*COPYING file in %doc
*spec in US/english and legible
*Include source matches upstream md5sum
*buildrequires look good
*no locales to worry about
*only gstreamer plugin so's no shared libs in default linker path
*does not create any unowned directories. Package requires
gstreamer-plugins-base which in turns requires gstreamer which owns
*no duplicates in %files
*file permissions seem fine
*install and clean sections look good
*macro use is consistent
*docs is good
*no -devel subpackage
*la files removed in install section
*not a gui, no desktop file needed
*appears to meet all packaging guidelines
One small thing I'd like to see changed. Can you rename the macro at from
majorminor to gst_majorminor? I was slightly confused initially because its the
same numerical value as the package version majorminor. Its clear now its a
macro to define the gstreamer majorminor, but it would be helpful for clarity to
change the name to gst_majorminor. This is not a blocker however, but it may
help save a little time if someone has to pick this package up from you later.
This package is APPROVED for entry into the development tree.
I haven't tested this yet, I'm be building stream-engine next so I can test this.
Jef, thanks for the review.
Package Change Request
Package Name: gstreamer-plugins-farsight
New Branches: OLPC2
Updated Fedora Owners: gdesmott,bpepple