Merge lp:~seb128/unity-control-center/user-graphical-sessions into lp:unity-control-center

Proposed by Sebastien Bacher
Status: Merged
Approved by: Sebastien Bacher
Approved revision: 12748
Merged at revision: 12750
Proposed branch: lp:~seb128/unity-control-center/user-graphical-sessions
Merge into: lp:unity-control-center
Diff against target: 15 lines (+2/-4)
1 file modified
panels/user-accounts/um-history-dialog.c (+2/-4)
To merge this branch: bzr merge lp:~seb128/unity-control-center/user-graphical-sessions
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Approve
Robert Ancell Approve
Review via email: mp+210510@code.launchpad.net

Commit message

don't list pts sessions in the user login history

Description of the change

don't list pts sessions in the user login history

To post a comment you must log in.
Revision history for this message
Robert Ancell (robert-ancell) wrote :

I think you mean pts not tty. You could also drop the tty sessions too now that is fixed in LightDM.

review: Approve
Revision history for this message
Sebastien Bacher (seb128) wrote :

> I think you mean pts not tty.

right, edited the commit messages, thanks

> You could also drop the tty sessions too now that is fixed in LightDM.

GNOME is listing them for some reason, let's stick to what they are doing?

Revision history for this message
Robert Ancell (robert-ancell) wrote :

> > You could also drop the tty sessions too now that is fixed in LightDM.
>
> GNOME is listing them for some reason, let's stick to what they are doing?

According to that bug report they only did that to work with LightDM

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
12748. By Sebastien Bacher

list only graphical sessions in the user login history

Revision history for this message
Sebastien Bacher (seb128) wrote :

ok, updated to only include graphical sessions

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'panels/user-accounts/um-history-dialog.c'
--- panels/user-accounts/um-history-dialog.c 2014-02-24 09:46:04 +0000
+++ panels/user-accounts/um-history-dialog.c 2014-03-12 17:38:55 +0000
@@ -237,10 +237,8 @@
237 break;237 break;
238 }238 }
239239
240 /* Display only x-session and tty records */240 /* Display only x-session records */
241 if (!g_str_has_prefix (history.type, ":") &&241 if (!g_str_has_prefix (history.type, ":")) {
242 !g_str_has_prefix (history.type, "tty")&&
243 !g_str_has_prefix (history.type, "pts")) {
244 continue;242 continue;
245 }243 }
246244

Subscribers

People subscribed via source and target branches