Re: svn: /php/php-src/ branches/PHP_5_2/NEWS branches/PHP_5_2/ext/pcre/pcrelib/AUTHORS branches/PHP_5_2/ext/pcre/pcrelib/ChangeLog
branches/PHP_5_2/ext/pcre/pcrelib/HACKING branches/PHP_5_2/ext/pcre/pcrelib/LICENCE branches/PHP_5_2/ext/pcre

2010/1/22 Johannes Schlüter <johannes@php.net>: > Hi, > > On Thu, 2010-01-21 at 17:49 +0000, Ilia Alshanetsky wrote: >> iliaa                                    Thu, 21 Jan 2010 17:49:38 +0000 >> >> Revision: >> http://svn.php.net/viewvc?view=revision&revision=293812 >> >> Log: >> Upgraded bundled PCRE to version 8.01 > > This seems to depend on some parts of C99, at least stdint.h, by that it > breaks the build for some compilers. > > On Solaris using suncc for instance: > > ext/pcre/pcrelib/pcre_internal.h", line 198: cannot find include file: > <stdint.h> > > Maybe adding -xc99=all helps there, but one might have to check which > compilers we support and how well they support C99 and if the code uses > other C99 features ... > > See also #50496 and #50753 for a similar issue... > > johannes > > ps. I think it might be beneficial for PHP at large to depend more on > C99 as the types are needed in multiple places and other C99 features > might be interesting ... The main problem is gcc thinking that adding random c99 features to default is a good thing to do. However, we could as well define these types on demand using the configure script. That will solve 99% of the resons why poeple use stdint.h. Cheers, -- Pierre @pierrejoye | http://blog.thepimp.net | http://www.libgd.org