[Bug 467507] Review Request: Rufscript-fonts - Rufscripts is a decorative handwriting based font

[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=467507

Paul Flo Williams <paul at frixxon.co.uk> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |paul at frixxon.co.uk

--- Comment #7 from Paul Flo Williams <paul at frixxon.co.uk> 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.