Re: gEDA-dev: missing #define HAVE_LOCALE_H, and gschem

[Peter Clifton <pcjc2-at-NOSPAMcam_dot_ac_dot_uk>]

On Wed, 2007-04-18 at 14:20 +0100, Peter Clifton wrote:
> On Wed, 2007-04-18 at 09:04 -0400, Dan McMahill wrote:
> > Peter Clifton wrote:
> > > On Wed, 2007-04-18 at 08:31 -0400, Ales Hvezda wrote:
> > > 
> > >> 	What version of gettext is causing this locale.h problem for
> people?

> There is a ChangeLog in Gettext 0.16.1 regarding removing an unused
> HAVE_LOCALE_H, between 0.14.5 and 0.15.0 but that may have been
> internal. Seems odd - as I've not seen this before, and I was on 0.15.0
> for some time. I didn't see a comment about an 0.14.6 release either.
> (not that I was looking, but I did look for the release number before
> and after the change).

It seems like we relied on HAVE_LOCALE_H which gettext used to define.
It doesn't seem to want or need to now, yet we still use the setlocale
call which requires including "locale.h".

So.. it isn't gettext's fault per-se, and we should either add
AC_CHECK_HEADERS(locale.h) manually, or persuede auto{conf|header|scan}
whatever, to recognise its need - and do it for us (does it work like
that?)

We should also depend on the test succeeding, as gschem won't compile
without it.

Peter




_______________________________________________
geda-dev mailing list
geda-dev@moria.seul.org
http://www.seul.org/cgi-bin/mailman/listinfo/geda-dev