Bug 486269 - Review Request: levien-inconsolata-fonts - Inconsolata fonts
Summary: Review Request: levien-inconsolata-fonts - Inconsolata fonts
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Kevin Fenzi
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On: 431368
Blocks: 477401
TreeView+ depends on / blocked
 
Reported: 2009-02-19 06:24 UTC by Kevin Fenzi
Modified: 2012-07-14 01:57 UTC (History)
3 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2009-03-02 21:52:02 UTC
Type: ---
Embargoed:
kevin: fedora-review?
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Kevin Fenzi 2009-02-19 06:24:08 UTC
Spec URL: http://www.scrye.com/~kevin/fedora/levien-inconsolata-fonts/levien-inconsolata-fonts.spec
SRPM URL: http://www.scrye.com/~kevin/fedora/levien-inconsolata-fonts/levien-inconsolata-fonts-1.01-1.fc11.src.rpm
Description: 

A monospace font, designed for code listings and the like, in print.

This is a rename/update of the existing inconsolata-fonts package.

Comment 3 Kevin Fenzi 2009-02-25 00:13:13 UTC
I've made those changes here: 

http://www.scrye.com/~kevin/fedora/levien-inconsolata-fonts/levien-inconsolata-fonts-1.01-2.fc11.src.rpm

However, it's not building here under mock. 
I can't seem to track down why. ;( 

Will keep looking, but if anyone has any ideas, let me know...

Comment 4 Nicolas Mailhot 2009-02-25 06:27:04 UTC
The package renaming changes %{fontname} and a define was changed to global. Maybe there's a conflict with your use of $fontname in the fontforge script?

Otherwise, can you post failing build logs?

Comment 5 Kevin Fenzi 2009-02-25 19:07:07 UTC
Yeah, the old font fails exactly the same way, so it must be a change somewhere in rawhide. ;( 

http://koji.fedoraproject.org/koji/getfile?taskID=1167442&name=build.log

+ fontforge -script - /builddir/build/SOURCES/Inconsolata.sfd
Copyright (c) 2000-2008 by George Williams.
 Executable based on sources from 16:11 GMT 24-Dec-2008.
 Library based on sources from 16:11 GMT 24-Dec-2008.
  File "<stdin>", line 2
    while ( i < $argc )
                ^
SyntaxError: invalid syntax
error: Bad exit status from /var/tmp/rpm-tmp.e4UUpX (%build)

any ideas?

Comment 6 Kevin Fenzi 2009-02-28 06:04:04 UTC
It was enabling the python bindings. When those are enabled, it assumes python... 

passing it '-lang=ff' causes it to use the 'legacy' fontforge scripting lang. 

Will get this imported and built now. 

Does the fontconfig file I have here look ok?

New Package CVS Request
=======================
Package Name: levien-inconsolata-fonts
Short Description: A monospace font, designed for code listings and the like, in print
Owners: kevin
Branches: devel
InitialCC:

Comment 7 Kevin Fenzi 2009-02-28 23:40:16 UTC
cvs done.

Comment 8 Kevin Fenzi 2009-03-02 21:52:02 UTC
Built in rawhide and old package marked dead.

Comment 9 Kevin Fenzi 2012-07-14 01:57:23 UTC
Going ahead with a EL6 branch

Package Change Request
======================
Package Name: levien-inconsolata-fonts
New Branches: el6
Owners: kevin
InitialCC:


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