[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: |
Tue, 09 Aug 2022 08:57:29 +0200 |
User-agent: |
Evolution 3.42.1 |
Am Montag, dem 08.08.2022 um 21:50 +0300 schrieb muradm:
> > Which user? Closely related, who acquires resources provided by
> > @code{seatd}? Just the greeter? A regular user logging in?
> > What access level is needed/provided? Read access? Write
> > access?
> While I understand what you are saying, for me user is fine, and I
> can't come up with better description, as my eyes too blurred on
> this subject. Anyway for now I specified it as "libseat user".
I don't think this really aids us here – it instead lets us ask who is
a "libseat user". Perhaps you want to specify "login managers" like
greetd or gdm/sddm/etc. explicitly here? Also, (when) do regular users
have to be in the seat group?
> > > + (group seatd-group (default "seat"))
> > > + (existing-group? seatd-existing-group? (default #f))
> > AFAIK this is not necessary. accounts-service-type can handle
> > multiple eq? groups, so as long as you're careful with what you put
> > into group, you shouldn't get an error.
> ok field removed
Note ‘eq?’ groups here. In other words, you should be able to take a
group (not just a group name) for the group field, sanitize the field
so that it will always be a group, and then use that group in seatd-
accounts (see the second option mentioned in
<79341a82bf9cd5fc6c2227255095f3fe2927dcbe.camel@ist.tugraz.at>). If
for instance instead of seat, you wanted the video group, you would
have to take the one from %base-groups, rather than creating a new one.
Cheers
- [bug#56690] [PATCH] gnu: seatd-service-type: Should use seat group., (continued)
- [bug#56690] [PATCH] gnu: seatd-service-type: Should use seat group., muradm, 2022/08/07
- [bug#56690] [PATCH] gnu: seatd-service-type: Should use seat group., muradm, 2022/08/07
- [bug#56690] [PATCH] gnu: seatd-service-type: Should use seat group., Liliana Marie Prikler, 2022/08/08
- [bug#56690] [PATCH] gnu: seatd-service-type: Should use seat group., muradm, 2022/08/08
- [bug#56690] [PATCH] gnu: seatd-service-type: Should use seat group.,
Liliana Marie Prikler <=
- [bug#56690] [PATCH] gnu: seatd-service-type: Should use seat group., muradm, 2022/08/09
- [bug#56690] [PATCH] gnu: seatd-service-type: Should use seat group., Liliana Marie Prikler, 2022/08/10
- [bug#56690] [PATCH] gnu: seatd-service-type: Should use seat group., muradm, 2022/08/13
- [bug#56690] [PATCH] gnu: seatd-service-type: Should use seat group., muradm, 2022/08/22