[Bug 486269] Review Request: levien-inconsolata-fonts - Inconsolata fonts

2009-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=486269


Kevin Fenzi ke...@tummy.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE




--- Comment #8 from Kevin Fenzi ke...@tummy.com  2009-03-02 16:52:02 EDT ---
Built in rawhide and old package marked dead.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 486269] Review Request: levien-inconsolata-fonts - Inconsolata fonts

2009-02-28 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=486269


Kevin Fenzi ke...@tummy.com changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #7 from Kevin Fenzi ke...@tummy.com  2009-02-28 18:40:16 EDT ---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 486269] Review Request: levien-inconsolata-fonts - Inconsolata fonts

2009-02-27 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=486269


Kevin Fenzi ke...@tummy.com changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #6 from Kevin Fenzi ke...@tummy.com  2009-02-28 01:04:04 EDT ---
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:

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 486269] Review Request: levien-inconsolata-fonts - Inconsolata fonts

2009-02-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=486269





--- Comment #5 from Kevin Fenzi ke...@tummy.com  2009-02-25 14:07:07 EDT ---
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=1167442name=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?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 486269] Review Request: levien-inconsolata-fonts - Inconsolata fonts

2009-02-24 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=486269





--- Comment #3 from Kevin Fenzi ke...@tummy.com  2009-02-24 19:13:13 EDT ---
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...

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 486269] Review Request: levien-inconsolata-fonts - Inconsolata fonts

2009-02-24 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=486269





--- Comment #4 from Nicolas Mailhot nicolas.mail...@laposte.net  2009-02-25 
01:27:04 EDT ---
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?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 486269] Review Request: levien-inconsolata-fonts - Inconsolata fonts

2009-02-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=486269


Nicolas Mailhot nicolas.mail...@laposte.net changed:

   What|Removed |Added

 CC||fedora-fonts-bugs-l...@redh
   ||at.com
 Blocks||477401
 Depends on||431368




--- Comment #1 from Nicolas Mailhot nicolas.mail...@laposte.net  2009-02-20 
04:46:06 EDT ---
http://fedoraproject.org/wiki/Shipping_fonts_in_Fedora_%28FAQ%29#What_if_the_new_naming_guidelines_require_me_to_rename_my_source_package.3F

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 486269] Review Request: levien-inconsolata-fonts - Inconsolata fonts

2009-02-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=486269


Nicolas Mailhot nicolas.mail...@laposte.net changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|ke...@tummy.com
   Flag||fedora-review+




--- Comment #2 from Nicolas Mailhot nicolas.mail...@laposte.net  2009-02-20 
05:29:02 EDT ---
1. a fontconfig ruleset would be nice
http://fedoraproject.org/wiki/Shipping_fonts_in_Fedora_%28FAQ%29#The_fontconfig_stuff_the_font_guidelines_suggest_seems_complex._Can_I_skip_it.3F

2. FPC will want you to use %global not %define soonish
http://fedoraproject.org/wiki/PackagingDrafts/global_preferred_over_define

3. Are you sure you want to provide the old name?
http://fedoraproject.org/wiki/Shipping_fonts_in_Fedora_%28FAQ%29#Do_I_need_to_Provide_my_old_package_names.3F

Anyway, nothing here is blocking

☎☎☎ APPROVED ☎☎☎

Please do not forget the other steps in

http://fedoraproject.org/wiki/Shipping_fonts_in_Fedora_%28FAQ%29#What_if_the_new_naming_guidelines_require_me_to_rename_my_source_package.3F

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review