search

tomsquest / docker-radicale

Docker image for Radicale calendar and contact server :calendar: + security :closed_lock_with_key: + addons :rocket:
GNU General Public License v3.0
562 stars 80 forks source link

Remove tini in favor of `--init` #24

Closed tomsquest closed 6 years ago

tomsquest commented 6 years ago

This PR removes Tini and instructs to use the --init flag when running the container.

Tini is no more needed to handle processes as it is build in docker since version 1.13. From Tini repo:

If you are using Docker 1.13 or greater, Tini is included in Docker itself. This includes all versions of Docker CE. To enable Tini, just pass the --init flag to docker run. Source: https://github.com/krallin/tini#using-tini

@robertbeal what do you think?

robertbeal commented 6 years ago

Awesome, I literally noticed a couple of weeks ago that tini was now part of docker. Makes perfect sense to do.

tomsquest commented 6 years ago

Tagged as 2.1.8.5

tomsquest commented 6 years ago

Thanks @robertbeal !