Re: nl_langinfo.m4: HAVE_LANGINFO_CODESET unquoted

2016-01-12 Thread Paul Eggert
On 01/12/2016 04:56 AM, Gavin Smith wrote: The code for this output is in func_emit_autoconf_snippets; it shouldn't be hard to swap them around. I can try to post a patch if others agree this is the reason for the problem. It might be simpler to change the modules/nl_langinfo file instead, by

Re: nl_langinfo.m4: HAVE_LANGINFO_CODESET unquoted

2016-01-12 Thread Gavin Smith
On 12 January 2016 at 12:28, Gavin Smith wrote: > func_gl_gnulib_m4code_nl_langinfo () > { > if ! $gl_gnulib_enabled_nl_langinfo; then > gl_FUNC_NL_LANGINFO > if test $HAVE_NL_LANGINFO = 0 || test $REPLACE_NL_LANGINFO = 1; then > AC_LIBOBJ([nl_langinfo]) > fi >

Re: nl_langinfo.m4: HAVE_LANGINFO_CODESET unquoted

2016-01-12 Thread Gavin Smith
On 12 January 2016 at 00:53, Gavin Smith wrote: > Maybe AC_REQUIRE isn't hoisting the code far enough? The code is > remaining in the function corresponding to a gnulib module's code, and > not going to the very top level. The code is expanded, but is executed > too late. So gnulib-m4.comp, which

Re: nl_langinfo.m4: HAVE_LANGINFO_CODESET unquoted

2016-01-11 Thread Gavin Smith
On 11 January 2016 at 23:08, Paul Eggert wrote: > On 01/11/2016 12:32 PM, Gavin Smith wrote: >> > Let's do the latter, since these variables should all be set to 0 or 1 by > then. > > The macro in question AC_REQUIREs gl_LANGINFO_SET, which should set > HAVE_LANGINFO_CODESET and HAVE_LANGINFO_ERA

Re: nl_langinfo.m4: HAVE_LANGINFO_CODESET unquoted

2016-01-11 Thread Paul Eggert
On 01/11/2016 12:32 PM, Gavin Smith wrote: In the nl_langinfo.m4 file, there's a test: if test $HAVE_LANGINFO_CODESET = 1 && test $HAVE_LANGINFO_ERA = 1 \ && test $FUNC_NL_LANGINFO_YESEXPR_WORKS = 1; then : else This gave an (apparently harmless) error message when config

nl_langinfo.m4: HAVE_LANGINFO_CODESET unquoted

2016-01-11 Thread Gavin Smith
In the nl_langinfo.m4 file, there's a test: if test $HAVE_LANGINFO_CODESET = 1 && test $HAVE_LANGINFO_ERA = 1 \ && test $FUNC_NL_LANGINFO_YESEXPR_WORKS = 1; then : else This gave an (apparently harmless) error message when configuring under OpenIndiana 11: ./configure[22339]: