coreutils
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] improve 'autotools-install'


From: Assaf Gordon
Subject: Re: [PATCH] improve 'autotools-install'
Date: Fri, 22 Feb 2013 11:15:02 -0500
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:10.0.4) Gecko/20120510 Icedove/10.0.4

Hello Stefano,

Stefano Lattarini wrote, On 02/22/2013 02:30 AM:
> On 02/22/2013 12:08 AM, Assaf Gordon wrote:
>>
> I think this explanation should go in the commit message of the second patch,
> as it makes clear why such patch is needed.
> 

Good idea, attached an improved patch.

>>
>> Sadly, autotools-install still doesn't complete, because gettext0.18.1
>> fails to compile with stpncpy() related problem (exactly as solved in
>> coreutils [2]) but that's is not a coreutil bug.
>>
> Is the issue still present with the latest gettext version (1.18.2)?  If not,
> you could update the '$tarballs' definition to point to that instead.

No, 0.18.2 doesn't compile either.
Eric Blake already found the fix for this, I'll just send the gettext people a 
bug report.

> 
> Also, I see that the Automake version referenced by '$tarballs' is still
> 1.12.3; I think it should be updated to the latest available version
> (1.13.2 at the moment of writing).
> 

I can send a separate patch for that, but perhaps others would chime in as to 
whether this should be done?
I assume changing version (1.12 vs 1.13) should be done when it's explicitly 
needed?

-gordon

Attachment: autotools_script_fix2.patch
Description: Text Data


reply via email to

[Prev in Thread] Current Thread [Next in Thread]