Hi again, On 06.09.2011, at 11:36, Barry Kauler wrote:
> I would like some help thanks. > > To compile Firefox and SeaMonkey with system Sqlite, Sqlite has to be > compiled with the options '-DSQLITE_SECURE_DELETE > -DSQLITE_ENABLE_UNLOCK_NOTIFY', as explained here: > > http://bkhome.org/blog/?viewDetailed=02287 > http://www.linuxfromscratch.org/blfs/view/cvs/server/sqlite.html > > A little while ago I modified sqlite.conf, and I thought that I had done it > ok: > > ------------------------------------------- > # BK refer http://bkhome.org/blog/?viewDetailed=02287 > # um, do i need both of these?... > configprefix='CFLAGS="$CFLAGS -O2 -DSQLITE_SECURE_DELETE > -DSQLITE_ENABLE_UNLOCK_NOTIFY"' > export CFLAGS="$CFLAGS -O2 -DSQLITE_SECURE_DELETE > -DSQLITE_ENABLE_UNLOCK_NOTIFY" > > atstage cross && var_append extraconfopt ' ' '--disable-tcl' > # apparently not correctly wired up in the Makefile, thus requiring > # the tcl based "amalgamation" build-style, ... > pkginstalled tcl && var_append GCC_WRAPPER_APPEND ' ' '-DSQLITE_ENABLE_FTS3=1' > ---------------------------------------------- > > However, SeaMonkey configure informed that they are not set, so cannot > use system sqlite. > > So, I added this to sqlite.conf, trying it right now: > > ------------------------------------------ > #BK try this also, seems above method not working... > var_append GCC_WRAPPER_APPEND ' ' '-DSQLITE_SECURE_DELETE > -DSQLITE_ENABLE_UNLOCK_NOTIFY' > ------------------------------------------ > > My question is, is the last method correct? Or is there another > correct way? What is wrong with the first two attempts? just exporting CFLAGS is generally not a too reliable way to build all kind of custom Makefiles, this is why we decided to not tinker with random variables a decade ago in the ROCK Linux days. Instead we wrap the compiler to reliably control what goes to the compiler, for any package, even with very strange build scripts, ... The configprefix also would only have effect during configure, not the later source files. > Rene, could you please put these options permanently into T2 sqlite.conf? Using the wrapper is good: Committed revision 39643. René -- René Rebe, ExactCODE GmbH, Jaegerstr. 67, DE-10117 Berlin DE Legal: Amtsgericht Charlottenburg HRB 105123B, Tax-ID#: DE251602478 Managing Directors: Susanne Klaus, René Rebe http://exactcode.com | http://t2-project.org | http://rene.rebe.de
----------------------------------------------------------- If you wish to unsubscribe from this mailing, send mail to [email protected] with a subject of: unsubscribe t2
