[DTrace-devel] [PATCH] dtprobed: don't chown systemd service files to root during install

[Nick Alcock]

On 11 Nov 2024, Nick Alcock said:

> I put this in while thinking vaguely that "systemd requires these
> to be root-owned, we're not using install(1) so we should fix that".
> If redirecting, the ownership will be the ownership of the running
> process unless the file already exists (and the thing to do there is
> to rm -f it first, not to chown it afterwards): and if we're not
> root, chowning isn't going to work anyway (and will abort the
> install).
>
> So take the chown out and replace it with an rm -f in the right place.

Patch withdrawn: Kris's patch is better.