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=507615 Nicolas Mailhot <nicolas.mailhot@xxxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED AssignedTo|nobody@xxxxxxxxxxxxxxxxx |sshedmak@xxxxxxxxxx Flag| |fedora-review+ --- Comment #3 from Nicolas Mailhot <nicolas.mailhot@xxxxxxxxxxx> 2009-06-24 17:22:24 EDT --- 1. not sure if it should be classified as sans-serif, I guess it's up for Telugu users to decide 2. Please work a bit more on the package summary (at minimum, start with "A free OpenType font" not "Its Free OpenType font") 3. Fontforge complains a lot about this TTF file. It seems it was created by copying Pothana2000, and it still declares Pothana2000 all over the place. Please ask politely upstream to clean up the font metadata (remove the restricted bit, change the Pothana2000 strings to Vemana2000, change the font style to Regular or whatever standard style is appropriate). You can even do it yourself with Fedora's fontforge and send the fixed file upstream if the author agrees (I see I had noted the non-standard style for Pothana2000 too, even though it's less broken metadata-side than Vemana200). However, none of this is strictly speaking rpm packaging, so I'll approve the package and let you deal with the TTF bugs as the font Fedora packager :p ☘☘☘ APPROVED ☘☘☘ This review is finished and you're now on you own to build, push and register your package everywhere it matters. Your next steps are documented on: http://fedoraproject.org/wiki/Font_package_lifecycle#3.a Thank you for packaging a new Fedora font! Do it again please :) If you still need sponsoring, apply to the Packager group in FAS and post your login here, so I can sponsor you. If you have questions or need more help, don't hesitate to ask questions in ##fonts or ##fedora-devel or on the fonts sig mailing list http://fedoraproject.org/wiki/Fonts_SIG_mailing_lists -- 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