[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Wheel group as polkit admins
From: |
Leo Prikler |
Subject: |
Re: Wheel group as polkit admins |
Date: |
Sat, 23 Nov 2019 20:08:39 +0100 |
User-agent: |
Evolution 3.30.5 |
Hi Ludo,
Am Samstag, den 23.11.2019, 18:17 +0100 schrieb Ludovic Courtès:
> Hi Leo,
>
> Leo Prikler <address@hidden> skribis:
>
> > Thanks for the hint. Since it's all just static text, I don't
> > really
> > need the whole Guile power of computed-file, so I've shortened it
> > to:
> >
> > (define polkit-wheel
> > (file-union
> > "polkit-wheel"
> > `(("share/polkit-1/rules.d/wheel.rules"
> > ,(plain-file
> > "wheel.rules"
> > "polkit.addAdminRule(function(action, subject) {
> > return [\"unix-group:wheel\"];
> > });
> > ")))))
>
> Neat.
>
> > > Should we make that the default, BTW? It would seem to make
> > > sense as
> > > that’s the whole point of the “wheel” group.
> > >
> > > What do people think?
> >
> > I'm probably biased as the author of this service, but I think it
> > would
> > probably make sense to include it in %desktop-services. Perhaps we
> > could even add wheel.rules to polkit-service-type itself, although
> > I'm
> > somewhat conflicted on that, as one could not opt out.
>
> Yeah, let’s make it a separate service like you did.
>
> Could you send a patch that does that?
>
> Thanks!
>
> Ludo’.
Sure. I've split it up into two patches – one for the service itself,
and one to add it to %desktop-services. Also I'm using a simple-
service instead of a service type, but it still serves the same
purpose.
Regards,
Leo
0001-services-Add-polkit-wheel-service.patch
Description: Text Data
0002-services-Add-polkit-wheel-service-to-desktop-service.patch
Description: Text Data
Re: Wheel group as polkit admins, Marius Bakke, 2019/11/17