bug-gnulib
[Top][All Lists]
Advanced

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

Re: [PATCH] lib/gettext.h: fix warning if gettext is already present


From: Bruno Haible
Subject: Re: [PATCH] lib/gettext.h: fix warning if gettext is already present
Date: Mon, 03 Feb 2020 02:07:07 +0100
User-agent: KMail/5.1.3 (Linux/4.4.0-171-generic; KDE/5.18.0; x86_64; ; )

Giulio Benetti wrote:
> > Ah! Most of the contents of this <libintl.h> is OK. But
> >    - gettext_noop
> >    - gettext_printf
> > don't belong there.
> 
> I see that usually glibc #define gettext_noop() in regex_internal.h that 
> in order #include libintl.h[1] so, the same usage as gnulib does.

The big difference is that libintl.h is a public header file (installed in
/usr/include), whereas regex_internal.h is not.

> > Therefore the right action is that the uClibc people remove their definition
> > of gettext_noop and gettext_printf from this file.
> 
> uClibc-ng embeds gettext-tiny which has that libintl.h flavour where 
> they define gettext_noop() and gettext_printf() [2].

Ah. Then your bug report should go to this 'gettext-tiny' project.

> So I end up with the 
> conclusion that libbytesize has the problem, none of you all(gnulib, 
> uClibc-ng, glibc). Basically the problem is that they define 
> -DENABLE_NLS in any case ...

There is no documentation that discourages or forbids defining ENABLE_NLS
always. Therefore libbytesize is right. Please see the more elaborate
explanation in an earlier mail of this thread.

Bruno




reply via email to

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