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 <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[email protected]

--- Comment #7 from Paul Flo Williams <[email protected]> 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
[email protected]
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to