Re: [ovs-dev] [PATCH] fedora-build: Remove %{build_number} from the configure line

2013-12-16 Thread Kyle Mestery (kmestery) 

On Dec 14, 2013, at 11:27 AM, Gurucharan Shetty  wrote:

> On Fri, Dec 13, 2013 at 6:53 PM, Kyle Mestery  wrote:
>> Fixes the OVS builds on Fedora, which are currently broken upstream.
>> 
>> Signed-off-by: Kyle Mestery 
> Thanks for fixing.
> I applied this to master.
> 
Thanks for the review and push Guru!

Kyle

>> ---
>> rhel/openvswitch-fedora.spec.in  | 2 +-
>> rhel/openvswitch-kmod-fedora.spec.in | 2 +-
>> 2 files changed, 2 insertions(+), 2 deletions(-)
>> 
>> diff --git a/rhel/openvswitch-fedora.spec.in 
>> b/rhel/openvswitch-fedora.spec.in
>> index 5384c32..197541f 100644
>> --- a/rhel/openvswitch-fedora.spec.in
>> +++ b/rhel/openvswitch-fedora.spec.in
>> @@ -35,7 +35,7 @@ traffic.
>> %setup -q -n openvswitch-%{version}
>> 
>> %build
>> -./configure --prefix=/usr --sysconfdir=/etc 
>> --localstatedir=%{_localstatedir} --enable-ssl %{build_number}
>> +./configure --prefix=/usr --sysconfdir=/etc 
>> --localstatedir=%{_localstatedir} --enable-ssl
>> make %{_smp_mflags}
>> 
>> %install
>> diff --git a/rhel/openvswitch-kmod-fedora.spec.in 
>> b/rhel/openvswitch-kmod-fedora.spec.in
>> index 16a8c73..ed38002 100644
>> --- a/rhel/openvswitch-kmod-fedora.spec.in
>> +++ b/rhel/openvswitch-kmod-fedora.spec.in
>> @@ -34,7 +34,7 @@ traffic. This package contains the kernel modules.
>> %setup -q -n openvswitch-%{version}
>> 
>> %build
>> -./configure --prefix=/usr --sysconfdir=/etc 
>> --localstatedir=%{_localstatedir} --with-linux=/lib/modules/%{kernel}/build 
>> --enable-ssl %{build_number}
>> +./configure --prefix=/usr --sysconfdir=/etc 
>> --localstatedir=%{_localstatedir} --with-linux=/lib/modules/%{kernel}/build 
>> --enable-ssl
>> make %{_smp_mflags} -C datapath/linux
>> 
>> %install
>> --
>> 1.8.3.1
>> 
>> ___
>> dev mailing list
>> dev@openvswitch.org
>> http://openvswitch.org/mailman/listinfo/dev

___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH] fedora-build: Remove %{build_number} from the configure line

2013-12-14 Thread Gurucharan Shetty 
On Fri, Dec 13, 2013 at 6:53 PM, Kyle Mestery  wrote:
> Fixes the OVS builds on Fedora, which are currently broken upstream.
>
> Signed-off-by: Kyle Mestery 
Thanks for fixing.
I applied this to master.

> ---
>  rhel/openvswitch-fedora.spec.in  | 2 +-
>  rhel/openvswitch-kmod-fedora.spec.in | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/rhel/openvswitch-fedora.spec.in b/rhel/openvswitch-fedora.spec.in
> index 5384c32..197541f 100644
> --- a/rhel/openvswitch-fedora.spec.in
> +++ b/rhel/openvswitch-fedora.spec.in
> @@ -35,7 +35,7 @@ traffic.
>  %setup -q -n openvswitch-%{version}
>
>  %build
> -./configure --prefix=/usr --sysconfdir=/etc 
> --localstatedir=%{_localstatedir} --enable-ssl %{build_number}
> +./configure --prefix=/usr --sysconfdir=/etc 
> --localstatedir=%{_localstatedir} --enable-ssl
>  make %{_smp_mflags}
>
>  %install
> diff --git a/rhel/openvswitch-kmod-fedora.spec.in 
> b/rhel/openvswitch-kmod-fedora.spec.in
> index 16a8c73..ed38002 100644
> --- a/rhel/openvswitch-kmod-fedora.spec.in
> +++ b/rhel/openvswitch-kmod-fedora.spec.in
> @@ -34,7 +34,7 @@ traffic. This package contains the kernel modules.
>  %setup -q -n openvswitch-%{version}
>
>  %build
> -./configure --prefix=/usr --sysconfdir=/etc 
> --localstatedir=%{_localstatedir} --with-linux=/lib/modules/%{kernel}/build 
> --enable-ssl %{build_number}
> +./configure --prefix=/usr --sysconfdir=/etc 
> --localstatedir=%{_localstatedir} --with-linux=/lib/modules/%{kernel}/build 
> --enable-ssl
>  make %{_smp_mflags} -C datapath/linux
>
>  %install
> --
> 1.8.3.1
>
> ___
> dev mailing list
> dev@openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH] fedora-build: Remove %{build_number} from the configure line

2013-12-13 Thread Ben Pfaff 
On Fri, Dec 13, 2013 at 09:53:25PM -0500, Kyle Mestery wrote:
> Fixes the OVS builds on Fedora, which are currently broken upstream.
> 
> Signed-off-by: Kyle Mestery 

Guru, will you review this (and apply it if you're happy with it)?

Thanks,

Ben.
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] [PATCH] fedora-build: Remove %{build_number} from the configure line

2013-12-13 Thread Kyle Mestery 
Fixes the OVS builds on Fedora, which are currently broken upstream.

Signed-off-by: Kyle Mestery 
---
 rhel/openvswitch-fedora.spec.in  | 2 +-
 rhel/openvswitch-kmod-fedora.spec.in | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/rhel/openvswitch-fedora.spec.in b/rhel/openvswitch-fedora.spec.in
index 5384c32..197541f 100644
--- a/rhel/openvswitch-fedora.spec.in
+++ b/rhel/openvswitch-fedora.spec.in
@@ -35,7 +35,7 @@ traffic.
 %setup -q -n openvswitch-%{version}
 
 %build
-./configure --prefix=/usr --sysconfdir=/etc --localstatedir=%{_localstatedir} 
--enable-ssl %{build_number}
+./configure --prefix=/usr --sysconfdir=/etc --localstatedir=%{_localstatedir} 
--enable-ssl
 make %{_smp_mflags}
 
 %install
diff --git a/rhel/openvswitch-kmod-fedora.spec.in 
b/rhel/openvswitch-kmod-fedora.spec.in
index 16a8c73..ed38002 100644
--- a/rhel/openvswitch-kmod-fedora.spec.in
+++ b/rhel/openvswitch-kmod-fedora.spec.in
@@ -34,7 +34,7 @@ traffic. This package contains the kernel modules.
 %setup -q -n openvswitch-%{version}
 
 %build
-./configure --prefix=/usr --sysconfdir=/etc --localstatedir=%{_localstatedir} 
--with-linux=/lib/modules/%{kernel}/build --enable-ssl %{build_number}
+./configure --prefix=/usr --sysconfdir=/etc --localstatedir=%{_localstatedir} 
--with-linux=/lib/modules/%{kernel}/build --enable-ssl
 make %{_smp_mflags} -C datapath/linux
 
 %install
-- 
1.8.3.1

___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev