Spec Name or Url: http://www.ocjtech.us/libeXosip2-1.9.1-0.1.pre16.spec SRPM Name or Url: http://www.ocjtech.us/libeXosip2-1.9.1-0.1.pre16.src.rpm Description: A library that hides the complexity of using the SIP protocol for mutlimedia session establishement. This protocol is mainly to be used by VoIP telephony applications (endpoints or conference server) but might be also useful for any application that wish to establish sessions like multiplayer games. Builds in mock on i386/devel and using plain ol' rpmbuild on x86_64/fc4. This is my first attempt at submitting something to FE, so please be gentle :).
* BuildRoot should be %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n) (wiki: PackagingGuidelines#BuildRoot) * You build the doxygen documentation but you don't include it in the package. Please add it to the -devel package (with a %doc tag). * In the spec file, %makeinstall should be replaced by "make install DESTDIR=%{buildroot}" * In the files list, "%dir %{_includedir}/eXosip2" is redundant (covered by the next line) A few questions: * Do you want this lib included for an application in particular ? * Why use the prerelease version instead of the stable 1.9.0 ?
New spec/srpm with problems pointed out by Aurelien fixed: Spec Name or Url: http://www.ocjtech.us/libeXosip2-1.9.1-0.2.pre16.spec SRPM Name or Url: http://www.ocjtech.us/libeXosip2-1.9.1-0.2.pre16.src.rpm I plan on eventually packaging OpenPBX when it is stable. OpenPBX is a fork of Asterisk and includes a new SIP channel that uses eXoSIP2, oSIP (already in extras), oRTP (already in extras). The prerelease is the one specified by the author of the new channel.
* BuildRoot should be %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n) (wiki: PackagingGuidelines#BuildRoot), you forgot the -n at the end. * From /usr/include/eXosip2/eXosip.h, I would say that libeXosip2-devel requires libosip2-devel (it includes files from /usr/include/osipparser2/)
New spec/srpm with issues from comment #3 fixed: Spec Name or Url: http://www.ocjtech.us/libeXosip2-1.9.1-0.3.pre16.spec SRPM Name or Url: http://www.ocjtech.us/libeXosip2-1.9.1-0.3.pre16.src.rpm
Review for release 0.3.pre16: * RPM name is OK * Source libeXosip2-1.9.1-pre16.tar.gz is the same as upstream * Builds fine in mock * rpmlint of libeXosip2 looks OK * rpmlint of libeXosip2-devel looks OK * File list of libeXosip2 looks OK * File list of libeXosip2-devel looks OK * Included binary sip_reg works. APPROVED Minor nitpick: after or before install, remove the INSTALL file from the %doc list, it's useless (since we use a package).