Bug 2031374

Summary: Review Request: rit-panmana-fonts - Open Type body text font for Malayalam traditional script
Product: [Fedora] Fedora Reporter: Rajeesh <rajeeshknambiar>
Component: Package ReviewAssignee: Parag AN(पराग) <panemade>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: package-review, panemade, vvijayra
Target Milestone: ---Flags: panemade: 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: 2021-12-28 13:41:05 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 Rajeesh 2021-12-11 12:31:51 UTC
Spec URL: https://rajeeshknambiar.fedorapeople.org/spec/rit-panmana-fonts.spec
SRPM URL: https://rajeeshknambiar.fedorapeople.org/srpm/rit-panmana-fonts-1.2-0.fc35.src.rpm
Description: A body text Malayalam Unicode font
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-22 08:28:51 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) You can add fontconfig file instead of just %{nil}
%global fontconfs meta/67-panmana-fonts.conf

Rest looks good.
APPROVED.

Comment 3 Gwyn Ciesla 2021-12-27 17:47:27 UTC
(fedscm-admin):  The Pagure repository was created at https://src.fedoraproject.org/rpms/rit-panmana-fonts