Bug 285591 - Review Request: lohit-fonts - font package for indic fonts
Review Request: lohit-fonts - font package for indic fonts
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
All Linux
medium Severity medium
: ---
: ---
Assigned To: Jens Petersen
Fedora Extras Quality Assurance
Depends On:
Blocks: 253158
  Show dependency treegraph
Reported: 2007-09-11 00:19 EDT by Rahul Bhalerao
Modified: 2007-11-30 17:12 EST (History)
4 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2007-09-24 09:34:45 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
petersen: fedora‑review+
petersen: fedora‑cvs+

Attachments (Terms of Use)
lohit-fonts.spec-1.patch (2.01 KB, patch)
2007-09-13 07:27 EDT, Jens Petersen
no flags Details | Diff
lohit-fonts.spec-2.patch (2.56 KB, patch)
2007-09-21 06:19 EDT, Jens Petersen
no flags Details | Diff
lohit-fonts.spec-3.patch (753 bytes, patch)
2007-09-21 12:44 EDT, Jens Petersen
no flags Details | Diff

  None (edit)
Description Rahul Bhalerao 2007-09-11 00:19:34 EDT
Spec URL: http://rbhalera.fedorapeople.org/lohit-fonts/lohit-fonts.spec
SRPM URL: http://rbhalera.fedorapeople.org/lohit-fonts/lohit-fonts-2.1.5-1.fc8.src.rpm
Description: This package provides the Hindi, Bengali, Gujarati, Punjabi, Tamil,
Kannada, Malayalam, Oriya, Telugu TrueType/Opentype fonts.
Comment 1 Jens Petersen 2007-09-11 02:52:44 EDT
rpmlint output:

 lohit-fonts.src: E: no-changelogname-tag

need a initial change entry explaining the package is derived from fonts-indic.

 lohit-fonts.src: W: invalid-license GPL

License field needs to be updated, see:

I guess it should be GPLv2.  Unless we have some explicit font embedding exception?
Comment 2 Rahul Bhalerao 2007-09-11 06:17:11 EDT
Updated the files with the required changes. 
Comment 3 Jens Petersen 2007-09-13 07:23:11 EDT
Thanks for the update.  Please bump the release number for each new revision.

The rpmlint output is clean now. :)

Presumably for Indic we don't need fonts.dir and fonts.scale?
How about a catalogue file?

My review follows:

+ rpmlint clean
+ package follows upstream project name
+ package is based on fonts-indic
+ following Packaging Guidelines except as noted below
+ license is good (GPLv2) and included
+ spec file is clearly written
+ md5sum is pristine
bb9497ee772062b97ff00a1a68b17c98  fonts-indic-2.1.5.tar.gz
+ builds correctly
+ filelists are correct
+ font install scriptlets are standard

- should not Requires fontconfig explicitly
Comment 4 Jens Petersen 2007-09-13 07:27:12 EDT
Created attachment 194541 [details]

- also should not obsolete fonts-indic packages - they will now require

and some other things I noticed.
Comment 5 Parag AN(पराग) 2007-09-13 08:13:05 EDT
from https://bugzilla.redhat.com/show_bug.cgi?id=281901#c5, retain old Source
tag instead one suggested in above patch.
Comment 6 Jens Petersen 2007-09-13 19:19:39 EDT
(In reply to comment #5)
> from https://bugzilla.redhat.com/show_bug.cgi?id=281901#c5, retain old Source
> tag instead one suggested in above patch.

Ok fair enough - I should have rebuilt before posting... ;)
(wget doesn't handle it well either - and we probably need a better
location of future releases anyway, having it on the wiki is a little

Please put the full URL as a comment above the Source line.
Comment 7 Rahul Bhalerao 2007-09-17 09:35:38 EDT
The changes have been made and the new srpm is available here:
Comment 8 Jens Petersen 2007-09-21 06:17:03 EDT
- please mention the actual changes made in the changelog
- should not require fontconfig (see discussion on fedora-packaging list)

rpmlint gives:
lohit-fonts-bengali.noarch: W: no-version-in-last-changelog
lohit-fonts-gujarati.noarch: W: no-version-in-last-changelog
lohit-fonts-hindi.noarch: W: no-version-in-last-changelog
lohit-fonts-kannada.noarch: W: no-version-in-last-changelog
lohit-fonts-malayalam.noarch: W: no-version-in-last-changelog
lohit-fonts-oriya.noarch: W: no-version-in-last-changelog
lohit-fonts-punjabi.noarch: W: no-version-in-last-changelog
lohit-fonts-tamil.noarch: W: no-version-in-last-changelog
lohit-fonts-telugu.noarch: W: no-version-in-last-changelog
Comment 9 Jens Petersen 2007-09-21 06:19:34 EDT
Created attachment 201881 [details]

Here is a bunch of cleanup and fixes.

Probably this should be sufficient for this review.
Please review and make a changelog for it.
Comment 10 Jens Petersen 2007-09-21 12:44:50 EDT
Created attachment 202561 [details]

additional patch to add lohit to fontdir names
Comment 11 Parag AN(पराग) 2007-09-24 03:19:00 EDT
Spec URL: http://paragn.fedorapeople.org/lohit-fonts.spec
SRPM URL: http://paragn.fedorapeople.org/lohit-fonts-2.1.5-3.fc7.src.rpm

please review this.
Comment 12 Jens Petersen 2007-09-24 03:47:24 EDT
Here is the review:

+ rpmlint clean
+ package is named after upstream project name
+ meets packaging guidelines
+ license is gpl and included
+ spec files is clearly written
+ source is pristine
bb9497ee772062b97ff00a1a68b17c98  fonts-indic-2.1.5.tar.gz
+ noarch package builds correctly
+ owns all dirs except /usr/share/fonts (owned by fontconfig)
+ macro usage is consistent

All must items are satisfied afaict.

Package is APPROVED.
Comment 13 Parag AN(पराग) 2007-09-24 04:21:11 EDT
New Package CVS Request
Package Name: lohit-fonts
Short Description: Lohit TrueType fonts for Indic languages
Owners: rbhalera
Branches: devel
InitialCC: petersen
Cvsextras Commits: yes

Comment 14 Parag AN(पराग) 2007-09-24 04:29:50 EDT
  Thanks for review. 
Comment 15 Jens Petersen 2007-09-24 05:10:02 EDT
cvsadmin done
Comment 16 Jens Petersen 2007-09-24 09:34:45 EDT
Thanks Parag for getting the package built.
Comment 17 Jens Petersen 2007-10-07 23:21:21 EDT
Maybe my indications on the license were not quite accurate.
According to http://fedoraproject.org/wiki/Licensing/FAQ we reach (4)
of the first question ("How do I figure out what version of the GPL/LGPL
my package is under?") and so COPYING implies GPLv2+, if I'm not mistaken.

Please update the License field to that in the next update.

We should really make the version explicit in the README file say for the next 
release of Lohit fonts though (the .ttf files themselves just say GPL).

Note You need to log in before you can comment on or make changes to this bug.