[Pkg-systemd-maintainers] Review of http://paste.debian.net/16099/

[Zbigniew Jędrzejewski-Szmek]

On Mon, Jul 15, 2013 at 05:58:00PM +0200, Michael Stapelberg wrote:
> Hi Shawn,
> 
> Thanks for posting your service file for review before submitting.
> 
> I am referring to http://paste.debian.net/16099/, which contains
> ExecStartPre=/bin/mkdir -p /var/run/iodine.
> 
> This is not idiomatic. All temporary directories which should be present
> before starting the service should be configured via the tmpfiles.d(5)
> mechanism.
Hm, I know that this is the current recommended approach,
but is that really better? It splits the configuration between two
different places, and also creates the directory unconditionally.
With enough services one'll have hundereds of empty directories
in /run/. I think that the mkdir-in-unit apporach is actually nicer.

Zbyszek

-- 
they are not broken. they are refucktored
                           -- alxchk