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
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 use XDG_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 values
XDG_SESSION_DESKTOP, set by systemd, only a single value
It 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:
deltragon
changed the title
Use standard XDG_CURRENT_SESSION for desktop detection
Use standardized environment variable for desktop detection
Jun 26, 2024
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: