janeczku / docker-dropbox

:whale: Dropbox in a Docker image. This works.
MIT License
150 stars 129 forks source link

Fix build problem when using docker-compose #2

Closed jefft closed 8 years ago

jefft commented 8 years ago

Hi,

Please find linked a Dockerfile patch that simply moves the VOLUME and EXPOSE to the end.

This is necessary if building with docker-compose build (rather than the usual docker build) with a docker-compose.yml containing volumes, as in this example). For some reason docker-compose build mounts the volumes, including /dbox/.dropbox-dist. The Dockerfile isn't expecting stuff in /dbox/.dropbox-dist, and so when it gets to the dropbox start -i step, it fails with permission errors.

The fix is simply to put the VOLUME mount at the end, after the dropbox start -i.

(PS: Thanks for merging my last pull request. Your changes all work nicely for me.)

janeczku commented 8 years ago

@jefft Good catch and thanks for the fix! What you are doing with docker-compose there looks interesting too :smile: