Bug 173028 - Review Request: ser - SIP Express Router
Summary: Review Request: ser - SIP Express Router
Keywords:
Status: CLOSED DUPLICATE of bug 180345
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: John Mahowald
QA Contact: David Lawrence
URL: http://iptel.org/ser
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2005-11-12 18:38 UTC by Peter Lemenkov
Modified: 2007-11-30 22:11 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2006-02-10 21:43:34 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Peter Lemenkov 2005-11-12 18:38:18 UTC
Spec Name or Url: http://paula.comtv.ru/ser.spec
SRPM Name or Url: http://paula.comtv.ru/ser-0.9.4-1.src.rpm
Description: Ser or SIP Express Router is a very fast and flexible SIP (RFC3621)
proxy server. Written entirely in C, ser can handle thousands calls
per second even on low-budget hardware. A C Shell like scripting language
provides full control over the server's behaviour. It's modular
architecture allows only required functionality to be loaded.
Currently the following modules are available: digest authentication,
CPL scripts, instant messaging, MySQL support, a presence agent, radius
authentication, record routing, an SMS gateway, a jabber gateway, a 
transaction module, registrar and user location

Comment 1 John Mahowald 2005-12-16 23:17:27 UTC
RFC is actually 3261

Needs a bit of work: (courtesy of fedora-qa script)

* RPM name is OK
* Source ser-0.9.4_src.tar.gz is the same as upstream
* This is the latest version
* Builds fine in mock


Needs work:
* Missing SMP flags. If it doesn't build with it, please add a comment
  (wiki: PackagingGuidelines#parallelmake)
* Spec file: tag Vendor is forbidden
  (wiki: PackagingGuidelines#tags)
* The BuildRoot must be cleaned at the beginning of %install
* rpmlint of ser: init script can be without no replace, and should be
executable. But it should not be enabled by default.
* rpmlint of ser-mysql: Simply remove the . from the summary.
* rpmlint of ser-jabber: Another period to remove.
* rpmlint of ser-cpl-c: Yet another period.
* rpmlint of ser-pa: Remove .
* rpmlint of ser-postgres: Remove .
* File list of ser: Move the /usr/etc heirarchy to /etc.
* Missing dependancy on service for %preun (package initscripts)
* Missing dependancy on chkconfig for %post (package chkconfig)
* Missing dependancy on chkconfig for %preun (package chkconfig)
* Scriptlets: missing "service" in %postun (wiki: ScriptletSnippets) (in
subpackage ser)

Comment 2 Jeffrey C. Ollie 2006-02-07 20:17:01 UTC
*** Bug 180345 has been marked as a duplicate of this bug. ***

Comment 3 Jeffrey C. Ollie 2006-02-08 03:58:35 UTC
Peter, do you want to keep SER or can it be turned over to Andreas?


Comment 4 Peter Lemenkov 2006-02-08 06:30:26 UTC
(In reply to comment #3)
> Peter, do you want to keep SER or can it be turned over to Andreas?

I willingly let Andreas to take ownership over SER. My only reques is to make
separate packages for ser-jabber, ser-cpl-c and ser-pa modules as well as for
ser-postgres and ser-mysql modules.

This patch would be useful for doing this:

http://paula.comtv.ru/ser-0.9.4-Makefile.patch


Comment 5 Andreas Thienemann 2006-02-08 09:36:29 UTC
Thx Peter.

About the different packages:

mysql and postgresql are already packaged as subpackages. This makes sense, as
both pull in quite a lot of dependencies, which is unnecessary if one is only
using one database and not the other.

But about the jabber, cpl-c and pa modules, I'm not so sure. All they depend on
are libxml2, expat and pthread.
These libraries are already installed on most systems by default, so it makes no
sense IMHO to split the package up any further.

Comments?

Comment 6 Jeffrey C. Ollie 2006-02-10 21:43:34 UTC

*** This bug has been marked as a duplicate of 180345 ***

Comment 7 Christian Iseli 2006-03-29 14:19:49 UTC
Removed FE-REVIEW blocker


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