Marin

@patrickallaert so is this going to be merged ? I would like a new pecl build which supports 7.3+ . Also a Windows dll build would be nice (latest is...

I guess it is related to this Pull request https://github.com/patrickallaert/php-apm/pull/74 So, tried this fork: https://github.com/VHDev/php-apm/tree/php7.3 Still 1 error left: ``` php-apm# ./configure --without-sqlite3 --with-mysql=/usr/include/mysql checking for grep that handles long...

So changed this, and now it compiles. ``` php-apm# git diff diff --git a/driver_mysql.c b/driver_mysql.c index 6db8ee0..f52e388 100644 --- a/driver_mysql.c +++ b/driver_mysql.c @@ -42,7 +42,7 @@ static void mysql_destroy(TSRMLS_D) {...

So I guess you need to merge that PR and fix the my_bool thing for mysql8...

@dunglas or anybody else that knows can reply questions I have about https://github.com/dunglas/frankenphp/blob/main/CONTRIBUTING.md: - So if I have a local checkout of dunglas/frankenphp I could do steps 1-3 (editing the...

@dunglas Thanks, I will give it a try .

> It looks like we'll have to maintain different syntax to start a transaction depending on the driver. DBAL currently calls beginTransaction directly on the PDO driver, thus not using...

So for my understanding . In the "original" code what would happen if you would get a connection from the pool and do a "start transaction" ? I guess after...

> I can make a pull request to this project if it makes it easier to test @marinhekman. @bartvanhoutte Yes please, put it in draft for now I guess until...

@mmoreram I guess you would like to support php7.4+ incl. php 8. If so, we cannot rely on php8 only, so we have to properly use psr/log version < 2...