Skip to content

Revert "Fix incorrect collation for MySQL 8"#6240

Merged
taylorotwell merged 1 commit intolaravel:10.xfrom
Jubeki:revert-6239-10.x
Sep 15, 2023
Merged

Revert "Fix incorrect collation for MySQL 8"#6240
taylorotwell merged 1 commit intolaravel:10.xfrom
Jubeki:revert-6239-10.x

Conversation

@Jubeki
Copy link
Copy Markdown
Contributor

@Jubeki Jubeki commented Sep 15, 2023

Reverts #6239

It seems this collation is not supported for MariaDB:
https://mariadb.com/kb/en/supported-character-sets-and-collations/

I also tested it with Laravel Sail + MariaDB and it doesn't work:

Screenshot 2023-09-15 at 16 22 29

In the original PR, @binaryfire asked if it possible to add a new config section for mariadb.

@taylorotwell taylorotwell merged commit 96d3ecf into laravel:10.x Sep 15, 2023
@taylorotwell
Copy link
Copy Markdown
Member

Hmm, I think we still need a solution for MySQL 8 here. It may be time to define a separate connection for Maria?

@Jubeki Jubeki deleted the revert-6239-10.x branch September 18, 2023 13:09
taylorotwell added a commit that referenced this pull request Sep 18, 2023
y-martini pushed a commit to webfucktory/laravel that referenced this pull request Sep 22, 2023
christopheredjohnson pushed a commit to TheYGSGroup/laravel that referenced this pull request Jan 5, 2024
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.

2 participants