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@xxxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED CC| |tcallawa@xxxxxxxxxx AssignedTo|nobody@xxxxxxxxxxxxxxxxx |nicolas.mailhot@xxxxxxxxxxx Flag| |fedora-review?, | |needinfo?(tcallawa@xxxxxxxx | |om) --- Comment #1 from Nicolas Mailhot <nicolas.mailhot@xxxxxxxxxxx> 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-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review