[Bug 529323] Review Request: gdouros-aegean-fonts - Font for ancient scripts in the Aegean vicinity

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


Nicolas Mailhot <nicolas.mailhot at laposte.net> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
                 CC|                            |tcallawa at redhat.com
         AssignedTo|nobody at fedoraproject.org    |nicolas.mailhot at laposte.net
               Flag|                            |fedora-review?,
                   |                            |needinfo?(tcallawa at redhat.c
                   |                            |om)




--- Comment #1 from Nicolas Mailhot <nicolas.mailhot at laposte.net>  2009-10-16 17:53:03 EDT ---
Wow, you don't know how long I've waited for someone to pick up this review
request (Antiquity is cool!)

Review:

1. don't really know if it's better to hang this on George Douros (gdouros-) or
Unicode Fonts for Ancient Scripts (ufas-). Probably a good idea to ask upstream
what they prefer. Please sync the wiki page name with the choice you make in
the rpm

2. You have a typo in the description (Aegea)

3. It'd be a good idea to convince the author to put his licensing info in the
zip (as a detached txt file). If his web site ever disappears the "Fonts in
this site are offered free for any use; they may be opened, edited, modified,
regenerated, posted, packaged and redistributed." statement will be lost (it's
in the "License" font metadata field but sadly our font viewers seem not to
display it right now so users will miss it

4. Since this is a non-standard licensing statement, please confirm with spot
that "Freely redistributable without restriction" is the right licensing string
to use

The rest of the package is pretty clean and was a pleasure to review, thank you

-- 
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.