Bug#1010247: ITP: greetd -- minimal and flexible login manager daemon

[Marc Dequènes (duck) <duck@duckcorp.org>]

Quack,

Thanks a lot for your feedback.

On 2022-07-16 14:56, Johannes Schauer Marin Rodrigues wrote:

> Issue 1: your choice of _greetd lets my postinst fail because my 
> NAME_REGEX in
> /etc/adduser.conf is "^[a-z][-a-z0-9_]*\$"

This naming scheme is becoming quite common in Debian now. Since you do 
not have problem with _apt or _chrony etc, I wondered if there was any 
adduser magic option and indeed apt is using --force-badname. I'll add 
the option but please not chrony, and probably other packages, do not 
necessarily use it and you'll probably have to make an exception (or 
fill bug reports).

> Issue 2: after installing it, I get a black screen printing "/bin/sh: 
> 1: exec:
> agreety: not found" -- maybe this is because the user "greeter" doesn't 
> have
> /usr/sbin/ in their PATH?

I did not have time to test agreety and sway is in the path, but I 
checked and agreety does not work when run as non-root (not sure how 
useful that would be anyway) so I'm in favour of changing the default 
config to use the absolute path. Did you try this solution?

> Issue 3: /etc/greetd/config.toml says:

I have not yet packaged wlgreet yet but that's coming; I added this 
config as example for the future.

The config I'm using is:
----------------------------------
exec "wlgreet; swaymsg exit"

bindsym Mod4+shift+e exec swaynag \
	-t warning \
	-m 'What do you want to do?' \
	-b 'Poweroff' 'systemctl poweroff -i' \
	-b 'Reboot' 'systemctl reboot -i'

output * bg /etc/greetd/background fill

include /etc/sway/config.d/*
----------------------------------

I guess I'll remove the background line, or maybe add another 
greetd-specific config directory for easy customization, I'm not sure 
yet.

[some time later…]

I pushed these changes, so hopefully that should help.

I could not get any reply from the Debian Rust team unfortunately, so 
enquote cannot be uploaded yet. I need to package greetd_ipc in the team 
too and that should be enough for wlgreet I think.
I'll keep you posted.

Regards.
\_o<

--
Marc Dequènes