The above urls don't seem to work.  Can you give me updated url info for these?

-jef

(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.

-jef

Here we go.

SRPM:
http://jspaleta.thecodergeek.com/Fedora%20SRPMS/gstreamer-plugins-farsight/gstreamer-plugins-farsight-0.10.2-1.src.rpm
SPEC:
http://jspaleta.thecodergeek.com/Fedora%20SRPMS/gstreamer-plugins-farsight/gstreamer-plugins-farsight.spec

Let me grab a beer and I'll start a mock build and if that passes I'll start a
formal review.

One thing, is there any way to test the functionality of this plugin currently?

-jef

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

-jef 

A couple of questions with regard to the configure options

--disable-jasper	
jasper is in Extras currently. Is there a specific problem associated with
enabling it?  

--with-plugins=rtpdemux,rtpjitterbuffer
Are there other optional plugins to explore?

-jef

(In reply to comment #6)
> A couple of questions with regard to the configure options
> 
> --disable-jasper	
> jasper is in Extras currently. Is there a specific problem associated with
> enabling it?  
> 
> --with-plugins=rtpdemux,rtpjitterbuffer
> 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:
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=218217

BTW, I've brought my server back online for now for these packages.

GOOD:
*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
77c92554c2bd57ad1b426e5ba50eb9a8  gst-plugins-farsight-0.10.2.tar.gz
*buildrequires look good
*no locales to worry about
*only gstreamer plugin so's no shared libs in default linker path
*not relocatable
*does not create any unowned directories. Package requires   
  gstreamer-plugins-base which in turns requires gstreamer which owns 
  /usr/lib/gstreamer-0.10
*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

CVS done.