Re: [PHP4BETA] zend-scanner.l does not compile with SUNs CC (4.2)

On Mon, 27 Dec 1999, Thies C. Arntzen wrote: > On Mon, 27 Dec 1999, Zeev Suraski wrote: > > > On Mon, 27 Dec 1999, Thies C. Arntzen wrote: > > > > > CFLAGS="-g" CXXFLAGS="-g" CC=c89 CXX=CC ../php4/configure blabla > > > > > > /bin/sh ../libtool --silent --mode=compile CC -DHAVE_CONFIG_H -I. -I../../php4/Zend > > > -I.. -I../Zend -I../../php4 -I/dot/dc4/zend/aolserver3//include > > > -I/dot/oracle/app/oracle/product/8.0.5/rdbms/demo > > > -I/dot/oracle/app/oracle/product/8.0.5/network/public > > > -I/dot/oracle/app/oracle/product/8.0.5/rdbms/demo > > > -I/dot/oracle/app/oracle/product/8.0.5/network/public -I/dot/dc4/zend/php-dev/expat//include > > > -I../TSRM -I../../php4/Zend -I../../php4/TSRM -D_POSIX_PTHREAD_SEMANTICS -D_REENTRANT -I../TSRM -g > > > -c zend-scanner-cc.cc > > > "../../php4/Zend/zend-scanner.l", line 204: Error: stdiofile is not a > > > member of streambuf. > > > > > > sascha said, this is related to the fact that the compiler does not > > > support full ISO C++. > > > > > > if i comment out the offending line everything compiles. but we leak fd's > > > - so it's not a valid fix. > > > > > > i think we should support this compiler, as gcc is not always a > > > choice. > > > > > > btw, it's not really the compiler that has the problem, it's the -lC + > > > headers that is somehow different to gcc's -lstdc++ or M$VC++. > > > > > > so the next logical question question is: how hard would it be to remove > > > the -lC / -lstdc++ completely? flex only want's some tiny class for it's > > > input - couldn't we just write our own? > > > > It wants an iostream derived class, not just any class, and for that you > > need the standard C++ library. > > i'm not a C++ guy, but i'm sure we could rewrite the few needed things so > that zend could live without the standard C++ library. the "new" op is in > the crt if i'm not wrong. I'm pretty sure we can't. I wasn't talking about the new operator, obviously there's not a single compiler out there that doesn't support this keyboard. iostream is in the standard C++ library; flex++ relies on it. > > > > I'm not too keen on supporting compilers that don't conform to the ANSI > > C++ standard. If you want to compile a thread safe build, you would need > > an ANSI C++ compliant compiler. > > that makes SUN Workshop 4.2 unusable for compiling Zend. > > as i'm not mr c++ nor mr flex there's not much i can do about this myself > right now. i'm sure were going to hear more about this issue as soon as > more people want the ZTS build in the future (Apache2). there are a lot > of people that are forced to use the vendor toolchain and cannot use gcc > for political (or whatsoever) reasons. I'm sure Sun must have an updated ANSI C++ compiler. If they don't, they should. There are plenty of different non-conforming C++ compilers out there (Visual C++ doesn't conform either but it's pretty close), we simply can't support them all. Zeev -- Zeev Suraski <zeev@zend.com> http://www.zend.com/