Bug 1190048 - Review Request: pecita-fonts - non-commercial font with OFL license
Summary: Review Request: pecita-fonts - non-commercial font with OFL license
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-02-06 07:37 UTC by Carlos Morel-Riquelme
Modified: 2015-03-03 16:57 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2015-03-03 16:57:33 UTC
panemade: fedora-review?


Attachments (Terms of Use)

Description Carlos Morel-Riquelme 2015-02-06 07:37:52 UTC
Spec URL: https://empateinfinito.fedorapeople.org/font/pecita-fonts/pecita-fonts.spec

SRPM URL: https://empateinfinito.fedorapeople.org/font/pecita-fonts/pecita-fonts-0.1-1.20140527.fc21.src.rpm

Description: Pecita is a typeface that mimics handwriting
The use of complex features makes this possible
The OpenType features Localized form substitution, Superscript ,
Subscript, Small Capitals, Standard Ligatures and Contextual Alternates
are implemented.

Fedora Account System Username: empateinfinito

FE-NEEDSPONSOR

Comment 1 Parag AN(पराग) 2015-02-06 17:00:40 UTC
Review:

+ Mock build is successful for F22(x86_64)

+ rpmlint output on all generated rpm looks good
2 packages and 0 specfiles checked; 0 errors, 0 warnings.

+ Source verified with upstream as (sha356sum)
upstream source: 5e0783209552d6a6d3f58b2fa8c191d8331fe60a29ed17576d6bdd5614a6eb2c
srpm source: 5e0783209552d6a6d3f58b2fa8c191d8331fe60a29ed17576d6bdd5614a6eb2c

+ License "OFL" is valid and included in source files.

+ fontconfig file looks good

+ font metadata information is present

Suggestions:
1) For Handwriting category font we use "cursive" name so change sans-serif to cursive in fontconfig file

2) version 4.3 is embedded in font metainfo so let's use that

3) $ appstream-util validate pecita.metainfo.xml 
pecita.metainfo.xml: FAILED:
• markup-invalid        : <id> does not have correct extension for kind
• tag-missing           : <extends> is not present
• style-invalid         : <summary> requires sentence case
• style-invalid         : <p> cannot contain a hyperlink
• style-invalid         : <p> does not end in '.|:|!'
• style-invalid         : Not enough <p> tags for a good description
Validation of files failed

=> fix messages 3,4,5 from above in metainfo xml file

4) Create wiki page for this font as given in your other font package review.

5) use oflb as prefix foundry

Comment 2 Parag AN(पराग) 2015-02-06 17:16:54 UTC
Just realized this font fits in fontconfig priority level 61-64. Other existing oflb LGC fonts are at 63 level. Choose 61 or higher maybe 63.

│ 61-64  │ Low priority LGC fonts                                             │

Comment 4 Parag AN(पराग) 2015-03-03 16:57:33 UTC
FAS user empateinfinito moved to new FAS account name iddnna


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