Skip to content
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

Populate user dir once (#44) #52

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

fanks4
Copy link

@fanks4 fanks4 commented Mar 19, 2023

I found another way to install grav by populating user dir if empty during container start. Therefore i skip user dir during build. A script will then populate /var/www/user as described in README.md
This commit also includes https://github.com/Freiheitswolke/docker-grav/tree/fix/volume and some changes to README.md to reflect the changed volumes. The commit will not break the old way with a mount for the whole '/var/www/html' for those who want to upgrade grav via the admin console. (If '/var/www/html' is mounted you need to populate the whole directory yourself after starting the container anyway.)
Building or pulling the container image with the new version of grav is a more proper way to perform upgrades in my opinion.

Closes #44

Kind regards and thank you for all the great work so far
Simon

I found another way to install grav by populating user dir if empty during container start.
Therefore i skip user dir during build. A script will then populate /var/www/user as described in README.md
This commit also includes https://github.com/Freiheitswolke/docker-grav/tree/fix/volume and some changes to README.md to reflect the changed volumes.
The commit will not break the old way with a mount for the whole '/var/www/html' for those who want to upgrade grav via the admin console.
Building or pulling the container image with the new version of grav is a more proper way to perform upgrades in my opinion.

Closes getgrav#44
@joshua-jerred
Copy link

This works great a year later still.

For anyone still looking to use this, use these changes along with a source image change to php:8.1.18-apache-bullseye. As long as the lines mentioned #49 are commented out, everything works great.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bind html to a directory doesn't work
2 participants