Re: [PATCH] gnu: gettext: Enable "xgettext --language=glade"

[ludo@gnu.org (Ludovic Courtès)]

Andreas Enge <andreas@enge.fr> skribis:

> On Mon, Nov 18, 2013 at 12:19:24AM +0100, Ludovic Courtès wrote:
>> Actually, it seems to be solvable without adding the dependency:
>> 
>> --8<---------------cut here---------------start------------->8---
>> $ xgettext -L glade /dev/null 
>> xgettext: Language "glade" is not supported. xgettext relies on expat.
>>           This version was built without expat.
>> 
>> $ LD_LIBRARY_PATH=$(guix build expat)/lib xgettext -L glade /dev/null 
>> xgettext: /dev/null:1:1: no element found
>> --8<---------------cut here---------------end--------------->8---
>
> I know, that was one of the things we tested early on; but...

OK, sorry for stating the obvious then (apologies for not following the
discussion closely enough!).

>> Would it work for you to just use that LD_LIBRARY_PATH trick where it’s
>> needed?
>> That way we avoid making gettext “fatter”.  I suppose we’ll seldom need
>> to do that trick anyway.
>
> ... might there not be users who want a fully functional xgettext? Now that
> we found a simple solution, I am rather in favour of implementing it.
> Not to mention that this solves the issue once and for all, and we do not
> need to remember the LD_LIBRARY_PATH trick later on...

Yeah, makes sense to me; so feel free to commit now.

Are there other optional dependencies that are dlopen’d, or is it the
only one?  It’s OK if it’s just expat, but we wouldn’t want to add
bigger things.

Thanks,
Ludo’.