Federico Simoncelli has submitted this change and it was merged. Change subject: spec: Simplify libvirt dependencies ......................................................................
spec: Simplify libvirt dependencies Replaced nested if-else structure with single ifs per ditro/version, which are easier to understand and maintain. On Fedora, we required libvirt instead of libvirt python - this looks like a mistake that works becuase we require also libvirt-daemon-xxx packages. Now we require libvirt-daemon like in rhel >= 7. Change-Id: I49ba0be799ffd23402536e7ef1e7d70f4a8aa797 Relates-To: https://bugzilla.redhat.com/1158563 Relates-To: https://bugzilla.redhat.com/1176673 Relates-To: https://bugzilla.redhat.com/1200718 Signed-off-by: Adam Litke <ali...@redhat.com> Signed-off-by: Nir Soffer <nsof...@redhat.com> Reviewed-on: https://gerrit.ovirt.org/39543 Reviewed-by: Allon Mureinik <amure...@redhat.com> Reviewed-by: Yaniv Bronhaim <ybron...@redhat.com> Reviewed-by: Federico Simoncelli <fsimo...@redhat.com> --- M vdsm.spec.in 1 file changed, 10 insertions(+), 6 deletions(-) Approvals: Nir Soffer: Verified Yaniv Bronhaim: Looks good to me, but someone else must approve Federico Simoncelli: Looks good to me, approved Allon Mureinik: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/39543 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I49ba0be799ffd23402536e7ef1e7d70f4a8aa797 Gerrit-PatchSet: 7 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Nir Soffer <nsof...@redhat.com> Gerrit-Reviewer: Adam Litke <ali...@redhat.com> Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Eyal Edri <ee...@redhat.com> Gerrit-Reviewer: Federico Simoncelli <fsimo...@redhat.com> Gerrit-Reviewer: Francesco Romani <from...@redhat.com> Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com> Gerrit-Reviewer: Yaniv Bronhaim <ybron...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches