Code review comment for lp:~xnox/upstart/user-log-dir

Revision history for this message
Steve Langasek (vorlon) wrote :

On Wed, Jan 16, 2013 at 12:02:21PM -0000, Dmitrijs Ledkovs wrote:

> Reading XDG specification, it sounds like instead of
> $XDG_CACHE_HOME/upstart/sessions, we instead should be using
> $XDG_RUNTIME_DIR/upstart/ to store session init's pids.

> """

> $XDG_RUNTIME_DIR defines the base directory relative to which
> user-specific non-essential runtime files and other file objects (such as
> sockets, named pipes, ...) should be stored. The directory MUST be owned
> by the user, and he MUST be the only one having read and write access to
> it. Its Unix access mode MUST be 0700.

> The lifetime of the directory MUST be bound to the user being logged in.
> It MUST be created when the user first logs in and if the user fully logs
> out the directory MUST be removed. If the user logs in more than once he
> should get pointed to the same directory, and it is mandatory that the
> directory continues to exist from his first login to his last logout on
> the system, and not removed in between. Files in the directory MUST not
> survive reboot or a full logout/login cycle.

This last paragraph is the problematic part, since up to this point AIUI
we've been discussing having one upstart session per login session, *not*
one upstart session per logged-in user. XDG_RUNTIME_DIR only has the
correct semantics for the latter.

« Back to merge proposal