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

Default installation requires php-mbstring #67

Closed
wants to merge 2 commits into
base: master
from

Conversation

Projects
None yet
3 participants
@sblondon
Copy link
Contributor

sblondon commented May 22, 2018

After the installation of the package, the web interface requires the install of php-mbstring package. This PR adds the package as a dependance to matomo.

Screenshot:
mbstring_error

/var/log/apt/history.log contains:

Start-Date: 2018-05-18 17:25:57
Commandline: apt install php-mbstring
Requested-By: stephane (1002)
Install: php7.0-mbstring:amd64 (7.0.27-0+deb9u1, automatic), php-mbstring:amd64 (1:7.0+49)
End-Date: 2018-05-18 17:25:57

@mattab

This comment has been minimized.

Copy link
Member

mattab commented May 24, 2018

cc @aureq

@aureq

This comment has been minimized.

Copy link
Collaborator

aureq commented Jun 23, 2018

@sblondon thanks for the pull request.

php-mbstring is only available on Debian 9. How would you recommend declaring the dependency to keep a backward compatibility with Debian 8?

@sblondon

This comment has been minimized.

Copy link
Contributor Author

sblondon commented Jun 24, 2018

It seems the package is not necessary for Jessie (Debian 8) because mbstring is embedded with php5-cli:
https://packages.debian.org/jessie/php5-cli

Something like php5-cli (>= 5.5.9) | (php7.0-cli, php-mbstring) could be a solution. However I doubt it's semantically correct because the parentheses are used for software versions:
https://www.debian.org/doc/debian-policy/#syntax-of-relationship-fields

@sblondon

This comment has been minimized.

Copy link
Contributor Author

sblondon commented Jul 28, 2018

This PR is superseded by new one (#70).

@sblondon sblondon closed this Jul 28, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.