search

NLeSC / structure-from-motion

Structure from Motion Pipeline
GNU General Public License v2.0
61 stars 17 forks source link

pipeline doesn't support spaces in cwd path #34

Closed cwmeijer closed 7 years ago

cwmeijer commented 7 years ago

/media/christiaan/UUI/2017 mei pointcloud$ sudo docker run -u $UID -v $PWD:/data nlesc/structure-from-motion [sudo] password for christiaan: Unable to find image 'mei:latest' locally docker: Error response from daemon: repository mei not found: does not exist or no pull access. See 'docker run --help'.

nielsdrost commented 7 years ago

The command does not seem to get to the pipeline at all. The $PWD expands into your path, misleading the docker command into thinking the name of your image is "mei" Perhaps surround the PWD with quotes?:

sudo docker run -u $UID -v "$PWD:/data" nlesc/structure-from-motion
nielsdrost commented 7 years ago

Also: thanks for reporting this :-)

cwmeijer commented 7 years ago

That works of course. I just copied and pasted from documentation without thinking :-) Created pull request for docker documentation.