https://bugzilla.redhat.com/show_bug.cgi?id=1814351
Bug ID: 1814351 Summary: Rename Request: google-carlito-fonts - Sans-serif font metric-compatible with Calibri font Product: Fedora Version: rawhide Status: NEW Component: Package Review Assignee: nobody@fedoraproject.org Reporter: pnemade@redhat.com QA Contact: extras-qa@fedoraproject.org CC: package-review@lists.fedoraproject.org Target Milestone: --- Classification: Fedora
Spec URL: https://download.copr.fedorainfracloud.org/results/pnemade/fedora-review/fed...
SRPM URL: https://download.copr.fedorainfracloud.org/results/pnemade/fedora-review/fed...
Description: Carlito is metric-compatible with Calibri font. Carlito comes in regular, bold, italic, and bold italic. The family covers Latin-Greek-Cyrillic (not a complete set, though) with about 2,000 glyphs. It has the same character coverage as Calibri. This font is sans-serif typeface family based on Lato.
Fedora Account System Username: pnemade
https://bugzilla.redhat.com/show_bug.cgi?id=1814351
Parag Nemade pnemade@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |nicolas.mailhot@laposte.net
--- Comment #1 from Parag Nemade pnemade@redhat.com --- Hi Nicolas,
Can you help with this package review? also other package review https://bugzilla.redhat.com/show_bug.cgi?id=1814349
Thanks, Parag.
https://bugzilla.redhat.com/show_bug.cgi?id=1814351
--- Comment #2 from Nicolas Mailhot nicolas.mailhot@laposte.net --- See also https://github.com/google/fonts/issues/1441#issuecomment-595684649
https://bugzilla.redhat.com/show_bug.cgi?id=1814351
Nicolas Mailhot nicolas.mailhot@laposte.net changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags| |fedora-review+
--- Comment #3 from Nicolas Mailhot nicolas.mailhot@laposte.net --- Wonderful clean up, thank you. It’s so nice to get font specs that can be diffed against a Fedora master template, saves hours in review.
Please switch to the github source when it gets available
Please try to use consistent vocabulary: font family, not font nor typeface (font is a single font file, typeface is a font face ie a single style). I know font makers mix them all and expect others to understand what that means, but users are not font buff jargon parsers. Font family has a clear definition in guidelines and is the term used in fontconfig.
Otherwise,
APPROVED
https://bugzilla.redhat.com/show_bug.cgi?id=1814351
Parag Nemade pnemade@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Summary|Rename Request: |Rename Request: |google-carlito-fonts - |google-carlito-fonts - |Sans-serif font |Carlito, a sans-serif font |metric-compatible with |family metric-compatible |Calibri font |with Calibri font family
https://bugzilla.redhat.com/show_bug.cgi?id=1814351
Parag Nemade pnemade@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED Assignee|nobody@fedoraproject.org |nicolas.mailhot@laposte.net
--- Comment #4 from Parag Nemade pnemade@redhat.com --- Thanks for this package review.
I will keep checking for github source availability.
Updated the Summary and description for this package as well for "font" to "font family".
https://download.copr.fedorainfracloud.org/results/pnemade/fedora-review/fed...
https://bugzilla.redhat.com/show_bug.cgi?id=1814351
--- Comment #5 from Gwyn Ciesla gwync@protonmail.com --- (fedscm-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/google-carlito-fonts
https://bugzilla.redhat.com/show_bug.cgi?id=1814351
Parag Nemade pnemade@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Fixed In Version| |google-carlito-fonts-1.103- | |0.13.20130920.fc33 Resolution|--- |RAWHIDE Last Closed| |2020-04-01 16:12:17
--- Comment #6 from Parag Nemade pnemade@redhat.com --- Thank you for this package review.
Built this package in google-carlito-fonts-1.103-0.13.20130920.fc33 build.
As this font has been specified as optional font in Fedora comps file, reported only https://pagure.io/fedora-comps/pull-request/479
package-review@lists.fedoraproject.org