Join GitHub today
GitHub is home to over 28 million developers working together to host and review code, manage projects, and build software together.
Sign upXDG_DATA_DIRS not set in Xenial desktop #2430
Comments
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
marmarek
Nov 12, 2016
Member
Why some gnome-specific script should do that? It looks /etc/X11/Xsession.d/60x11-common_xdg_path also set it, if $DESKTOP_SESSION is set to anything. Is $DESKTOP_SESSION set?
|
Why some gnome-specific script should do that? It looks |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
unman
Nov 12, 2016
Member
On Fri, Nov 11, 2016 at 05:42:11PM -0800, Marek Marczykowski-Górecki wrote:
Why some gnome-specific script should do that? It looks
/etc/X11/Xsession.d/60x11-common_xdg_pathalso set it, if$DESKTOP_SESSIONis set to anything. Is$DESKTOP_SESSIONset?You're right it should.
No, $DESKTOP_SESSION isn't set.
|
On Fri, Nov 11, 2016 at 05:42:11PM -0800, Marek Marczykowski-Górecki wrote:
|
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
unman
Nov 12, 2016
Member
It could be set in /etc/X11/Xsession.d/25xdg-qubes-settings.
Any issue with that?
|
It could be set in /etc/X11/Xsession.d/25xdg-qubes-settings. |
andrewdavidwong
added
bug
C: Ubuntu
labels
Nov 13, 2016
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
andrewdavidwong
Nov 13, 2016
Member
I'm not sure how to categorize this issue, since I don't know what "Xenial-desktop" is. So, I googled it, and it appears to be a version of Ubuntu. Since we didn't have an Ubuntu label, I created one and applied it to this issue. If any of that is incorrect, please let me know.
andrewdavidwong
commented
Nov 13, 2016
•
edited
Edited 1 time
-
andrewdavidwong
edited Nov 13, 2016 (most recent)
edited
-
Nov 13, 2016 (most recent)
|
I'm not sure how to categorize this issue, since I don't know what "Xenial-desktop" is. So, I googled it, and it appears to be a version of Ubuntu. Since we didn't have an Ubuntu label, I created one and applied it to this issue. If any of that is incorrect, please let me know. |
andrewdavidwong
added this to the Release 3.2 milestone
Nov 13, 2016
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
marmarek
Nov 13, 2016
Member
I'm trying to find where this variable is set "normally", but could not. Any idea?
|
I'm trying to find where this variable is set "normally", but could not. Any idea? |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
unman
Nov 13, 2016
Member
@andrewdavidwong That's a good idea. I'm working on Qubes providing a ubuntu template, so having a separate label makes sense.
@marmarek No: I haven't been able to find that. It doesn't seem to be set in other templates
|
@andrewdavidwong That's a good idea. I'm working on Qubes providing a ubuntu template, so having a separate label makes sense. |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
marmarek
Nov 13, 2016
Member
I meant bare metal Ubuntu installation. Anyway, if we can't find how it should be set, /etc/X11/Xsession.d/25xdg-qubes-settings may be fine. But make sure to not override it if already set.
|
I meant bare metal Ubuntu installation. Anyway, if we can't find how it should be set, |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
unman
Nov 13, 2016
Member
@andrewdavidwong On reflection it would be better to put Ubuntu and Debian issues together under a Debian tag. I think almost all issues will be common. If there comes a stage where we have substantial Ubuntu specific problems then we could reinstate a separate tag.
|
@andrewdavidwong On reflection it would be better to put Ubuntu and Debian issues together under a Debian tag. I think almost all issues will be common. If there comes a stage where we have substantial Ubuntu specific problems then we could reinstate a separate tag. |
andrewdavidwong
added
C: Debian
and removed
C: Ubuntu
labels
Nov 14, 2016
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
|
@unman: Done. |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
unman
Nov 15, 2016
Member
So this could go as a conditional patch in gui-agent-linux or in linux-template-builder: as it's the first release of the template we could put it there to have from the start. Thoughts?
|
So this could go as a conditional patch in gui-agent-linux or in linux-template-builder: as it's the first release of the template we could put it there to have from the start. Thoughts? |
marmarek
modified the milestones:
Release 3.2,
Release 3.2 updates
Nov 19, 2016
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
marmarek
Nov 28, 2016
Member
Definitely not linux-template-builder. It should be as minimal as possible - all actual files, modifications etc should be done by some package. Otherwise it will be hard to change that in the future using ordinary update.
|
Definitely not linux-template-builder. It should be as minimal as possible - all actual files, modifications etc should be done by some package. Otherwise it will be hard to change that in the future using ordinary update. |
unman commentedNov 12, 2016
Qubes OS version (e.g.,
R3.1):R3.2
Affected TemplateVMs (e.g.,
fedora-23, if applicable):Xenial-desktop
Expected behavior:
Gnome Applications open.
Actual behavior:
Applications do not open.
Starting from command line gives error:
Steps to reproduce the behavior:
General notes:
XDG_DATA_DIRS is set to :/var/lib/snapd/desktop -
the value should be set in /etc/X11/Xsession.d/55gnome-session_gnomerc
It isn't set because the session isn't set to gnome-session*
Related issues:
#2065 ?