On Mon, Sep 30, 2013 at 7:17 PM, Lennart Poettering
<lenn...@kemper.freedesktop.org> wrote:
>  units/u...@.service.in |    1 +
>  1 file changed, 1 insertion(+)
>
> New commits:
> commit e6dca814412f17db05910acedf76d36d3b7f1355
> Author: Evan Callicoat <a...@propter.net>
> Date:   Mon Sep 23 21:01:04 2013 -0500
>
>     units: Add SHELL environment variable
>
>     With the advent of systemd --user sessions, it's become very interesting 
> to spawn X as a user unit, as well as accompanying processes that may have 
> previously been in a .xinitrc/.xsession, or even just to replace a collection 
> of XDG/GDM/KDM/etc session files with independent systemd --user units. The 
> simplest case here would be to login on a tty, with the traditional 
> /usr/sbin/login "login manager".
>
>     However, systemd --user (spawned by user@.service) is at the top level of 
> the slice for the user, and does not inherit any environment variables from 
> the login process. Given the number of common applications which rely on 
> SHELL being set in the environment, it seems like the cleanest way to provide 
> this variable is to set it to %s in the user@.service.
>
>     Ideally in the long-term, applications which rely on SHELL being set 
> should be fixed to just grab it from getpwnam() or similar, but until that 
> becomes more common, I propose this simple change to make user sessions a 
> little bit nicer out of the box.

sadly, that's going to take a very long time.

so, +1 from me on this change.

Thanks,

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

Reply via email to