Re: [PHP-DEV] [HEADS UP] pecl/phar is now ext/phar

2008-05-14 Thread Richard Quadling
2008/5/13 Antony Dovgal [EMAIL PROTECTED]: On 13.05.2008 01:45, Gregory Beaver wrote: Thanks to all who have contributed, particularly Marcus, Steph, Lars, and the others who chimed in with ideas on the list. phar_detect_phar_fname_ext() fails if is_complete = 1 and filename contains ..

Re: [PHP-DEV] [HEADS UP] pecl/phar is now ext/phar

2008-05-14 Thread Gregory Beaver
Richard Quadling wrote: 2008/5/13 Antony Dovgal [EMAIL PROTECTED]: On 13.05.2008 01:45, Gregory Beaver wrote: Thanks to all who have contributed, particularly Marcus, Steph, Lars, and the others who chimed in with ideas on the list. phar_detect_phar_fname_ext() fails if

Re: [PHP-DEV] [HEADS UP] pecl/phar is now ext/phar

2008-05-14 Thread Richard Quadling
2008/5/14 Gregory Beaver [EMAIL PROTECTED]: Richard Quadling wrote: 2008/5/13 Antony Dovgal [EMAIL PROTECTED]: On 13.05.2008 01:45, Gregory Beaver wrote: Thanks to all who have contributed, particularly Marcus, Steph, Lars, and the others who chimed in with ideas on the list.

Re: [PHP-DEV] [HEADS UP] pecl/phar is now ext/phar

2008-05-13 Thread Hannes Magnusson
On Mon, May 12, 2008 at 11:45 PM, Gregory Beaver [EMAIL PROTECTED] wrote: * code coverage is pushing 80%, up from about 63% Are these known failures? Number of tests : 377 364 Tests skipped : 13 ( 3.4%) Tests warned:0 ( 0.0%) ( 0.0%) Tests failed:

Re: [PHP-DEV] [HEADS UP] pecl/phar is now ext/phar

2008-05-13 Thread Antony Dovgal
On 13.05.2008 01:45, Gregory Beaver wrote: Thanks to all who have contributed, particularly Marcus, Steph, Lars, and the others who chimed in with ideas on the list. phar_detect_phar_fname_ext() fails if is_complete = 1 and filename contains .. For example: Breakpoint 1,

Re: [PHP-DEV] [HEADS UP] pecl/phar is now ext/phar

2008-05-13 Thread Steph Fox
Hannes, Are these known failures? Guess the configure line? I can't reproduce this with default settings. - Steph Number of tests : 377 364 Tests skipped : 13 ( 3.4%) Tests warned:0 ( 0.0%) ( 0.0%) Tests failed: 256 ( 67.9%) ( 70.3%) Tests

Re: [PHP-DEV] [HEADS UP] pecl/phar is now ext/phar

2008-05-13 Thread Antony Dovgal
On 13.05.2008 15:57, Steph Fox wrote: Hannes, Are these known failures? Guess the configure line? I can't reproduce this with default settings. See my mail about phar_detect_phar_fname_ext(), it might be the same problem. -- Wbr, Antony Dovgal -- PHP Internals - PHP Runtime Development

Re: [PHP-DEV] [HEADS UP] pecl/phar is now ext/phar

2008-05-13 Thread Hannes Magnusson
On Tue, May 13, 2008 at 1:57 PM, Steph Fox [EMAIL PROTECTED] wrote: Hannes, Are these known failures? Guess the configure line? I can't reproduce this with default settings. './configure' \ '--enable-debug' \ '--with-zend-vm=GOTO' \ '--without-pear' \ '--with-zlib' \ '--with-bz2' \

Re: [PHP-DEV] [HEADS UP] pecl/phar is now ext/phar

2008-05-13 Thread Steph Fox
and default php.ini. Yes. I did ./cvsclean and obviously ./buildconf Looks like the buildtest-process script (or php.qa.reports) is rejecting the failure report.. probably due to the size of the report Sorry Hannes, should've kept it on-list. The clue was in Tony's mail, I'm just testing a

Re: [PHP-DEV] [HEADS UP] pecl/phar is now ext/phar

2008-05-13 Thread Steph Fox
Sorry Hannes, should've kept it on-list. The clue was in Tony's mail, I'm just testing a fix now. Access denied, insufficient karma. Fix is in PECL CVS, what do I do now? Mail in a patch against 5_3? - Steph -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit:

Re: [PHP-DEV] [HEADS UP] pecl/phar is now ext/phar

2008-05-13 Thread Steph Fox
You should try again, karma should be there, now. Thanks Johannes :) Hannes, let me know if. - Steph -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php

Re: [PHP-DEV] [HEADS UP] pecl/phar is now ext/phar

2008-05-13 Thread Johannes Schlüter
On Tue, 2008-05-13 at 14:41 +0100, Steph Fox wrote: Sorry Hannes, should've kept it on-list. The clue was in Tony's mail, I'm just testing a fix now. Access denied, insufficient karma. Fix is in PECL CVS, what do I do now? Mail in a patch against 5_3? You should try again, karma

Re: [PHP-DEV] [HEADS UP] pecl/phar is now ext/phar

2008-05-13 Thread Hannes Magnusson
On Tue, May 13, 2008 at 4:14 PM, Steph Fox [EMAIL PROTECTED] wrote: You should try again, karma should be there, now. Thanks Johannes :) Hannes, let me know if. Munch better, only 42 failures now. 001+ Content-type: text/html; charset=utf-8 001- Content-type: text/html Looks like all

Re: [PHP-DEV] [HEADS UP] pecl/phar is now ext/phar

2008-05-13 Thread Steph Fox
Hi Hannes, Munch better, only 42 failures now. 001+ Content-type: text/html; charset=utf-8 001- Content-type: text/html Looks like all of them are failing like this. That's really not a lot to go on. Which are 'all of them'? The frontcontrollers? Thanks, - Steph -- PHP Internals -

Re: [PHP-DEV] [HEADS UP] pecl/phar is now ext/phar

2008-05-13 Thread Steph Fox
Hi Greg, I would guess all tests that have --EXPECT_HEADERS-- and text/html. We need to put default_charset= in the --INI-- section and then the tests will pass. Do you want to do this Steph, or should I? Would you mind? I'm caught up in HEAD right now :\ - Steph Greg -- PHP Internals

[PHP-DEV] [HEADS UP] pecl/phar is now ext/phar

2008-05-12 Thread Gregory Beaver
Hi, It's time for helly's birthday present from me (and indirectly, Derick, who did the cp -r) :). As Johannes requested, pecl/phar has been copied to php-src/ext/phar, development will continue there. I suspect there may need to be some changes at gcov.php.net to keep things running smoothly,

Re: [PHP-DEV] [HEADS UP] pecl/phar is now ext/phar

2008-05-12 Thread Nuno Lopes
It's time for helly's birthday present from me (and indirectly, Derick, who did the cp -r) :). Happy Birthday! I suspect there may need to be some changes at gcov.php.net to keep things running smoothly, so this is a heads up message. OK, done. Nuno -- PHP Internals - PHP Runtime