Re: SQLite 3.14

Argh! Apologies for autocomplete adding that 'e' to your name! - Davey On Sat, Aug 20, 2016 at 03:09 Davey Shafik <davey@php.net> wrote: > Christophe, > > I got the failure multiple times in my Debian Jessie docker container that > I use for builds - you can check it out yourself at > https://github.com/dshafik/php-build to see the setup. > > Thanks for looking into this! > > - Davey > On Sat, Aug 20, 2016 at 01:35 Christoph M. Becker <cmbecker69@gmx.de> > wrote: > >> Hi Davey! >> >> On 19.08.2016 at 15:32, Davey Shafik wrote: >> >> > I saw this failure while packaging 7.1.0beta3, and assume it might be >> > related to your update: >> > >> > FAIL SQLite3 open_basedir checks >> > [ext/sqlite3/tests/sqlite3_21_security.phpt] >> > >> > ========DIFF======== >> > 006- >> > 007- Warning: SQLite3::__construct(): open_basedir restriction in >> effect. >> > File(%s) is not within the allowed path(s): (.) in >> > %ssqlite3_21_security.php on line %d >> > 008- Exception: open_basedir prohibits opening %s in >> > %ssqlite3_21_security.php:%d >> > 009- Stack trace: >> > 010- #0 %ssqlite3_21_security.php(%d): SQLite3->__construct('%s') >> > 011- #1 {main} >> > ========DONE======== >> > >> > Can you please look into this in time for RC1? >> >> I've just checked again with the tagged PHP-7.1.0beta3, but the test >> succeeds on my machine. Therefore it's hard for me to assess what is >> wrong. According to the diff, it appears that the second DB which >> shouldn't be created according to the open_basedir restriction, is >> actually successfully created. >> >> Anyway, it's rather unlikely that an open_basedir related failure is >> caused by updating SQLite, as this check is part of the PHP binding[1], >> which has not been affected by this commit. The issue might be caused >> by commit cc125f27[2], but that's also somewhat unlikely, because the >> Travis checks usually succeed generally. >> >> Can you reproduce the test failure? In which enviroment? >> >> [1] <gïgðz›IE~Jn' >> ó¬https://github.com/php/php-src/blob/PHP-7.1.0beta3/ext/sqlite3 >> /sqlite3.c#L125 >> >> <https://github.com/php/php-src/blob/PHP-7.1.0beta3/ext/sqlite3/sqlite3.c#L125> >> > >> [2] <https://github.com/php/php-src/commit/cc125f27> >> >> -- >> Christoph M. Becker >> >