Bug 236816 - Package Review: hal-info - Device information files for HAL
Package Review: hal-info - Device information files for HAL
Status: CLOSED RAWHIDE
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Matthias Clasen
Fedora Extras Quality Assurance
:
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2007-04-17 15:40 EDT by David Zeuthen
Modified: 2013-04-15 01:07 EDT (History)
5 users (show)

See Also:
Fixed In Version: 20070402-1
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2007-04-17 21:57:47 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
mclasen: fedora‑review+


Attachments (Terms of Use)

  None (edit)
Description David Zeuthen 2007-04-17 15:40:05 EDT
As stated in bug 230707 we should split hal-info. Please see

 http://people.redhat.com/davidz/hal-info/

rpmlint output

 $ rpmlint ../RPMS/noarch/hal-info-20070402-1.noarch.rpm
 W: hal-info no-documentation
 W: hal-info invalid-license AFL/GPL

both of these warnings can be waived I think.
Comment 1 Matthias Clasen 2007-04-17 15:55:29 EDT
- rpmlint output looks ok, those two warnings are waivable
- package name is tarball name, ok
- The version is a bit unusual, but thats the upstream versioning convention, 
  so ok, I guess
- spec file name is ok
- packaging guidelines appear to be followed 
- license is ok

To be continued...
Comment 2 Matthias Clasen 2007-04-17 17:28:34 EDT
mustfix:
- COPYING must be added as %doc
- %install must rm -rf $RPM_BUILD_ROOT

nice to have:
- add AUTHORS, NEWS, ChangeLog to %doc
- use recommended BuildRoot

Comment 3 Matthias Clasen 2007-04-17 18:20:40 EDT
builds fine in mock
Comment 4 David Zeuthen 2007-04-17 20:35:55 EDT
Fixed but mustfix and nice-to-have items mentioned in comment 2.
Comment 5 Matthias Clasen 2007-04-17 20:39:49 EDT
good to go then. approved.
Comment 6 David Zeuthen 2007-04-17 21:57:47 EDT
Thanks for the review. I've got Fedora releng to add'n'tag this for Fedora 7 so
closing this bug. Thanks.
Comment 7 Parag AN(पराग) 2013-04-15 01:07:44 EDT
package review status script is failing to parse this bug as this is not following proper summary, correcting this.

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