Bug 239165

This builds fine and rpmlint is quiet.

According to
http://fedoraproject.org/wiki/Packaging/NamingGuidelines#PreReleasePackages
release should be 0.1.beta7%{?dist}.

This is another package that links statically with libnet.  Not that it has any
choice, but this requires a trip through FESCo.  Either that or the libnet
maintainer needs to get whatever patches debian uses to build libnet dynamically.

Review:
* source files match upstream:
   aed5b163ed4886f04242b46005a6cb4876ef38ad72001a94facb62a99dc99c57  
   tcptraceroute-1.5beta7.tar.gz
X package doesn't meet versioning guidelines for prerelease packages.
* specfile is properly named, is cleanly written and uses macros consistently.
* summary is OK.
* description is OK.
* dist tag is present.
* build root is OK.
* license field matches the actual license.
* license is open source-compatible.
* license text included in package.
* latest version is being packaged.
* BuildRequires are proper.
* compiler flags are appropriate.
* %clean is present.
* package builds in mock (development, x86_64).
* package installs properly
* debuginfo package looks complete.
* rpmlint is silent.
* final provides and requires are sane:
   tcptraceroute = 1.5-0.beta7.1.fc8
  =
   libpcap.so.0.9()(64bit)
* %check is not present; no test suite upstream.
* no shared libraries are added to the regular linker search paths.
* owns the directories it creates.
* doesn't own any directories it shouldn't.
* no duplicates in %files.
* file permissions are appropriate.
* no scriptlets present.
* code, not content.
* documentation is small, so no -docs subpackage is necessary.  (Actually the 
  documentation is nearly twice as large as the lone executable, but the package 
  is only 142K.)
* %docs are not necessary for the proper functioning of the package.
* no headers.
* no pkgconfig files.
* no static libraries.
* no libtool .la files.

Changed the release

http://folk.ntnu.no/sindrb/packages/green_nyc/tcptraceroute.spec
http://folk.ntnu.no/sindrb/packages/green_nyc/tcptraceroute-1.5-0.1.beta7.fc7.src.rpm

Will I have to do something about the libnet issue, if so, what?

OK, the libnet maintainer has no interest in allowing libnet to be dynamic, so
there's nothing you can do.  It looks like everything will just sort itself with
a rebuild if libnet magically becomes dynamic in the future.  So I think we're
done here.

APPROVED

New Package CVS Request
=======================
Package Name: tcptraceroute
Short Description: A traceroute implementation using TCP packets
Owners: foolish
Branches: FC-6 F-7 EL-4 EL-5

cvs done. 
Note that the bugzilla sync process isn't yet caught up to the merge of core and
extras components, so it might not be until tomorrow for this to have a bugzilla
component. 

Sindre,

Some of the documentation files are being packaged twice (this package has two
different doc directories; it should only have doc files in the
/usr/share/doc/tcptraceroute-1.5  directory).

jpo

----------
$ rpm -qpl tcptraceroute-1.5-0.1.beta7.fc6.i386.rpm 
/usr/bin/tcptraceroute
/usr/share/doc/tcptraceroute
/usr/share/doc/tcptraceroute-1.5
/usr/share/doc/tcptraceroute-1.5/AUTHORS
/usr/share/doc/tcptraceroute-1.5/COPYING
/usr/share/doc/tcptraceroute-1.5/ChangeLog
/usr/share/doc/tcptraceroute-1.5/README
/usr/share/doc/tcptraceroute/AUTHORS
/usr/share/doc/tcptraceroute/COPYING
/usr/share/doc/tcptraceroute/ChangeLog
/usr/share/doc/tcptraceroute/NEWS
/usr/share/doc/tcptraceroute/README
/usr/share/doc/tcptraceroute/examples.txt
/usr/share/doc/tcptraceroute/tcptraceroute.1.html
/usr/share/doc/tcptraceroute/tcptraceroute.lsm
/usr/share/man/man1/tcptraceroute.1.gz
-------------

Fixed the issue with the duplicate doc files.

Created attachment 157646 [details]
Specfile patch

Sindre,

The patch makes the following changes to the specfile:

  * re-introduces some of the documentation files
    ( tcptraceroute.1.html tcptraceroute.lsm examples.txt )

  * adds two perl scripts to the documentation files list
    ( tcptra-testsuite.pl traceroute.cgi )

    Note: the execution bit is cleared in the %setup section

  * removes several white spaces

jpo