search

librenms / docker

LibreNMS Docker image
MIT License
692 stars 280 forks source link

Update to php8.3 #459

Closed ohai89 closed 1 month ago

ohai89 commented 1 month ago

Fixes #458

Had to manually link php83 -> php That can probably be improved.

CLAassistant commented 1 month ago

CLA assistant check
All committers have signed the CLA.

crazy-max commented 1 month ago

Had to manually link php83 -> php

Yes this is is necessary as default is 8.2 in current alpine 3.19. This is similar to https://github.com/librenms/docker/pull/324/files#diff-dd2c0eb6ea5cfc6c4bd4eac30934e2d5746747af48fef6da689e85b752f39557R74

That can probably be improved.

We could remove version from dirs like https://github.com/crazy-max/docker-nextcloud/pull/78/files but don't really like it tbh