drupalwxt / docker-scaffold

Docker Scaffold for Drupal WxT
4 stars 7 forks source link

Fix PHP Redis install #3

Closed patheard closed 3 years ago

patheard commented 3 years ago

Creates the missing parent directories that are breaking the PHP Redis install step of the Docker image build.

This is currently breaking CI that relies on the docker-scaffold project: https://github.com/drupalwxt/site-wxt/runs/1443350406?check_suite_focus=true#step:5:823

Happy to tweak this PR if you want to make the change in a different way!

sylus commented 3 years ago

Hey @patheard sorry i had to force push I forgot to push last night when added this:

https://github.com/drupalwxt/docker-scaffold/commit/873bf808c0de74fa40dd5caae643676423758c9a

It should work now :D

I pushed some containers with these customizations till the next version:

docker pull drupalwxt/site-wxt:4.0.0-rc1-redis-nginx docker pull drupalwxt/site-wxt:4.0.0-rc1-redis

https://github.com/drupalwxt/helm-drupal already works with it :D

Can't test this locally via:

https://github.com/drupalwxt/helm-drupal/blob/master/docs/local-dev-kind.md

patheard commented 3 years ago

All good - thanks for the fix :)