-
Notifications
You must be signed in to change notification settings - Fork 168
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use standardized environment variable for desktop detection #588
Comments
XDG_CURRENT_SESSION
for desktop detection
I am running Kubuntu 24.04 (Plasma 5.27 with Wayland), and |
|
I typoed it - it's supposed to be |
FTR, on Fedora 41/Plasma 6 Wayland I get:
|
Ubuntu 24.04.2 LTS, X-Windows, Ubuntu flawored GNOME
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Currently, we (mostly) use
DESKTOP_SESSION
- however, that seems to be something that only happened to be supported, but never actually standardized. (Eg. sway already doesn't support it, for it we already useXDG_CURRENT_DESKTOP
.)Additionally, this non-standardization leads to issues like #546 - where distros just change
DESKTOP_SESSION
as they see fit.There are two options here:
XDG_CURRENT_DESKTOP
, standardized by the desktop entry spec, allows multiple valuesXDG_SESSION_DESKTOP
, set by systemd, only a single valueIt seems slightly preferable to use
XDG_CURRENT_DESKTOP
, as it is more standard. However, then we need to handle the multi-value case (probably best to just use the first value we recognize).The text was updated successfully, but these errors were encountered: