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
