[Bug 533919] Review Request: mplus fonts - The M+ family of fonts designed by Coji Morishita

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
                 CC|                            |fedora-fonts-bugs-list@redh
                   |                            |at.com
         AssignedTo|nobody@xxxxxxxxxxxxxxxxx    |nicolas.mailhot@xxxxxxxxxxx
               Flag|                            |fedora-review?,
                   |                            |needinfo?(igshaan.mesias@gm
                   |                            |ail.com)




--- Comment #1 from Nicolas Mailhot <nicolas.mailhot@xxxxxxxxxxx>  2009-11-28 09:57:38 EDT ---
Here is my package review. Sorry it took longer than usual, was busy hacking a
packager checker script (and given m+ is composed of 43 files, I'm so glad I
took the time to finish the script before looking at it; could not have done
the same level of review manually)

This is an awesome font set, thank you very much for packaging it!

Anyway, the review:

1. (not blocking) you don't include any fontconfig files, they would make your
user's life a lot easier (however, CJK is quite a mess in fontconfig, so maybe
it's better that way for now. Let's see what CJK users think of it)

2. (not blocking) rpmlint complains at the licensing string. I know the MPlus
license has been approved by spot, so it's not a legal problem. However I fear
all the repo-checking scripts will complain at you because of that. Please open
a fedora rpmlint bug so the license gets added to its list. (however upstream
would have made our life easier by using a standard font license such as the
OFL or the GPL with font exception)

3. (not blocking) rpmlint complains some of the lines you used in descriptions
are too long. Please break them in smaller (<80) bits

4. (blocking) same problem for at least one for your summaries

5. (not blocking) it seems rpmlint grew a spell-checker in fedora-devel. Please
fix the spelling mistakes it has identified (ie everything which is not a human
name)

6. (blocking) please make sure your Source0 is a full URL pointing to the
source file. That will make your future packager life a lot easier (we have
many sf-hosted projects in Fedora, so it can be done)

7. (not blocking) you don't need to repeat the "Group:    User Interface/X"
line in Fedora releases with a recent rpm

8. (not blocking) it would be nice if each sub-package had a different summary.
What is the difference between 1c and 2c ?

9. (blocking) m++ipa.pe is a build script, it has nothing to do in %doc

10. (not blocking) you don't need to specify %dir %{_fontdir} in the common
subpackage, it will be added automatically to each font subpackage

11. (not blocking) it's mostly cosmetic, but it would be nice if the font files
cased the style names they declare (ie "Bold" instead of "bold"). Apps are not
always smart enough to correct this

12. (not blocking) a few font files declare an "heavy" weight. The standard
qualifier for "heavy" is "Black". That means weight selection won't work as
expected in apps not smart enough to try every possible "Black" legacy alias.
Please ask upstream to change this (also please have upstream check they did
mean "Black" and not some other weight). Standard style qualifiers and their
meaning have been described by Microsoft in the following whitepaper:
http://blogs.msdn.com/text/attachment/2249036.ashx

13. (not blocking) the fonts fall just short of complete coverage for several
languages and unicode blocks (we only test if a language/block needs less than
10 glyphs to be finished). Please relay upstream so it gets a chance to
complete them (for example, ab is incomplete because the Unicode consortium
added two glyphs to it this year IIRC)

14. (not blocking) it would be nice if upstream included the text of their
license in the fonts copyright field, and not just "Copyright(c) 2009 M+ FONTS
PROJECT" without licensing info

15. (not blocking) it would be nice if upstream released a source archive with
source files and build scripts, so we can re-build in Fedora using our own
fontforge version (that helps finding bugs in fontforge, and as our fontforge
improves, so do the fonts we build with it)

Please fix whatever you can and relay the requests to upstream when
appropriate. I'll attach the output of latest repo-font-audit for your package
so you have a copy of rpmlint errors, font naming warnings, and font coverage
analysis.

There are a lot of requests here but your package is quite complex. Given the
number of packaged font files both you and upstream did a very good job, I've
seen much worse in packages that only included a couple font files.

NEEDINFO for now

-- 
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

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]