Change in vdsm[master]: spec: split qemu-kvm* requirement to its own %if
nsoffer at redhat.com
nsoffer at redhat.com
Thu Sep 25 18:53:43 UTC 2014
Nir Soffer has posted comments on this change.
Change subject: spec: split qemu-kvm* requirement to its own %if
......................................................................
Patch Set 1:
(1 comment)
Nice, It would be nice to get some more info on the commit message.
http://gerrit.ovirt.org/#/c/33378/1//COMMIT_MSG
Commit Message:
Line 8:
Line 9: qemu-kvm* dependency is about to become even more complex, and as such
Line 10: it is granted a special %if section. One of the three occurences of
Line 11: qemu-kvm-rhev can be dropped, since el6's current requirement equally
Line 12: applies to el7.
Can you explain in the commit message why we did not require specific version for qemu-kvm (still true for other packages)?
Experience with recent builds of non-intel rhel7 show that rhel 7 packages (lvm) may be *older* than rhel 6 packages.
Line 13:
Line 14: Change-Id: I088eee894b631d9f864d6b7563877bf3ac76a9f3
--
To view, visit http://gerrit.ovirt.org/33378
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I088eee894b631d9f864d6b7563877bf3ac76a9f3
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer 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