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

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|[email protected] |[email protected]
               Flag|fedora-review?              |fedora-review+

--- Comment #7 from Nicolas Mailhot <[email protected]> 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
[email protected]
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to