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=467507 Paul Flo Williams <paul@xxxxxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |paul@xxxxxxxxxxxxx --- Comment #7 from Paul Flo Williams <paul@xxxxxxxxxxxxx> 2011-05-08 11:38:02 EDT --- Is Rahul with you taking over this package? (The passage of time suggests yes, but you never know.) 1. The spec name must be the same as the package name. 2. The license in the source file is GPLv3+ with exceptions 3. The SFD contains the line "Version: 0.1.0" and this is consistent with Hiran's other font under review at the moment, Perizia, but version control of fonts is notoriously bad so 0.100, pulled from the name table, is OK. 4. The fontconfig is very wrong, as you've said that the font "Sans" is the same as the font "Rufscript". Please use the basic fontconfig template and declare this as a cursive font. 5. From Nicolas's previous comment #2, points 6 and 7 still apply. Other than those, this is a simple package, and almost ready for approval. -- 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. _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review