mriedmann / humhub-docker

Alpine-based PHP-FPM and NGINX HumHub docker-container
MIT License
95 stars 79 forks source link

update to php8 #268

Closed mriedmann closed 1 year ago

blackandred commented 1 year ago

Bump 😉

mriedmann commented 1 year ago

help? ;)

sorry almost no time for this project ... any PRs are highly appreciated.

blackandred commented 1 year ago

Sure. I can help with this one as a next PR after #308 - if you could please take a look.

blackandred commented 1 year ago

This can possibly conflict with the Alpine bump PR, need to check if the PHP version if satisifed for all listed versions in versions.txt

The compatiblity table is there: https://docs.humhub.org/docs/admin/requirements/#php-environment

That's a pitty we do not have precise control over the PHP version, but it would require a lot more effort.

blackandred commented 1 year ago

Just checked - Alpine 3.17 contains PHP 8.1 (https://pkgs.alpinelinux.org/packages?name=php*&branch=v3.17&repo=&arch=&maintainer=) and HumHub 1.11 - 1.13 is compatible with PHP 8.1 (https://docs.humhub.org/docs/admin/requirements/#php-environment).

mriedmann commented 1 year ago

Wow thank you so much for your help! I think it should be no problem to drop the support for versions < 1.11 and continue with PHP8 only.