On 18.07.2017 at 12:15, Anatol Belski wrote: >> -----Original Message----- >> From: Dan Ackroyd [mailto:dan...@basereality.com] >> Sent: Tuesday, July 18, 2017 10:42 AM >> To: Christoph M. Becker <cmbecke...@gmx.de> >> Cc: internals@lists.php.net >> Subject: [PHP-DEV] Re: Bundled libraries upgrade 'process' >> >> On 17 July 2017 at 20:09, Christoph M. Becker <cmbecke...@gmx.de> wrote: >>> On 17.07.2017 at 16:12, Dan Ackroyd wrote: >>> >>>> I just investigated an alleged bug related to the SQLite3 extension. >>> >>> Can you please report the bug and assign it to me. I can't promise >>> that I'll be able to fix it, but I would take a look. >> >> It's already been reported and fixed upstream. >> >> Reported 'bug': https://bugs.php.net/bug.php?id=74883 >> >> Fixed in commit: https://www.sqlite.org/src/info/f27b6370407842e2 > > Thanks for the info. The fix in mainstream doesn't look critical, as the > return code is already being caught in PHP. Clear, there are explicit option > checks in the linked patch, but the currently bundled Sqlite would bail out > anyway. A possible improvement in the end is a different error text. The > version jump seems to be big and barely justified, furthermore I'd be > hesitant to bundle a just released version. Except there are some other bugs > that are fixed by the newer version, I would rather go by applying the patch > to the current bundled lib (if possible) or letting it go.
I agree that updating all supported PHP versions to Sqlite 3.20.0 isn't justified here, particularly since I think this is a userland application error – why would one create a database in read-only mode? There wouldn't be much to read, and most likely they'd get an error for a following `select`. IMHO it would even be sufficient to integrate user note 121337[1] into the docs. At the most we could detect the erroneous flag combination and throw an exception with a better message. [1] <http://php.net/manual/en/sqlite3.construct.php#121337> -- Christoph M. Becker -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php
