Request for review: scrub

[Matthias Saou]

Tom 'spot' Callaway wrote :

> Again, a really small package. One binary, a man page, and some docs.
> 
> Please review for approval.

Things to fix :
- Don't strip the binary, or debuginfo package will be useless
- Don't manually gzip the man page, rpm magic does that for you

Things that could be improved :
- Patch for CFLAGS isn't needed, you can just override at make time
- You can use -D with install to have the directory tree created

Attached is a quick patch to change all this ;-)

Matthias

-- 
Clean custom Red Hat Linux rpm packages : http://freshrpms.net/
Fedora Core release 3 (Heidelberg) - Linux kernel 2.6.11-1.27_FC3
Load : 0.72 1.01 1.20
-------------- next part --------------
A non-text attachment was scrubbed...
Name: scrub.spec.patch
Type: application/octet-stream
Size: 1441 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/fedora-extras-list/attachments/20050520/a6d421ca/attachment.obj>