Change in vdsm[master]: spec: Simplify libvirt dependencies

nsoffer at redhat.com nsoffer at redhat.com
Sun Apr 5 17:52:22 UTC 2015


Nir Soffer has posted comments on this change.

Change subject: spec: Simplify libvirt dependencies
......................................................................


Patch Set 5:

(1 comment)

https://gerrit.ovirt.org/#/c/39543/5/vdsm.spec.in
File vdsm.spec.in:

Line 199: %if 0%{?rhel} == 6
Line 200: Requires: libvirt >= 0.10.2-29.el6_5.4
Line 201: %endif
Line 202: 
Line 203: %if 0%{?fedora} >= 20
> We're now not requiring any libvirt for fedora < 20 (it probably require so
Do you want to add a comment here? According to Yaniv, we are not supporting fedora 20 any more in master, and I plan to send another patch removing this condition. I kept it here only to make backporting 3.5 easier.
Line 204: Requires: libvirt-daemon >= 1.2.9
Line 205: %endif
Line 206: 
Line 207: Requires: libvirt-python, libvirt-lock-sanlock, libvirt-client


-- 
To view, visit https://gerrit.ovirt.org/39543
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I49ba0be799ffd23402536e7ef1e7d70f4a8aa797
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Adam Litke <alitke at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Eyal Edri <eedri at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list