@mrsdizzie How am I able to test your fix? I get the same issue on wp db import or wp db export after updating to PHP 8.4 and new bundles on the server.

@mrsdizzie How am I able to test your fix? I get the same issue on wp db import or wp db export after updating to PHP 8.4 and new bundles on the server.

This PR hasn't been merged yet so outside of checking out my personal branch here you'd need to wait until it gets reviewed and merged

If you create branch in this repo instead of forking, people could install it via wp package or composer with the dev-<branchname> syntax

I just updated to the latest db_command package and ran a wp db reset. I still get the same warning and the command fails. I think WP CLI doesn't get the correct system information. Check the screenshot below, especially the MySQL binary in the info section. The fix of this pull request is still running with mysql instead of mariadb in my case for that reason, I guess.

I don't think the PR actually touched wp db reset itself. It only changed mysqlcheck and mysqldump, as that's what the original report was about. It did not touch mysql. We use that in a few places still:

^ this one is ultimately used by wp db reset

@mrsdizzie wanna do a follow-up PR?

I don't think the PR actually touched wp db reset itself. It only changed mysqlcheck and mysqldump, as that's what the original report was about. It did not touch mysql. We use that in a few places still:

@mrsdizzie wanna do a follow-up PR?

Yes, I'll think on the best way to do this since wp cli really does assume and use the mysql binary in many places. Probably an initial PR to wp-cli itself for a new utils function that returns the correct binary based on some heuristics (presumably people will have both installed at some point if they are intentionally separating to not be a drop in replacement anymore).

I still getting Deprecated warnings as far as I know I am using the latest wp cli docker image...

$ wp cli info
OS: Linux 6.8.0-1020-azure #23-Ubuntu SMP Mon Dec 9 16:58:58 UTC 2024 x86_64
Shell:
PHP binary: /usr/local/bin/php
PHP version: 8.2.27
php.ini used:
MySQL binary: /usr/bin/mysql
MySQL version: mysql from 11.4.4-MariaDB, client 15.2 for Linux (x86_64) using readline 5.1
SQL modes:
WP-CLI root dir: phar://wp-cli.phar/vendor/wp-cli/wp-cli
WP-CLI vendor dir: phar://wp-cli.phar/vendor
WP_CLI phar path: /var/www/html
WP-CLI packages dir:
WP-CLI cache dir: /tmp/.wp-cli/cache
WP-CLI global config:
WP-CLI project config:
WP-CLI version: 2.11.0

Yeah I think we still need that follow-up PR

I just updated and can confirm that the issues are fixed in version 2.12. Thanks for all the work.