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.richard...@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.gua...@intel.com
> >      Cc: [3]sta...@dpdk.org
> >      Signed-off-by: Bruce Richardson <[4]bruce.richard...@intel.com>
> > 
> >    Tested on FreeBSD 10.4.
> >    Reviewed-by: David Marchand <[5]david.march...@redhat.com>

Applied, thanks


Reply via email to