-
Notifications
You must be signed in to change notification settings - Fork 2
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
Some PHP packages from packages.sury.org have changed their policy #40
Comments
it fails with php 7.1.
|
Same trouble with 7.2
Any suggestion, @zerodeux ? |
What are the steps to reproduce the pb ? I tried a "make 7.2" and it succeeded. /assign @athoune |
I just realized that the APT pinning will not work for Docker containers since we don't upgrade in situ. We only build the container from scratch. Give it up. Thus the original problem should only be solved by making sure that the Sury-based builds are using version-prefixed names while installing packages. Here is the list of packages that should be checked (occurence "php-xxx" means : please replace with "install php7.x-xxx" when targeting Sury pacakges) :
In order to realize it's tricky, here is an example. If you want to install the Redis or Memcached extensions which are very popular, you would :
Note that I did not found any occurences of those in the Dockerfile7.x* files, thus I think this ticket can be closed. |
In packages.sury.org : previously a package like 'php-memcached' would come from Debian official repository and pull the default version (ex. memcached module for PHP 7.3 in Buster). If you used Sury's repo to grab a non-Debian version like 7.2, you would install php7.2-memcached.
Now in packages.sury.org (since early dec. 2020, coinciding with PHP 8.0 release) 'php-memcached' is a meta-package which installs ALL versions of the memcached modules from its repository (5.6, 7.0, 7.1, 7.2, 7.3, 7.4, 8.0). Thus if you upgrade you get a vast inflation of unused packages.
The fix is to add some APT pinning information :
The text was updated successfully, but these errors were encountered: