Change in vdsm[ovirt-3.2]: vdsm.spec: Don't require python-ordereddict on fedora

dougsland at redhat.com dougsland at redhat.com
Tue Jan 15 16:21:04 UTC 2013


Hello Mark Wu,

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

    http://gerrit.ovirt.org/11057

to review the following change.

Change subject: vdsm.spec: Don't require python-ordereddict on fedora
......................................................................

vdsm.spec: Don't require python-ordereddict on fedora

It is a regression introduced by commit bb0620f. The condition "0%{?rhel} < 7"
also holds true on fedora, so it causes to require python-ordereddict on fedora

Change-Id: I87e167f207669f4dda5bb71d7809fe301ea7e905
Signed-off-by: Mark Wu <wudxw at linux.vnet.ibm.com>
Signed-off-by: Douglas Schilling Landgraf <dougsland at redhat.com>
---
M vdsm.spec.in
1 file changed, 2 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/57/11057/1

diff --git a/vdsm.spec.in b/vdsm.spec.in
index c01d29f..b518045 100644
--- a/vdsm.spec.in
+++ b/vdsm.spec.in
@@ -44,7 +44,7 @@
 BuildRequires: genisoimage
 BuildRequires: openssl
 BuildRequires: m2crypto
-%if 0%{?rhel} < 7
+%if 0%{?rhel} == 6
 BuildRequires: python-ordereddict
 %endif
 
@@ -174,7 +174,7 @@
 BuildArch:      noarch
 
 Requires: %{name}-python = %{version}-%{release}
-%if 0%{?rhel} < 7
+%if 0%{?rhel} == 6
 Requires: python-ordereddict
 %endif
 


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I87e167f207669f4dda5bb71d7809fe301ea7e905
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.2
Gerrit-Owner: Douglas Schilling Landgraf <dougsland at redhat.com>
Gerrit-Reviewer: Mark Wu <wudxw at linux.vnet.ibm.com>


More information about the vdsm-patches mailing list