Bug 429700

Summary:	Review Request: bing - bandwidth ping
Product:	[Fedora] Fedora	Reporter:	Adam Jackson <ajax>
Component:	Package Review	Assignee:	Rafał Psota <rafalzaq>
Status:	CLOSED RAWHIDE	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	rawhide	CC:	fedora-package-review, notting
Target Milestone:	---	Flags:	rafalzaq: fedora-review+ kevin: fedora-cvs+
Target Release:	---
Hardware:	All
OS:	Linux
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2008-02-21 18:04:45 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

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.