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=533919 Nicolas Mailhot <nicolas.mailhot@xxxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|nicolas.mailhot@xxxxxxxxxxx |igshaan.mesias@xxxxxxxxx Flag|fedora-review? |fedora-review+ --- Comment #7 from Nicolas Mailhot <nicolas.mailhot@xxxxxxxxxxx> 2010-02-22 12:49:45 EST --- This is awesome packaging, thank you for your work. I'm sure it was at least as scary to make as it was to review, you'll probably find other font sets easier to package in the future (superfamilies are not that common) Anyway 1. (not blocking) add some fontconfig rules someday 2. (not blocking) spell-checking fixes: They are x weights ⇒ Includes x weights emphasize the balance ⇒ emphasizes the balance But for such a complex package that's nothing ㆫㆫㆫ APPROVED ㆫㆫㆫ Attached the a repo-font-audit check of your last packages. Naming and coverage problems only, to relay upstream ⇒ RE-ASSIGING -- 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. _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review