On 7/6/17, 2:36 PM, "[email protected] on behalf of Ben Pfaff"
<[email protected] on behalf of [email protected]> wrote:
Can someone review this please?
On Mon, May 01, 2017 at 10:30:27AM -0700, Ben Pfaff wrote:
> Hemant, does this fix the problem you reported?
>
> On Fri, Apr 14, 2017 at 09:14:55PM -0700, Ben Pfaff wrote:
> > The check for rte_config.h in acinclude.m4 used AC_CHECK_FILE, but this
> > macro is intended to check for a file on the host system, not the build
> > system, which means that it fails unconditionally in a cross-compilation
> > environment. However, the intended check here is for a header file,
> > which is part of the build system. To check for part of the build
system,
> > we can just use "test", so this commit makes that change.
> >
> > Reported-by: Hemant Agrawal <[email protected]>
> > Reported-at:
https://urldefense.proofpoint.com/v2/url?u=https-3A__mail.openvswitch.org_pipermail_ovs-2Ddev_2017-2DMarch_329994.html&d=DwICAg&c=uilaK90D4TOVoH58JNXRgQ&r=BVhFA09CGX7JQ5Ih-uZnsw&m=9SEip3J9loq15hpoutcyARnoiI9CS9RRVfT0oFE5C8k&s=dOQYeMGyHRUBxuPzkJLkr2oUHFTObc-qjwZhHCDFTrU&e=
> > Signed-off-by: Ben Pfaff <[email protected]>
> > ---
> > acinclude.m4 | 7 ++++---
> > 1 file changed, 4 insertions(+), 3 deletions(-)
> >
> > diff --git a/acinclude.m4 b/acinclude.m4
> > index 744d8f89525c..842469455914 100644
> > --- a/acinclude.m4
> > +++ b/acinclude.m4
> > @@ -180,9 +180,10 @@ AC_DEFUN([OVS_CHECK_DPDK], [
> > DPDK_INCLUDE="$with_dpdk/include"
> > # If 'with_dpdk' is passed install directory, point to headers
> > # installed in $DESTDIR/$prefix/include/dpdk
> > - AC_CHECK_FILE([$DPDK_INCLUDE/rte_config.h], [],
> > - [AC_CHECK_FILE([$DPDK_INCLUDE/dpdk/rte_config.h],
> > -
[DPDK_INCLUDE=$DPDK_INCLUDE/dpdk], [])])
> > + if test ! -e "$DPDK_INCLUDE/rte_config.h" && \
> > + test -e "$DPDK_INCLUDE/dpdk/rte_config.h"; then
> > + DPDK_INCLUDE=$DPDK_INCLUDE/dpdk/rte_config.h
Did you mean
DPDK_INCLUDE=$DPDK_INCLUDE/dpdk
rather than
DPDK_INCLUDE=$DPDK_INCLUDE/dpdk/rte_config.h
?
> > + fi
> > DPDK_LIB_DIR="$with_dpdk/lib"
> > ;;
> > esac
> > --
> > 2.10.2
> >
_______________________________________________
dev mailing list
[email protected]
https://urldefense.proofpoint.com/v2/url?u=https-3A__mail.openvswitch.org_mailman_listinfo_ovs-2Ddev&d=DwICAg&c=uilaK90D4TOVoH58JNXRgQ&r=BVhFA09CGX7JQ5Ih-uZnsw&m=9SEip3J9loq15hpoutcyARnoiI9CS9RRVfT0oFE5C8k&s=5mgaD8RH43sN-x5q37TpOgF-sG-M7rA0u3U4L9eOrP0&e=
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev