https://bugzilla.redhat.com/show_bug.cgi?id=2122022
Akira TAGOH <tagoh(a)redhat.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |tagoh(a)redhat.com
--- Comment #1 from Akira TAGOH <tagoh(a)redhat.com> ---
Before starting the formal review:
1. the foundry is supposed to be set from the foundry property in fontconfig
though, "PfEd" is too generic. if upstream doesn't have too much fonts
shipped
by them, you can simply omit that field. or if "Terminus" is a unique name
enough, you can use it instead, like DejaVu.
2. please set correct family name to the family. that should be "Terminus
(TTF)". this affects the AppStream file and the wrong name there makes harder
to find out the exact font name through AppStream.
3. You don't need "<test
name="lang"...><string>en</string>" thing in config.
--
You are receiving this mail because:
You are always notified about changes to this product and component
You are on the CC list for the bug.
https://bugzilla.redhat.com/show_bug.cgi?id=2122022