Hello Saggi Mizrahi,

I'd like you to do a code review.  Please visit

    http://gerrit.ovirt.org/21385

to review the following change.

Change subject: spec: Don't depend on unreleased RPMs.
......................................................................

spec: Don't depend on unreleased RPMs.

The requested package is not yet available in RHEL and will only be
available in 6.5.

Change-Id: I4a753dd4fdb8f32e53ac3231b37c60fbafeb82b3
Signed-off-by: Saggi Mizrahi <smizr...@redhat.com>
---
M vdsm.spec.in
1 file changed, 11 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/85/21385/1

diff --git a/vdsm.spec.in b/vdsm.spec.in
index 17564ea..c225c23 100644
--- a/vdsm.spec.in
+++ b/vdsm.spec.in
@@ -131,6 +131,17 @@
 %endif
 Requires: libvirt-python, libvirt-lock-sanlock, libvirt-client
 
+# iscsi-intiator versions
+%if 0%{?rhel}
+%if 0%{?rhel} >= 7
+Requires: iscsi-initiator-utils
+%else
+Requires: iscsi-initiator-utils >= 6.2.0.872-15
+%endif # rhel 7
+%else # fedora
+Requires: iscsi-initiator-utils >= 6.2.0.873-3
+%endif
+
 %if 0%{?rhel}
 Requires: python-ethtool >= 0.6-3
 %if 0%{?rhel} >= 7
@@ -138,7 +149,6 @@
 Requires: device-mapper-multipath
 Requires: e2fsprogs
 Requires: fence-agents-all
-Requires: iscsi-initiator-utils
 Requires: lvm2
 Requires: python
 Requires: policycoreutils-python
@@ -152,7 +162,6 @@
 # Update the qemu-kvm requires when block_stream will be included
 Requires: qemu-kvm >= 2:0.12.1.2-2.295.el6_3.4
 Requires: qemu-img >= 2:0.12.1.2-2.295.el6_3.4
-Requires: iscsi-initiator-utils >= 6.2.0.873-3
 Requires: device-mapper-multipath >= 0.4.9-52
 Requires: e2fsprogs >= 1.41.12-11
 Requires: fence-agents
@@ -184,7 +193,6 @@
 %if 0%{?fedora} >= 18
 Requires: initscripts >= 9.42.2-1
 %endif
-Requires: iscsi-initiator-utils >= 6.2.0.872-14
 Requires: device-mapper-multipath >= 0.4.9-18
 Requires: e2fsprogs >= 1.41.14
 Requires: sanlock >= 2.4-2, sanlock-python


-- 
To view, visit http://gerrit.ovirt.org/21385
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I4a753dd4fdb8f32e53ac3231b37c60fbafeb82b3
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <ybron...@redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizr...@redhat.com>
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to