acl-devel
[Top][All Lists]
Advanced

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

Re: [PATCH] configure.ac: crank up gettext version


From: Sam James
Subject: Re: [PATCH] configure.ac: crank up gettext version
Date: Sat, 19 Nov 2022 23:36:09 +0000


> On 19 Nov 2022, at 23:24, Guillem Jover <guillem@hadrons.org> wrote:
> 
> On Wed, 2022-11-16 at 15:40:42 +0000, Sam James wrote:
>> Even if newer gettext is installed, the version used & paths taken
>> are gated on the macro's argument. Older gettext can't handle musl's
>> libintl situation correctly and end up silently skipping it.
>> 
>> There's also a bunch of modernisation which avoids some warnings
>> in newer versions. 0.21 is in Debian stable so should be fine.
>> 
>> Signed-off-by: Sam James <sam@gentoo.org>
>> ---
>> configure.ac | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>> 
>> diff --git a/configure.ac b/configure.ac
>> index 69b6fca..4966e40 100644
>> --- a/configure.ac
>> +++ b/configure.ac
>> @@ -37,7 +37,7 @@ AC_SYS_LARGEFILE
>> AM_PROG_AR
>> LT_INIT
>> 
>> -AM_GNU_GETTEXT_VERSION([0.18.2])
>> +AM_GNU_GETTEXT_VERSION([0.21])
>> AM_GNU_GETTEXT([external])
> 
> Instead of this, it's probably better to do:
> 
> ,---
> # Minimal version supporting AM_GNU_GETTEXT_REQUIRE_VERSION.
> AM_GNU_GETTEXT_VERSION([0.19.6])
> # Require at least the following version, but use the latest available one.
> AM_GNU_GETTEXT_REQUIRE_VERSION([0.19.6])
> `---

Good suggestion -- I was reluctant to mention it because of an issue
we'd had before with older autoconf though
https://git.savannah.gnu.org/cgit/gdbm.git/commit/?id=71f1b46d41d91ebdca975c8d6cfa1865982198e1.

Best,
sam

Attachment: signature.asc
Description: Message signed with OpenPGP


reply via email to

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