lando / core

Current Lando v3 runtime
https://docs.lando.dev/core/v3
GNU General Public License v3.0
4 stars 18 forks source link

Make `lando db-export` use `mariadb-dump`, if it exists #148

Closed christopher-b closed 1 month ago

christopher-b commented 1 month ago

mysqldump is deprecated in current versions of MariaDB, and lando db-export generates a deprecation warning in these versions. This change allows db-export to use mariadb-dump if that command exists and the MARIADB_DATABASE env var is set.

Resolves lando/mariadb#47.

Instead of worrying about which versions of MariaDB use the new command, I'm checking if the command exists and using it if it does.

I have created a test for the MariaDB service.

netlify[bot] commented 1 month ago

Deploy Preview for lando-core failed. Why did it fail? →

Name Link
Latest commit 185a8736502b861f3482db36eab2be7ad14967e0
Latest deploy log https://app.netlify.com/sites/lando-core/deploys/6633ed9feb2bc000086a3d13