Disable avahi daemon by default, enable with docker run -e AVAHI=1, minor cleanup, don't rm locales

cptactionhank / docker-netatalk

Docker container running Netatalk

https://hub.docker.com/r/cptactionhank/netatalk/

MIT License

134 stars 75 forks source link

Disable avahi daemon by default, enable with docker run -e AVAHI=1, minor cleanup, don't rm locales #5

Closed rrva closed 8 years ago

rrva commented 8 years ago

Due to security concerns and that the most-common usage might not need service discovery, I disabled the avahi daemon by default but it's still runnable by doing `docker run -e AVAHI=1 ...``

Also removed an unused env variable at the top of the Dockerfile and reinstated /usr/share/locale files as they might be needed for some charset settings

rrva commented 8 years ago

@samalba

cptactionhank commented 8 years ago

LGTM

rrva commented 8 years ago

Fixed conflicts