Skip to content
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

LightDM background no longer can be set to a simple color, conflicting with the documentation #90558

Closed
8573 opened this issue Jun 16, 2020 · 5 comments
Labels
0.kind: bug 2.status: stale https://github.com/NixOS/nixpkgs/blob/master/.github/STALE-BOT.md

Comments

@8573
Copy link
Contributor

8573 commented Jun 16, 2020

Describe the bug
Commit 3d1706c changed the type of the NixOS configuration option services.xserver.displayManager.lightdm.background from str to path, requiring that it be set to the path of an image that is to be used as the desktop background. Formerly it also was allowed to set the background to a solid color, e.g., I have it set to "#000000", and the option documentation still claims that this is allowed:

description = ''
The background image or color to use.
'';

Given that the option documentation still claims that setting the background to a color is allowed, I suppose commit 3d1706c was in error.

Notify maintainers
@jtojnar @worldofpeace

Metadata

  • system: "x86_64-linux"
  • host os: Linux 5.4.43-hardened, NixOS, 20.09pre228622.029a5de0839 (Nightingale)
  • multi-user?: yes
  • sandbox: yes
  • version: nix-env (Nix) 2.3.6
  • channels(root): "nixos-20.09pre229574.0a146054bdf"
  • nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos

Maintainer information:

# a list of nixpkgs attributes affected by the problem
attribute: []
# a list of nixos modules affected by the problem
module: [services.xserver.displayManager.lightdm]
@jtojnar
Copy link
Contributor

jtojnar commented Jun 16, 2020

Thanks for reporting.

Looks like the option actually affects several packages, each having a slightly different semantics:

Not sure if we should update the description to respect the lowest common denominator or support both path and colour (either without any check as before or add an assertion).

@8573
Copy link
Contributor Author

8573 commented Jun 17, 2020

Thanks for the information. With this information I was able to get my system building again easily enough by instead using services.xserver.displayManager.lightdm.greeters.gtk.extraConfig. As an easy fix, maybe using ….greeters.*.extraConfig should be suggested, but then, as NixOS/rfcs#42 may be moving slowly towards acceptance, maybe not.

Not sure if we should update the description to respect the lowest common denominator or support both path and colour (either without any check as before or add an assertion).

Considering other NixOS options, it would seem typical to me to have two options, say backgroundFile and backgroundColour, with assertions as necessary. On another hand, maybe such options should be under ….greeters.*, as the format in which a color must be specified could depend on the greeter.

(Edit, 2020-08-02: A more drastic solution in which some users might be interested is to switch to GDM, primarily for other benefits.)

@macalimlim
Copy link

I would prefer having two separate configurations for setting the background image and the background color...

@stale
Copy link

stale bot commented Jan 30, 2021

I marked this as stale due to inactivity. → More info

@stale stale bot added the 2.status: stale https://github.com/NixOS/nixpkgs/blob/master/.github/STALE-BOT.md label Jan 30, 2021
@Zimmi48
Copy link
Member

Zimmi48 commented Aug 26, 2023

Fixed in #138336.

@Zimmi48 Zimmi48 closed this as completed Aug 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0.kind: bug 2.status: stale https://github.com/NixOS/nixpkgs/blob/master/.github/STALE-BOT.md
Projects
None yet
Development

No branches or pull requests

4 participants