[Bug 589162] Review Request: stress - A tool to put given subsystems under a specified load

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=589162

Terje Røsten <terjeros at phys.ntnu.no> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|fedora-review?              |fedora-review+

--- Comment #8 from Terje Røsten <terjeros at phys.ntnu.no> 2010-05-07 12:57:55 EDT ---
ok rpmlint, just spelling warnings
ok sources
 sha1sum stress-1.0.4.tar.gz*
 7ccb6d76d27ddd54461a21411f2bc8491ba65168  stress-1.0.4.tar.gz
 7ccb6d76d27ddd54461a21411f2bc8491ba65168  stress-1.0.4.tar.gz.spec
 However use wget _n to preserve timestamps on the tarball
ok license GPLv2+, see comment above
ok dirs, perms and owns
ok correct language
 ! spec file
 remove a strange rm -f in %install under
 rm -f $RPM_BUILD_ROOT%{_infodir}/dir
ok docs and man page
ok buildopts
ok macros
ok utf-8 file names
ok koji:
 http://koji.fedoraproject.org/koji/taskinfo?taskID=2172740

Fix the rm issue on import.

  The package stress is APPROVED.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.