Skip to content

Conversation

cmb69
Copy link
Member

@cmb69 cmb69 commented Sep 21, 2018

It is possible to pass flags when opening an SQLite database. For
Sqlite < 3.5.0 these are ignored, since sqlite3_open doesn't support
flags. Neither a warning or notice is raised in this case, nor is this
behavior documented in the PHP manual. Instead of fixing it either
way, we lift the requirement to SQLite 3.5.0 (released on 2007-09-04)
instead of the former SQLite 3.3.9 (released on 2007-01-04).

@cmb69 cmb69 changed the title Require SQLite ≥ 3.5.0 [WIP] Require SQLite ≥ 3.5.0 Sep 21, 2018
@cmb69
Copy link
Member Author

cmb69 commented Sep 21, 2018

pdo_sqlite has the same issue. Should tackle this as well.

@cmb69
Copy link
Member Author

cmb69 commented Sep 23, 2018

I'll wait on the outcome of PR #3548, before pursuing this further.

It is possible to pass flags when opening an SQLite database.  For
Sqlite < 3.5.0 these are ignored, since `sqlite3_open` doesn't support
flags.  Neither a warning or notice is raised in this case, nor is this
behavior documented in the PHP manual.  Instead of fixing it either
way, we lift the requirement to SQLite 3.5.0 (released on 2007-09-04)
instead of the former SQLite 3.3.9 (released on 2007-01-04).
@cmb69
Copy link
Member Author

cmb69 commented Oct 6, 2018

Rebased and overhauled to cater to pdo_sqlite as well.

@cmb69 cmb69 changed the title [WIP] Require SQLite ≥ 3.5.0 Require SQLite ≥ 3.5.0 Oct 6, 2018
This has been introduced by a bad rebase conflict resolution.
@cmb69
Copy link
Member Author

cmb69 commented Oct 9, 2018

Closing and re-opening to trigger CI, which failed for unrelated reasons.

@cmb69 cmb69 closed this Oct 9, 2018
@cmb69 cmb69 reopened this Oct 9, 2018
@cmb69
Copy link
Member Author

cmb69 commented Oct 9, 2018

Hmm, this time AppVeyor failed for unrelated reasons, but Travis succeeded.

Anyhow, if there are no objections, I'll merge this PR on the next weekend.

@php-pulls
Copy link

Comment on behalf of petk at php.net:

Labelling

@php-pulls
Copy link

Comment on behalf of cmb at php.net:

Applied via 87392ef.

@php-pulls php-pulls closed this Oct 13, 2018
@cmb69 cmb69 deleted the sqlite-3.5.0 branch October 13, 2018 10:20
@cmb69
Copy link
Member Author

cmb69 commented Oct 13, 2018

salathe pushed a commit to salathe/phpdoc-en that referenced this pull request Oct 13, 2018
svn2github pushed a commit to svn2github/phpdoc_en that referenced this pull request Oct 14, 2018
heiglandreas pushed a commit to phpdoctest/en that referenced this pull request Feb 4, 2020
salathe pushed a commit to salathe/phpdoc-en that referenced this pull request Sep 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants