Related: #102225
I will do a module later, I'll need some help

Should I move the packages to pkgs/applications/display-managers/?
greetd isn't really a display manager, but it's close...

Argh you beat me to greetd!

WRT

Should I move the packages to pkgs/applications/display-managers/?
greetd isn't really a display manager, but it's close...

It does manage the actual session managers and so is a part of that system, I'd say it's fine to put it in display-managers (that's where I had it in my implemented nix package). If you need any assistance let me know (I'm ultra new to Nix so I think you've definitely got this) and I'll see if I can assist :)

I will do a module later, I'll need some help

@luc65r I have a module ready to go: greetd.nix.
Would it be ok for you if I create a PR for that?

[...] mobule possibly needs to be moved (where do wayland based DMs go?)

Technically greetd is not wayland-based and makes no assumptions about either the greeter or the session (relevant quote from the docs: "If you can run it from your shell in a TTY, greetd can start it").

That doesn't really make classification easier though. I guess display-managers is fine? There is a hard dependency on PAM, so os-specific/linux would also be correct.

Would it be ok for you if I create a PR for that?

@Qzle Sure!
I wrote something, but I've been too lazy to test it.

That doesn't really make classification easier though. I guess display-managers is fine? There is a hard dependency on PAM, so os-specific/linux would also be correct.

That was my reasoning, I guess it doesn't really matter where it is.

If possible and agreeable I would put all of the greeters (gtkgreet, wlgreet, etc..) under the greetd. prefix so it the greeters would just be greetd.gtkgreet or greetd.wlgreet, as imo it makes it cleaner in the long run

Aside from the conflict, what else is stopping this from getting merged?

I would put all of the greeters (gtkgreet, wlgreet, etc..) under the greetd. prefix

Great idea! I will do it as soon as possible.

what else is stopping this from getting merged?

After that, nothing.

Wow, @luc65r, that's awesome stuff! Especially adding all those greeters! Fantastic, thank you! 😄

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.

Result of nixpkgs-review pr 102242 run on x86_64-linux 1

This is awesome @luc65r, looking forward to the modules landing!

I second @lovesegfault, this is awesome. Thank you so much!