[Bug 1162148] Review Request: labiryntowy-fonts - Conscript (artifical font) letters consist of vertical and horizontal bars. No matter their thickness.

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

 



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



--- Comment #24 from Parag AN(पराग) <panemade@xxxxxxxxx> ---
1) Right its optional but I recommend to have fontconfig files for fonts.
Though this font is not covering any non CGL languages (only Latin) so it can
be optional but I will say add generic names rule file. You can use this font
style as fantasy in fontconfig.

2) Just by looking into this line
%{_font_pkg} %{_fontdir}
one can easily say its going to give duplicate dir installation warning. 

Let's check on koji for this and I can see in build.log this
warning: File listed twice: /usr/share/fonts/labiryntowy

recommended way is 
%_font_pkg *.ttf -f %{fontconf}

3) Marwin,
    I observed either something is broken in this packaged font or in ttname
package itself. But I have ran ttname on some fonts in fedora and it worked
well. It helped to fix copyright as well as license metadata in font files but
for this font its failing. Maybe you want to report this to upstream and get
the font metadata fixed as given in fonts guidelines.
   Also, When you do Ctrl+Shift+f in fontforge and click on "TTF names" in left
side, you will see on right side there is no license or license url field so
indeed that information is missing.

Check this ->
https://fedoraproject.org/wiki/Packaging:FontsPolicy#Correcting_the_metadata_with_ttname

4) I have recently added appdata metainfo file to many fonts packages in F21
and F22. This package also need to add metainfo file. I know this is not again
in Guidelines but Richard Hughes wants this. See
https://lists.fedoraproject.org/pipermail/devel/2014-October/203394.html

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review





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