[cairo] [PATCH] Pull -lz for the script backend

Olivier Blin olivier.blin at softathome.com
Wed Aug 31 21:42:22 UTC 2016


Thank you all for the review.

Adrian, do you want to send a follow-up patch to error out if libz is 
missing?
Or should I do it?

Cheers

Le 31/08/2016 à 23:36, Bryce Harrington a écrit :
> On Tue, Aug 30, 2016 at 04:37:49PM -0700, Behdad Esfahbod wrote:
>> Adrian is right.  Original patch lgtm.
> Thanks.  I'll interpret that as an acked-by if you don't mind, and go
> ahead and land it.
>
> Pushed:
>      1057487..35a1761  master -> master
>
> Bryce
>   
>> (i just remembered how much fun I had writing those macros...:-)
>>
>> On Aug 30, 2016 3:28 PM, "Adrian Johnson" <ajohnson at redneon.com> wrote:
>>
>>> On 31/08/16 06:38, Bryce Harrington wrote:
>>>> On Fri, Aug 12, 2016 at 03:48:57PM +0200, Olivier Blin wrote:
>>>>> Le 12/08/2016 à 15:08, Enrico Weigelt, metux IT consult a écrit :
>>>>>> On 12.08.2016 11:04, Olivier Blin wrote:
>>>>>>
>>>>>>>> Looks like this would silently disable the script backend if libz
>>>>>>>> is missing. IMHO, a very bad idea - instead it should break with a
>>>>>>>> proper error message.
>>>>>>> This is actually copy/paste of what is already properly done for the
>>>>>>> PostScript and PDF backends.
>>>>>>>
>>>>>>> CAIRO_ENABLE_SURFACE_BACKEND(pdf, PDF, yes, [
>>>>>>>      # The pdf backend requires zlib.
>>>>>>>      use_pdf=$have_libz
>>>>>>>      pdf_NONPKGCONFIG_LIBS=-lz
>>>>>>> ])
>>>>>> Right, there seems to be the same problem.
>>>>>> I'd suggest emitting an error here.
>>>>> Ok, but this is a separate issue that was already present in the
>>>>> previous configure file.
>>>>> This can be fixed regardless of this patch.
>>>>>
>>>>> Thanks for your input
>>>> Would you mind sending a 2-patch set that fixes both issues for us?
>>> There is no problem with this patch.
>>>
>>> https://lists.cairographics.org/archives/cairo/2016-August/027701.html
>>>
>>>> Thanks,
>>>> Bryce
>>>>
>>>>> --
>>>>> Olivier Blin - SoftAtHome
>>>>>
>>>>> --
>>>>> cairo mailing list
>>>>> cairo at cairographics.org
>>>>> https://lists.cairographics.org/mailman/listinfo/cairo
>>> --
>>> cairo mailing list
>>> cairo at cairographics.org
>>> https://lists.cairographics.org/mailman/listinfo/cairo


-- 
Olivier Blin - SoftAtHome



More information about the cairo mailing list