-
Notifications
You must be signed in to change notification settings - Fork 329
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
Update to smarty4 #850
Comments
We also need to see if Smarty is a system dependency or a composer dependency |
smarty4 is a system dependency on debian. @xavierba is working on a smarty4 package on rh-like systems |
NOTE: When (if) updating requirement to smarty4, we must also list the new version in deb package (replacement of smarty path in config files in |
smarty4 is not available on debian 11 and debian 10. We may require it with as a OR dependency: |
php-Smarty 4 not yet available on EPEL/Fedora, but as we don't require explicitely v3, it will use Smarty 4 as soon it is available. |
Some time ago (see #773) we decided to remove smarty3 from direct dependencies, because
1/ it was broken on ubuntu (https://bugs.launchpad.net/ubuntu/+source/smarty3/+bug/1986916) and
2/ it was unavailable on some rh-like systems
I think it is interesting to now consider the upgrade to smarty4, for multiple reasons:
The text was updated successfully, but these errors were encountered: