[Top][All Lists]

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

Re: Gettext macros (and a couple other ones)

From: Christian Grothoff
Subject: Re: Gettext macros (and a couple other ones)
Date: Sun, 5 Sep 2021 19:01:19 +0200
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.12.0

Hi Alessio,

I checked, and this was done (a long time ago) to support OS X builds.
There doesn't seem a pressing reason to keep it, I doubt anyone has
build GNUnet on OS X for a while, and there might be a much better
solution today for OS X anyway.

My 2 cents


On 8/29/21 7:15 PM, Alessio Vanni wrote:
> Christian Grothoff <> writes:
>> Good point, but that's a specific issue with that macro, which I agree
>> should not use "_" but dgettext(). I've fixed this (and a few related)
>> instance(s) in Git master just now.
> Hello,
> thanks for the fix.
> Related to this issue: since those macros explicitly use `dgettext' I
> thought I'd try to place the "gettext.h" header directly inside
> "gnunet_common.h", so that `dgettext' is always defined, but I
> discovered that "platform.h" contains this snippet:
> #include <locale.h>
> #include "gettext.h"
> /**
>  * GNU gettext support macro.
>  */
> #define _(String) dgettext (PACKAGE, String)
> #define LIBEXTRACTOR_GETTEXT_DOMAIN "libextractor"
> #else
> #include "libintlemu.h"
> #define _(String) dgettext ("org.gnunet.gnunet", String)
> #define LIBEXTRACTOR_GETTEXT_DOMAIN "org.gnunet.libextractor"
> #endif
> It appears that FRAMEWORK_BUILD is never defined anywhere, even though
> it's part of `'.  Is it safe to remove it?
> In case it can be removed, then "gettext.h" can be simply included by
> "gnunet_common.h" too since multiple inclusions are guarded by the
> preprocessor, meaning that GNUnet will keep using the `_' macro defined
> by "platform.h" while other applications don't have to explicitly add
> "#include <gnunet/gettext.h>" before including the "gnunet_util_lib.h"
> header.
> If it can't be removed, things are going to be a bit inconsistent, since
> "gettext.h" would be included even when it shouldn't. The #ifndef check
> can't be reliably performed inside "gnunet_common.h" for the same
> reasons why GNUNET_AGPL_URL can give an incorrect value, i.e. it's a
> public header which can get values from anything included before it,
> like a "config.h" generated by Autoconf and thus FRAMEWORK_BUILD might
> be defined for unrelated reasons, making things break unexpectedly.
> Thanks,
> A.V.

Attachment: signature.asc
Description: OpenPGP digital signature

reply via email to

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