Fix dockerfile tiagocoutinho#7 (tiagocoutinho#15)

* Return proper error when no command line arguments are passed

* Fix tiagocoutinho#14: broken Dockerfile

Author: tiagocoutinho

Dockerfile

@@ -1,15 +1,13 @@
 FROM python:3.8-alpine
 
-WORKDIR /src
-
-COPY *.md modbus_proxy.py setup.py *.conf ./
+COPY . /src
 
 # install dependencies to the local user directory
 RUN pip --disable-pip-version-check --no-input --no-cache-dir --timeout 3 \
-    install .[yaml]
+    install src/[yaml]
 
 # clean up
-RUN rm *.md modbus_proxy.py setup.py
+RUN rm /src -r
 
 ENTRYPOINT ["modbus-proxy"]
-CMD ["--help"]
+CMD ["--help"]

src/modbus_proxy.py

@@ -23,10 +23,7 @@
         "standard": {"format": "%(asctime)s %(levelname)8s %(name)s: %(message)s"}
     },
     "handlers": {
-        "console": {
-            "class": "logging.StreamHandler",
-            "formatter": "standard",
-        }
+        "console": {"class": "logging.StreamHandler", "formatter": "standard"}
     },
     "root": {"handlers": ["console"], "level": "INFO"},
 }
@@ -286,7 +283,11 @@ def parse_args(args=None):
         type=str,
         help="log configuration file. By default log to stderr with log level = INFO",
     )
-    return parser.parse_args(args=args)
+    options = parser.parse_args(args=args)
+
+    if not options.config_file and not options.modbus:
+        parser.exit(1, "must give a config-file or/and a --modbus")
+    return options
 
 
 def create_config(args):