Merge lp:~bfiller/upstart/desktop-mode into lp:ubuntu/trusty/upstart

Proposed by Bill Filler
Status: Rejected
Rejected by: Steve Langasek
Proposed branch: lp:~bfiller/upstart/desktop-mode
Merge into: lp:ubuntu/trusty/upstart
Diff against target: 13 lines (+3/-0)
1 file modified
debian/user-conf/xsession-init.conf (+3/-0)
To merge this branch: bzr merge lp:~bfiller/upstart/desktop-mode
Reviewer Review Type Date Requested Status
Steve Langasek Disapprove
Dimitri John Ledkov Pending
Review via email: mp+212075@code.launchpad.net

Description of the change

set global env var DESKTOP_MODE needed by touch apps when running on non-touch sessions like the desktop

To post a comment you must log in.
Revision history for this message
Ricardo Salveti (rsalveti) wrote :

Change looks fine, but would like someone from the foundations team to review this as well.

Just to know, are we able to change the variable name or it is already being used by our applications?

Revision history for this message
Steve Langasek (vorlon) wrote :

Per discussion on IRC, we should do this differently by having apps detect an existing feature of the X environment (e.g., $DISPLAY being set) instead of setting another variable.

review: Disapprove

Unmerged revisions

1546. By Bill Filler

set DESKTOP_MODE env var on all session but ubuntu-touch

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'debian/user-conf/xsession-init.conf'
--- debian/user-conf/xsession-init.conf 2013-03-14 16:30:39 +0000
+++ debian/user-conf/xsession-init.conf 2014-03-21 01:16:51 +0000
@@ -6,6 +6,9 @@
66
7pre-start script7pre-start script
8 [ -z "$DESKTOP_SESSION" ] && stop || true8 [ -z "$DESKTOP_SESSION" ] && stop || true
9 if [ $DESKTOP_SESSION != "ubuntu-touch" ]; then
10 initctl set-env --global DESKTOP_MODE=1
11 fi
9end script12end script
1013
11script14script

Subscribers

People subscribed via source and target branches

to all changes: