> -----Original Message-----
> From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Thomas Monjalon
> Sent: Thursday, January 17, 2019 10:27 PM
> To: Zhang, Roy Fan <roy.fan.zh...@intel.com>
> Cc: dev@dpdk.org; akhil.go...@nxp.com; De Lara Guarch, Pablo
> <pablo.de.lara.gua...@intel.com>
> Subject: Re: [dpdk-dev] [PATCH] crypto/aesni_mb: fix compile
>
> 16/01/2019 14:21, Fan Zhang:
> > This patch fixes the inflexible compile issue AESNI-MB PMD.
> > Originally the compile of the PMD will fail if IPSec_MB is
> > not installed in default location.
> >
> > Fixes: c68d7aa354f6 ("crypto/aesni_mb: use architecture independent macros")
> >
> > Signed-off-by: Fan Zhang <roy.fan.zh...@intel.com>
> > ---
> > -IMB_HDR = /usr/include/intel-ipsec-mb.h
> > +IMB_HDR = $(shell echo "\#include <intel-ipsec-mb.h>" | \
>
> What is the backslash? Looks like a typo.
Wouldn't make get confused without it?
Konstantin
>
> > + $(CC) -E $(EXTRA_CFLAGS) - | grep 'intel-ipsec-mb.h' | \
> > + head -1 | cut -d'"' -f2)
>
> I think the most standard invocation of head is "head -n1".
> You can also use "sed 1q".
>
