Re: [PHP4BETA] ZTS on unix.

On Tue, Aug 31, 1999 at 08:28:02PM +0200, Stig Bakken wrote: > On Tue, 31 Aug 1999, Thies C. Arntzen wrote: > > > hi mr. autoconf (stig?), > > > > we'll need to make a few adjustments to the configure for > > --enable-thread-safety mode. > > > > (i'm kinda lost here) > > > > libzend/zend-scanner.l needs to create zend-scanner.cc (c++ version) and > > that needs to be complied: > > > > my ugly hack (libzend/Makefile) did: > > > > hack: > > $(LEX) -+ -S$(srcdir)/flex.skl -Pzend -ozend-scanner.cc -i $(srcdir)/zend-scanner.l > > g++ -g -Wall $(DEFS) -c zend-scanner.cc -o zend-scanner.o > > > > so i could say: > > # make hack > > .... > > # make > > > > to get my ZTS version of libzend. > > > > also -I../TSRM -I../../TSRM needs to be in some of the makefiles > > Is what you're saying that we should use different [f]lex commands when > compiling with and without ZTS? Yes. For ZTS, a C++ flexer must be created. Could be easily solved by adding a new make rule and filling in the values on configure time, depending on --*-thread-safety. if on AC_SUBST(zend-scanner, zend-scanner.cc) AC_SUBST(zend-scanner-compile, '$(CXX) $(CXXFLAGS) $< -o $@') else AC_SUBST(zend-scanner, zend-scanner.c) AC_SUBST(zend-scanner-compile '$(CC) $(CCFLAGS) $< -o $@') endif Then we need another check for C++ compiler as well. -- Regards, Sascha Schumann Consultant

Index: Makefile.am =================================================================== RCS file: /repository/libzend/Makefile.am,v retrieving revision 1.9 diff -u -r1.9 Makefile.am --- Makefile.am 1999/07/03 19:37:55 1.9 +++ Makefile.am 1999/08/31 18:37:39 @@ -14,6 +14,9 @@ # automake isn't too clever about "non-standard" use of lex and yacc +zend-scanner.o: @zend-scanner@ + @zend-scanner-compile@ + zend-scanner.c: zend-scanner.l $(LEX) -Pzend -ozend-scanner.c -i $(srcdir)/zend-scanner.l