[Bug 225761] Merge Review: fonts-chinese

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Merge Review: fonts-chinese


https://bugzilla.redhat.com/show_bug.cgi?id=225761


petersen at redhat.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEEDINFO                    |ASSIGNED
               Flag|needinfo?(petersen at redhat.co|
                   |m)                          |




------- Additional Comments From petersen at redhat.com  2007-09-28 02:40 EST -------
Basically looks fine to me.

A few comments:

- better to use: %defattr(-,root,root,-)
- I'd put taipeifonts after cjkunifonts in the requires.
- please describe in the changelog that the change is that
  taipei fonts was moved to a separate package taipeifonts.


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.