On 04.10.2019 19:31, Ben Pfaff wrote:
On Fri, Oct 04, 2019 at 04:37:38PM +0200, Ilya Maximets wrote:
Our CI scripts uses OVS_CFLAGS variable that is intended for internal
usage by 'configure' script only.  Usual CFLAGS should be used instead
to avoid giving bad example to users.

Additionally, '-m32' flag passed directly to CC variable to avoid
splitting it from the compiler invocations and force same API/ABI for
invocations of 'configure' and 'make'.
'BUILD_ENV' dropped as not needed anymore.

Before this patch 'configure' always checked for 64bit libraries
regardless of fact that we're going to build 32bit binary.  This
caused issues if only 64bit version of desired library was available.

Suggested-by: Ben Pfaff <b...@ovn.org>
Signed-off-by: Ilya Maximets <i.maxim...@ovn.org>
---

Version 2:
   * CFLAGS passed to './configure' where possible (except 'distcheck' case).

Acked-by: Ben Pfaff <b...@ovn.org>

Thanks! I gave it one more run and applied to master.

Best regards, Ilya Maximets.
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to