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/lightdm-gtk-greeter: make it overridable by other greeters #44926
Conversation
8b087b6
to
fe159ac
Compare
04d06ed
to
ea69845
Compare
6a859f0
to
b30d000
Compare
b30d000
to
c446b9b
Compare
Is this build error on the second commit relevant to this PR? Do I need to fix something? |
This is definitely not the way to implement this. I think this can be done by simply removing |
That's better! Remaining problem is that unnecessary |
c446b9b
to
ee4139c
Compare
@infinisil Rewrote it based on your idea. Thanks. |
|
||
environment.etc."lightdm/lightdm-gtk-greeter.conf".source = gtkGreeterConf; | ||
}) | ||
(mkIf (ldmcfg.greeter.name == "lightdm-gtk-greeter") { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't this also be conditioned on ldmcfg.enable
and cfg.enable
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed.
ee4139c
to
f6fb824
Compare
f6fb824
to
7fe4021
Compare
Rebased this PR. I removed |
Are there any updates on this pull request, please? |
Thank you for your contributions.
|
I marked this as stale due to inactivity. → More info |
Is it still relevant? |
Motivation for this change
So that we do not have to care about the order to import greeters modules:
nixpkgs/nixos/modules/services/x11/display-managers/lightdm.nix
Lines 70 to 76 in bb3f7d1
Moreover, I added the second commit in which gtk greeter is disabled automatically if any other greeter is enabled.Currently, every time when adding a greeter module, one has to addin the config section. By this PR, no need to do it.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)