Alon Bar-Lev has posted comments on this change. Change subject: Package must own all directories that it creates. ......................................................................
Patch Set 2: (1 inline comment) .................................................... File ovirt-image-uploader.spec.in Line 50 Line 51 Line 52 Line 53 Line 54 What about this one? by description... the /etc/ovirt-engine is created by you and not by any of your dependencies... so it should also be %dir... but there is a conflict, as when you install engine then it also own this... Correct solution is to install install this under /etc/ovirt-image-uploader... -- To view, visit http://gerrit.ovirt.org/14040 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I6df07d9d7f39014447ba42b15d90d652c61781aa Gerrit-PatchSet: 2 Gerrit-Project: ovirt-image-uploader Gerrit-Branch: master Gerrit-Owner: Kiril Nesenko <[email protected]> Gerrit-Reviewer: Alon Bar-Lev <[email protected]> Gerrit-Reviewer: Federico Simoncelli <[email protected]> Gerrit-Reviewer: Keith Robertson <[email protected]> Gerrit-Reviewer: Kiril Nesenko <[email protected]> Gerrit-Reviewer: Sandro Bonazzola <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
