Re: [systemd-devel] [PATCH 2/3] core: Put transient user units in XDG_RUNTIME_DIR instead of XDG_CONFIG_HOME.

[Hristo Venev]

On Fri, 2014-03-28 at 19:02 +0100, Kay Sievers wrote:
> On Fri, Mar 28, 2014 at 6:38 PM, Hristo Venev <hri...@venev.name> wrote:
> > They are temporary and should not clutter the configuration directory.
> > ---
> >  src/core/unit.c | 32 +++++++++++++++-----------------
> >  1 file changed, 15 insertions(+), 17 deletions(-)
> >
> > diff --git a/src/core/unit.c b/src/core/unit.c
> 
> > +                        c = strappend(getenv("XDG_RUNTIME_DIR"), 
> > "/systemd/user");
> 
> Nesting function calls without error checking should be avoided for
> systemd code.
> 
> Kay

This is executed only if the manager is running as user. In case
XDG_RUNTIME_DIR isn't set, systemd would fail to start.

9f5bbfe354c52cd9e28cca32c35596b73e8d738b:src/core/main.c:1510

signature.asc
Description: This is a digitally signed message part

_______________________________________________
systemd-devel mailing list
systemd-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/systemd-devel