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