Re: [Bug-wget] [PATCH] Use u8_check() instead our own utf8 checking

[Darshit Shah]

I think we should go with Angel's patch. configure.ac should check for
the libidn version and decide which code to use.

We should use the fixed libidn code for all updated machines, but
there will always be some ancient machines that will not update, let's
support Angel's patch on those ones.

On Fri, Aug 14, 2015 at 7:27 PM, Ander Juaristi <address@hidden> wrote:
>
>
> On 07/12/2015 04:28 PM, Tim Rühsen wrote:
>>
>> libidn 1.31 fixes this issue. (It's already available for Debian
>> unstable.)
>>
>> Should we simply revert the (my) patch/commit or is it worth checking for
>> libidn <= 1.30 in configure.ac. If the second, I would like to see Ángel's
>> patch going into Wget.
>>
>> Any thoughts ?
>>
>
> I would go for the first one (apply Angel's patch).
>
> I can confirm at the time of writing libidn has recently released version
> 1.32 (at 2015-08-01). It's available at unstable, as you say. But I think
> it's too early to link against it yet.
>
>> Tim
>>
>
> Regards,
> - AJ
>



-- 
Thanking You,
Darshit Shah