Hi!

Looks good. One glitch though, I don't see the version being provided by upstream anywhere. In such cases it's preferred to use the time stamp.


# Do not trust font metadata versionning unless you've checked upstream does
# update versions on file changes. When in doubt use the timestamp of the most
# recent file as version. “1.0” versions especially are suspicious.

Please check this up. I'll do a complete review later today. 

Thanks,
Ankur

Thanks for doing this.

Updated spec file: http://susmit.fedorapeople.org/packaging/kanotf/kanotf-fonts.spec

Update SRPM: http://susmit.fedorapeople.org/packaging/kanotf/kanotf-fonts-20050515-1.fc14.src.rpm

+ Package meets naming and packaging guidelines
+ Spec file matches base package name.
+ Spec has consistant macro usage.
+ Meets Packaging Guidelines.
+ License
+ License field in spec matches
+ License file included in package
+ Spec in American English
+ Spec is legible.
+ Sources match upstream md5sum:

- Package needs ExcludeArch
+ BuildRequires correct
- Spec handles locales/find_lang
- Package is relocatable and has a reason to be.
+ Package has %defattr and permissions on files is good.
- Package has a correct %clean section.
- Package has correct buildroot
%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
- Package is code or permissible content.
- Doc subpackage needed/used.
+ Packages %doc files don't affect runtime.

- Headers/static libs in -devel subpackage.
- Spec has needed ldconfig in post and postun
- .pc files in -devel subpackage/requires pkgconfig
- .so files in -devel subpackage.
- -devel package Requires: %{name} = %{version}-%{release}
- .la files are removed.

- Package is a GUI app and has a .desktop file

+ Package compiles and builds on at least one arch.
+ Package has no duplicate files in %files.
+ Package doesn't own any directories other packages own.
+ Package owns all the directories it creates.
? No rpmlint output.
+ final provides and requires are sane:
(include output of for i in *rpm; do echo $i; rpm -qp --provides $i; echo =; rpm -qp --requires $i; echo; done
manually indented after checking each line.  I also remove the rpmlib junk and anything provided by glibc.)
[ankur@ankur result]$ review-req-check 
== kanotf-fonts-20050515-1.fc16.noarch.rpm ==
Provides:
config(kanotf-fonts) = 20050515-1.fc16
font(kedage)  
font(mallige)  
kanotf-fonts = 20050515-1.fc16

Requires:
/bin/sh  
/bin/sh  
config(kanotf-fonts) = 20050515-1.fc16
fontpackages-filesystem  

== kanotf-fonts-20050515-1.fc16.src.rpm ==
Provides:

Requires:
fontpackages-devel  

SHOULD Items:

+ Should build in mock.
- Should build on all supported archs
- Should function as described.
- Should have sane scriptlets.
- Should have subpackages require base package with fully versioned depend.
+ Should have dist tag
+ Should package latest version
- check for outstanding bugs on package. (For core merge reviews)

Issues:

1. rpmlint output:

[ankur@ankur SRPMS]$ rpmlint ../SPECS/kanotf-fonts.spec kanotf-fonts-20050515-1.fc14.src.rpm /var/lib/mock/fedora-rawhide-i386/result/*.rpm
../SPECS/kanotf-fonts.spec: W: invalid-url Source0: http://sourceforge.net/projects/brahmi/files/Brahmi%20OpenType%20Fonts/OpenType%20font%20for%20Kannada%20-%20Kedgae%20and%20Mallige%20Ver%201.0/kanotf.zip <urlopen error timed out>
kanotf-fonts.noarch: W: incoherent-version-in-changelog 20050515 ['20050515-1.fc16', '20050515-1']
kanotf-fonts.noarch: W: wrong-file-end-of-line-encoding /usr/share/doc/kanotf-fonts-20050515/readme.txt
3 packages and 1 specfiles checked; 0 errors, 3 warnings.


You forgot to add the release tag in the ChangeLog :)
Please do also correct the wrong-file-end-of-line-encoding error. 

The rest looks okay. Please make these corrections and I'll approve the package

Ankur

Done.
Thanks.

All right then. XXX Approved XXX

Thank you very much Ankur.

New Package SCM Request
=======================
Package Name: kanotf-fonts
Short Description: OpenType Kannada fonts
Owners: susmit
Branches: f14
InitialCC:

Did you really not want an f15 branch?

Oops..I forgot that rawhide is now f16.
Thanks for the correction.

New Package SCM Request
=======================
Package Name: kanotf-fonts
Short Description: OpenType Kannada fonts
Owners: susmit
Branches: f14, f15
InitialCC:

New Package SCM Request
=======================
Package Name: kanotf-fonts
Short Description: OpenType Kannada fonts
Owners: susmit
Branches: f14 f15
InitialCC:

Git done (by process-git-requests).

Package kanotf-fonts-20050515-1.fc15:
* should fix your issue,
* was pushed to the Fedora 15 updates-testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=updates-testing kanotf-fonts-20050515-1.fc15'
as soon as you are able to, then reboot.
Please go to the following url:
https://admin.fedoraproject.org/updates/kanotf-fonts-20050515-1.fc15
then log in and leave karma (feedback).

Thanks Jason.
Closing as NEXTRELEASE.

kanotf-fonts-20050515-1.fc15 has been pushed to the Fedora 15 stable repository.  If problems still persist, please make note of it in this bug report.