Bug 1237281 - Review Request: bashithacomplex-fonts - Sinhala fonts
Summary: Review Request: bashithacomplex-fonts - Sinhala fonts
Keywords:
Status: CLOSED NOTABUG
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: 2015-06-30 17:33 UTC by Danishka Navin
Modified: 2017-12-22 07:50 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2017-12-22 07:50:21 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Comment 1 Parag AN(पराग) 2015-07-03 03:08:29 UTC
Review:
+ Package built fine for F23 x86_64

+ rpmlint on all generated rpms gave output
bashithacomplex-fonts.noarch: W: spelling-error Summary(en_US) Bashitha -> Bashing
bashithacomplex-fonts.noarch: W: no-documentation
bashithacomplex-fonts.src: W: spelling-error Summary(en_US) Bashitha -> Bashing
2 packages and 0 specfiles checked; 0 errors, 3 warnings.

+ Source verified with upstream as (sha256sum)
Source in srpm:8460b239ee8fd0d92fcb2762b411d2560ba91b45958d49a24b1ee5a052d47008
Upstream source:8460b239ee8fd0d92fcb2762b411d2560ba91b45958d49a24b1ee5a052d47008

+ follows fonts packaging guidelines

Suggestions:
1) Correct the release tag as
Release:        0.1.%{checkout}%{?dist}

2) You should also add metainfo file, see sample at http://pkgs.fedoraproject.org/cgit/lklug-fonts.git/tree/

and also do changes in spec file to install this bashithacomplex.metainfo.xml file

3) Note following lines are optional now in Fedora packages
rm -fr %{buildroot}
and
%clean
rm -fr %{buildroot}

4) Ask upstream to provide license somewhere on their website which we can download and add to this package. License is not clear just by downloading ttf file. Also, on upstream website they used word "BhashitaScreen" and it is not related to the font file we are packaging so not sure what license is for this BhashitaComplex font.

Comment 2 Danishka Navin 2015-07-03 05:35:16 UTC
Spec URL: https://snavin.fedorapeople.org/packages/bashithacomplex-fonts/bashithacomplex-fonts.spec

SRPM URL: https://snavin.fedorapeople.org/packages/bashithacomplex-fonts/bashithacomplex-fonts-1.000-0.2.20150628.fc21.src.rpm

Fixed succession 1 and 2.


I already contacted upstream to provide license file, before start packaging. I hope they will provide it soon.

Comment 3 Parag AN(पराग) 2017-06-13 11:14:16 UTC
Any updates? Otherwise I will prefer to close this. Once any update is available either reopen this review or file a new package review.

Will close this in 2 weeks.


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