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

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
                 CC|                            |[email protected]
         AssignedTo|[email protected]    |[email protected]
               Flag|                            |fedora-review?,
                   |                            |needinfo?([email protected]
                   |                            |om)




--- Comment #1 from Nicolas Mailhot <[email protected]>  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.

_______________________________________________
Fedora-fonts-bugs-list mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/fedora-fonts-bugs-list

Reply via email to