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

MariaDB support #412

Open
robertsilen opened this issue Jul 29, 2024 · 3 comments
Open

MariaDB support #412

robertsilen opened this issue Jul 29, 2024 · 3 comments

Comments

@robertsilen
Copy link
Contributor

Any particular reason MariaDB is not mentioned as supported database in the README? Would it be ok to add it with a PR?

This is part of my efforts I describe in my blog Improving MariaDB Support in Open Source Projects.

@robertsilen
Copy link
Contributor Author

Made PR #414 now, hope it is ok?

@MaineK00n
Copy link
Collaborator

MariaDB is already supported.
Set DBType to MySQL and write DBPath and it will work fine.

$ docker run --rm --name mariadb -d -e MARIADB_ROOT_PASSWORD=password -e MARIADB_DATABASE=test -p 3306:3306 mariadb
$ goval-dictionary fetch ubuntu 24.04 --dbtype mysql --dbpath "root:password@tcp(127.0.0.1:3306)/test?parseTime=true"

@robertsilen
Copy link
Contributor Author

robertsilen commented Aug 20, 2024

MariaDB is already supported. Set DBType to MySQL and write DBPath and it will work fine.

$ docker run --rm --name mariadb -d -e MARIADB_ROOT_PASSWORD=password -e MARIADB_DATABASE=test -p 3306:3306 mariadb
$ goval-dictionary fetch ubuntu 24.04 --dbtype mysql --dbpath "root:password@tcp(127.0.0.1:3306)/test?parseTime=true"

Awesome, thank you @MaineK00n . So it should be fine to add a mention of MariaDB to the README as per #414 ? :)

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