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 201806 - Review Request: mbuffer
Summary: Review Request: mbuffer
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Paul Howarth
QA Contact: Fedora Package Reviews List
URL:
Whiteboard:
Depends On:
Blocks: FE-ACCEPT
TreeView+ depends on / blocked
 
Reported: 2006-08-08 22:20 UTC by Alexander Dalloz
Modified: 2007-11-30 22:11 UTC (History)
0 users

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2006-08-11 00:32:41 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Alexander Dalloz 2006-08-08 22:20:38 UTC
Spec URL: http://www.uni-x.org/review/mbuffer.spec
SRPM URL: http://www.uni-x.org/review/mbuffer-20060728-1.src.rpm
Description: mbuffer is a tool for buffering data streams. Its special feature is to show the I/O rate and summary to the user.

Comment: The package had already been under review, in "pre-bugzilla_review" times. But for some reasons I never committed it into CVS. So I feel it is better to again offer a review. Reference for previous review:
http://www.redhat.com/archives/fedora-extras-list/2005-August/msg00046.html
http://www.redhat.com/archives/fedora-extras-list/2005-August/msg00152.html

Comment 1 Parag AN(पराग) 2006-08-09 12:58:11 UTC
rpmlint on SRPM is not silent
W: mbuffer mixed-use-of-spaces-and-tabs
The specfile mixes use of spaces and tabs for indentation, which is a
cosmetic annoyance.  Use either spaces or tabs for indentation, not both.


Comment 2 Paul Howarth 2006-08-09 13:22:16 UTC
Given that I already approved this package back in the old days, I'll take
another quick look at it later today when I a free moment or two; shouldn't take
too long.

Comment 3 Alexander Dalloz 2006-08-09 17:58:33 UTC
(In reply to comment #1)
> rpmlint on SRPM is not silent
> W: mbuffer mixed-use-of-spaces-and-tabs
> The specfile mixes use of spaces and tabs for indentation, which is a
> cosmetic annoyance.  Use either spaces or tabs for indentation, not both.

Thanks, did rpmlint just against the binary rpm. Now fixed it without version
bump (the 2 compile options needed another tab indent).
 



Comment 4 Paul Howarth 2006-08-10 12:43:15 UTC
Review
======

- rpmlint clean
- package and spec file name OK
- package meets guidelines
- license is GPL, matches spec, text included
- spec file written in English and is legible
- sources match upstrem
- package builds OK in mock for FC5.x86_64
- BR's OK
- no locales, libraries, subpackages or pkgconfigs to worry about
- not relocatable
- no directory ownership or permissions issues
- no duplicate files
- %clean section present and correct
- macro usage is consistent
- code, not content
- no large docs
- docs don't affect runtime
- no desktop file needed for CLI app
- package appears to function correctly
- no scriptlets

Needswork:

 * No point including the NEWS file as it just says to look in ChangeLog

Are you sponsored? I don't see your bugzilla address in owners.list.


Comment 5 Alexander Dalloz 2006-08-10 17:03:51 UTC
Thanks Paul! I removed the NEWS file from %docs and bumped release. The new

SRPM URL: http://www.uni-x.org/review/mbuffer-20060728-2.src.rpm
Spec URL: http://www.uni-x.org/review/mbuffer.spec

Yes, I was sponsored last year by Warren and maintain 2 packages (keychain +
pam_abl) in Extas. My FE mail address is different to my BZ account - please see
the %changelog for the FE address.


Comment 6 Paul Howarth 2006-08-10 17:28:09 UTC
(In reply to comment #5)
> Thanks Paul! I removed the NEWS file from %docs and bumped release. The new
> 
> SRPM URL: http://www.uni-x.org/review/mbuffer-20060728-2.src.rpm
> Spec URL: http://www.uni-x.org/review/mbuffer.spec

OK, APPROVED.

> Yes, I was sponsored last year by Warren and maintain 2 packages (keychain +
> pam_abl) in Extas. My FE mail address is different to my BZ account - please see
> the %changelog for the FE address.

Hmm, I thought the FE and BZ addresses had to match or things got very confused.
As long as you receive your bug reports it's OK though :-)


Comment 7 Alexander Dalloz 2006-08-10 18:30:47 UTC
Thanks Paul for the work and approval.
I will ask at other place about the mail address. So far I am listed in the
owners.list with my bugzilla address, and I am getting bug tickets about my
packages properly.

Comment 8 Alexander Dalloz 2006-08-11 00:32:41 UTC
Though not a requirement, I changed my bugzilla email address to prevent any
future confusion.
The devel branch build properly, so I am closing this ticket.


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