[ovs-dev,1/2] rhel: Make python 2.7 a requirement of build.
diff mbox

Message ID 1442421031-11694-1-git-send-email-gshetty@nicira.com
State Accepted
Headers show

Commit Message

Gurucharan Shetty Sept. 16, 2015, 4:30 p.m. UTC
Reported-by: Dotslash Lu <dotslash.lu@gmail.com>
Signed-off-by: Gurucharan Shetty <gshetty@nicira.com>
---
 AUTHORS                  |    1 +
 rhel/openvswitch.spec.in |    2 +-
 2 files changed, 2 insertions(+), 1 deletion(-)

Comments

Flavio Leitner Sept. 22, 2015, 12:24 a.m. UTC | #1
On Wed, Sep 16, 2015 at 09:30:30AM -0700, Gurucharan Shetty wrote:
> diff --git a/rhel/openvswitch.spec.in b/rhel/openvswitch.spec.in
> index dad3e04..642986b 100644
> --- a/rhel/openvswitch.spec.in
> +++ b/rhel/openvswitch.spec.in
> @@ -22,7 +22,7 @@ License: ASL 2.0
>  Release: 1
>  Source: openvswitch-%{version}.tar.gz
>  Buildroot: /tmp/openvswitch-rpm
> -Requires: logrotate, python
> +Requires: logrotate, python >= 2.7

I think this will break CentOS-6.x and RHEL-6.x because they don't
have python 2.7 RPMs yet.

fbl
Gurucharan Shetty Sept. 22, 2015, 2:22 p.m. UTC | #2
On Mon, Sep 21, 2015 at 5:24 PM, Flavio Leitner <fbl@sysclose.org> wrote:
> On Wed, Sep 16, 2015 at 09:30:30AM -0700, Gurucharan Shetty wrote:
>> diff --git a/rhel/openvswitch.spec.in b/rhel/openvswitch.spec.in
>> index dad3e04..642986b 100644
>> --- a/rhel/openvswitch.spec.in
>> +++ b/rhel/openvswitch.spec.in
>> @@ -22,7 +22,7 @@ License: ASL 2.0
>>  Release: 1
>>  Source: openvswitch-%{version}.tar.gz
>>  Buildroot: /tmp/openvswitch-rpm
>> -Requires: logrotate, python
>> +Requires: logrotate, python >= 2.7
>
> I think this will break CentOS-6.x and RHEL-6.x because they don't
> have python 2.7 RPMs yet.

So, OVS master needs python 2.7. (I was able to install python 2.7
from sources on CentOS 6). We have the same problem for Xenserver
platforms too. There was a long discussion sometime back where the
decision for made to move to python 2.7. I don't know what is a good
way to solve this predicament.


>
> fbl
>

Patch
diff mbox

diff --git a/AUTHORS b/AUTHORS
index a7f40bb..95d2949 100644
--- a/AUTHORS
+++ b/AUTHORS
@@ -254,6 +254,7 @@  David Palma             palma@onesource.pt
 Derek Cormier           derek.cormier@lab.ntt.co.jp
 Dhaval Badiani          dbadiani@vmware.com
 DK Moon                 dkmoon@nicira.com
+Dotslash Lu             dotslash.lu@gmail.com
 Edwin Chiu              echiu@vmware.com
 Eivind Bulie Haanaes
 Eric Lopez              elopez@nicira.com
diff --git a/rhel/openvswitch.spec.in b/rhel/openvswitch.spec.in
index dad3e04..642986b 100644
--- a/rhel/openvswitch.spec.in
+++ b/rhel/openvswitch.spec.in
@@ -22,7 +22,7 @@  License: ASL 2.0
 Release: 1
 Source: openvswitch-%{version}.tar.gz
 Buildroot: /tmp/openvswitch-rpm
-Requires: logrotate, python
+Requires: logrotate, python >= 2.7
 BuildRequires: openssl-devel
 
 %bcond_without check