Dennis- I'm not able to duplicate this here, and it apparently worked for the OP. What OS and shell are you running configure in? Can you check for stray /r/n line endings in the configure script. That can sometimes confuse the shell, and I may have left a one in.
I can't comment on the suggested lemon changes (--help support and usage updates) -- perhaps open a ticket and see what DRH says. -Shane On 7/23/08, Dennis Cote <[EMAIL PROTECTED]> wrote: > > Shane Harrelson wrote: > > I checked in some updates to the "configure" support that will hopefully > do > > the right thing and pass any OMIT options to lemon and mkkeywordhash. > > There was also a minor fix to handle SQLITE_OMIT_VIEW being defined while > > SQLITE_OMIT_SUBQUERY is undefined in select.c (something you probably > rand > > into with your example.) Also note that in your example, -DTEMP_STORE=2 > > should probably be replaced with some variation of the --enable-tempstore > > configure option. > > > > Shane, > > I just looked at your changes, and it seems to me the format of the > options being passed to lemon is incorrect. > > The lemon program take options of the form D=SQLITE_OMIT* as shown in > the usage information. > > $ ./lemon --help > Command line syntax error: undefined option. > c:\sqlite\SQLite3\build\lemon.exe --help > here --^ > Valid command line options for "c:\sqlite\SQLite3\build\lemon.exe" are: > -b Print only the basis in report. > -c Don't compress the action table. > D=<string> Define an %ifdef macro. > -g Print grammar without actions. > -m Output a makeheaders compatible file > -q (Quiet) Don't print the report file. > -s Print parser stats to standard output. > -x Print the version number. > > I have tested that these options work as expected by manually editing > the old Makefile and passing such options using the OPTS variable. > > Further testing reveals that lemon also seems to accept option in the > form -DSQLITE_OMIT* just like the compiler, so this is probably a > non-issue (except that the lemon usage documentation should be updated > and the command should be changed to accept a --help option as well). > > When I use configure to build a new makefile after your changes the > OPT_FEATURE_FLAGS are set incorrectly. > > I ran this command: > > $ ../sqlite/configure CFLAGS="-DSQLITE_OMIT_TRIGGER" > > During the run the following messages were generated (near the end): > > ../sqlite/configure: OPT_FEATURE_FLAGS+= -DSQLITE_OMIT_TRIGGER: command > not found > ../sqlite/configure: ac_temp_CFLAGS+= -DSQLITE_OS_WIN=1: command not found > > The generated makefile contains the following line: > > OPT_FEATURE_FLAGS = -DSQLITE_OMIT_LOAD_EXTENSION=1 > > with no mention of the SQLITE_OMIT_TRIGGER option. > > When make is executed lemon is called to generate the parser like this: > > ./lemon.exe -DSQLITE_OMIT_LOAD_EXTENSION=1 parse.y > > and the generated parser still contains all the trigger related code. > > It looks like this feature still needs some more work. > > Dennis Cote > > _______________________________________________ > sqlite-users mailing list > sqlite-users@sqlite.org > http://sqlite.org:8080/cgi-bin/mailman/listinfo/sqlite-users > _______________________________________________ sqlite-users mailing list sqlite-users@sqlite.org http://sqlite.org:8080/cgi-bin/mailman/listinfo/sqlite-users
