cocur/slugify

I had to install php7.1-mbstring

lcmartinezdev opened this issue ยท 7 comments

It is php7.1-mbstring a requirement?

If true, I can add on readme

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?

I used

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

on Ubuntu 16.10

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

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

Adding the ext to the composer.json is indeed the best approach to avoid further problems ๐Ÿ‘๐Ÿป

@wandersonwhcr could you provide a PR?

Of course! Just a minute ;)

Done.