I spoke with Ilia and he aproved the patch. Really the patch changes nothing for 32-bit and UNIX 64-bit.
Thanks. Dmitry. > -----Original Message----- > From: Marcus Boerger [mailto:[EMAIL PROTECTED] > Sent: Tuesday, April 17, 2007 11:11 PM > To: Dmitry Stogov > Cc: Derick Rethans; php-cvs@lists.php.net > Subject: Re: [PHP-CVS] cvs: php-src(PHP_5_2) /ext/standard > proc_open.c /main SAPI.c main.c php_ini.c /main/streams > plain_wrapper.c /win32 select.c /win32/build config.w32 TSRM > TSRM.h tsrm_win32.c ZendEngine2 zend_API.c zend_API.h zend_alloc.c ze > > > Hello Dmitry, > > same here. Itis kind of a big change. Are we in any > desperate need to do this right now? Basically we have no > experience with Win64/PHP at all. So we shouldn't go for it > in the middle of a release process. > > best regards > marcus > > Tuesday, April 17, 2007, 8:52:01 PM, you wrote: > > > On Mon, 16 Apr 2007, Dmitry Stogov wrote: > > >> dmitry Mon Apr 16 08:09:57 2007 UTC > >> > >> Modified files: (Branch: PHP_5_2) > >> /TSRM TSRM.h tsrm_win32.c > >> /ZendEngine2 zend_API.c zend_API.h zend_alloc.c > >> zend_builtin_functions.c > zend_compile.c zend_execute.c > >> zend_execute.h zend_execute_API.c zend_ini.c > >> zend_opcode.c zend_types.h zend_vm_def.h > >> zend_vm_execute.h > >> /php-src/ext/standard proc_open.c > >> /php-src/main SAPI.c main.c php_ini.c > >> /php-src/main/streams plain_wrapper.c > >> /php-src/win32 select.c > >> /php-src/win32/build config.w32 > >> Log: > >> WIN64 support > > > This patch looks sort of dangerous, are you sure you want > to add this > > to > > PHP 5.2.2 after the first RC has already been released? IMO > this would > > fit much better in an upcoming PHP 5.3. > > > Derick > > -- > > Derick Rethans > > http://derickrethans.nl | http://ez.no | http://xdebug.org > > > > > Best regards, > Marcus > > -- > PHP CVS Mailing List (http://www.php.net/) > To unsubscribe, visit: http://www.php.net/unsub.php > -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php
