[Feat] added display manager entry - #77
Conversation
areofyl
left a comment
There was a problem hiding this comment.
This only works on systemd, which is a problem since fetch runs on non-systemd distros too (prime example is Gentoo). Could you detect the DM by scanning running processes (like we do for WM) instead? Also the substring search on the service file could false-positive on comments or paths: matching the ExecStart= line specifically would be more reliable. And it probably shouldn't be in the default field list since many tiling WM users don't use a DM.
also removed displaymanager from defaults
|
I have changed it to process scanning instead, but on systems that have a lot of running processes I think this is quite slow. Maybe we should rather check if the distro is running systemd and do the |
I have added a display manager entry. So users can now add
displaymanagerto get something like: