Re: [PATCH 1/4] gnu: Add potrace.

[Ludovic Courtès]

Eric Bavier <address@hidden> skribis:

> Ludovic Courtès writes:
>
>> Eric Bavier <address@hidden> skribis:
>>
>>> From f178a122f3805cbc23119b20c2144cf2a59bed3f Mon Sep 17 00:00:00 2001
>>> From: Eric Bavier <address@hidden>
>>> Date: Tue, 7 Oct 2014 11:13:35 -0500
>>> Subject: [PATCH 1/4] gnu: Add potrace.
>>>
>>> * gnu/packages/fontutils.scm (potrace): New variable.
>>
>> This part looks good to me, but...
>
> Ok.
>
>>
>>>  gnu/packages/fontutils.scm |   28 ++++++++++++++
>>>  gnu/packages/giflib.scm    |   90 
>>> ++++++++++++++++++++++++++++++++++++++++++++
>>
>> What about the giflib.scm changes?  Were they intended?
>
> No, those changes were not intended.

OK.

>> Perhaps giflib 5.x can be added in image.scm?
>
> I was experimenting with the latest version of fontforge, which can use
> giflib 5.x.  For the time being it can be left out.
>
> An updated patch is attached.
>
>
> From 6721d6e9e31af8aff5e781d813e55347dea378c9 Mon Sep 17 00:00:00 2001
> From: Eric Bavier <address@hidden>
> Date: Tue, 7 Oct 2014 12:27:37 -0500
> Subject: [PATCH] gnu: Add potrace.
>
> * gnu/packages/fontutils.scm (potrace): New variable.

LGTM!

Ludo'.