[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#714428: marked as done (killer: Should it consult ck-list-sessions to find local sessions?)



Your message dated Sun, 28 Sep 2014 17:50:18 +0000
with message-id <E1XYIby-0001so-3H@franck.debian.org>
and subject line Bug#714428: fixed in killer 0.90-10
has caused the Debian Bug report #714428,
regarding killer: Should it consult ck-list-sessions to find local sessions?
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@bugs.debian.org
immediately.)


-- 
714428: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=714428
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message ---
Package:  killer
Version:  0.90-8
Severity: wishlist
User:     debian-edu@lists.debian.org
Usertags: debian-edu

As can be seen in <URL: http://bugs.debian.org/648604 > and
<URL: http://bugs.debian.org/709521 >, some times display managers fail
to update the utmp database of currently logged in users, and this cause
killer to throw out logged in users.

While this definitely is a bug in lightdm and other display managers to
not update utmp and wtmp, we could consider asking the consolekit module
to list local users when utmp is empty.

Something similar to this might work (the patch is incomplete and
untested):

--- killer-0.90/killer  2000-02-10 16:55:57.000000000 +0100
+++ killer      2013-06-29 09:03:51.000000000 +0200
@@ -871,8 +871,7 @@
 
     $consoleuser = "";
 
-    open ( W, "$whocmd|") || return;
-
+    if (open ( W, "$whocmd|")) {
     while ( <W> ) {
        chop;
        @parts = split(/[ \t]+/);
@@ -883,6 +882,24 @@
     }
     close(W);
 }
+    my $consolekitcmd = '/usr/bin/ck-list-sessions'
+    if ( ! $consoleuser && -x $consolekitcmd) {
+        open ( W, "$consolekitcmd|") || return;
+        my $user;
+        while ( <W> ) {
+            chomp;
+            if (/^\tunix-user = '(\d+)'/) {
+                $user = $1;
+            }
+            if (/^\tis-local = (.+)$/) {
+                if ('TRUE' eq $1) {
+                    $consoleuser = uid2username($user);
+                }
+            }
+        }
+        close(W);
+    }
+}
 
 =head2 showConsoleUser
 
The output from ck-list-sessions look like this:

$ ck-list-sessions 
Session1:
	unix-user = '10001'
	realname = 'Some User'
	seat = 'Seat1'
	session-type = ''
	active = TRUE
	x11-display = ':0'
	x11-display-device = '/dev/tty7'
	display-device = ''
	remote-host-name = ''
	is-local = TRUE
	on-since = '2013-06-28T07:41:00.536509Z'
	login-session-id = ''
%

Should we try to extend killer like this?

-- 
Happy hacking
Petter Reinholdtsen

--- End Message ---
--- Begin Message ---
Source: killer
Source-Version: 0.90-10

We believe that the bug you reported is fixed in the latest version of
killer, which is due to be installed in the Debian FTP archive.

A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed.  If you
have further comments please address them to 714428@bugs.debian.org,
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Petter Reinholdtsen <pere@debian.org> (supplier of updated killer package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing ftpmaster@ftp-master.debian.org)


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Format: 1.8
Date: Sun, 28 Sep 2014 19:44:10 +0200
Source: killer
Binary: killer
Architecture: source all
Version: 0.90-10
Distribution: unstable
Urgency: medium
Maintainer: Debian Edu Developers <debian-edu@lists.debian.org>
Changed-By: Petter Reinholdtsen <pere@debian.org>
Description: 
 killer     - Background job killer
Closes: 673371 714428 714477
Changes: 
 killer (0.90-10) unstable; urgency=medium
 .
   * Update empty-domain.dpatch to prefer /etc/mailname over the
     dnsdomainname output when generating mail addresses (Closes:
     673371).
   * Refresh all patches to reflect the new line numbers.
   * Restructure control file lists to be more friendly to line based
     version control systems.
   * Update standard-version from 3.9.5 to 3.9.6.  No changes needed.
   * Add new patch consoleuser-consolekit.patch to fall back to
     ck-list-sessions if it exist for finding local sessions if 'who'
     do not list any (Closes: #714428).
   * Add new patch consoleuser-logind.patch to use loginctl to find
     local sessions if 'who' and consolekit both fail.
   * Use xprintidle to figure out how long the X session have been
     unused (Closes: #714477).
Checksums-Sha1: 
 1fd5bc7d98b38682c9c7ea745c420a2663d7a3d7 1311 killer_0.90-10.dsc
 08e0a1077b0a87937526dae493c83329005fac9a 8679 killer_0.90-10.debian.tar.gz
 dcf49d8302fda7510b7a6fbfca431a3d86b891fe 27620 killer_0.90-10_all.deb
Checksums-Sha256: 
 dc8dd8983c08ee627e6703d5252f9feb842b3dc006ec61f4d1f1b96a23ae6328 1311 killer_0.90-10.dsc
 40119d8babe72a881883c9fef3f757362094b5816be9aa4c274f160effb3afe3 8679 killer_0.90-10.debian.tar.gz
 bcaf3a831fa08c7c34edda61320a2f61c5c8891aaeb45b4520f3e3be288805b5 27620 killer_0.90-10_all.deb
Files: 
 e9613facaa4d582fef2f6b29c9435ca6 1311 misc optional killer_0.90-10.dsc
 3afd2353f2024b44f648b35277287357 8679 misc optional killer_0.90-10.debian.tar.gz
 cd925dd0bd64ca6c12234226c68d182d 27620 misc optional killer_0.90-10_all.deb

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.12 (GNU/Linux)

iD8DBQFUKEkh20zMSyow1ykRAuFBAJ0U+cD++JTc3AJ7iSo3LbHIU4VRYgCgjqUF
/NKmmXuM6L5o0Dy4/rP7xtk=
=ohAo
-----END PGP SIGNATURE-----

--- End Message ---

Reply to: