-
Notifications
You must be signed in to change notification settings - Fork 123
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
Upgrade Dockerfile from PHP 7.4 to PHP 8.0 #49
Comments
Hi @johnnieclapper, thanks for your work!
You also need to add |
Hey there, this Dockerfile was a great starting point for my use case. Thank you for that!
Nevertheless I was forced to upgrade php because some of my project dependencies needed php 8.0.
Fortunately GRAV 1.7.x perfectly runs with php 8.0.
In case anybody has to match the same requirements I'm sharing my experience with you. Maybe it helps.
What I did:
1. Change PHP Version
In first line of Dockerfile I simply changed the image FROM php:7.4-apache to FROM php:8.0-apache. That's the beauty of the php docker image.
2. Build fails
Then you get an error during build for pecl install apcu:
The reason should be the following:
√php-apcu-bc: This can be dropped, nothing depends on it any longer.: https://wiki.ubuntu.com/ServerTeam/Transition/Php8.0
I simply commented the lines out.
3. Add JIT
Everything seems to be working fine but I'm wondering if in matters of performance it makes sense to enable JIT in PHP 8.0. Therefore I added two JIT settings at the end of ini settings. Not tested jet if it really improves performance.:
Can anybody give me feedback if I'm doing it right? And maybe my suggestions are useful for future updates of this repository. It seems that in the future GRAV will make the step to php 8.0 as a minimum requirement (https://getgrav.org/blog/raising-php-requirements-2020).
Have a great day,
Johnnie
The text was updated successfully, but these errors were encountered: