Re: Building and testing Bison 3.0.2 with MinGW [3/5]

[Eli Zaretskii]

> From: Akim Demaille <address@hidden>
> Date: Fri, 16 Jan 2015 14:53:33 +0100
> Cc: Bison Bugs <address@hidden>
> 
> 
> > Le 7 oct. 2014 à 20:14, Eli Zaretskii <address@hidden> a écrit :
> > 
> > A coupe of tests failed and revealed a problem with l10n: the Windows
> > 'setlocale' doesn't cater to the LC_* environment variables, so
> > isprint returns values that are not necessarily consistent with the
> > locale that Bison is instructed to use (the test in question sets
> > LC_ALL=C).  Here's a patch to fix that:
> 
> Hi Eli,
> 
> Shouldn't this be done by gettext?

I don't think isprint has something to do with gettext.

> Or maybe something in gnulib?

Last time I checked even gnulib's 'setlocale' didn't solve this
problem.

> Or maybe the tests should do something else than LC_ALL=C to disable
> internationalization?

The only thing that will work on Windows is to call 'setlocale' with
LC_ALL=C, but I don't see how you can do that since Bison is a
program, not a library.

What is the problem with my suggested fix?  If you don't like it in
main.c, we could make it a function in some Windows-specific source
file.