Re: Build error: "ENABLE_NLS" is not defined

[Grégoire Sutre]

Robert Millan wrote:

> This affects gnulib/error.c and gnulib/gettext.h which would be much better
> not to change, as they're being imported semi-automatically.

I understand.

But could this be actually a bug in gnulib?  The problem only occurs 
when gettext is not found and when compiling with -Wundef -Werror, so 
maybe it went unnoticed?

> Perhaps you could solve this at its source?  (i.e. by defining ENABLE_NLS to
> 0 when gettext is unavailable).

Indeed, the problem is gone when the following line is added before the 
call to AM_GNU_GETTEXT in configure.ac:

AC_DEFINE([ENABLE_NLS], [0])

It does not really solve the problem at its source, though.  ENABLE_NLS 
is defined in AM_GNU_GETTEXT and the documentation of this macro [1] 
does not require ENABLE_NLS to be defined when gettext is not available.

Best regards,

Grégoire

[1] 
http://www.gnu.org/software/hello/manual/gettext/AM_005fGNU_005fGETTEXT.html