Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 173823 - Review Request: nuttcp
Summary: Review Request: nuttcp
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Adrian Reber
QA Contact: David Lawrence
URL: http://people.redhat.com/rvokal/nuttc...
Whiteboard:
Depends On:
Blocks: FE-ACCEPT 173744
TreeView+ depends on / blocked
 
Reported: 2005-11-21 17:50 UTC by Radek Vokál
Modified: 2007-11-30 22:11 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2005-11-28 14:52:01 UTC
Type: ---
Embargoed:

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

Description Radek Vokál 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 Vokál 2005-11-21 18:36:05 UTC 
Some packaging changes in 

http://people.redhat.com/rvokal/nuttcp/nuttcp-5.1.11-2.src.rpm

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

Changes:
- 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 Vokál 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
+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
+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 Vokál 2005-11-22 13:04:32 UTC 
point taken, here's patched version 

http://people.redhat.com/rvokal/nuttcp/nuttcp-5.1.11-3.src.rpm
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

APPROVED

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 Vokál 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. 

http://people.redhat.com/rvokal/nuttcp/nuttcp-5.1.11-4.src.rpm

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 Vokál 2005-11-28 14:52:01 UTC 
Yep, got the sponsor already :) The package is in CVS - devel branch. Closing
this as NEXTRELEASE
Note You need to log in before you can comment on or make changes to this bug.