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=483434
--- Comment #5 from Christian Krause chkr@plauener.de 2009-02-24 15:04:47 EDT --- Hello Jess,
good: - all mentioned issues solved - content of the binary packages is now ok
very minor issue - just for completeness: - omit the macro in the changelog of the spec file, otherwise rpmlint will complain: SPECS/argtable2.spec:66: W: macro-in-%changelog _defaultdocdir
- you've added the spec file to the sources, so the sources in the provided src.rpm are slightly different from the ones on sf.net
Besides these two little minor issues I think the package looks very good. Since I'm not an "offical" reviewer I could only help you up to this point and so somebody with that status has to do the final formal review to give you the approval for the new package.