-
Notifications
You must be signed in to change notification settings - Fork 760
Missing composer vendor folder #28
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
Comments
Hi @larron, I'm not sure why the VOLUME declaration conflicts with running composer. Can you give an example? |
@TrafeX I'm seeing the same issue. Sample of what I'm trying to do...
After running compose, the vendor folder is never created. Now, if I start up the Dockerfile, and execute into the running container, I can run composer install from the |
@whnunlife that's a perfect example of what is happening. Removing the VOLUME declaration from the base will fix the issue. |
Thanks, you both are absolutely right. It's because the I've removed the VOLUME instruction in https://github.com/TrafeX/docker-php-nginx/releases/tag/1.6.0 |
@TrafeX Thanks! |
When attempting to build from this image and use composer as the documentation suggests, you'll come up short and discover that the vendor folder created by composer will never be written into /var/www/html
This is broken because of the VOLUME /var/www/html in the base image, IE this repository.
Removing the VOLUME declaration from the base image will fix this issue.
The text was updated successfully, but these errors were encountered: