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

I had to install php7.1-mbstring #184

Closed
lcmartinezdev opened this issue Sep 11, 2017 · 7 comments
Closed

I had to install php7.1-mbstring #184

lcmartinezdev opened this issue Sep 11, 2017 · 7 comments

Comments

@lcmartinezdev
Copy link

lcmartinezdev commented Sep 11, 2017

It is php7.1-mbstring a requirement?

If true, I can add on readme

@florianeckerstorfer
Copy link
Member

Slugify uses mb_strtolower(), which should be part of the core PHP library (see http://php.net/manual/en/function.mb-strtolower.php). Which method did you use to install PHP?

@lcmartinezdev
Copy link
Author

I used

sudo apt-get install php7.1 php7.1-cli php7.1-common

on Ubuntu 16.10

@florianeckerstorfer
Copy link
Member

You're right, you need to add --enable-mbstring when compiling PHP. We should add this to the docs.

@wandersonwhcr
Copy link
Contributor

wandersonwhcr commented Sep 15, 2017

IMHO, I think we must add ext-mbstring into require attribute from composer.json. This change will work with all PHP installations with mbstring compiled, too.

@pycarlos you must not recompile PHP, but must install php7.1-mbstring using apt-get.

sudo apt-get install php7.1-mbstring

@OskarStark
Copy link
Contributor

Adding the ext to the composer.json is indeed the best approach to avoid further problems 👍🏻

@wandersonwhcr could you provide a PR?

@wandersonwhcr
Copy link
Contributor

Of course! Just a minute ;)

@wandersonwhcr
Copy link
Contributor

Done.

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

No branches or pull requests

4 participants