[Bug 285561] Review Request: wqy-unibit-fonts - a dual-width bitmap font for maximum unicode coverage

bugzilla at redhat.com bugzilla at redhat.com
Mon Sep 24 08:58:25 UTC 2007


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

Summary: Review Request: wqy-unibit-fonts - a dual-width bitmap font for maximum unicode coverage


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





------- Additional Comments From petersen at redhat.com  2007-09-24 04:58 EST -------
Ok, thanks, that is all ok.  Sorry to keep you waiting so long.

Below is my review.

Good:
+ rpmlint is ok
wqy-unibit-fonts.noarch: W: symlink-should-be-relative
/etc/X11/fontpath.d/wqy-unibit-fonts /usr/share/fonts/wenquanyi/wqy-unibit
(this can be waived or you can make it relative by prepending "../.." if you wish.
+ naming is based on upstream name
+ meets packaging guidelines
+ license is gplv2 and included
+ spec file is clear
+ source is pristine
751dacd1326cd49b44486b45c592cfa6  wqy-unibit-bdf-1.1.0-1.tar.gz
+ noarch package builds correctly
+ buildreqs listed
+ filelist ok

Except the following, all must items are satisfied AFAICS.

Needs attention:
- both wqy-bitmap-fonts and this package own /usr/share/fonts/wenquanyi
Personally I don't feel it is a blocker, but it is against the packaging
guidelines: you should probably consider just using
/usr/share/fonts/wenquanyi-unibit for this package and
similarly for the other package.

- Do you think there will be a wqy-unibit-ttf say one day?
(just worrying if the name were to be too general)


-- 
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, or are watching someone who is.




More information about the Fedora-package-review mailing list