Re: [PHP-DEV] [Fwd: [PHP-CVS] cvs: php-src /ext/standard formatted_print.c]

2007-08-17 Thread Gregory Beaver
Antony Dovgal wrote: > > Ilia? Greg? > Any comments? > > On 14.08.2007 15:42, Antony Dovgal wrote: >> This patch breaks PEAR phar install on 64bit. >>> From what I can see in the source code, it compares data in the phar >>> with >> the result of sprintf("%u"..), which is different on 32bit and

Re: [PHP-DEV] [Fwd: [PHP-CVS] cvs: php-src /ext/standard formatted_print.c]

2007-08-17 Thread Ilia Alshanetsky
I think the C behaviour is correct and we need to make a (hopefully) slight adjustment to the pear installer code. On 17-Aug-07, at 8:14 AM, Antony Dovgal wrote: Ilia? Greg? Any comments? On 14.08.2007 15:42, Antony Dovgal wrote: This patch breaks PEAR phar install on 64bit. From what I c

Re: [PHP-DEV] [Fwd: [PHP-CVS] cvs: php-src /ext/standard formatted_print.c]

2007-08-17 Thread Antony Dovgal
Ilia? Greg? Any comments? On 14.08.2007 15:42, Antony Dovgal wrote: This patch breaks PEAR phar install on 64bit. From what I can see in the source code, it compares data in the phar with the result of sprintf("%u"..), which is different on 32bit and 64bit because of this patch. Either phar

Re: [PHP-DEV] [Fwd: [PHP-CVS] cvs: php-src /ext/standard formatted_print.c]

2007-08-14 Thread Cristian Rodriguez
On 8/14/07, Antony Dovgal <[EMAIL PROTECTED]> wrote: > > This patch breaks PEAR phar install on 64bit. > From what I can see in the source code, it compares data in the phar with > the result of sprintf("%u"..), which is different on 32bit and 64bit because > of this patch. > > Either phar should