Re: [ovs-dev] [PATCH] m4: Fix 32bit build with 64bit version of libunwind-dev installed.

2019-10-02 Thread William Tu
On Wed, Oct 02, 2019 at 06:46:36PM +0300, Ilya Maximets wrote: > On 02.10.2019 17:46, Ben Pfaff wrote: > >On Tue, Oct 01, 2019 at 06:40:10PM -0700, William Tu wrote: > >>On Tue, Oct 1, 2019 at 4:19 PM William Tu wrote: > >>> > >>>On Tue, Oct 01, 2019 at 08:29:41PM +0300, Ilya Maximets wrote: >

Re: [ovs-dev] [PATCH] m4: Fix 32bit build with 64bit version of libunwind-dev installed.

2019-10-02 Thread Ben Pfaff
On Wed, Oct 02, 2019 at 06:46:36PM +0300, Ilya Maximets wrote: > I think that we need both solutions. Conditional installing of > 32bit library will allow us to test with it. However, I think, > that we need to eliminate usage of OVS_CFLAGS in our CI scripts > too, because this gives a bad

Re: [ovs-dev] [PATCH] m4: Fix 32bit build with 64bit version of libunwind-dev installed.

2019-10-02 Thread Ilya Maximets
On 02.10.2019 17:46, Ben Pfaff wrote: On Tue, Oct 01, 2019 at 06:40:10PM -0700, William Tu wrote: On Tue, Oct 1, 2019 at 4:19 PM William Tu wrote: On Tue, Oct 01, 2019 at 08:29:41PM +0300, Ilya Maximets wrote: On 01.10.2019 19:56, Ben Pfaff wrote: On Tue, Oct 01, 2019 at 07:46:10PM +0300,

Re: [ovs-dev] [PATCH] m4: Fix 32bit build with 64bit version of libunwind-dev installed.

2019-10-02 Thread Ben Pfaff
On Tue, Oct 01, 2019 at 06:40:10PM -0700, William Tu wrote: > On Tue, Oct 1, 2019 at 4:19 PM William Tu wrote: > > > > On Tue, Oct 01, 2019 at 08:29:41PM +0300, Ilya Maximets wrote: > > > On 01.10.2019 19:56, Ben Pfaff wrote: > > > >On Tue, Oct 01, 2019 at 07:46:10PM +0300, Ilya Maximets wrote: >

Re: [ovs-dev] [PATCH] m4: Fix 32bit build with 64bit version of libunwind-dev installed.

2019-10-01 Thread William Tu
On Tue, Oct 1, 2019 at 4:19 PM William Tu wrote: > > On Tue, Oct 01, 2019 at 08:29:41PM +0300, Ilya Maximets wrote: > > On 01.10.2019 19:56, Ben Pfaff wrote: > > >On Tue, Oct 01, 2019 at 07:46:10PM +0300, Ilya Maximets wrote: > > >>On 01.10.2019 19:17, Ben Pfaff wrote: > > >>>On Tue, Oct 01, 2019

Re: [ovs-dev] [PATCH] m4: Fix 32bit build with 64bit version of libunwind-dev installed.

2019-10-01 Thread William Tu
On Tue, Oct 01, 2019 at 08:29:41PM +0300, Ilya Maximets wrote: > On 01.10.2019 19:56, Ben Pfaff wrote: > >On Tue, Oct 01, 2019 at 07:46:10PM +0300, Ilya Maximets wrote: > >>On 01.10.2019 19:17, Ben Pfaff wrote: > >>>On Tue, Oct 01, 2019 at 02:22:20PM +0300, Ilya Maximets wrote: > 32 and 64 bit

Re: [ovs-dev] [PATCH] m4: Fix 32bit build with 64bit version of libunwind-dev installed.

2019-10-01 Thread Ilya Maximets
On 01.10.2019 19:56, Ben Pfaff wrote: On Tue, Oct 01, 2019 at 07:46:10PM +0300, Ilya Maximets wrote: On 01.10.2019 19:17, Ben Pfaff wrote: On Tue, Oct 01, 2019 at 02:22:20PM +0300, Ilya Maximets wrote: 32 and 64 bit versions of libunwind-dev package can not be installed at the same time at

Re: [ovs-dev] [PATCH] m4: Fix 32bit build with 64bit version of libunwind-dev installed.

2019-10-01 Thread Ben Pfaff
On Tue, Oct 01, 2019 at 07:46:10PM +0300, Ilya Maximets wrote: > On 01.10.2019 19:17, Ben Pfaff wrote: > > On Tue, Oct 01, 2019 at 02:22:20PM +0300, Ilya Maximets wrote: > > > 32 and 64 bit versions of libunwind-dev package can not be installed > > > at the same time at least in Debian and Ubuntu.

Re: [ovs-dev] [PATCH] m4: Fix 32bit build with 64bit version of libunwind-dev installed.

2019-10-01 Thread Ilya Maximets
On 01.10.2019 19:17, Ben Pfaff wrote: On Tue, Oct 01, 2019 at 02:22:20PM +0300, Ilya Maximets wrote: 32 and 64 bit versions of libunwind-dev package can not be installed at the same time at least in Debian and Ubuntu. AC_CHECK_LIB macro doesn't know if we have '-m32' in OVS_CFLAGS, so we need

Re: [ovs-dev] [PATCH] m4: Fix 32bit build with 64bit version of libunwind-dev installed.

2019-10-01 Thread Ben Pfaff
On Tue, Oct 01, 2019 at 02:22:20PM +0300, Ilya Maximets wrote: > 32 and 64 bit versions of libunwind-dev package can not be installed > at the same time at least in Debian and Ubuntu. AC_CHECK_LIB macro > doesn't know if we have '-m32' in OVS_CFLAGS, so we need to > additionally check for actual