Merge pull request #26 from dirkcjelli/dockerfix

Some minimal Dockerfile fixes, readme
arnisoph · Mar 18, 2018 · ad47e27 · ad47e27
2 parents 8608b28 + dca4098
commit ad47e27
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/Dockerfile b/Dockerfile
@@ -14,7 +14,12 @@ COPY requirements/base.txt /tmp/requirements.txt
 COPY tabellarius/ /tabellarius
 
 RUN \
+  pip install --upgrade pip && \
   pip install -r /tmp/requirements.txt && \
   rm -rf \
     /tmp/* \
     /var/tmp/*
+
+ENTRYPOINT ["python", "/tabellarius/tabellarius.py"]
+
+CMD ["-V"]
diff --git a/README.rst b/README.rst
@@ -190,7 +190,7 @@ Run in Docker container:
 
 ::
 
-    $ docker run -it -v /path/to/config:/config:ro bechtoldt/tabellarius:<VERSION> python /tabellarius/tabellarius.py --confdir=/config
+    $ docker run -it -v /path/to/config:/config:ro bechtoldt/tabellarius:<VERSION> --confdir=/config
 
 If you prefer running Tabellarius on arbitrary computers you should consider using `virtualenv <https://pypi.python.org/pypi/virtualenv>`_ with or without `virtualenvwrapper <https://pypi.python.org/pypi/virtualenvwrapper/>`_.