-
Notifications
You must be signed in to change notification settings - Fork 26
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
Supported PHP version, automated builds, better tags? #3
Comments
Many require 7.2? Would love to see those stats. However, I agree we could setup automated builds. |
Well, I didn't meant these are OS projects ;) but ocramius/proxy is very popular package, which is in almost every PHP project I am working on. I'll try to build some alpine and debian/ubuntu based images and semi-automate them like in official php library (full-auto isn't possible due to tag naming AFAIK). |
@marcj loce -> lose what is it with the stats loosing? i think a good project and maintainer should not need to worry about this. About the 7.2 being difficult, i think it is non sense with a pinch of salt. I mean what is the error that you are getting rather? The image should obviously be alpine, and not ubuntu mainly. |
php-pm project is very interesting, but indeed, allow multiple PHP version usage (7.2, 7.3) would unlock some usages. |
Looking at your file: php-pm-docker/build/Dockerfile-nginx Line 18 in fa9b3d1
Instead of using alpine and the provided php packages, why not just use the php official images? With some automation, you could deliver multiple version thanks to this method. Would you consider a merge request about that? |
I have found the PHP packages to be very complex. If you find a multi-arch working php approach I'd be happy to consider a PR. |
It is more a question than an issue. Containers on dockerhub looks a bit outdated - maybe it is a good idea to make some automated builds on webhooks? Also supporting more specified tags (with php version - I've tried with updating php version to 7.2 but it is quite complicated) would be nice - many projects require 7.2 php version due to
object
typehinting.I may help, but don't want to make something which won't be accepted later :)
The text was updated successfully, but these errors were encountered: