Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 218217 - Review Request: telepathy-stream-engine - Telepathy client to handle media streaming
Summary: Review Request: telepathy-stream-engine - Telepathy client to handle media st...
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Jef Spaleta
QA Contact: Fedora Package Reviews List
URL:
Whiteboard:
Depends On:
Blocks: FE-ACCEPT
TreeView+ depends on / blocked
 
Reported: 2006-12-03 17:16 UTC by Brian Pepple
Modified: 2008-04-23 16:21 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2006-12-28 00:31:25 UTC
Type: ---
Embargoed:
j: fedora-cvs+


Attachments (Terms of Use)

Description Brian Pepple 2006-12-03 17:16:39 UTC
Spec URL: http://piedmont.homelinux.org/fedora/telepathy/telepathy-stream-engine.spec
SRPM URL: http://piedmont.homelinux.org/fedora/telepathy/telepathy-stream-engine-0.3.16-1.src.rpm
Description: A Telepathy client that uses Farsight and GStreamer to handle media
streaming for channels with the IceSignalling interface.

Comment 1 Mamoru TASAKA 2006-12-09 08:35:00 UTC
I cannot read your spec/srpm. Is piedmont.homelinux.org alive?

Comment 2 Jef Spaleta 2006-12-23 00:45:59 UTC
yes it looks like the piedmont.homelinux.org server is down again.

Brian email me the spec and src.rpm and I can host them for you.

-jef

Comment 4 Jef Spaleta 2006-12-23 02:27:26 UTC
Uhm looks like they bumped up to 0.3.18 in the meantime.

Would you like me to spin up an updated srpm and spec for 0.3.18 and place them
on the website for you?

-jef

Comment 5 Jef Spaleta 2006-12-23 03:50:19 UTC
URL updated
http://jspaleta.thecodergeek.com/Fedora%20SRPMS/telepathy-stream-engine/telepathy-stream-engine-0.3.18-1.src.rpm

I will be starting the formal review of 0.3.18 srpm tonite as time allows.

Comment 6 Jef Spaleta 2006-12-23 04:32:26 UTC
It looks good. Here is the formal review.
I'm going to start a clock on this, and if noone brings up any blockers I'll
approve tomorrow evening.

-jef

GOOD:
*Built in mock on i386 against fedora-development
*rpmlint runs cleanly against it
*follows naming guidelines
*licnese is LGPL and spec tag matches
*COPYING file in %doc
*spec in US/english and legible
*Include source matches upstream md5sum
52c0b48905f11cac71df1f8560c8f4d7  telepathy-stream-engine-0.3.18.tar.gz

*buildrequires look good
*no locales to worry about
*no shared libraries
*not relocatable
*does not create any unowned directories. Package requires   
  libdbus-1.so.3 which is provided by dbus which owns 
  %{_datadir}/dbus-1/services/
*no duplicates in %files
*file permissions seem fine
*install and clean sections look good
*macro use is consistent
*docs is good
*no -devel subpackage
*no la files
*not a gui, no desktop file needed
*appears to meet all packaging guidelines



Comment 7 Mamoru TASAKA 2006-12-23 05:27:38 UTC
Well, 
* please check if the following is what you expect.
---------------------------------------------------
checking for pic2plot... no
checking for gs... no
checking for convert... no
configure: WARNING: Imagemagick not found, drawing sequence diagrams will be
disabled.
configure: WARNING: Ghostscript not found, drawing sequence diagrams will be
disabled.
configure: WARNING: GNU plotutils not found, drawing sequence diagrams will be
disabled.
----------------------------------------------------
Note: all of them are available in Fedora Core/Extras and
a people who has already installed them beforehand may create
different binary rpm of telepathy-stream-engine.

Comment 8 Brian Pepple 2006-12-23 06:16:04 UTC
Yes, this was on purpose since the sequence diagrams generation support isn't
complete yet.

Comment 9 Mamoru TASAKA 2006-12-23 07:29:18 UTC
Okay, then this package can be approved IMO.

Comment 10 Jef Spaleta 2006-12-24 02:56:51 UTC
(In reply to comment #8)
> Yes, this was on purpose since the sequence diagrams generation support isn't
> complete yet.

Does the configure script let you explicitly disable imagemagik,gs, and
plotutils support?  It would be best if these features were explicitly disabled
so we can get consistent rebuilds of the srpm even in build environments where
these exist.
I'm not sure this should count as a blocker or not, but I think Mamoru has a
point concerning binary reproducibility.

-jef

Comment 11 Brian Pepple 2006-12-24 03:36:30 UTC
(In reply to comment #10)
> Does the configure script let you explicitly disable imagemagik,gs, and
> plotutils support?  It would be best if these features were explicitly disabled
> so we can get consistent rebuilds of the srpm even in build environments where
> these exist.

I believe there is a '--enable-sequence-diagrams=no' option we can add to
disable the building of this.



Comment 12 Jef Spaleta 2006-12-25 09:46:11 UTC
Approved,
just add --enable-sequencec-diagrams=no  to the %configure before you build.

-jef

Comment 13 Brian Pepple 2006-12-28 00:31:25 UTC
Jef, thanks for the review.

Comment 14 Brian Pepple 2008-04-23 15:45:19 UTC
Package Change Request
======================
Package Name: telepathy-stream-engine
New Branches: OLPC2
Updated Fedora Owners: gdesmott,bpepple

Comment 15 Jason Tibbitts 2008-04-23 16:21:07 UTC
CVS done.


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