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][email protected]> 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][email protected] > > Cc: [3][email protected] > > Signed-off-by: Bruce Richardson <[4][email protected]> > > > > Tested on FreeBSD 10.4. > > Reviewed-by: David Marchand <[5][email protected]> Applied, thanks
