[dpdk-dev] [dpdk-stable] [PATCH] app/test: fix missing header

Thomas Monjalon thomas at monjalon.net
Fri Apr 19 23:17:16 CEST 2019


19/04/2019 14:12, Bruce Richardson:
> On Fri, Apr 19, 2019 at 02:09:51PM +0200, David Marchand wrote:
> >    nit: "test/compress:" for the title prefix ?
> 
> maybe so, half the time I never know what to prefix autotest stuff like
> this with! :-) Thomas, please fix on apply, if possible.

In this case, you can just copy the prefix from the "Fixes:" line :)

> >    On Fri, Apr 19, 2019 at 2:03 PM Bruce Richardson
> >    <[1]bruce.richardson at intel.com> wrote:
> > 
> >      Build of test_compressdev.c fails on FreeBSD due to the missing
> >      include of unistd.h, to provide the function usleep.
> >      Fixes: b06aa643cac4 ("test/compress: add initial unit tests")
> >      Cc: [2]pablo.de.lara.guarch at intel.com
> >      Cc: [3]stable at dpdk.org
> >      Signed-off-by: Bruce Richardson <[4]bruce.richardson at intel.com>
> > 
> >    Tested on FreeBSD 10.4.
> >    Reviewed-by: David Marchand <[5]david.marchand at redhat.com>

Applied, thanks




More information about the dev mailing list