Re: new module bison-i18n

[Paul Eggert]

Good idea, thanks.  Two minor points:

>     if test -n "$BISON_LOCALEDIR"; then
>       USER_LINGUAS="${LINGUAS-%UNSET%}"
>       if test -n "$USER_LINGUAS"; then

What is the significance of the %UNSET% here?  I.e., why is an unset
LINGUAS treated as nonempty?  Maybe a comment would help?

>     if test -n "$YACC"; then
>       case "$YACC" in
>         *bison*)
>           if ($YACC --print-localedir) >/dev/null 2>&1; then

This doesn't look quite right, since "yacc" is an alias for "bison" on
some implementations, so "yacc --print-localedir" will work.  For
example, I've installed Bison 2.3 in my home directory, so:

   $ yacc -print-localedir
   /home/eggert/opt/Linux-i686/bison-2.3/share/locale

How about something like the following code instead?  It's a bit
shorter and easier for me to read, and takes the above into account.

  BISON_USE_NLS=no
  BISON_LOCALEDIR=
  if test "$USE_NLS" = yes; then
    dnl AC_PROG_YACC sets the YACC variable; other macros set the BISON 
variable.
    if (${YACC-${BISON-:}} --print-localedir) >/dev/null 2>&1; then
       BISON_LOCALEDIR=`${YACC-${BISON-:}} --print-localedir`
    fi
    AC_SUBST([BISON_LOCALEDIR])
    if test -n "$BISON_LOCALEDIR"; then
      USER_LINGUAS="${LINGUAS-%UNSET%}"
      if test -n "$USER_LINGUAS"; then
        BISON_USE_NLS=yes
      fi
    fi
  fi