lightdm/gdm needs to set $XDG_CURRENT_DESKTOP
Bug #1212408 reported by
Marc Deslauriers
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Light Display Manager |
Fix Released
|
Medium
|
Unassigned | ||
| gdm (Ubuntu) |
Fix Released
|
Undecided
|
Unassigned | ||
| gnome-session (Ubuntu) |
Fix Released
|
Medium
|
Robert Ancell | ||
| lightdm (Ubuntu) |
Fix Released
|
Medium
|
Robert Ancell | ||
Bug Description
Since the switch to upstart, some processes in the user environment, such as gnome-screensaver, now get launched without $XDG_CURRENT_
This used to get set by gnome-session, but it is no longer high enough in the process tree to set it for the complete user session.
Consensus in #ubuntu-desktop was that lightdm should be setting it.
Related branches
lp:~robert-ancell/lightdm/xdg-current-desktop
- PS Jenkins bot: Approve (continuous-integration)
- Michael Terry: Approve
- LightDM Development Team: Pending requested
-
Diff: 176 lines (+59/-1)8 files modifiedsrc/seat.c (+4/-0)
src/session-config.c (+12/-0)
src/session-config.h (+2/-0)
tests/Makefile.am (+2/-0)
tests/data/sessions/named.desktop (+5/-0)
tests/scripts/xdg-current-desktop.conf (+28/-0)
tests/src/test-session.c (+4/-1)
tests/test-xdg-current-desktop (+2/-0)
lp:~darkxst/ubuntu/saucy/gdm/lp1212408
- Ubuntu branches: Pending requested
-
Diff: 114 lines (+94/-0)3 files modifieddebian/changelog (+7/-0)
debian/patches/series (+1/-0)
debian/patches/ubuntu_export_XDG_CURRENT_DESKTOP.patch (+86/-0)
lp:~darkxst/ubuntu/saucy/gnome-shell-extensions/lp1212408-fix
- Ubuntu branches: Pending requested
-
Diff: 53 lines (+17/-0)5 files modified.pc/applied-patches (+1/-0)
data/gnome-classic.desktop.in (+1/-0)
debian/changelog (+7/-0)
debian/patches/fix-ubuntu-sessions.patch (+7/-0)
debian/patches/series (+1/-0)
| Changed in lightdm (Ubuntu): | |
| assignee: | nobody → Robert Ancell (robert-ancell) |
| tags: | added: rls-s-incoming |
Revision history for this message
| Robert Ancell (robert-ancell) wrote | #1 |
| Changed in lightdm (Ubuntu): | |
| status: | New → Triaged |
| importance: | Undecided → Medium |
Revision history for this message
| Robert Ancell (robert-ancell) wrote | #2 |
Revision history for this message
| Robert Ancell (robert-ancell) wrote | #3 |
| Changed in lightdm (Ubuntu): | |
| status: | Triaged → In Progress |
| Changed in lightdm (Ubuntu): | |
| status: | In Progress → Fix Committed |
| Changed in lightdm: | |
| status: | New → Fix Committed |
| importance: | Undecided → Medium |
| Changed in gnome-session (Ubuntu): | |
| status: | New → Fix Committed |
| importance: | Undecided → Medium |
| Changed in gnome-session (Ubuntu): | |
| assignee: | nobody → Robert Ancell (robert-ancell) |
| status: | Fix Committed → In Progress |
Revision history for this message
| Launchpad Janitor (janitor) wrote | #4 |
| Changed in gnome-session (Ubuntu): | |
| status: | In Progress → Fix Released |
| Changed in lightdm: | |
| milestone: | none → 1.7.10 |
| status: | Fix Committed → Fix Released |
Revision history for this message
| Launchpad Janitor (janitor) wrote | #5 |
| Changed in lightdm (Ubuntu): | |
| status: | Fix Committed → Fix Released |
Revision history for this message
| Robert Ancell (robert-ancell) wrote | #6 |
| summary: |
- lightdm needs to set $XDG_CURRENT_DESKTOP + lightdm/gdm needs to set $XDG_CURRENT_DESKTOP |
| Changed in gdm (Ubuntu): | |
| status: | New → Confirmed |
Revision history for this message
| Launchpad Janitor (janitor) wrote | #7 |
| Changed in gdm (Ubuntu): | |
| status: | Confirmed → Fix Released |
Revision history for this message
| Yves-Alexis Perez (corsac) wrote | #8 |
Revision history for this message
| Robert Ancell (robert-ancell) wrote | #9 |
Revision history for this message
| Robert Ancell (robert-ancell) wrote | #10 |
To post a comment you must log in.
Makes sense. As previously raised by mterry it can't be done without putting the name in the xsession file [1] but his proposal makes sense.
[1] http://