diff options
author | Lennart Poettering <lennart@poettering.net> | 2009-08-11 03:57:27 +0200 |
---|---|---|
committer | Lennart Poettering <lennart@poettering.net> | 2009-08-12 04:00:04 +0200 |
commit | 76f0578e7a497e569438316703df254f84b548a0 (patch) | |
tree | ad8e5e828178e229cb6f8db92193834a660d6c55 /tools/ck-launch-session.c | |
parent | 8535db29009c48a37fc2559ca0fcc5b366e1e122 (diff) |
get rid of session.d's session_active_changed callout
The 'session_active_changed' callout is an invitaton to racy misuses
since it splits up the session switches into two events. This patch gets
rid of it.
At this point there are two known users of the session.d/ callouts:
- Suse/Ubuntu ship a script for emulating pam_atconsole style
/var/run/console management for supporting D-Bus' at_console feature
properly. Both Martin Pitt and Kay Sievers however are happy with
having this feature go away. In addition I prepared a patch for
D-Bus which makes the need for these scripts go away entirely. That
patch is currently awaiting review.
- udev's udev-acl tool currently uses this. However this doesn't work
correctly anyway since it relies on that the CK database is correctly
dumped before the tool is invoked, which CK does not handle properly
at this time. In fact fixing this problem is the main reason why I
came up with this patch series. Kay is also very interested in
seeing the 'session_active_changed' logic go away.
A later commit introduces a replacement functionality that is less racy.
Please note that this patch does not drop session.d/ in its entirety.
Only the 'session_active_changed' type callout is removed. Scripts that
only rely on 'session_added'/'session_removed' will continue to work
fine, and will continue to be supported.
Diffstat (limited to 'tools/ck-launch-session.c')
0 files changed, 0 insertions, 0 deletions