Re: RE: [PHP-CVS] Re: [PHP-DEV] Re: [PHP-CVS] com php-src: Fix dba configuration for Windows: ext/dba/config.w32
| From: | Christoph M. Becker | Date: | Wed, 12 Oct 2016 13:57:48 +0000 |
| Subject: | Re: RE: [PHP-CVS] Re: [PHP-DEV] Re: [PHP-CVS] com php-src: Fix dba configuration for Windows: ext/dba/config.w32 | ||
| References: | 1 2 3 4 5 6 7 | Groups: | php.cvs php.internals |
| Request: | Send a blank email to php-cvs+get-94939@lists.php.net to get a copy of this message | ||
Hi Anatol!
On 08.10.2016 at 20:56, Anatol Belski wrote:
> The support for libqdbm is now added to master, the current
> dependency builds are fetchable here
> http://windows.php.net/downloads/php-sdk/deps/vc14/ or from the
> full
> package.
Thanks!
> I was looking for other possible components to support, but seems
> libqdbm is the only what is viable ATM. Though some more recent
> components can be implemented, but that's another story. Two tests
> are failing for me in ext\dba, that'll need some attention.
I can confirm that dba_flatfile.phpt and dba_qdbm.phpt are failing. I
had already filed <https://bugs.php.net/bug.php?id=72885> a while
ago
which appears to be the reason for the dba_flatfile.phpt failure.
Running the supplied reproduce script results in different test.txt
files; namely, the deletion marker (NUL) is inserted in the wrong place
when run under Windows.
It seems that php_stream_seek()[1] has different behavior in "a+" mode
on Windows than on Linux. On Linux it appears to successfully set the
position, but on Windows it appears to fail. However, in both cases
php_stream_seek() returns 0.
[1]
<https://github.com/php/php-src/blob/PHP-7.0.12/ext/dba/libflatfile/flatfile.c#L131>
--
Christoph M. Becker