[Pkg-systemd-maintainers] Bug#724668: Bug#717386: pending

[Michael Stapelberg]

Hi Michael,

thanks for all your comments.

Michael Biebl <email at michaelbiebl.de> writes:
> You should probably run setfacl with -R, so the ACLs are set
> recursively, which might be necessary if files already exist.
Good point, done.

I also pushed a commit that reverts the deletion of the group in postrm.

> But as already mentioned, we don't install /var/log/journal by default,
> so I'm not sure if we should run setfacl at all.
> If a user can mkdir /var/log/journal, he can certainly also run the
> setfacl call.
Alternatively, could we just ship /var/log/journal, so that enabling
persistent logging for the users who wish to do so is easier? Running
setfacl is certainly non-obvious to me :).

-- 
Best regards,
Michael