Bug 1825183
| Summary: | Review Request: hanamin-fonts - Japanese Mincho-typeface TrueType font | ||
|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Akira TAGOH <tagoh> |
| Component: | Package Review | Assignee: | Nicolas Mailhot <nicolas.mailhot> |
| Status: | CLOSED NEXTRELEASE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
| Severity: | medium | Docs Contact: | |
| Priority: | medium | ||
| Version: | rawhide | CC: | fonts-bugs, package-review, pnemade, ppisar |
| Target Milestone: | --- | Flags: | nicolas.mailhot:
fedora-review+
|
| Target Release: | --- | ||
| Hardware: | All | ||
| OS: | Linux | ||
| Whiteboard: | |||
| Fixed In Version: | Doc Type: | If docs needed, set a value | |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | 2020-05-28 05:41:25 UTC | Type: | --- |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | --- | Target Upstream Version: | |
| Embargoed: | |||
|
Description
Akira TAGOH
2020-04-17 10:35:20 UTC
This is to rename hanazono-fonts. Ok, quick review: 1. the spec applies official Fedora font templates 2. you do not strictly need the common_description indirection when generating a single font package, but it does no wrong either 4. since we only ship OpenType fonts nowadays, maybe it is not useful to remind the font family format in Summary and description 5. usual rpmlint checking as required in review: $ rpmlint /srv/rpm/fonts-rpm-macros/hanamin-fonts-20170904.fc33.noarch.rpm hanamin-fonts.noarch: W: spelling-error %description -l en_US Hanazono -> Shannon hanamin-fonts.noarch: W: spelling-error %description -l en_US glyphwiki -> glyph wiki, glyph-wiki, hieroglyph hanamin-fonts.noarch: W: wrong-file-end-of-line-encoding /usr/share/doc/hanamin-fonts/README.txt hanamin-fonts.noarch: W: wrong-file-end-of-line-encoding /usr/share/doc/hanamin-fonts/THANKS.txt Please fix the wrong-file-end-of-line-encoding,warnings using %linuxtext or any other method Thanks for the conversion APPROVED I thought this can be packaged like this https://download.copr.fedorainfracloud.org/results/pnemade/fedora-review/fedora-rawhide-x86_64/01345510-hanamin-fonts/hanamin-fonts.spec Hi Parag
spec cleanliness aside (and I don’t pretend to be a CJK expert) I think Akira is right, we’re not in presence of two font families, but a single one, split over two files to workaround the OpenType pre-file glyph number limit. And that will work fine because fontconfig will merge the files as a single family.
While
fc-scan -f "%{family[0]};%{style[0]};%{fullname[0]};%{width};%{weight};%{slant};%{fontversion};%{file}\n" /usr/share/fonts/clm-* |sort -t ';' -k1,1d -k4,4n -k5,5n -k6,6n -k2,2d -k7,7dr | uniq | column --separator ';' -t
is horrible from a usability POW, its results are useful
Thank you, I understood this font family packaging now. Fixed doc thing and requested creating new repo for this now. Thank you for the review. (fedscm-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/hanamin-fonts The package can't be built: https://kojipkgs.fedoraproject.org//work/tasks/7968/44477968/build.log Okay, the package has been built on rawhide: https://koji.fedoraproject.org/koji/buildinfo?buildID=1516776 |