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://mail.openvswitch.org/pipermail/ovs-dev/2017-March/329994.html
> 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
> +     fi
>          DPDK_LIB_DIR="$with_dpdk/lib"
>          ;;
>      esac
> -- 
> 2.10.2
> 
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to