Re: Fwd: [PHP-CVS] cvs: php-src(PHP_5_3) /ext/standard/tests/general_functions var_export-locale.phpt

From: Date: Thu, 21 Aug 2008 14:59:48 +0000
Subject: Re: Fwd: [PHP-CVS] cvs: php-src(PHP_5_3) /ext/standard/tests/general_functions var_export-locale.phpt
References: 1 2  Groups: php.qa 
Request: Send a blank email to php-qa+get-64421@lists.php.net to get a copy of this message
Hi Pierre Very good point indeed. The issue might also be that it's not that common to install the build environment, run and compile php on windows. I think the best starting point on where to look if you also want to run phpt tests on windows is http://wiki.php.net/internals/windows. Some other blogposts (thx to Elizabeth): http://elizabethmariesmith.com/2007/10/building-php-on-windows-unconference-slides/ http://c7y.phparch.com/c/entry/1/art,building_php_on_windows Greetings, Felix Op 21-aug-08, om 14:43 heeft Pierre Joye het volgende geschreven:
Hi! A quick note about locale and windows support: As you may noticed, I fixed a couple of locale related tests on windows and enabled some other. The idea is to have more portable tests instead of platform specific versions. The latter is harder to debug and are often not updated or synced (be solaris, sun or windows). I need some help to review other tests and do the same tricks I did. It will be amazingly helpful to improve PHP portability. Feel free to ping me if you have any question, Example: ---------- Forwarded message ---------- -if (!setlocale(LC_ALL, "de","de_DE","de_DE.ISO8859-1","de_DE.ISO_8859-1","de_DE.UTF-8")) { +if (!setlocale(LC_ALL, "german", "de","de_DE","de_DE.ISO8859-1","de_DE.ISO_8859-1","de_DE.UTF-8")) {
       die("skip locale needed for this test is not supported on this
platform"); } Other example: ext/standard/tests/strings strtolower.phpt (using 'C' on windows for US/ASCII) Cheers, --Pierre http://blog.thepimp.net | http://www.libgd.org --PHP Quality Assurance Mailing List <http://www.php.net/> To unsubscribe, visit: http://www.php.net/unsub.php


« previous php.qa (#64421) next »