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

Switch to mysql-client #118

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Switch to mysql-client #118

wants to merge 1 commit into from

Conversation

Synchro
Copy link

@Synchro Synchro commented Feb 4, 2021

Add MySQL's key and repo and use it to install mysql-client in place of mariadb-client, should fix #117.

I have no idea how to test this, but I've tested the individual commands.

@edbizarro
Copy link
Owner

Thanks @Synchro

Can you take a look on the alpine images as well? Thanks.

@Synchro
Copy link
Author

Synchro commented Feb 4, 2021

That's not quite so easy. Alpine doesn't have a MySQL package, or rather it does, but it actually installs MariaDB. It seems that MySQL itself contains support for Alpine, but Alpine does not in turn provide a packaged version of it. There is an open request to add a real MySQL package to Alpine, but it's not getting much love, so I commented on it to try to push things along.

@Synchro
Copy link
Author

Synchro commented Feb 9, 2021

Any chance of merging this? All projects on gitlab using Laravel 8 schema dumps will be breaking because of this issue. What's here won't fix Alpine, but it will at least fix Debian, which I expect is the majority.

@CptOrange16
Copy link

This would be very useful for us, we recently updated our system to mysql 8 and it would be nice to be able to have the same version running ok for gitlab

@Synchro
Copy link
Author

Synchro commented Jul 29, 2022

For anyone running into this, I solved this by simply switching to a different image, no other changes required, and it just works:

image: lorisleiva/laravel-docker:8.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

MySQL 8 support broken
3 participants