Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 177096 - Review Request: fonttools: A tool to convert True/OpenType fonts to XML and back
Summary: Review Request: fonttools: A tool to convert True/OpenType fonts to XML and back
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Eric Tanguy
QA Contact: David Lawrence
URL:
Whiteboard:
Depends On:
Blocks: FE-ACCEPT
TreeView+ depends on / blocked
 
Reported: 2006-01-06 10:20 UTC by Roozbeh Pournader
Modified: 2014-09-29 11:56 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2010-07-29 04:47:59 UTC
Type: ---
Embargoed:
gwync: fedora-cvs+

Attachments (Terms of Use)

Description Roozbeh Pournader 2006-01-06 10:20:17 UTC 
Spec Url: http://guava.farsiweb.info/~roozbeh/fonttools.spec
SRPM Url: http://guava.farsiweb.info/~roozbeh/fonttools-2.0-0.1.20060103cvs.src.rpm
Description:
TTX/FontTools is a tool for manipulating TrueType and OpenType fonts. It is
written in Python and has a BSD-style, open-source license. TTX can dump
TrueType and OpenType fonts to an XML-based text format and vice versa.

Additional notes:
1. The reason for this being a CVS version is that the upstream's latest release has been in September 2002, while the maintainer has continually referred to the CVS version for bugfixes since then.
2. I prefer to name the package "ttx" (the name of the only binary the package provides) instead of "fonttools", because "ttx" is the name most users know this by. I stayed with "fonttools" because the Extras naming policy leans toward upstream names, and also because SUSE ships it (the September 2002 version) under the name "fonttools".
3. This is practically a binary and a module. I don't think any other application uses the module, but if that's the case, I may be able to make this two packages, one named "ttx" and another named "python-fonttools", or if I can name the package "ttx", I can add a "Provides" of "python-fonttools".
Comment 1 Roozbeh Pournader 2006-01-08 10:07:58 UTC 
Updated version, now using the %{?dist} tag:

Spec Url: http://guava.farsiweb.info/~roozbeh/fonttools.spec
SRPM Url: http://guava.farsiweb.info/~roozbeh/fonttools-2.0-0.2.20060103cvs.src.rpm
Comment 2 Eric Tanguy 2006-01-31 21:27:36 UTC 
Review for release 0.2.20060103cvs:
* RPM name is OK
* Builds fine in mock
* rpmlint of fonttools looks OK
* File list of fonttools looks OK

Needs work:
* Encoding should be UTF-8
* No downloadable source. Please give the full URL in the Source tag.
I know the source from cvs are not downloadable but it is a MUST item. Maybe you
can ask maintainer to make a snapshot ?
Comment 3 Roozbeh Pournader 2006-02-01 18:09:21 UTC 
(In reply to comment #2)
> Needs work:
> * Encoding should be UTF-8

Encoding of which file is not UTF-8?!

> * No downloadable source. Please give the full URL in the Source tag.
> I know the source from cvs are not downloadable but it is a MUST item. Maybe you
> can ask maintainer to make a snapshot ?

It seems that the developer has created snapshots not long before stopping the
development.

Updated version:

Spec Url: http://guava.farsiweb.info/~roozbeh/fonttools.spec
SRPM Url: http://guava.farsiweb.info/~roozbeh/fonttools-2.0-0.3.20050624cvs.src.rpm

%changelog
- Use upstream snapshots, moving the difference into a patch
- Change alphatag time to the latest change in CVS
- Use %%{python_sitearch} instead of %%{python_sitelib} (for x86_86)
- Use sed instead of a patch to remove shebang
Comment 4 Eric Tanguy 2006-02-01 21:44:46 UTC 
(In reply to comment #3)
> (In reply to comment #2)
> > Needs work:
> > * Encoding should be UTF-8
> 
> Encoding of which file is not UTF-8?!

Sorry it was a mistake!

> 
> > * No downloadable source. Please give the full URL in the Source tag.
> > I know the source from cvs are not downloadable but it is a MUST item. Maybe you
> > can ask maintainer to make a snapshot ?
> 
> It seems that the developer has created snapshots not long before stopping the
> development.
> 
> Updated version:
> 
> Spec Url: http://guava.farsiweb.info/~roozbeh/fonttools.spec
> SRPM Url:
http://guava.farsiweb.info/~roozbeh/fonttools-2.0-0.3.20050624cvs.src.rpm
> 
> %changelog
> - Use upstream snapshots, moving the difference into a patch
> - Change alphatag time to the latest change in CVS
> - Use %%{python_sitearch} instead of %%{python_sitelib} (for x86_86)
> - Use sed instead of a patch to remove shebang
> 
Review for release 0.3.20050624cvs:
* RPM name is OK
* Source fonttools-2005-03-15.210812.tar.bz2 is the same as upstream
* Builds fine in mock
* rpmlint of fonttools looks OK
* File list of fonttools looks OK

APPROVED
Comment 5 Roozbeh Pournader 2006-02-02 17:06:24 UTC 
Thanks a lot for the review. Imported and built.
Comment 6 Parag Nemade 2010-07-09 09:20:34 UTC 
Roozbeh,
   I need this package in EPEL6 as its dependency on python-robofab. If you are interested to maintain this in EPEL6 then you are always welcome but if not please allow me to own EPEL6 branch.
  We have left less time to build python-robofab(and fonttools) in EPEL6.

Please provide your comments here.
Comment 7 Parag Nemade 2010-07-19 06:17:35 UTC 
ping Roozbeh?
Comment 8 Parag Nemade 2010-07-26 05:40:24 UTC 
I suppose its enough waiting here. Also, no reply to personal mails.

Please process my cvs request and if in future Roozbeh will be back then I am willing to give ownership to him in EPEL6.

Package Change Request
======================
Package Name: fonttools
New Branches: EL-6
Owners: pnemade
InitialCC: Roozbeh

I need this package in EPEL6 as its dependency of python-robofab.
Comment 9 Kevin Fenzi 2010-07-26 22:24:23 UTC 
cvs done.
Comment 10 Parag Nemade 2010-07-29 04:47:59 UTC 
Thanks kevin. I have built this in EPEL6.
Comment 11 Parag Nemade 2014-09-28 06:28:15 UTC 
Package Change Request
======================
Package Name: fonttools
New Branches: epel7
Owners: pnemade
Comment 12 Gwyn Ciesla 2014-09-29 11:56:03 UTC 
Git done (by process-git-requests).
Note You need to log in before you can comment on or make changes to this bug.