Re: phar update

Hi Greg 2009/2/21 Greg Beaver <greg@chiaraquartet.net>: > Hi, > > [snip] Good work! > > I do need some help verifying phar as working with the following > configurations: > > any endian any unix with intel CC > any Windows Vista > 64-bit Windows XP > any other supported OS/compiler/processor combo I didn't list. I tested phar under 5.3-cvs and HEAD with Windows Vista 32bit little endian and VC9. Theres only one compilation warning which only exists in HEAD: ext\phar\func_interceptors.c(101) : warning C4101: 'newlen' : unreferenced local variable However tests wise then the following tests fails in 5.3-cvs: Phar front controller with mounted external file [C:\php\src\ext\phar\tests\front.phar.phpt] Phar::getSupportedSignatures() [C:\php\src\ext\phar\tests\phar_get_supported_signatures_002.phpt] Phar::setStub() (zip-based) [C:\php\src\ext\phar\tests\zip\phar_stub.phpt] and in 124 tests fails for in HEAD, instead of writing an insanely long list here, I have zipped both the test log and diffs for 5.3 and HEAD which can be downloaded here: > > Verification is as simple as compiling PHP 5.3 from CVS or snap and > running the tests. > > Also thanks to those raising concerns about phar, it has served to make > it stronger. > > Thanks, > Greg > > -- > PHP Internals - PHP Runtime Development Mailing List > To unsubscribe, visit: http://www.php.net/unsub.php > > -- Kalle Sommer Nielsen kalle@php.net