Super -> exec vs Alt-F2 -> exec have different environment
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
gnome-session (Ubuntu) |
Fix Released
|
Medium
|
Marco Trevisan (Treviño) | ||
Xenial |
Fix Released
|
Undecided
|
Marco Trevisan (Treviño) | ||
upstart (Ubuntu) |
Fix Released
|
Medium
|
Dimitri John Ledkov | ||
Trusty |
Confirmed
|
Medium
|
Unassigned | ||
Utopic |
Won't Fix
|
Medium
|
Unassigned | ||
Vivid |
Won't Fix
|
Medium
|
Unassigned | ||
Xenial |
Fix Released
|
Undecided
|
Dimitri John Ledkov |
Bug Description
upstart in user session manages environmental variables and most things have them, however Alt-f2 started processes lack them.
[Impact]
Some environment variables are missing from apps that are launcher from Alt+f2 command lens, or in general with dbus-activation (an example might be gnome-terminal, global menu not working).
[Test case]
Super Key -> "xterm" -> Enter -> env | grep SSH
it should return something like:
SSH_AUTH_
Alt+F2 -> "xterm" -> Enter -> env | grep SSH
The returned values should match the results got from the xterm instance that has been ran from unity dash application lens (the one opened with Super).
The same should be checked for the variable XDG_MENU_PREFIX
Before this fix happened we had this result:
via Super Key -> the environment is correct
via Alt-F2 -> it is not
[Regression potential]
Nothing known. Some variables could be overridden, but this is expected.
Related branches
- Marco Trevisan (Treviño) (community): Approve
- Upstart Reviewers: Pending requested
-
Diff: 255 lines (+83/-2)4 files modifieddbus/Makefile.am (+1/-0)
dbus/org.freedesktop.DBus.xml (+9/-0)
init/Makefile.am (+33/-0)
init/control.c (+40/-2)
- VCS imports: Pending requested
-
Diff: 106 lines (+86/-0)3 files modifieddebian/changelog (+7/-0)
debian/patches/export_env_to_upstart.patch (+78/-0)
debian/patches/series (+1/-0)
- VCS imports: Pending requested
-
Diff: 13147 lines (+12538/-0)121 files modified.bzr-builddeb/default.conf (+2/-0)
debian/50-ubuntu.conf (+2/-0)
debian/55gnome-session_gnomerc (+23/-0)
debian/README.Debian (+45/-0)
debian/changelog (+3170/-0)
debian/compat (+1/-0)
debian/control (+130/-0)
debian/control.in (+126/-0)
debian/copyright (+52/-0)
debian/defaults.list (+346/-0)
debian/gnome-session-bin.install (+7/-0)
debian/gnome-session-bin.postinst (+12/-0)
debian/gnome-session-bin.prerm (+9/-0)
debian/gnome-session-bin.user-session.upstart (+31/-0)
debian/gnome-session-common.dirs (+1/-0)
debian/gnome-session-common.install (+6/-0)
debian/gnome-session-common.links (+1/-0)
debian/gnome-session-wayland.install (+2/-0)
debian/gnome-session.install (+3/-0)
debian/patches/103_kill_the_fail_whale.patch (+41/-0)
debian/patches/13_display_session_properties.patch (+18/-0)
debian/patches/22_support_autostart_delay.patch (+239/-0)
debian/patches/50_ubuntu_sessions.patch (+65/-0)
debian/patches/51_remove_session_saving_from_gui.patch (+66/-0)
debian/patches/53_add_sessionmigration.patch (+26/-0)
debian/patches/95_dbus_request_shutdown.patch (+99/-0)
debian/patches/export_env_to_upstart.patch (+76/-0)
debian/patches/revert_remove_gnome_session_properties.patch (+3992/-0)
debian/patches/series (+9/-0)
debian/po-up/POTFILES.in (+2/-0)
debian/po-up/af.po (+42/-0)
debian/po-up/am.po (+42/-0)
debian/po-up/ar.po (+47/-0)
debian/po-up/az.po (+41/-0)
debian/po-up/be.po (+42/-0)
debian/po-up/be@latin.po (+41/-0)
debian/po-up/bg.po (+48/-0)
debian/po-up/bn.po (+41/-0)
debian/po-up/bn_IN.po (+46/-0)
debian/po-up/bs.po (+44/-0)
debian/po-up/ca.po (+43/-0)
debian/po-up/cs.po (+50/-0)
debian/po-up/cy.po (+41/-0)
debian/po-up/da.po (+42/-0)
debian/po-up/de.po (+52/-0)
debian/po-up/dz.po (+44/-0)
debian/po-up/el.po (+69/-0)
debian/po-up/en_CA.po (+42/-0)
debian/po-up/en_GB.po (+41/-0)
debian/po-up/eo.po (+45/-0)
debian/po-up/es.po (+53/-0)
debian/po-up/et.po (+49/-0)
debian/po-up/eu.po (+45/-0)
debian/po-up/fa.po (+43/-0)
debian/po-up/fi.po (+45/-0)
debian/po-up/fr.po (+50/-0)
debian/po-up/fur.po (+41/-0)
debian/po-up/ga.po (+42/-0)
debian/po-up/gl.po (+51/-0)
debian/po-up/gu.po (+54/-0)
debian/po-up/he.po (+44/-0)
debian/po-up/hi.po (+48/-0)
debian/po-up/hr.po (+40/-0)
debian/po-up/hu.po (+48/-0)
debian/po-up/hy.po (+41/-0)
debian/po-up/id.po (+41/-0)
debian/po-up/is.po (+42/-0)
debian/po-up/it.po (+41/-0)
debian/po-up/ja.po (+49/-0)
debian/po-up/ka.po (+41/-0)
debian/po-up/kn.po (+41/-0)
debian/po-up/ko.po (+41/-0)
debian/po-up/ku.po (+43/-0)
debian/po-up/lt.po (+48/-0)
debian/po-up/lv.po (+46/-0)
debian/po-up/mg.po (+40/-0)
debian/po-up/mi.po (+42/-0)
debian/po-up/mk.po (+44/-0)
debian/po-up/ml.po (+45/-0)
debian/po-up/mn.po (+49/-0)
debian/po-up/mr.po (+49/-0)
debian/po-up/ms.po (+43/-0)
debian/po-up/nb.po (+40/-0)
debian/po-up/ne.po (+46/-0)
debian/po-up/nl.po (+53/-0)
debian/po-up/nn.po (+50/-0)
debian/po-up/nso.po (+42/-0)
debian/po-up/oc.po (+43/-0)
debian/po-up/or.po (+44/-0)
debian/po-up/pa.po (+49/-0)
debian/po-up/patches.pot (+42/-0)
debian/po-up/pl.po (+41/-0)
debian/po-up/pt.po (+42/-0)
debian/po-up/pt_BR.po (+49/-0)
debian/po-up/ro.po (+41/-0)
debian/po-up/ru.po (+41/-0)
debian/po-up/rw.po (+50/-0)
debian/po-up/si.po (+43/-0)
debian/po-up/sk.po (+43/-0)
debian/po-up/sl.po (+47/-0)
debian/po-up/sq.po (+43/-0)
debian/po-up/sr.po (+45/-0)
debian/po-up/sr@latin.po (+45/-0)
debian/po-up/sv.po (+45/-0)
debian/po-up/ta.po (+50/-0)
debian/po-up/te.po (+43/-0)
debian/po-up/th.po (+41/-0)
debian/po-up/tk.po (+42/-0)
debian/po-up/tr.po (+48/-0)
debian/po-up/uk.po (+41/-0)
debian/po-up/vi.po (+43/-0)
debian/po-up/wa.po (+49/-0)
debian/po-up/xh.po (+42/-0)
debian/po-up/zh_CN.po (+41/-0)
debian/po-up/zh_HK.po (+49/-0)
debian/po-up/zh_TW.po (+49/-0)
debian/po-up/zu.po (+42/-0)
debian/rules (+24/-0)
debian/source/format (+1/-0)
debian/ubuntu-session.install (+3/-0)
debian/watch (+2/-0)
summary: |
- upstart user session variables are not in dbus activated processes + Super -> exec vs Alt-F2 -> exec have different environment |
Changed in upstart (Ubuntu): | |
importance: | Undecided → Medium |
Changed in unity-lens-applications (Ubuntu): | |
importance: | Undecided → Medium |
Changed in unity (Ubuntu): | |
importance: | Undecided → Medium |
no longer affects: | unity-lens-applications (Ubuntu Trusty) |
no longer affects: | unity-lens-applications (Ubuntu Utopic) |
no longer affects: | unity-lens-applications (Ubuntu Vivid) |
Changed in unity-lens-applications (Ubuntu): | |
status: | New → Won't Fix |
no longer affects: | unity (Ubuntu Trusty) |
no longer affects: | unity (Ubuntu Utopic) |
no longer affects: | unity (Ubuntu Vivid) |
Changed in unity (Ubuntu): | |
status: | New → Won't Fix |
Changed in unity: | |
status: | New → Won't Fix |
tags: | added: rls-w-incoming |
tags: |
added: rls-w-notfixing removed: rls-w-incoming |
no longer affects: | unity-lens-applications (Ubuntu) |
affects: | unity → ubuntu-translations |
no longer affects: | ubuntu-translations |
no longer affects: | unity (Ubuntu) |
Changed in upstart (Ubuntu Trusty): | |
importance: | Undecided → Medium |
Changed in upstart (Ubuntu Utopic): | |
importance: | Undecided → Medium |
Changed in upstart (Ubuntu Vivid): | |
importance: | Undecided → Medium |
Changed in upstart (Ubuntu): | |
assignee: | nobody → Dimitri John Ledkov (xnox) |
description: | updated |
description: | updated |
Changed in gnome-session (Ubuntu): | |
status: | New → In Progress |
importance: | Undecided → Medium |
assignee: | nobody → Marco Trevisan (Treviño) (3v1n0) |
tags: | added: desktop-trello-import |
tags: |
added: verification-done removed: verification-needed |
Changed in upstart (Ubuntu Xenial): | |
status: | New → In Progress |
assignee: | nobody → Dimitri John Ledkov (xnox) |
tags: |
added: verification-done removed: verification-needed |
So further experimentation revealed that we should be keeping the DBUS activation environment up to date with all the initctl set-env calls.
Thus initctl set-env calls should call into org.freedesktop .DBus.UpdateAct ivationEnvironm ent in the user session upstart case.