Bug 429700 - Review Request: bing - bandwidth ping
Summary: Review Request: bing - bandwidth ping
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Rafał Psota
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2008-01-22 15:01 UTC by Adam Jackson
Modified: 2008-02-21 18:04 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2008-02-21 18:04:45 UTC
Type: ---
Embargoed:
rafalzaq: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Adam Jackson 2008-01-22 15:01:46 UTC
Spec URL: http://ajax.fedorapeople.org/bing/bing.spec
SRPM URL: http://ajax.fedorapeople.org/bing/bing-1.1.3-1.fc9.src.rpm

Description:

Bing is a point-to-point bandwidth measurement tool (hence the 'b'), based on
ping.

Bing determines the real (raw, as opposed to available or average) throughput
on a link by measuring ICMP echo requests' round trip times for different
packet sizes at each end of the link.

Comment 1 Rafał Psota 2008-01-22 15:11:13 UTC
I'll take this one.

Comment 2 Adam Jackson 2008-01-22 15:15:24 UTC
Please be sure to grab the latest version of the spec, I fixed a non-utf8 %doc bug.

Comment 3 Rafał Psota 2008-01-22 16:24:03 UTC
rpmlint is not silent:
bing.src: W: mixed-use-of-spaces-and-tabs (spaces: line 1, tab: line 12)

I think you should include ChangeLog file as %doc.


Please fix the above issues and I'll approve this package.

Comment 4 Adam Jackson 2008-01-23 18:04:57 UTC
I think I shouldn't.  The changelog hasn't changed since 1997.

Comment 5 Rafał Psota 2008-01-24 18:12:33 UTC
Yes it's pretty old. Ok, forget about it. Just fix that rpmlint warning before
uploading to cvs.

APPROVED

Comment 6 Adam Jackson 2008-02-18 15:47:07 UTC
New Package CVS Request
=======================
Package Name: bing
Short Description: bandwidth ping
Owners: ajax
Branches: 
InitialCC: 
Cvsextras Commits: yes


Comment 7 Kevin Fenzi 2008-02-18 17:28:22 UTC
cvs done.

Comment 8 Adam Jackson 2008-02-21 18:04:45 UTC
Imported and built, thanks all.


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