Bug 192867

disclaimer: I hold none of the official fedora credentials, so take my comments
as random joe observations.

Your spec file fails many of the basic requirements listed in
http://fedoraproject.org/wiki/Packaging/Guidelines
See "Writing a package from scratch" section for how to produce a basic specfile
using an existing template which should help produce something not so far off
the mark.

Here are some of the more major omissions:
- no URL tag
- source tag is not a url to the canonical upstream source.
- no explicit buildroot
- the description spills over 80 characters a line and has text mostly comprised
of advocacy rather than descritopn.
- doesn't use standard macros as %configure, %makeinstall  thus resulting in an
install to /usr/local, which is a big no no for fedora. 
- no changelog
- no spaces, resulting i poor readability.

A more detailed review will possibly follow after these basic things are taken
care of.

Problems Fixed, new .spec and .src.rpm files changed.
Thanks for reporting it.

More notes regarding 1.3.2-1:
- the canonical sf source url used in most fedora specs is
http://download.sourceforge.net/projectname/.. or http://dl.sf.net/projectname/..
  so in this case http://download.sourceforge.net/ctorrent/ctorrent-1.3.2.tar.gz

- you don't need the BuildRequires: rpm, see the section
BuildRequires/Exceptions in the Packaging guidelines. It includes rpm-build
which implies rpm.

- you need a BuildRequires: openssl-devel

- you don't need the Requires: libc.so.6 libc.so.6(GLIBC_2.0)
libc.so.6(GLIBC_2.1). All shared library dependencies are automatically
generated by rpm. This includes openssl. See the "Requires" section in the
Packaging guidelines.

- the first line of your description still exceeds 80, please wrap it.

- the secton Macros from the Packaging guidelines says "Use macros instead of
hard-coded directory names (see Extras/RPMMacros)" which would mean
%{_bindir}/ctorrent  instead of /usr/bin/ctorrent in the files manifest

When changing the spec in result of comments made in the review bump the Release
tag and add an appropriate changelog so that there is some tracking of the
review process.

I can not officially review or sponsor this so thats all from me. Hope it helps.

.spec file changed, new link to release 2:
http://www.netservers.org/packages/rpm/ctorrent-1.3.2-2.src.rpm

Anyway adding %{_bindir}/ctorrent doesn't work as using the correct path, that
I've chosen before. By the way thanks for the complete review unfortunately you
can't sponsor me, but i think that soon you'll this possibility too.

- Requires: gcc  should be removed. If there is a runtime dependancy on any gcc
lib its automatically genereated. 

- %{_bindir}/ctorrent  should work. If it doesn't you either made a typo or your
build enviroment is not quite standard fedora fare. custom macros and such...?

- your changelog is not formatted properly. The first line should end with - 
E-V-R of the package. There should be an empty newline after each new changelog
section. "Changes to .spec file" is not an actual description of what changed
therefore useles to reveiwers and users. Provide a little bit more details on
what actually changed. Something like:
 - fixed URL
 - removed unneeded Reqiures, added openssl-devel to BR.

- please remove all the trailing spaces, they might confuse rpm and other spec
handling tools. The spaces in the template are there not to trail but to align
the position of the values of the tags.

rpmlint as mandated by the guidelines should have spotted most of these

Spec URL: http://www.netservers.org/packages/rpm/ctorrent.spec
SRPM URL: http://www.netservers.org/packages/rpm/ctorrent-1.3.2-3.src.rpm

Release 3, problems fixed.

MD5Sums:
1bc787df91285a9cec8509617c3152d6  ctorrent-1.3.2.tar.gz

Good:
* Source URL is canonical
* Upstream source tarball verified
* Buildroot has all required elements
* All necessary BuildRequires listed.
* All desired features are enabled
* Make succeeds even when %{_smp_mflags} is defined
* Files have appropriate permissions and owners
* Package builds fine in Mock.
* Package installs and uninstalls cleanly on FC5.

Bad:
* Group Tag is not from official list. http://fedoraproject.org/wiki/RPMGroups
* Not all paths begin with macros. In the file section, '/usr/bin' should be
replaced with '%{_bindir}'.  Refer to http://fedoraproject.org/wiki/Extras/RPMMacros
* We need to verify that the license is really GPL, since neither the tarball or
the website mention the license at all.

Minor:
* rpmlint gives the following error, which is addressed in the first item of the
Bad section:
  W: ctorrent non-standard-group Net
* I wouldn't even bother to package the COPYING file, since it only contains the
authors name and nothing else.  The AUTHOR file contains the exact same information.

Done, i fixed all errors listed above.
http://www.netservers.org/packages/rpm/ctorrent.spec
http://www.netservers.org/packages/rpm/ctorrent-1.3.2-3.src.rpm
The license is GPL, I'm sure about it. The author have written a presentation of
his program on a Debian list and he writes that license is GPL. Anyway i've
found some more websites that confirm this.

You've still got a problem with the %file section.  Looks like you changed
'/usr/bin/ctorrent' to just '%{_bindir}'.  It needs to be '%{_bindir}/ctorrent'
or '%{_bindir}/%{name}'

ops yes, my mistake sorry.
Anyway fixed now.

+1 APPROVE.

Fixed errors noted in comment #7.

I'll also be your sponser.  It looks like your next step is:
http://fedoraproject.org/wiki/Extras/Contributors#head-a89c07b5b8abe7748b6b39f0f89768d595234907

Package Change Request
======================
Package Name: ctorrent
New Branches: EL-5

cvs done.