Yedidyah Bar David has posted comments on this change. Change subject: packaging: do not use /var/tmp for engine ......................................................................
Patch Set 2: Verified+1 > Please ensure that spec file add %dir on this directory and no other package > relies on $(LOCALSTATE_DIR)/tmp/$(ENGINE_NAME) (dwh and reports at least) None look there, as far as I can tell. Still I left it untouched following Alon's request on the bug. Reports currently writes to /var/lib/ovirt-engine/reports.xml (not /var/tmp), but even this is going to change soon for separate hosts - see http://gerrit.ovirt.org/29657 . AFAICT that is irrelevant to this change. > Also verify it works with selinux enforcing. Done, on fedora 19. -- To view, visit http://gerrit.ovirt.org/29708 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I88563c8c0d319eb3356b47233a31c82d9e8b84c7 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <[email protected]> Gerrit-Reviewer: Sandro Bonazzola <[email protected]> Gerrit-Reviewer: Yedidyah Bar David <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: No _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
