[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: bison-1.29 test failures
From: |
Len Makin |
Subject: |
Re: bison-1.29 test failures |
Date: |
Mon, 15 Oct 2001 09:44:02 +1000 |
Hello utils/gettext maintainers,
Here's a minor bug report, which showed up when compiling bison
intl code.
Cheers,
Len Makin
--
address@hidden:+61 3 9669 8109: CSIRO/Bureau of Meteorology
High Performance Computing and Communications Centre
24th floor, 150 Lonsdale St | GPO Box 1289K, Melbourne, VIC 3001 Australia
On Thursday, 11 October 2001, at 11:52,
Pascal Bart <address@hidden> wrote:
> we don't maintain intl, this a part of the gettext library, you
> should see with address@hidden
> Thanks.
> On Thursday, 11 October 2001, at 9:14,
> Len Makin <address@hidden> wrote:
> >Another very minor point (which may belong to the gettext people
> >rather than bison) is that I got warnings from the intl subdirectory like:
> >In file included from system.h:119,
> > from output.c:93:
> >/usr/include/locale.h:56: warning: `LC_MESSAGES' redefined
> >../intl/libintl.h:31: warning: this is the location of the previous
> >definition
> >
> >Checking, I found the following code in libintl.h :
> >....
> >#include <locale.h>
> >
> >/* The LC_MESSAGES locale category is the category used by the functions
> > gettext() and dgettext(). It is specified in POSIX, but not in ANSI C.
> > On systems that don't define it, use an arbitrary value instead.
> > On Solaris, <locale.h> defines __LOCALE_H then includes <libintl.h> (i.e.
> > this file!) and then only defines LC_MESSAGES. To avoid a redefinition
> > warning, don't define LC_MESSAGES in this case. */
> >#if !defined LC_MESSAGES && !defined __LOCALE_H
> ># define LC_MESSAGES 1729
> >#endif
> >....
> >but in our version of locale.h it says:
> >#ifndef _LOCALE_H
> >#define _LOCALE_H
> >
> >i.e. there is only a single "_" leading character. Changing the test to
> >#if !defined LC_MESSAGES && !defined _LOCALE_H
> >got rid of the warnings.
>
> Pascal Bart (address@hidden)
>