[Bug 1805725] Review Request: gfs-orpheus-fonts - A 21st century Greek font family

2020-02-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1805725

Nicolas Mailhot  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |NEXTRELEASE
Last Closed||2020-02-24 23:04:56



--- Comment #4 from Nicolas Mailhot  ---
Done for F32 & F33. Thanks Parag and Gwyn.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1805725] Review Request: gfs-orpheus-fonts - A 21st century Greek font family

2020-02-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1805725



--- Comment #3 from Gwyn Ciesla  ---
(fedscm-admin):  The Pagure repository was created at
https://src.fedoraproject.org/rpms/gfs-orpheus-fonts

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1805725] Review Request: gfs-orpheus-fonts - A 21st century Greek font family

2020-02-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1805725



--- Comment #2 from Nicolas Mailhot  ---
Fixed everything except the font tech speak.

Thanks for the review!

https://pagure.io/releng/fedora-scm-requests/issue/22482

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1805725] Review Request: gfs-orpheus-fonts - A 21st century Greek font family

2020-02-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1805725

Parag AN(पराग)  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||panem...@gmail.com
   Assignee|nob...@fedoraproject.org|panem...@gmail.com
  Flags||fedora-review+



--- Comment #1 from Parag AN(पराग)  ---
Review:

This package follows new fonts packaging guidelines.

Rpmlint
---
Checking: gfs-orpheus-fonts-20161102-1.fc33.noarch.rpm
  gfs-orpheus-fonts-doc-20161102-1.fc33.noarch.rpm
  gfs-orpheus-fonts-20161102-1.fc33.src.rpm
gfs-orpheus-fonts.noarch: W: spelling-error %description -l en_US descenders ->
descender, descender s, descends
gfs-orpheus-fonts-doc.noarch: W: summary-not-capitalized C gfs-orpheus-fonts
optional documentation files
gfs-orpheus-fonts-doc.noarch: E: description-line-too-long C This package
provides optional documentation files shipped with gfs-orpheus-fonts.
gfs-orpheus-fonts-doc.noarch: E: non-readable
/usr/share/doc/gfs-orpheus-fonts-doc/GFS_Orpheus_Specimen.pdf 640
gfs-orpheus-fonts.src: W: spelling-error %description -l en_US descenders ->
descender, descender s, descends
3 packages and 0 specfiles checked; 2 errors, 3 warnings.

Source checksums

http://www.greekfontsociety-gfs.gr/_assets/fonts/GFS_Orpheus.zip :
  CHECKSUM(SHA256) this package :
7bfe591a3d31d13edfe82f6ba968ad1b0fcd2ee5ecb579b9837181949679752c
  CHECKSUM(SHA256) upstream package :
7bfe591a3d31d13edfe82f6ba968ad1b0fcd2ee5ecb579b9837181949679752c


Requires

gfs-orpheus-fonts (rpmlib, GLIBC filtered):
config(gfs-orpheus-fonts)
fontpackages-filesystem

gfs-orpheus-fonts-doc (rpmlib, GLIBC filtered):



Provides

gfs-orpheus-fonts:
config(gfs-orpheus-fonts)
font(gfsorpheus)
gfs-orpheus-fonts
metainfo()
metainfo(org.fedoraproject.gfs-orpheus-fonts.metainfo.xml)

gfs-orpheus-fonts-doc:
gfs-orpheus-fonts-doc


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

Issues:
1) Fix the rpmlint warnings/errors before this package import

APPROVED

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1805725] Review Request: gfs-orpheus-fonts - A 21st century Greek font family

2020-02-21 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1805725

Nicolas Mailhot  changed:

   What|Removed |Added

 CC||fonts-bugs@lists.fedoraproj
   ||ect.org
   Doc Type|--- |If docs needed, set a value



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org