Bug 398551

Summary:	Review Request: gfs-theokritos-fonts - GFS Theokritos decorative font
Product:	[Fedora] Fedora	Reporter:	Nicolas Mailhot <nicolas.mailhot>
Component:	Package Review	Assignee:	Parag AN(पराग) <panemade>
Status:	CLOSED RAWHIDE	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	rawhide	CC:	fedora-package-review, fonts-bugs, notting
Target Milestone:	---	Flags:	panemade: fedora-review+ kevin: fedora-cvs+
Target Release:	---
Hardware:	All
OS:	Linux
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2007-11-27 21:38:35 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Nicolas Mailhot 2007-11-25 18:12:42 UTC

Spec URL: http://nim.fedorapeople.org/gfs-theokritos-fonts.spec
SRPM URL: http://nim.fedorapeople.org/gfs-theokritos-fonts-20070415-1.fc9.src.rpm

Description:
Yannis Kefallinos (1894–1958) was one of the most innovative engravers of his 
generation and the first who researched methodicaly the aesthetics of book and 
typographic design in Greece. He taught at the Fine Arts School of Athens and 
established the first book design workshop from which many practising artists 
of the 60's and 70's had graduated.

In the late 50's Kefallinos designed and published an exquisite book with 
engraved illustrations of the ancient white funerary pottery in Attica in 
collaboration with Varlamos, Montesanto, Damianakis. For the text of 
Kefallinos' Δέκα λευκαί λήκυθοι (1956) the artist used a typeface which he 
himself had designed a few years before for an unrealised edition of 
Theocritos' Idyls. Its complex and heavily decorative design does point to 
aesthetic codes which preoccupied his artistic expression and, although 
impractical for contemporary text setting, it remains an original display face, 
or it can be used as initials.

The book design workshop of the Fine Arts School of Athens has been recently 
reorganised, under the direction of professor Leoni Vidali, and with her 
collaboration George D. Matthiopoulos has redesigned digitaly this historical 
font which is now available as GFS Theokritos.

Comment 1 Parag AN(पराग) 2007-11-26 17:14:11 UTC

gfs-theokritos-fonts.noarch: I: checking
gfs-theokritos-fonts.noarch: E: description-line-too-long Kefallinos' Δέκα
λευκαί λήκυθοι (1956) the artist used a typeface which he
Your description lines must not exceed 79 characters. If a line is exceeding
this number, cut it to fit in two lines.

gfs-theokritos-fonts.noarch: W: no-version-in-last-changelog
The last changelog entry doesn't contain a version. Please insert the
version that is coherent with the version of the package and rebuild it.

Comment 2 Nicolas Mailhot 2007-11-26 18:48:18 UTC

Updated version posted at the usual place to cut lines at 79 columns.

Note that rpmlint does not know how to count UTF-8 characters, so if it still
complains do check in an UTF-8 aware editor he's actually right (he isn't)

On the other rpmlint warning:

rpmlint wants to force you to put the version at the end of the first changelog
line, however the official approved Fedora changelog format list includes a
style where the version is not on this line
(http://fedoraproject.org/wiki/Packaging/Guidelines#Changelogs, 3rd style) so
rpmlint's warning is bogus

Comment 3 Parag AN(पराग) 2007-11-27 03:01:26 UTC

I agree it was big work for you to ask to fix small issue but still I prefer to
follow what rpmlint says till rpmlint tool gives it as Error and not Warning.

Comment 4 Parag AN(पराग) 2007-11-27 03:26:55 UTC

Dunno, But with updated version I still see
gfs-theokritos-fonts.src: E: description-line-too-long Kefallinos' Δέκα λευκαί
λήκυθοι (1956) the artist used a typeface which he

Otherwise Packaging looks ok.
APPROVED.

I don't want you to waste your time again now so if you want to fix it then fix
at time of cvs import.

Comment 5 Nicolas Mailhot 2007-11-27 07:00:59 UTC

rpmlint is wrong. I actually cheched the line it tags manually - it's less than
79 columns in UTF-8, which is our default encoding.

New Package CVS Request
=======================
Package Name: gfs-theokritos-fonts
Short Description: GFS Theokritos decorative font
Owners: nicolas.mailhot
Branches: F-7, F-8, devel
InitialCC: fedora-fonts-bugs-list
Cvsextras Commits: Yes

Comment 6 Kevin Fenzi 2007-11-27 19:57:49 UTC

cvs done.