daybreak
daybreak copied to clipboard
Update sail dependency
The image fails at building because PHP binaries are not found:

docker-compose.yml is using dockerfiles from ./vendor/laravel/sail/runtimes/8.0, reading composer.lock you were using laravel/sail 1.12.12 but the issue is laravel/sail 1.12.12 is using Ubuntu 21.04 (see https://github.com/laravel/sail/blob/v1.12.12/runtimes/8.0/Dockerfile) and Ubuntu 21.04 is deprecated See http://old-releases.ubuntu.com/releases/. However since laravel/sail v1.13.4 they use Ubuntu 21.10 for the 8.0 php flavor and ubuntu 22.04 for the 8.1 php flavor.
So I removed composer.lock and modified composer.json
-"laravel/sail": "^1.0.1",
+"laravel/sail": "^1.15.0",
Now it builds.
However I had the issue when generating the app key.

Update note: this is because I removed composer.lock so when I relaunched composer install with my PHP which is in 8.1 it required this version.
So I modified the docker-compose.yml too.
- context: ./vendor/laravel/sail/runtimes/8.0
+ context: ./vendor/laravel/sail/runtimes/8.1
dockerfile: Dockerfile
args:
WWWGROUP: '${WWWGROUP}'
- image: sail-8.0/app
+ image: sail-8.1/app
Another solution would be to only update composer.json then run composer update "laravel/sail" to update the composer.lock only for sail. But since moving to PHP 8.1 works, it's the occasion to move forward.
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.