Bug 481409 - Review Request: chisholm-to-be-continued-fonts - A stylized font
Summary: Review Request: chisholm-to-be-continued-fonts - A stylized font
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Clint Savage
QA Contact: Fedora Extras Quality Assurance
Depends On:
TreeView+ depends on / blocked
Reported: 2009-01-24 06:59 UTC by Clint Savage
Modified: 2009-09-24 20:00 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2009-01-29 23:03:58 UTC
Type: ---
nicolas.mailhot: fedora-review+
kevin: fedora-cvs+

Attachments (Terms of Use)

Description Clint Savage 2009-01-24 06:59:47 UTC
SRPM URL: http://herlo.fedorapeople.org/rpms/2bc-fonts-19990319-1.fc10.src.rpm
SPEC URL: http://herlo.fedorapeople.org/rpms/2bc-fonts.spec

ToBeContinued is a pfb font providing a fun fantasy-like script.  This package was on the Fonts SIG request for packaging.

Comment 1 Clint Savage 2009-01-24 07:04:35 UTC
I wanted to add that the page for this font has been created and is available at: https://fedoraproject.org/wiki/To_Be_Continued_fonts

Comment 2 Clint Savage 2009-01-24 07:36:10 UTC
So confusing, setting for review again.

Comment 3 Nicolas Mailhot 2009-01-24 11:08:36 UTC
1. As noted on IRC, the font file claims "© Alex Chisholm. All rights reserved.", while the web site claims "© Matt Chisholm, OFL" so you have a licensing discrepancy. I doubt it is acceptable as-is, please have it clarified upstream

2. The TTF version is probably better for Fedora than the Type1 version. The days of Type1 being the primary Linux font format are long past

3. Please remove comments from your spec file

4. I'd have named the package chisholm-to-be-continued-fonts, both to have a common prefix to use with other fonts on that site and to respect the name the font declares

Comment 4 Clint Savage 2009-01-24 20:33:10 UTC
1. Sent an email to Matt Chisholm about the legal discrepancy.  

2. Packaged the ttf version for review instead of the Type1 font.

3. Spec file is clean.

4. Package has been renamed to accommodate such credit.

I've uploaded two new files to represent the spec and srpm respective to the requested changes:

SRPM: http://herlo.fedorapeople.org/rpms/chisholm-to-be-continued-fonts-19990319-1.fc10.src.rpm
SPEC: http://herlo.fedorapeople.org/rpms/chisholm-to-be-continued.spec

Comment 5 Nicolas Mailhot 2009-01-24 21:34:24 UTC
1. you need to rename your spec too
2. your summary is too generic, add graffiti or cursive or decorative in there
3. why install -m 0643 (is this a reviewer-awake test?)
4. why fantacy in the fontconfig file (is this another test?)

Apart from that, you only need confirmation in the licensing in %doc to finish the package

Comment 6 Clint Savage 2009-01-25 17:00:32 UTC
1. Thanks for that catch, I missed the -fonts.
2. Added Decorative to the summary
3. I honestly don't know how that happened.  Fixed.
4. Again, I must have been sleepy at some point. I didn't see that change. Fixed.

The new font is licensed correctly in the font itself, thus no documentation needed.  Because of the license change (a new version of the font) and renaming the spec, two new files exist again.

SRPM: http://herlo.fedorapeople.org/rpms/chisholm-to-be-continued-fonts-20090124-1.fc10.src.rpm
SPEC: http://herlo.fedorapeople.org/rpms/chisholm-to-be-continued-fonts.spec

Comment 7 Nicolas Mailhot 2009-01-25 17:15:53 UTC
The SPEC still includes a 
install -m 0643 -p %{SOURCE1}

I'll approve the package, but please fix this before import in cvs.

✧✧✧ APPROVED ✧✧✧

You can now continue from
Please do not forget the wiki and comps bits.

Thank your for packaging a new font for Fedora.

Comment 8 Clint Savage 2009-01-25 17:48:23 UTC
New Package CVS Request
Package Name: chisholm-to-be-continued-fonts
Short Description: To Be Continued is a decorative/graffiti sans-serif font.
Owners: herlo
Branches: F-9 F-10
InitialCC: fonts-sig

Comment 9 Kevin Fenzi 2009-01-26 22:49:54 UTC
cvs done.

Comment 10 Fedora Update System 2009-01-29 23:03:55 UTC
chisholm-to-be-continued-fonts-20090124-2.fc9 has been pushed to the Fedora 9 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 11 Fedora Update System 2009-01-29 23:07:28 UTC
chisholm-to-be-continued-fonts-20090124-2.fc10 has been pushed to the Fedora 10 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 12 Jason Tibbitts 2009-09-24 17:53:22 UTC
How is this package in the distribution when this ticket is still blocking FE-Legal?

Comment 13 Clint Savage 2009-09-24 20:00:17 UTC
That block should have been removed some time ago.  The author changed the license to a free license.

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