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. > 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]> > --- > I sent this previously, but I don't think it made the public list, > so resending. > Apologies to anyone who gets this twice! > --- > app/test/test_compressdev.c | 1 + > 1 file changed, 1 insertion(+) > diff --git a/app/test/test_compressdev.c > b/app/test/test_compressdev.c > index 404b98f60..121514af5 100644 > --- a/app/test/test_compressdev.c > +++ b/app/test/test_compressdev.c > @@ -5,6 +5,7 @@ > #include <zlib.h> > #include <math.h> > #include <stdlib.h> > +#include <unistd.h> > #include <rte_cycles.h> > #include <rte_malloc.h> > -- > 2.20.1 > > Tested on FreeBSD 10.4. > Reviewed-by: David Marchand <[5][email protected]> > -- > David Marchand > Thanks, /Bruce
