Request 1201545 (superseded)

This request is superseded by request 1201646 (Show diff)

Overview

Request 1201545 superseded

No description set

Created by wfrisch about 2 months ago
In state superseded
Superseded by 1201646
Open review for openSUSE:Factory:Staging:N

Submit audit

Comments for request 1201545 4

Dominique Leuenberger (dimstar) - about 2 months ago

reviewer

-%dir %attr(750,root,root) %{_sysconfdir}/audit
-%attr(750,root,root) %dir %{_sysconfdir}/audit/plugins.d
+%dir %attr(750,root,root) %{_sysconfdir}/audit/plugins.d

that means /etc/audit is no longer owned by audit? bad practice to create directories below a non-onwed, non-system-owned directory

Wolfgang Frisch (wfrisch) - about 2 months ago

author

Hi dimstar, I reinstated %dir ... /etc/audit for the main audit package. Is it acceptable now? https://build.opensuse.org/request/show/1201646

Enzo Matsumiya (ematsumiya) - about 2 months ago

source maintainer

/etc/audit is now owned by the audit-rules subpackage:

+%dir %attr(750,root,root) %{_sysconfdir}/audit

Let me know if you think this (or introducing the audit-rules subpackage) should be a problem -- I thought it would cause more trouble, but on my (controlled/ideal) test environment all went good.

Wolfgang Frisch (wfrisch) - about 2 months ago

author

Hi Enzo, thanks for looking into it. I left your change and additionally added %dir %attr(750,root,root) %{_sysconfdir}/audit in the main audit package. This doesn't seem to be a problem. I can install all RPMs just fine.