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
nixos/display-managers: Add DesktopNames parameter to generated desktop session files #79830
Conversation
Actually forgot to request this when I reviewed the PR that added it. |
@ilya-fedin your commit message don't match the contributing guidelines https://github.com/NixOS/nixpkgs/blob/master/.github/CONTRIBUTING.md#submitting-changes. It should be prefixed with the package name or module path.
|
Good catch. We should probably also check whether the downstream session names match the upstream ones. |
…op session files Some display managers (e.g. SDDM) set the XDG_CURRENT_DESKTOP variable accroding to this parameter. If this variable is not defined, there will be some problems (e.g. MATE doesn't have icons on the desktop). Fixes NixOS#71427
04266c8
to
f7768c9
Compare
@worldofpeace oops, thanks |
@ilya-fedin Can you open a backport of this to release-20.03 https://github.com/NixOS/nixpkgs/blob/master/.github/CONTRIBUTING.md#backporting-changes? |
Okay |
Motivation for this change
Some display managers (e.g. SDDM) set the XDG_CURRENT_DESKTOP variable accroding to this parameter.
If this variable is not defined, there will be some problems (e.g. MATE doesn't have icons on the desktop).
Fixes #71427
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)