[Bug 1142445] Review Request: wireone-fonts - Wire One font by Alexei Vanyashin and Gayaneh Bagdasaryan

bugzilla at redhat.com bugzilla at redhat.com
Wed Sep 17 05:50:09 UTC 2014


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



--- Comment #1 from Parag AN(पराग) <panemade at gmail.com> ---
Review:

+ Package builds fine in mock in F22

+ rpmlint on generated rpms gave output
wireone-fonts.noarch: W: spelling-error %description -l en_US monoline -> mono
line, mono-line, monolingual
wireone-fonts.noarch: W: spelling-error %description -l en_US nouveau -> Noumea
wireone-fonts.noarch: W: spelling-error %description -l en_US px -> PX, ox, p
wireone-fonts.src: W: spelling-error %description -l en_US monoline -> mono
line, mono-line, monolingual
wireone-fonts.src: W: spelling-error %description -l en_US nouveau -> Noumea
wireone-fonts.src: W: spelling-error %description -l en_US px -> PX, ox, p
2 packages and 0 specfiles checked; 0 errors, 6 warnings.


+ source verified with upstream as (sha256sum)
packaged source:
1cf739c8fc17663059c2544be19cf590404e17fc9ec1778a080e13de5282e9fc
upstream source:
1cf739c8fc17663059c2544be19cf590404e17fc9ec1778a080e13de5282e9fc

+ License is OFL and included in its own text file OFL.txt

+ fontconfig file looks good

+ follows fonts packaging guidelines.

Suggestions:
1) cp command should be used with option '-p'

2) I think here you can use cyreal foundry as can be seen in output
ttname WireOne.ttf  | grep "#8"

APPROVED.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=2vVX4zXt5I&a=cc_unsubscribe


More information about the fonts-bugs mailing list