[GRASS5] PATCH: sql expression parser

[Paul Kelly]

On Fri, 10 Jun 2005, Daniel Calvelo Aros wrote:

> From: Paul Kelly <paul-grass at stjohnspoint.co.uk>
>> It didn't work anyway with Solaris lex and yacc, so probably not a
>> problem. Below is the error (nothing to do with your changes)---a
>> lot of it seems to be mis-placed comments but I'm not sure what
>> proper lex syntax for this is.
> [snip]
>
> Ouch. Are you resorting to using bison under solaris anyway?

No---to get it to compile I deleted all the comments from lex.l; also I 
had to delete a further few misplaced spaces and tabs before it would run 
properly.

> Might including
> the generated files (lex.yy.c and y.tab.c) in the "source" distribution be a
> solution?

That sounds like quite a good idea... I've noticed in some free software 
projects they don't even include the generated configure script in CVS and 
the user has to generate it themselves with autoconf. In GRASS the 
developers always generate the configure script; maybe "pre-generating" 
the lex and yacc files and storing in CVS would be a worthwhile idea.

Paul