Subject: [bug#49969] [PATCH v9 4/7] gnu: base: Add greetd-service-type.

From: Lars-Dominik Braun <[email-masked]>
Date: Tue, 7 Jun 2022 11:44:15 +0200

Hi,

overall this works quite well for me, thanks!

A few comments:

> +                  #:log-file
> +                  (string-append "/tmp/greetd." #$greetd-vt ".log")))
Why log to /tmp and not /var/log here?

> +                   #:log-file "/tmp/seatd.log"))
Same.

> * gnu/packages/admin.scm (libseat): New variable
> * gnu/packages/admin.scm (seatd): New variable
> * gnu/packages/freedesktop.scm (seatd): Remove variable
(gnu packages wm) references seatd, but does not use (gnu packages admin),
thus `guix pull` fails.

Cheers,
Lars