> -----Original Message-----
> From: dev [mailto:[email protected]] On Behalf Of Thomas Monjalon
> Sent: Thursday, January 17, 2019 10:27 PM
> To: Zhang, Roy Fan <[email protected]>
> Cc: [email protected]; [email protected]; De Lara Guarch, Pablo 
> <[email protected]>
> 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 <[email protected]>
> > ---
> > -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".
> 

Reply via email to