Marko Korhonen comments

Results 96 comments of


                                            Marko Korhonen

We created a new new fork

@ifsnop I already tried that `PDO::ATTR_STRINGIFY_FETCHES` for PDO connection (pdo options) without success.

We created a new new fork

@garas I tried it with setAttribute. And now also with pdoOptions of the constructor. Still the same.

We created a new new fork

Now my test matrix reflects current situation: https://github.com/druidfi/mysqldump-php/actions/runs/2911009428

We created a new new fork

We have now internally tested this with Alpine, Debian and Ubuntu images and in all of them PHP 8.1 fails with that double value. We have tried to change, add,...

We created a new new fork

I added extra pdo_checks file to do clean pdo call before starting mysqldump-php processing: https://github.com/druidfi/mysqldump-php/runs/7973950319?check_suite_focus=true

We created a new new fork

I created following "fix" to overcome the issue: https://github.com/druidfi/mysqldump-php/pull/8 It passes all the tests now.

WiP: One Dockerfile to rule them all

I got all 4 variants build with `docker buildx bake...` but `buster` variant: ``` docker run --rm --name frankenphp-demo \ -e FRANKENPHP_CONFIG="worker ./public/index.php" \ -e APP_RUNTIME=Runtime\\FrankenPhpSymfony\\Runtime \ -v $PWD:/app \...

WiP: One Dockerfile to rule them all

I would also wait with this one. I'll keep it updated with `main` in a meanwhile. About Debian Buster, I don't if that error ^^ above is gonna be fixed...

WiP: One Dockerfile to rule them all

@withinboredom I'm more than willing to cut out older stuff but I was in an impression that we should/could/would support the same distros as the main php docker library? @dunglas...

WiP: One Dockerfile to rule them all

Maybe cutting out Buster and Alpine3.15 then?