You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
snapcraft/wrappers/remote-viewer: do not export XDG_ var
The spec [1] contains this:
> If $XDG_DATA_HOME is either not set or empty, a default equal to $HOME/.local/share should be used.
> If $XDG_CONFIG_HOME is either not set or empty, a default equal to $HOME/.config should be used.
> If $XDG_STATE_HOME is either not set or empty, a default equal to $HOME/.local/state should be used.
So let's not export those since applications should simply rely on the
default/empty. Exporting those without any corresponding directory caused issue
with `nano` as can be seen in the previous commit.
Also, on a "normal" desktop environment, none of XDG_DATA_HOME, XDG_CONFIG_HOME
and XDG_STATE_HOME are exported:
```
$ env | grep ^XDG_
XDG_CONFIG_DIRS=/etc/xdg/xdg-ubuntu:/etc/xdg
XDG_MENU_PREFIX=gnome-
XDG_SESSION_DESKTOP=ubuntu
XDG_SESSION_TYPE=wayland
XDG_CURRENT_DESKTOP=ubuntu:GNOME
XDG_SESSION_CLASS=user
XDG_RUNTIME_DIR=/run/user/1000
XDG_DATA_DIRS=/usr/share/ubuntu:/usr/local/share/:/usr/share/:/var/lib/snapd/desktop
```
[1]: https://specifications.freedesktop.org/basedir-spec/basedir-spec-latest.html#variables
Signed-off-by: Simon Deziel <simon.deziel@canonical.com>
(cherry picked from commit 1c07fcf)
0 commit comments