[Buildroot] [PATCH v6 3/6] package/polkit: add systemd service file.

Adam Duskett aduskett at gmail.com
Sun Dec 8 18:46:58 UTC 2019


Thomas

On Sun, Dec 8, 2019 at 7:04 AM Thomas Petazzoni
<thomas.petazzoni at bootlin.com> wrote:
>
> On Fri,  6 Dec 2019 15:46:46 -0800
> aduskett at gmail.com wrote:
>
> > From: Adam Duskett <Aduskett at gmail.com>
> >
> > - Add polkit.service for systemd based systems.
>
> Why aren't you using the one from polkit itself? There is one in
> data/polkit.service.in, which gets compiled to polkit.service, and
> apparently installed if HAVE_SYSTEMD is enabled. There also a D-Bus
> service file at data/org.freedesktop.PolicyKit1.service.in.
I guess I could.

>
> > - Add a polkitd user upon which /usr/lib/polkit-1/polkitd relies.
> > - Set appropriate directory permissions to allow the polkitd user to
> >   access rules and actions.
> > - Set appropriate default attributes for the pkexec binary.
>
> These could be done separately from the systemd service file addition.
>
No, they can't. The user is required to access those particular
directories during
the service startup.

I explained that in IRC.

> Thomas
> --
> Thomas Petazzoni, CTO, Bootlin
> Embedded Linux and Kernel engineering
> https://bootlin.com

I don't have time to make a new patch set, so I am going to just mark the entire
patch series as rejected and move on.

If anybody else wants to resurrect this patch series they may do so.

Adam


More information about the buildroot mailing list