Bug 603632 - Review Request: typemade-josefinsansstd-light-fonts - This typeface was to made for geometric, elegant titling and kind of vintage
Summary: Review Request: typemade-josefinsansstd-light-fonts - This typeface was to ma...
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Pravin Satpute
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2010-06-14 08:24 UTC by Parag Nemade
Modified: 2010-07-09 14:02 UTC (History)
5 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2010-07-09 14:02:31 UTC
Type: ---
Embargoed:
psatpute: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)
EULA file included in upstream release (11.25 KB, application/pdf)
2010-06-25 09:07 UTC, Pravin Satpute
no flags Details

Description Parag Nemade 2010-06-14 08:24:27 UTC
Spec URL: http://paragn.fedorapeople.org/fedora-work/SPECS/typemade-josefinsansstd-light-fonts.spec
SRPM URL: http://paragn.fedorapeople.org/fedora-work/SRPMS/typemade-josefinsansstd-light-fonts-1.000-2.fc13.src.rpm
Description: 
The idea for creating this typeface was to make it geometric, elegant and
kind of vintage, especially for titling. It is based on Rudolf Koch's
Kabel (1927), Rudolf Wolf's Memphis (1930),Paul Renner's Futura (1927?).

Comment 1 Pravin Satpute 2010-06-25 09:07:24 UTC
Created attachment 426804 [details]
EULA file included in upstream release

i tried to search for license, 
nothing is mentioned in
1) font file .ttf
2) on website

i am not very much clear from the EULA file about the license

Comment 2 Pravin Satpute 2010-06-25 09:09:35 UTC
blocking on FE-LEGAL for clarification on license

Comment 3 Tom "spot" Callaway 2010-06-25 12:41:15 UTC
That EULA is clearly non-free. There is explicit restrictions on modification and redistribution.

Out of curiosity, why did you think this was FOSS? Perhaps there is another license?

Comment 4 Parag Nemade 2010-06-25 13:04:01 UTC
I got to know this font from http://code.google.com/p/googlefontdirectory/source/browse/josefin/ but saw some instability in google font directory so used original upstream source.

If source include in above SRPM is not acceptable then will ttf file from google link is ok to package with OFL licesne?

Comment 5 Tom "spot" Callaway 2010-06-25 13:19:53 UTC
We need to get clarification from Raph Levien, as the OFL license text is a template, and does not contain the appropriate information. This all seems suspicious, but it could simply be sloppy. :/

Comment 6 Parag Nemade 2010-06-25 13:33:01 UTC
Ok. I will send email to him giving above comment as a reference.

Comment 7 Parag Nemade 2010-06-28 06:10:21 UTC
Upstream of this font replied me as
==================================================
Hallo!

You can ignore that EULA, I've changed the license to SIL Open Font
License, 1.1, you can view it here,
http://code.google.com/webfonts/family?family=Josefin+Sans+Std+Light
I'll change it on my site ASAP.

Cheers!
==================================================

Can I use upstream font source then or wait till upstream release new source with updates license?

Comment 8 Pravin Satpute 2010-06-28 07:03:32 UTC
might be you can add this mail reference in this package for now and update it when upstream will do stuff from there side

see #568229 smc-fonts or sniglet review

Comment 9 Tom "spot" Callaway 2010-06-28 12:37:28 UTC
Copy of the email reply as a Source file is sufficient.

Comment 11 Pravin Satpute 2010-07-01 04:57:11 UTC
1) please drop EULA from spec file

+ package builds in mock (rawhide i686).
koji Build =>http://koji.fedoraproject.org/koji/taskinfo?taskID=2285029
+ rpmlint is silent for SRPM and for RPM.
typemade-josefinsansstd-light-fonts.noarch: W: obsolete-not-provided josefinsansstd-light-fonts
-> since obsoluted package built only in devel branch, no need for provides here

typemade-josefinsansstd-light-fonts.src: W: no-buildroot-tag
-> this is not require now

+ source files match upstream url
22f58c425e0775e5234e53ec7bf94116  typemade-JosefinSansStdLight.zip
+ package meets naming and packaging guidelines.
+ specfile is properly named, is cleanly written
+ Spec file is written in American English.
+ Spec file is legible.
+ dist tag is present.
+ license is open source-compatible.
+ License text is included in package.
+ %doc is present.
+ BuildRequires are proper.
+ %clean is present.
+ package installed properly.
+ Macro use appears rather consistent.
+ Package contains code, not content.
+ no headers or static libraries.
+ no .pc file present.
+ no -devel subpackage
+ no .la files.
+ no translations are available
+ Does owns the directories it creates.
+ fonts scriptlets present.
+ no duplicates in %files.
+ file permissions are appropriate.
+ Not a GUI application 

+ Approved

Comment 12 Parag Nemade 2010-07-05 05:19:08 UTC
Thanks for the review!

New Package CVS Request
=======================
Package Name: typemade-josefinsansstd-light-fonts
Short Description: This typeface was to made for geometric, elegant titling and kind of vintage
Owners: pnemade
Branches: 
InitialCC: i18n-team fonts-sig

Comment 13 Rakesh Pandit 2010-07-05 11:21:37 UTC
I see you are asking for fedora cvs. Accordingly set flag.

Comment 14 Kevin Fenzi 2010-07-08 01:19:27 UTC
CVS done (by process-cvs-requests.py).


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