Bug 2031375 - Review Request: rit-sundar-fonts - A traditional orthography display font for Malayalam script
Summary: Review Request: rit-sundar-fonts - A traditional orthography display font for...
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Parag AN(पराग)
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2021-12-11 12:33 UTC by Rajeesh
Modified: 2021-12-28 13:33 UTC (History)
3 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2021-12-28 13:33:11 UTC
Type: ---
Embargoed:
panemade: fedora-review+

Attachments (Terms of Use)

Description Rajeesh 2021-12-11 12:33:20 UTC 
Spec URL: https://rajeeshknambiar.fedorapeople.org/spec/rit-sundar-fonts.spec
SRPM URL: https://rajeeshknambiar.fedorapeople.org/srpm/rit-sundar-fonts-2.1.0-0.fc35.src.rpm
Description: A traditional orthography display font for Malayalam script
Fedora Account System Username: rajeeshknambiar
Comment 1 Parag Nemade 2021-12-18 04:56:55 UTC 
I prefer different bugzilla account for package reviews.
Comment 2 Parag AN(पराग) 2021-12-23 06:12:12 UTC 
Review Suggestion:
1) When new font packaging approved in Fedora, 'fontpackages' got obsoleted by 'fonts-rpm-macros' package. Hence, drop below from SPEC file
BuildRequires:    fontpackages-devel
Requires:    fontpackages-filesystem

2) The font family name in font file is "RIT Sundar" so correct it as
%global fontfamily RIT Sundar
in SPEC file

as the foundry and prefix/start of font family name is RIT (which is same word), the font package name remain same rit-sundar-fonts.

Rest looks good to me.

APPROVED.
Comment 3 Gwyn Ciesla 2021-12-27 16:54:07 UTC 
(fedscm-admin):  The Pagure repository was created at https://src.fedoraproject.org/rpms/rit-sundar-fonts
Note You need to log in before you can comment on or make changes to this bug.