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 189655 - Review Request: mftrace - convert TeX bitmap fonts to TrueType fonts
Summary: Review Request: mftrace - convert TeX bitmap fonts to TrueType fonts
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Jason Tibbitts
QA Contact: Fedora Package Reviews List
URL:
Whiteboard:
Depends On:
Blocks: FE-ACCEPT 189656
TreeView+ depends on / blocked
 
Reported: 2006-04-22 05:11 UTC by Quentin Spencer
Modified: 2007-11-30 22:11 UTC (History)
0 users

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2006-04-26 01:08:04 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Quentin Spencer 2006-04-22 05:11:36 UTC
Spec URL: http://webpages.charter.net/qspencer/rpm/mftrace.spec
SRPM URL: http://webpages.charter.net/qspencer/rpm/mftrace-1.1.19-1.src.rpm
Description: 
mftrace is a small Python program that lets you trace a TeX bitmap font into a PFA or PFB font (A PostScript Type1 Scalable Font) or TTF (TrueType) font.


This is a build dependency of GNU Lilypond, the automated music typesetting system that I am also submitting for review.

Comment 1 Jason Tibbitts 2006-04-23 05:00:48 UTC
It seems that version 1.2.0 is out; any reason not to submit the updated version?

Comment 2 Quentin Spencer 2006-04-23 06:21:47 UTC
I see now that the author's web page has been updated to reflect a 1.2.0
release, and I will be happy to update the spec file and SRPM to the new
release, except that the link on the web page is broken, so the new tarball has
apparently not actually been uploaded yet. I'll check again later, but at the
moment, it seems the SRPM I created is still the most current available release.

Comment 3 Jason Tibbitts 2006-04-23 19:24:11 UTC
Odd, but I'm sure it will work itself out.  I'll review the current version.

Issues:
The empty %doc; you should probably package README.txt and ChangeLog.
Generally .pyo files are %ghost'ed instead of being packaged (see
http://fedoraproject.org/wiki/Packaging/Python), although now that RPM
byte-compiles everything in sight I'm wondering if the guidelines shouldn't be
changed.  In any case, I would suggest ghosting the .pyo files but if you don't
want to then I won't insist.

* package meets naming and packaging guidelines.
* specfile is properly named, is cleanly written and uses macros consistently.
* license field matches the actual license.
* license is open source-compatible.  It's not included separately in the
package, but this is not necessary as the upstream tarball does not include it.
* source files match upstream:
   89d5f70cdfad6ed7be146dd1b548cec1  mftrace-1.1.19.tar.gz
   89d5f70cdfad6ed7be146dd1b548cec1  mftrace-1.1.19.tar.gz-srpm
* BuildRequires are proper.
* package builds in mock (development, i386).
* rpmlint is silent.
* final provides and requires are sane.
* no shared libraries are present.
* package is not relocatable.
* owns the directory it creates.
* doesn't own any directories it shouldn't.
* no duplicates in %files.
* file permissions are appropriate.
* %clean is present.
* %check is not present; package has no test suite.
* code, not content.
* documentation is small, so no -docs subpackage is necessary.
* %docs are not necessary for the proper functioning of the package.
* no headers.
* no pkgconfig files.
* no libtool .la droppings.
* not a GUI app.

Comment 4 Quentin Spencer 2006-04-25 18:46:29 UTC
OK, I fixed the empty docs (oops!) and ghosted the .pyo files. This is my first
python package, so I wasn't aware of that. Updated packages at:

http://webpages.charter.net/qspencer/rpm/mftrace.spec
http://webpages.charter.net/qspencer/rpm/mftrace-1.1.19-2.src.rpm


Comment 5 Jason Tibbitts 2006-04-25 22:52:10 UTC
I must admit that I don't fully understand the .pyo issue either; I package
denyhosts which is also in Python and the variable behavior with respect to .pyo
files has been nothing but trouble for me.  Currently I ghost them all.

Anyway, I just rebuilt in mock and everything now looks good.

APPROVED


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