search

tdeckers / docker-openhab

Docker image for Openhab
33 stars 41 forks source link

Redundant apt-get upgrade #22

Closed hjnijlunsing closed 7 years ago

hjnijlunsing commented 8 years ago

The dockerfile contains a duplicate entry for apt-get the same packages. One of them could be removed; to increase performance.

Overall it is advised not to run apt-get upgrade inside the container: https://docs.docker.com/engine/userguide/eng-image/dockerfile_best-practices/

tdeckers commented 7 years ago

Removed in next PR. Thanks.