@Oliver Grawert:  I don't think that 65snappy is fine based on my
understanding of the spec.  As per:
https://specifications.freedesktop.org/basedir-spec/basedir-spec-
latest.html

"$XDG_DATA_DIRS defines the preference-ordered set of base directories
to search for data files in addition to the $XDG_DATA_HOME base
directory. The directories in $XDG_DATA_DIRS should be seperated with a
colon ':'.

----> If $XDG_DATA_DIRS is either not set or empty, a value equal to
/usr/local/share/:/usr/share/ should be used. <----"

So it sounds like it's perfectly legal for XDG_DATA_DIRS to be empty/not
set, in which case a default is assumed.  But if you set it, it does not
say to append any defaults to the list - therefore if you set it, you
would want to manually ensure those entries are in the list.

In light of the above spec, 65snappy has two problems: (1) it just
appends to the list, leaving a leading ":" even if there was something
in there, (2) if the list is empty, it does not put in these defaults
before assigning it.

Examining my copy of 60x11-common_xdg_path indicates the following:
 * "Add additionnal xdg paths depending on selected desktop session" --- 
clearly the purpose is to add some non-default paths, not specify the default 
ones for other broken packages like snapd.
 * If you follow the logic, it's clear that the ONLY time this script will 
modify XDG_DATA_DIRS is if it is needing to add "/usr/share/"$DESKTOP_SESSION"" 
to the list of paths.  However, unlike snapd, it checks to see if the variable 
is empty, and if so, is careful to add the default paths as per the spec.

In no way do I interpret 60x11-common_xdg_path as being a script to work
around other scripts that don't correctly follow the XDG base dir spec.

Asking 60x11-common_xdg_path, which is from a different package, to work
around the brokenness of another package (snapd), sounds very fragile to
me.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1575014

Title:
  /etc/X11/Xsession.d/65snappy cause XRDP error

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/plasma-desktop/+bug/1575014/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to