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-enso-os-greeter: init at 0.2.1 #44920
Conversation
./lightdm-greeters/mini.nix | ||
./lightdm-greeters/gtk.nix |
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.
Not sure if I interpreted the comment here wrong, from what I understand, gtk.nix should always be on the bottom?
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.
With #44926, it doesn't seem to be a problem, so I'll just remove the reordering.
@@ -0,0 +1,72 @@ | |||
{ stdenv, fetchgit, pkgconfig | |||
, dbus, pcre, epoxy, libXdmcp, at-spi2-core, libxklavier, libxkbcommon, libpthreadstubs |
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.
These packages (2nd line) are just to resolve warnings(?) raised by pkgconfig. Seems to compile fine without them.
51344d8
to
84338fc
Compare
84338fc
to
71a52c4
Compare
|
||
greeters = { | ||
gtk = { | ||
enable = false; |
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.
enable = mkDefault false;
is better I guess.
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.
Done
''; | ||
}; | ||
|
||
wallpaper = mkOption { |
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.
This option might be redundant as we already have ldmcfg.background
?
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.
Oh, yeah it is.
71a52c4
to
1ee9932
Compare
1ee9932
to
2d1ecc4
Compare
Motivation for this change
Gtk greeter was too plain for me and couldn't get lightdm-webkit2-greeter to work so the next best thing.
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)