Bug 173823 - Review Request: nuttcp
Summary: Review Request: nuttcp
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Adrian Reber
QA Contact: David Lawrence
URL: http://people.redhat.com/rvokal/nuttc...
Depends On:
Blocks: FE-ACCEPT 173744
TreeView+ depends on / blocked
Reported: 2005-11-21 17:50 UTC by Radek Vokal
Modified: 2007-11-30 22:11 UTC (History)
2 users (show)

Clone Of:
Last Closed: 2005-11-28 14:52:01 UTC

Attachments (Terms of Use)
spec patch (2.12 KB, patch)
2005-11-22 12:07 UTC, Adrian Reber
no flags Details | Diff

Description Radek Vokal 2005-11-21 17:50:40 UTC
Spec Name or Url: nuttcp
SRPM Name or Url: http://people.redhat.com/rvokal/nuttcp/nuttcp-5.1.11-1.src.rpm
Description: nuttcp is possible replacement for ttcp package from Core.

       nuttcp  is  a  network performance measurement tool intended for use by
       network and system managers.  Its most basic usage is to determine  the
       raw  TCP  (or  UDP)  network  layer  throughput  by transferring memory
       buffers from a source system across an  interconnecting  network  to  a
       destination  system,  either  transferring  data  for  a specified time
       interval, or alternatively transferring a specified number of  buffers.
       In  addition  to  reporting  the  achieved  network throughput in Mbps,
       nuttcp also provides additional useful information related to the  data
       transfer  such  as  user,  system, and wall-clock time, transmitter and
       receiver CPU utilization, and loss percentage (for UDP transfers).

       nuttcp is based on nttcp, which in turn was an enhancement  by  someone
       at  Silicon  Graphics  (SGI) on the original ttcp, which was written by
       Mike Muuss at BRL sometime before December 1984, to compare the perfor-
       mance of TCP stacks by U.C. Berkeley and BBN to help DARPA decide which
       version to place in the first BSD Unix  release.   nuttcp  has  several
       useful  features beyond those of the basic ttcp/nttcp, such as a server
       mode, rate limiting, multiple parallel streams, and timer based  usage.
       nuttcp is also continuing to evolve to meet new requirements that arise
       and to add desired new features.  nuttcp has been  successfully  tested
       and  used  on a variety of Solaris, SGI, and PPC/X86 Linux systems, and
       should probably work fine on most flavors of Unix.

Comment 1 Radek Vokal 2005-11-21 18:36:05 UTC
Some packaging changes in 


xinted service is installed and unnecessary files from tarball were removed.

Comment 2 Adrian Reber 2005-11-22 11:07:34 UTC
Will attach a patch with required changes.

Comment 3 Adrian Reber 2005-11-22 12:07:02 UTC
Created attachment 121340 [details]
spec patch

- added a URL
- removed wrong URL from Source
- fixed summary according to guidlines
- removed bogus build require
- disabled xinetd services
- using correct path in xinetd files
- removed unnecessary checks for BUILD_ROOT
- replaced /etc with macro
- added noreplace flag to %config

Comment 4 Radek Vokal 2005-11-22 12:15:55 UTC
Ok, I will grab the changes. Why don't you like this?

-[ "$RPM_BUILD_ROOT" != "/" ] && rm -fr $RPM_BUILD_ROOT

I like to double-check things .. 

Comment 5 Adrian Reber 2005-11-22 12:27:11 UTC
-[ "$RPM_BUILD_ROOT" != "/" ] && rm -fr $RPM_BUILD_ROOT

This has been discussed already many times on the fedora-extras list.
As buildroot is already set this makes no sense and what happens if it is
/usr /opt // /etc. But the best would be to check mailing list archives for this.

Comment 6 Radek Vokal 2005-11-22 13:04:32 UTC
point taken, here's patched version 


Comment 7 Rahul Sundaram 2005-11-25 16:50:14 UTC
(In reply to comment #5)
> -[ "$RPM_BUILD_ROOT" != "/" ] && rm -fr $RPM_BUILD_ROOT
> +rm -fr $RPM_BUILD_ROOT
> This has been discussed already many times on the fedora-extras list.
> As buildroot is already set this makes no sense and what happens if it is
> /usr /opt // /etc. But the best would be to check mailing list archives for this.

Would be good to get this into the guidelines then

Comment 8 Adrian Reber 2005-11-28 10:47:31 UTC
* builds in mock (development and FC4)
* clean installation and removal
* rpmlint is happy
* md5sum of nuttcp-v5.1.11.c matches upstream
* spec looks good


I would like to see a comment above Source0 stating how the tarball was created
and that upstream doesn't provide one.

Comment 9 Radek Vokal 2005-11-28 12:29:01 UTC
Ok, the comment about Source is back. I've also did some gcc clean-up, so now
there are only few harmless warnings. 


Adrian, I don't have a sponsor, this is my first package in extras. Can you
sponsor my account? 

Comment 10 Adrian Reber 2005-11-28 14:48:27 UTC
I would have sponsored you but it looks like I was to slow.

Comment 11 Radek Vokal 2005-11-28 14:52:01 UTC
Yep, got the sponsor already :) The package is in CVS - devel branch. Closing

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