Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PDO_SQLITE version #1833

Closed
kevincerro opened this issue Jun 27, 2024 · 4 comments
Closed

PDO_SQLITE version #1833

kevincerro opened this issue Jun 27, 2024 · 4 comments

Comments

@kevincerro
Copy link
Contributor

kevincerro commented Jun 27, 2024

Its possible to upgrade pdo_sqlite version in bref docker images?

Seems that installed version of pdo_sqlite in bref docker images are too old (3.7.17)

I'm encountering following error with phpunit tests

Doctrine\DBAL\Exception\SyntaxErrorException: An exception occurred while executing a query: SQLSTATE[HY000]: General error: 1 near "(": syntax error
Executing tests from my machine 

Trying in my host machine works because I have a more recent version of pdo_sqlite (3.46.0)

Sample dockerfile

FROM composer:2 AS composer
FROM bref/php-83-fpm-dev
COPY --from=composer /usr/bin/composer /usr/bin/composer

# Install unzip and git (required by composer)
RUN yum install -y unzip git

WORKDIR /var/task

Command to print pdo_sqlite version

php -r 'echo "SQLite version: " . SQLite3::version()["versionString"] . PHP_EOL;'

@kevincerro
Copy link
Contributor Author

kevincerro commented Jun 27, 2024

Tryed to upgrade pdo_sqlite on my own without success.
I cannot find where gets pdo_sqlite installed in bref source Dockerfiles

@kevincerro
Copy link
Contributor Author

With AL2023 this will be solved because provides a more recent version of sqlite-devel

brefphp/aws-lambda-layers#122

@mnapoli
Copy link
Member

mnapoli commented Jun 28, 2024

Yes AL2023 would come with dependency updates, however this will be a major version change for Bref (or at least an opt-in change).

Having sqlite work today (with current layers) is definitely something we want. It should work, it's interesting that you're facing issues 🤔 Do you know what specific query causes the problem?

@mnapoli
Copy link
Member

mnapoli commented Jul 22, 2024

Should be fixed by 2.3.2, this was also necessary to support Laravel 11.

@mnapoli mnapoli closed this as completed Jul 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants