[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#56690] [PATCH] gnu: seatd-service-type: Should use seat group.
From: |
Liliana Marie Prikler |
Subject: |
[bug#56690] [PATCH] gnu: seatd-service-type: Should use seat group. |
Date: |
Fri, 05 Aug 2022 10:10:40 +0200 |
User-agent: |
Evolution 3.42.1 |
Am Freitag, dem 22.07.2022 um 07:27 +0300 schrieb muradm:
> * gnu/services/desktop.scm (seatd-service-type): Uses "seat" group.
> [extensions]: Added account-service-type with %seatd-accounts.
> (%seatd-accounts): List with "seat" group.
> (<seatd-configuration>): [group] Change default value to "seat".
> * doc/guix.texi: Mention that users may need to become members of
> "seat" group and update default value for group field.
Note, that your current patch adds a little asymmetry. Even if you
configure seatd to use a group different from seat, a (now useless)
seat group will be created.
There are (at least) two possible fixes for this:
1. Disable configuration for the group altogether, marking the field as
deprecated.
2. Change the field into one that accepts a group. Also sanitize the
field so that if a string such as "seat" is provided, it is turned into
a group. Then make seatd-accounts return this group.
Cheers