Re: [Bison-Announce] Bison 3.5.90 released [beta]

Akim Demaille wrote:

> > So the only thing missing before it worked for me was:
> > 
> >  #define YYENABLE_NLS 1
> >  #define ENABLE_NLS 1
> 
> Yes, you are right, you need both.
> 
> > I suppose they're normally set by autoblah, so I need to do this
> > manually. But maybe I should rather set them on the command-line
> > than in the source!?
> 
> I would probably keep them close to BISON_LOCALEDIR, so out
> of the file, but...
> 
> > Anyway, not a big deal,
> 
> yes, both are ok.

I was still thinking about it. While I need the defines above when
compiling the Bison generated file, BISON_LOCALEDIR is required
where the bindtextdomain call is, typically in the main program.
(I guess I could also do it when first entering the parser, but this
seems strange to me and requires another check to avoid calling it
every time.) -- Well, I think I'll put all of them in a single make
variable and pass them to both compilations, shouldn't really hurt.

> > though I think it should at least be briefly
> > mentioned, unless you argue using gettext without autoblah is a
> > non-supported mode of operation (which I've been doing in other
> > projects for a long time and it seems to be working well, and
> > doesn't require defining ENABLE_NLS).
> 
> Well, you may well use another system that gettext for i18n, it is
> not mandatory.  The point of this stack of macros is to provide
> magic _if_ the user wants to use what's already inside the generated
> code.

And that's all fine. All I was asking for is to mention these
required defines in the manual (I had to RTFS to find them).

> So you are in a way "in between": not using the Autotools magic,
> but willing to use the machinery inside.
> 
> I'm saying it's wrong.

Not saying, I guess!? ;)

Regards,
Frank