Bug 1805797 - Review Request: sil-andika-new-basic-fonts - A font family for literacy and beginning readers
Summary: Review Request: sil-andika-new-basic-fonts - A font family for literacy and b...
Keywords:
Status: CLOSED NEXTRELEASE
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: 1805921
TreeView+ depends on / blocked
 
Reported: 2020-02-21 15:02 UTC by Nicolas Mailhot
Modified: 2020-02-25 21:47 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2020-02-25 21:47:18 UTC
Type: ---
Embargoed:
panemade: fedora-review+


Attachments (Terms of Use)

Description Nicolas Mailhot 2020-02-21 15:02:13 UTC
Spec URL: https://download.copr.fedorainfracloud.org/results/nim/fonts-rpm-macros/fedora-rawhide-x86_64/01240860-sil-andika-new-basic-fonts/sil-andika-new-basic-fonts.spec

SRPM URL: https://download.copr.fedorainfracloud.org/results/nim/fonts-rpm-macros/fedora-rawhide-x86_64/01240860-sil-andika-new-basic-fonts/sil-andika-new-basic-fonts-5.500-1.fc33.src.rpm

Description:
Andika New Basic is a limited-character-set (no extended IPA or Cyrillic)
version of Andika that includes regular, bold, italic and bold-italic faces.

Andika is a sans serif, Unicode-compliant font family designed especially for
literacy use, taking into account the needs of beginning readers. The focus is
on clear, easy-to-perceive letterforms that will not be readily confused with
one another.

A sans serif font is preferred by some literacy personnel for teaching people
to read. Its forms are simpler and less cluttered than those of most serif
fonts. For years, literacy workers have had to make do with fonts that were
not really suitable for beginning readers and writers. In some cases, literacy
specialists have had to tediously assemble letters from a variety of fonts in
order to get all of the characters they need for their particular language
project, resulting in confusing and unattractive publications. Andika
addresses those issues.

Fedora Account System Username: nim



One of SIL’s Non-Roman Script Initiative.
https://scripts.sil.org/cms/scripts/page.php?site_id=nrsi&id=Home

SIL is a specialist of rare human scripts and contributes its expertise to the Unicode consortium. The objective is to get 100% of SIL’s font catalog in Fedora to improve our i18n coverage.

Due to $#@= variations on how SIL publishes its fonts, the spec uses some SIL-specific macros to hide those variations and limit the changes between the specs of SIL fonts.



The packaging conforms to https://pagure.io/packaging-committee/issue/935
as approved by FPC on 2020-02-13.

It is one of the test packages that were used to refine the new packaging guidelines
https://copr.fedorainfracloud.org/coprs/nim/fonts-rpm-macros/

The new fonts packaging build chain is now live in koji. For example:
https://koji.fedoraproject.org/koji/buildinfo?buildID=1468243

If the review is fast enough the package may make the FC32 100% Code Complete Deadline (2020-02-25)
https://fedorapeople.org/groups/schedule/f-32/f-32-key-tasks.html

Comment 1 Parag AN(पराग) 2020-02-23 10:13:25 UTC
Review:

Not going for full package review.

This package follows new fonts packaging guidelines.

Rpmlint
-------
Checking: sil-andika-new-basic-fonts-5.500-1.fc33.noarch.rpm
          sil-andika-new-basic-fonts-5.500-1.fc33.src.rpm
sil-andika-new-basic-fonts.noarch: W: spelling-error %description -l en_US letterforms -> letter forms, letter-forms, letterbombs
sil-andika-new-basic-fonts.src: W: spelling-error %description -l en_US letterforms -> letter forms, letter-forms, letterbombs
2 packages and 0 specfiles checked; 0 errors, 2 warnings.

Source checksums
----------------
https://software.sil.org/downloads/r/andika/AndikaNewBasic-5.500.zip :
  CHECKSUM(SHA256) this package     : 18308284a3e98cd7712d057742d8d6cd040a52ec0479a1dcf92d57815f8d368a
  CHECKSUM(SHA256) upstream package : 18308284a3e98cd7712d057742d8d6cd040a52ec0479a1dcf92d57815f8d368a


Requires
--------
sil-andika-new-basic-fonts (rpmlib, GLIBC filtered):
    config(sil-andika-new-basic-fonts)
    fontpackages-filesystem



Provides
--------
sil-andika-new-basic-fonts:
    config(sil-andika-new-basic-fonts)
    font(andikanewbasic)
    metainfo()
    metainfo(org.fedoraproject.sil-andika-new-basic-fonts.metainfo.xml)
    sil-andika-new-basic-fonts


All other things like license, config, metainfo files looks good.

APPROVED.

Comment 2 Nicolas Mailhot 2020-02-23 13:23:35 UTC
Thanks for the review!
https://pagure.io/releng/fedora-scm-requests/issue/22393

Comment 3 Nicolas Mailhot 2020-02-25 00:27:04 UTC
And now https://pagure.io/releng/fedora-scm-requests/issue/22513

Comment 4 Mohan Boddu 2020-02-25 21:19:05 UTC
(fedscm-admin):  The Pagure repository was created at https://src.fedoraproject.org/rpms/sil-andika-new-basic-fonts

Comment 5 Nicolas Mailhot 2020-02-25 21:46:57 UTC
Built for F32 (update) and F33

Thanks Parag and Gwyn


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