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

Reply via email to