[systemd-devel] [PATCH] unit: Set KillMode=mixed for services which use sulogin

[Michael Biebl]

2015-05-12 0:03 GMT+02:00 Lennart Poettering <lennart at poettering.net>:
> On Tue, 12.05.15 00:01, Lennart Poettering (lennart at poettering.net) wrote:
>
>> On Mon, 11.05.15 23:50, Michael Biebl (mbiebl at gmail.com) wrote:
>>
>> > 2015-05-08 17:43 GMT+02:00 Michael Biebl <mbiebl at gmail.com>:
>> > > See http://lists.freedesktop.org/archives/systemd-devel/2015-May/031536.html
>> >
>> > Lennart, are you ok if I commit this?
>>
>> Yes, please!
>
> But actually, I agree with Zbigniew, can you please replace all
> occurences of Killmode=process with KillMode=mixed, with the exception
> of debug-shell.service. Also, if you'd add a comment there explaining
> that that file sticks to KillMode=process to not affect backgrounded
> debugging process, things would even be better!

Hm, I just tested KillMode=mixed with getty at .service on installations
which didn't have libpam-systemd enabled.
This broke the usage of screen, since it killed the screen sessions.
I don't think we should change the behaviour as users might rely on that.

For sulogin using services, like emergency.service, I guess one can
argue, that screen is not something you typically use here.

Michael


-- 
Why is it that all of the instruments seeking intelligent life in the
universe are pointed away from Earth?