At the moment users have to run the docker container with /app/rs as a command. Because robosat always goes through the rs wrapper and the docker containers only package up robosat it's a better idea to make it the default entrypoint.
Before
docker run --rm -it daniel-j-h/robosat:latest-cpu /app/rs extract ..
After
docker run --rm -it daniel-j-h/robosat:latest-cpu extract ..
Without sub-command we show the help printout.
```
docker build -t daniel-j-h/robosat:latest-cpu -f docker/Dockerfile.cpu .
docker run --rm -it daniel-j-h/robosat:latest-cpu
usage: ./rs [-h] ...
optional arguments:
-h, --help show this help message and exit
```
jqtrde
commented
5 years ago
At the moment users have to run the docker container with
/app/rsas a command. Because robosat always goes through therswrapper and the docker containers only package up robosat it's a better idea to make it the default entrypoint.Before
docker run --rm -it daniel-j-h/robosat:latest-cpu /app/rs extract ..After
docker run --rm -it daniel-j-h/robosat:latest-cpu extract ..Without sub-command we show the help printout.
Supersedes https://github.com/mapbox/robosat/pull/140 - can be closed.
@jacquestardie please merge.