Bug 771837

Summary:	Review Request: gubbi-fonts - Free Kannada Opentype serif font
Product:	[Fedora] Fedora	Reporter:	Pravin Satpute <psatpute>
Component:	Package Review	Assignee:	Parag AN(पराग) <panemade>
Status:	CLOSED RAWHIDE	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	16	CC:	i18n-bugs, notting, package-review, panemade, svenkate
Target Milestone:	---	Keywords:	i18n
Target Release:	---	Flags:	panemade: fedora-review+ gwync: fedora-cvs+
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2012-01-10 07:12:47 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:

Description Pravin Satpute 2012-01-05 07:34:38 UTC

SPEC Url: http://pravins.fedorapeople.org/gubbi-fonts.spec
SRPM Url: http://pravins.fedorapeople.org/gubbi-fonts-1.0-1.fc16.src.rpm

Scratch build : http://koji.fedoraproject.org/koji/taskinfo?taskID=3621137

Comment 1 Parag AN(पराग) 2012-01-06 08:30:48 UTC

Review:

+ koji scratch build ->http://koji.fedoraproject.org/koji/taskinfo?taskID=3621137

+ rpmlint on rpms gave
gubbi-fonts.noarch: W: spelling-error %description -l en_US opentype -> open type, open-type, phenotype
gubbi-fonts.src: W: spelling-error %description -l en_US opentype -> open type, open-type, phenotype
2 packages and 0 specfiles checked; 0 errors, 2 warnings.

+ source verified with upstream as (sha1sum)
91dd2e35ab6ac58dd3efcf0ee63109e1901a54bb  gubbi-1.0.tar.gz
91dd2e35ab6ac58dd3efcf0ee63109e1901a54bb  ../SOURCES/gubbi-1.0.tar.gz

+ follows fonts packaging guidelines


Suggestions:
1) you should drop the versioned dependency. Change
BuildRequires: fontforge >= 20080429
to
BuildRequires: fontforge

2)you don't need to clean buildroot now. remove following from spec
BuildRequires: fontforge >= 20080429


APPROVED.

Comment 2 Pravin Satpute 2012-01-09 07:32:50 UTC

Thank you Parag for review, i will do the suggested changes during git import.

New Package SCM Request
=======================
Package Name: gubbi-fonts
Short Description: Free Kannada Opentype serif font
Owners: pravins
Branches: f16
InitialCC: i18n-team

Comment 3 Gwyn Ciesla 2012-01-09 13:07:51 UTC

Git done (by process-git-requests).

Comment 4 Pravin Satpute 2012-01-10 07:12:47 UTC

Thanks Jon for Git process. done built for rawhide and F16